CODE_RECOMMENDATIONS.md

Node.js code recommendations

Here are some of the recommendations we have come up based on perf-quest.

This document is a output of this issue

It is always interesting to read about what we should or should not do related to performance. Doing so based on test results is even better.

Results

• Give preference to the use of ES6 classes. We have 4 tests related with ES6 classes, and they win on the first three cases.

  • Class and Classless
  • Class and 'UnfrozenClassless'
  • Class and module
  • Class and prototype

• Avoid forEach

  • forEach and for

• Avoid the creation of extra hidden classes

  • Hidden classes

• Filter the array 'by hand' instead array.filter

  • Array filter

• Map the array 'by hand' instead array.map

  • Array map

• Use array.reduce instead do it 'by hand'

  • Array reduce

• If you are using WeakMap for data hiding, then prefer Symbol

  • WeakMap and Symbol

How to contribute

• Clone perf-quest
• Create a new issue
• Follow the readme

Sources used to make the tests

Our current plan on this issue is to write test cases to get results based on:

1. Our doubts and project needs.
2. Try to cover affirmations like 'Don't do X, don't do Y' found in videos and blogs.

example: "Don't use prototypes because this has a bad performance"

• http://debuggable.com/posts/understanding-hidden-classes-in-v8:4c7e81e4-1330-4398-8bd2-761bcbdd56cb
• https://www.smashingmagazine.com/2012/11/writing-fast-memory-efficient-javascript/
• https://gist.github.com/twokul/9501770
• https://www.html5rocks.com/en/tutorials/speed/v8/
• https://coderwall.com/p/kvzbpa/don-t-use-array-foreach-use-for-instead
• https://www.youtube.com/watch?v=UJPdhx5zTaw
• http://lanceball.com/words/2016/10/14/data-hiding-in-es6/