Pull to refresh

Comments 5

Вы уж простите но сейчас буду критиковать.
1) Свои модули стоит оформлять как модули npm, даже если и не выкладывать в сам npm
2) Определять методы внутри функции конструктора, это антипатер приводящий к лишнему расходу памяти. на каждый вызов new VK, у вас будет создаваться куча функций, причем объективных причин так писать я не вижу.
3) Запрашивать и получать результат весьма неудобно на мой взгляд. Куда проще событий выглядит такой код:
vk.getProfile({ ... }, function onGetProfile(err, result) {
   ....
});

4) То же самое для методов setToken
5) Давать произвольные имена событиям, лично мне кажется совсем плохой «фичей», можно такую кашу намутить, а необходимость в этом сомнительна
6) Тесты?
Ну, со многим я соглашусь. Будет еще версия с правками, значит.
События, скорее всего, останутся, а вот коллбеки я все же сделаю, чтобы можно было выбирать, кому что удобней будет.
Недавно написал библиотеку, для создания rest api клиентов под js, правда пока еще не успел обкатать, но вы можете посмотреть идею, да и саму реализацию на github, вдруг понравится или будут идеи по улучшению.
Если честно, я бы использовал другой подход (fermata), но либа должна еще работать в браузерах, а в решении по ссылке используется новый es6 Proxy.
Тоже немножко критики:
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.

Articles