[M.....s]

Mircy, robię bibliotekę wykopapi w nodejs, będzie działała zarówno na promises (Bluebird) jak i na zwykłych callbackach.
Czy ta metoda jest ok, czy coś jest do poprawki? Chcę się upewnić zanim przerobię wszystko w ten sposób

http://pastebin.com/kRhZNudx

#programowanie #nodejs #webdev #javascript

[G.....n]

@MirkoStats: W JavaScripcie stosujemy Klasa.takaKonwencja, a nie Klasa.taka_konwencja. ( ͡° ͜ʖ ͡°)

[G.....n]

@MirkoStats: PS. Wrzuć kod na GitHuba, to postaram Ci się w miarę możliwości zrobić jakieś pull requesty.

[Marmite]

@MirkoStats: A myślałeś o użyciu po prostu o użyciu promisifyAll?

[M.....s]

@Ginden: ok, wieczorem wrzucę.

W JavaScripcie stosujemy Klasa.takaKonwencja, a nie Klasa.taka_konwencja. ( ͡° ͜ʖ ͡°)

zasadniczo tak, ale istnieje już sobie sdk w pythonie https://github.com/p1c2u/wykop-sdk
i nie chciałem tworzyć nowych nazw metod. No ale to oczywiście można pozmieniać

[M.....s]

@Marmite: funkcja apiRequest zwraca 3 rzeczy: (error, response, body)
mogą być 3 różne errory:
obiekt: error
response.statusCode różne od 2**
obiekt: body.error

tu to sobie ładnie wszytko zrobiłem, a w promisifyAll nie niem jak to to miałby wyglądać

[karbz0ne]

@MirkoStats: wołaj jak wrzucisz

[M.....s]

jutro wrzucę, muszę przetestować wszystkie funkcje i spisać wstępną dokumentację

[G.....n]

@MirkoStats: Nie musisz. Wrzucaj od razu. :P

[M.....s]

@Ginden: @karbz0ne: @Marmite: jestem jestem https://github.com/bnt44/wykop-api

[Marmite]

@MirkoStats: Czy przy każdej akcji będę musiał sam ręcznie sprawdzać

!error && response.statusCode === 200 && !body.error

? Jeśli tak to to w ogóle nie jest pomocne, biblioteka powinna dostarczać abstrakcję do tego.

[M.....s]

@Marmite: tak miałem zrobić, ale nie wiedziałem jak ładnie przekazać te errory, będzie tak:

api.addEntry(body, embed, function(error, body) {
    if (!error) { //do stuff } };

no i przy okazji spojrzałem,że niepotrzebnie powtarzam kod

if (error) {
 reject(error);
 } else if (!(/^2/.test("" + response.statusCode))) {
 reject(response.statusCode);
 } else if (body.error) {
 reject(body.error);
 } else {
 self.userkey = body.userkey;
 fulfill(body);
 }

 if (callback) {

 return callback(error, response, body);

 }

gdzie mogę to

[M.....s]

zaraz pójdą zmiany

[M.....s]

@Marmite: ok, popróbuj ( ͡° ͜ʖ ͡°)
https://github.com/bnt44/wykop-api

var WykopAPI = require('./wykop-api.js');

var api1 = new WykopAPI(appkey, secretkey); // opcjonalnie (appkey, secretkey, oputput, format)

api.authenticate(accountkey, function(error, body) {

if (!error) {
api.addEntry(body, embed, function(error, body) {
if (!error) { //do stuff } }
});
};

});

[Marmite]

@MirkoStats: Ja i tak nie używam nigdy callbacków, brzydzę się nimi :P Bluebird FTW