You are viewing a single comment's thread from:

RE: Actifit BOT Development Updates: Exchange Functionality, Historical Data, Endpoints & More

in #actifit5 years ago

Thank you for your contribution.

  1. there are quite a lot of console.log, as I understand this might be useful sometimes (are they visible to the end-users?), you might want to extract the logging function into a separate Logger class, which could be configured to write to log files (persistent for debuging purpose)
  2. You might want to format your code, as some are not properly styled.
  3. for Javascript, you might want to cache array length in a for loop such as for (let i = 0, len = groupedTxs[index].length; i < len; i++) {
  4. Inconsistent usage of semi-colon
  5. var could be completely replaced by either let or const
  6. commented out code that is misleading. such as
  7. Missing exception handling when e.g. MongoDB is not accessible.
  8. Nested try-catch all catch general exceptions is not a good idea e.g. code-smell.
  9. Regarding this commit not sure why it has shown changes even quite a lot of code are actually un-changed.

Your contribution has been evaluated according to Utopian policies and guidelines, as well as a predefined set of questions pertaining to the category.

To view those questions and the relevant answers related to your post, click here.


Need help? Chat with us on Discord.

[utopian-moderator]

Sort:  

Thank you for your review, @justyy! Keep up the good work!

Coin Marketplace

STEEM 0.18
TRX 0.16
JST 0.031
BTC 60354.53
ETH 2608.25
USDT 1.00
SBD 2.54