This project contains a collection of demo modules that showcase different techniques and features of Angular development and how to implement them.

Please know that this project follows Angular style guid, and all of our future Angular application should follow Angular style guid.

This project utilized Angular feature modules to help separate codes related to different features in their own area (each module also utilized lazy loading to decrease load time):

• Complex Forms Module: Demonstrate how to create a complex form the span across different components
• End-To-End Testing Module: Demonstrates how to perform end-to-end testing on an Angular application using Protractor
• Inputs And Outputs Module: General guideline for using inputs and outputs for Angular components
• Internationalization Module: Demonstrate how to use ngx-translate
• RxJS Module: Demo the more common operators, and subjects; It does not cover everything within the RxJS library
• Unit Testing Module: Demonstrate what we are expecting when creating unit tests for different class types
• Web Storage Module: Demonstrate how to add, update, delete, and get items using the Web Storage Api

To learn more about feature module, please read:

• What is feature module: https://angular.io/guide/feature-modules
• Different module type: https://angular.io/guide/module-types
• What is lazy loaded module: https://angular.io/guide/lazy-loading-ngmodules
• What is shared module: https://angular.io/guide/sharing-ngmodules

On top of the configurations created by Angular CLI, we also require some additional configurations. This section will go over all the additional configurations expected to be in every future Angular project and on everyone's machine.

It is absolutely necessary for every Angular workspace to have launch and task configurations that allows for debugging npm start, npm run test, and npm run e2e. To create these configurations follow these steps:

1. Navigate to the debugger tab with Ctrl+Shift+D or by clicking the debugger icon
2. If a launch.json file doesn't already exists you should see a create a launch.json file link. Click that link, and wait until vscode gives you some options, then select one of them (it doesn't matter which one because we will be replacing the contents anyways).
3. Copy the following json and replace the contents of the launch.json file with them

   {
     "version": "0.2.0",
     "configurations": [
       {
         "name": "ng serve",
         "type": "chrome",
         "request": "launch",
         "preLaunchTask": "npm: start",
         "url": "http://localhost:4200/#",
         "webRoot": "${workspaceFolder}",
         "sourceMapPathOverrides": {
           "webpack:/*": "${webRoot}/*",
           "/./*": "${webRoot}/*",
           "/src/*": "${webRoot}/*",
           "/*": "*",
           "/./~/*": "${webRoot}/node_modules/*"
         }
       },
       {
         "name": "ng test",
         "type": "chrome",
         "request": "launch",
         "url": "http://localhost:9876/debug.html",
         "webRoot": "${workspaceFolder}",
         "sourceMaps": true,
         "pathMapping": {
           "/_karma_webpack_": "${workspaceFolder}"
         },
         "sourceMapPathOverrides": {
           "webpack:/*": "${webRoot}/*",
           "/./*": "${webRoot}/*",
           "/src/*": "${webRoot}/*",
           "/*": "*",
           "/./~/*": "${webRoot}/node_modules/*"
         }
       },
       {
         "name": "ng e2e",
         "type": "node",
         "request": "launch",
         "program": "${workspaceFolder}/node_modules/protractor/bin/protractor",
         "protocol": "inspector",
         "args": ["${workspaceFolder}/e2e/protractor.conf.js"]
       }
     ]
   }

4. Create a task.json file in the same .vscode folder as your launch.json then copy and paste the following json into it

   {
     "version": "2.0.0",
     "tasks": [
       {
         "type": "npm",
         "script": "start",
         "isBackground": true,
         "presentation": {
           "focus": true,
           "panel": "dedicated"
         },
         "group": {
           "kind": "build",
           "isDefault": true
         },
         "problemMatcher": {
           "owner": "typescript",
           "source": "ts",
           "applyTo": "closedDocuments",
           "fileLocation": ["relative", "${cwd}"],
           "pattern": "$tsc",
           "background": {
             "activeOnStart": true,
             "beginsPattern": {
               "regexp": "(.*?)"
             },
             "endsPattern": {
               "regexp": "Compiled |Failed to compile."
             }
           }
         }
       }
     ]
   }

Once this is done you should see new configuration options in your debug tab that you can choose to run. The ng serve and ng e2e configurations will work the same as the npm start and npm run e2e commands. The ng test command however will require you to still run the npm run test command because it requires a test instance to attach to. The benefit you get from all of these configurations is that you can now add breakpoints to your code base to pause code execution. Allowing you to inspect the state of your application at a given point in time. This makes debugging much easier.

Since We expect everyone to utilize Visual Studio Code for Angular application development, the following are all the required VScode plugins:

• Angular Essentials: johnpapa.angular-essentials
• Prettier - Code formatter: esbenp.prettier-vscode
• npm: eg2.vscode-npm-script
• npm Intellisense: christian-kohler.npm-intellisense
• Node.js Modules Intellisense: leizongmin.node-module-intellisense
• Code Spell Checker: streetsidesoftware.code-spell-checker
• GitLens — Git supercharged: eamodio.gitlens

The list can also be found in .vscode/extensions.json of this solution, please copy extensions.json file to other Angular application's .vscode folder.

Since we need our code formatting to be the same for everyone, .prettierrc file is expected to be present in each Angular project and the file should contain the following:

{
  "singleQuote": true,
  "printWidth": 140
}

We expect each Angular project to contain .gitignore file and .gitattributes file to standardize our git configuration.

.gitattributes file should contain * text=auto to normalize line endings

Besides the following packages:

• dependency:
  • angular material
  • ngx-logger
  • ngx-translate
• dev dependency:
  • compodoc
  • spectator
  • faker

All packages need to be approved before added to the project.

When logging messages, we want to keep a standard format to make it easier for our future selves to debug/troubleshoot. Therefore, we need to make sure the log message follow the format below:

<Timestamp> <LogLevel> <ClassName>.<MethodName>: <Message>

• Timestamp is the ISO string representation of when the message is logged
• LogLevel is the uppercase string representation of the message's log level
• ClassName is the name of the class that logged the message
• MethodName is the name of the method that logged the message, can be omitted if and only if the method is constructor
• Message is the actual log message; If necessary, log message should contain data to aid the debugging process

The following are some example log message:

# Not using ngx-logger:
2020-04-01T14:48:36.580Z DEBUG FooClass.Bar: This is an example

# Using ngx-logger:
2020-04-01T14:48:36.580Z DEBUG [vendor.js:282391] FooClass.Bar: This is an example

Note: Timestamp and LogLevel will be automatically added to your log message when using ngx-logger

Do keep in mind that, most likely than not, the Angular application will be deployed remotely. Thus, log messages might be our only way to figure out what went wrong. So try to keep the log messages as informative as possible while not drowning the reader.

Basic understanding of how to create comment and different comment type is required for this section. If you need a refresher, please check out JSDoc Style Guide.

When documenting code, it is crucial to meet the following expectations so that people can understand what the code suppose to do without polluting it.

Class comment should only be created for class that is a service, pipe, guard, or interceptor to help others understand purpose of the class. Following is an example class comment:

/**
 * This service offers basic mathematical operations
 */
export class MathService{...}

Comment should be created for properties of a component with @Input() or @Output() attribute to help others understand the purpose of those properties. Below is an example comment:

/**
 * User ID required to retrieve user information
 */
@Input() id: string;

Every method should have a method comment, which should include:

• An description of what the method does
  • If the method is complex, consider also explain why method works this way
  • If the method is a simple wrapper, the description should be simple
• If applicable, @param, @returns, and @throws should also be present
  • Do NOT leave empty comment tag around! (Except the cases were the comment tag meant to be empty)

The following is an example method comment:

/**
 * Subtract b from a
 *
 * @param {number} a The first number
 * @param {number} b The second number
 *
 * @returns {number} Difference between a and b
 *
 * @throws {RangeError} If a or b is less than zero
 */
subtract(a: number, b: number): number { ... }

This sections contains some general code documentation expectations that should be followed, and some additional examples.

In addition to following Angular naming conventions, we should also make sure the names accurately reflect the purpose and functionality of the class, method, variable, etc in a short and concise manner. The following is an example of bad names and potential fixes:

// Bad Example:
// Function name and parameter names not reflecting purpose and functionality of each
a(u: number[], q: number): bool {
  return u.indexOf(q) > -1;
 }

// Potential Fixes:
// Rename method name and parameter names to be more descriptive
isTargetInArray(array: number[], target: number): bool {
  return array.indexOf(target) > -1;
 }

If code cannot be explained properly within the comments and cannot be simplified, by refactoring or otherwise, then a document should be created and linked in the comment of the code. An example can be find in AsyncMessageProcessor of old OrbitalBus.

If the code is complex enough to warrant an inline comment, consider refactor that pieces of code into its own method. Below is an example of how to avoid inline comment:

// Bad Example:
// Inline comment for code that is too complex
blur(origin: number, x: number): number {
  ...
  // calculates Gaussian probability of x
  const prob = Math.exp(- ((origin - x) ** 2) / (sigma ** 2) / 2.0) / Math.sqrt(2.0 * Math.PI * (sigma ** 2));
  ...
}

// Potential Fix:
// Move complex logic to another method and add comment to that method
blur(origin: number, x: number): number {
  ...
  const prob = gaussian(x, origin, sigma);
  ...
}

/**
 * Calculate Gaussian probability of x
 *
 * @param {number} x Target
 * @param {number} mu Mean of the distribution
 * @param {number} sigma Variance of the distribution
 *
 * @returns {number} Gaussian probability
 */
gaussian(x:number, mu: number, sigma: number) {
  return Math.exp(- ((origin - x) ** 2) / (sigma ** 2) / 2.0) / Math.sqrt(2.0 * Math.PI * (sigma ** 2));
}

If refactoring cannot be done or the code is already as simple as it can be, consider following advise of create document for complex logic section.

People are able to extract the information provided by the following comment from the class name alone, thus the comment is useless:

/**
 * Service that do math
 */
export class MathService{...}

Below is an ABSOLUTELY TERRIBLE comment! DON'T ever do this!

/**
 * Join the provided two whole numbers, can be positive, negative, or zero,
 * to compute the total sum
 *
 *  Algorithm: Works by utilizing the cascade of adders (an arithmetic logic units)
 *     within the processor to perform addition on the 32 bits two's complement binary
 *     number together then translate the binary number back to a base 10 number
 *
 *  Modification History: last edited by Bob on April 1st to increase performance by
 *     removing extra space
 *
 * @param {number} a The first whole number that can be positive, negative, or zero
 * @param {number} b The second whole number that can be positive, negative, or zero
 *
 * @returns {number} The total amount of the two natural numbers combined
 */
add(a: number, b: number): number {
  return a + b;
 }

Too much unnecessary information! The code already tell us how it works, we don't need paragraphs to explain what a + b does and we don't need to know Bob remove a space (we have git).