Comments 5
Вы уж простите но сейчас буду критиковать.
1) Свои модули стоит оформлять как модули npm, даже если и не выкладывать в сам npm
2) Определять методы внутри функции конструктора, это антипатер приводящий к лишнему расходу памяти. на каждый вызов new VK, у вас будет создаваться куча функций, причем объективных причин так писать я не вижу.
3) Запрашивать и получать результат весьма неудобно на мой взгляд. Куда проще событий выглядит такой код:
4) То же самое для методов setToken
5) Давать произвольные имена событиям, лично мне кажется совсем плохой «фичей», можно такую кашу намутить, а необходимость в этом сомнительна
6) Тесты?
1) Свои модули стоит оформлять как модули npm, даже если и не выкладывать в сам npm
2) Определять методы внутри функции конструктора, это антипатер приводящий к лишнему расходу памяти. на каждый вызов new VK, у вас будет создаваться куча функций, причем объективных причин так писать я не вижу.
3) Запрашивать и получать результат весьма неудобно на мой взгляд. Куда проще событий выглядит такой код:
vk.getProfile({ ... }, function onGetProfile(err, result) {
....
});
4) То же самое для методов setToken
5) Давать произвольные имена событиям, лично мне кажется совсем плохой «фичей», можно такую кашу намутить, а необходимость в этом сомнительна
6) Тесты?
Недавно написал библиотеку, для создания rest api клиентов под js, правда пока еще не успел обкатать, но вы можете посмотреть идею, да и саму реализацию на github, вдруг понравится или будут идеи по улучшению.
Если честно, я бы использовал другой подход (fermata), но либа должна еще работать в браузерах, а в решении по ссылке используется новый es6 Proxy.
Если честно, я бы использовал другой подход (fermata), но либа должна еще работать в браузерах, а в решении по ссылке используется новый es6 Proxy.
Тоже немножко критики:
1) Не нравятся события типа
2) Почитайте мне очень нравится nodeguide.com/style.html
3) На мой взгляд лучше так
4) И опять же почитайте п.2
5) Спасибо за труды
1) Не нравятся события типа
vk.on('done:getProfiles', function(_o) {
console.log(_o);
});
лучше делать проще и название переменной не самое лучшее2) Почитайте мне очень нравится nodeguide.com/style.html
3) На мой взгляд лучше так
var requestArray = [];
чем var requestArray = new Array();
4) И опять же почитайте п.2
5) Спасибо за труды
Sign up to leave a comment.
ВКонтакте Node JS SDK