javascript-blowfish-unique/code-review.txt

Blowfish encryption library Javascript, jquery,coffeescript (blowfish.js)

Форматирование
 + большинство кода соответсвует
 - лучше использовать упрощенные конструкции условий с выходом (см. encrypt и decrypt)

Документирование
  + описания полные
  - документирование должно соответсвовать http://en.wikipedia.org/wiki/JSDoc
  - к методу decipher не было доков

Перменные и объявления функций
  - new Array (ровно как и new Object лучше не использовать, вместо них [] и {})
  - array.join(delimetr) может коряво работать в разных браузерах (не буду показывать пальцем)
    при пустых элементах массива, лучше явно его заполнять
  - в конструкторе не обязательно return this;
  - объявление переменных через var очень внимательно:
    var c = c1 = c2 = 0;// - не объявит c1 и c2, а создаст их в глобальном окружении

    // надо так
    var c, c1, c2;
    c = c1 = c2 = 0;

  - объявление переменных к классам (а так же JSDoc'и к ним) необходимо вынести из конструктора и привязать
    к прототипу (там же можно инициализировать простые типы: числа, true/false, а так же строки);
    причина - не придется при вызове конструктора повторно инициализировать данные

Javascript and Coffeescript
  - желательно для строк использовать ", а ' использовать в иных случаях (типа 'это "строка"') и объявляя символы
    в js нет такого типа как char, но можно визуально отличить его (ch = 'a')
  - в js нет синтаксиса [var1, var2] = /*array*/ someFunctionReturnsArray(), а в coffee есть