[M.....s]

Moje SDK do API Wykopu takie piękne https://github.com/bnt44/wykop-api

Wrzucac na npm?
#programowanie #nodejs #naukaprogramowania #javascript

[moon_bluebird]

@MirkoStats: npm?

[rbielawski]

@moon_bluebird: node packet manager

[M.....s]

@moon_bluebird: https://www.npmjs.com/
wtedy wystarczy npm install wykop-api i już masz w swoim folderze (oczywiście dotyczy to nodejs)

[rbielawski]

@moon_bluebird: tfu, *package

[moon_bluebird]

@MirkoStats: dzieki za fachowe wytłumaczenie :)

[bonus]

@MirkoStats: DRY - słyszałeś kiedyś? XD ten kod można skrócić spokojnie o ponad połowę + w JS używamy 2 spacji zamiast 4 w tabie i nie robimy entera po każdej linije, fatalnie czyta się ten kod

[G.....n]

@MirkoStats: Ogólnie to radzę wykorzystywać obiekty.
Np. masz metodę

WykopAPI.prototype.buryLink = function (param1, param2, callback) {

Nie lepiej zrobić
WykopAPI.prototype.buryLink = function ({id, reason}, callback) {

[y.....s]

https://github.com/bnt44/wykop-api

@MirkoStats: gdzie pracujesz cumplu? :)

[M.....s]

@yourboss: nigdzie, uczę się xD

[m.....i]

w JS używamy 2 spacji zamiast 4

@bonus: Można prosić źródło?

[M.....s]

@Ginden: no ale to utrudniasz użycie, zamiast wywołania

api.buryLink(, ).then(.... 

to

api.buryLink( { id: , reason:

[G.....n]

@MirkoStats: Wyobraź sobie, że do API dochodzi dodatkowy wymagany parametr na np. stream/index
Nie dość, że ludzie stary kod przestanie działać, to aktualizacja biblioteki będzie wymagała przepisania kodu, by działał z nową wersją.

[M.....s]

Nie dość, że ludzie stary kod przestanie działać

@Ginden: jeżeli parametr będzie wymagany to tak czy inaczej będą musieli przepisać swój kod i dodać wszędzie ten parametr, co za różnica czy w obiekcie options (bez konieczności update biblioteki)

var options = {

[M.....s]

DRY - słyszałeś kiedyś? XD ten kod można skrócić spokojnie o ponad połowę

@bonus: słyszałem, skróciłem co się dało, więcej mi do głowy nie przychodzi. Pokaż przykład co byś tu skrócił, może mi się otworzy klapka w mózgu

[G.....n]

@MirkoStats: Wcale, że nie, bo biblioteka może mieć defaultowe wartości, np. brak podania page zwraca stronę = 1.
Naprawdę, to jest znacznie wygodniejsze, zwłaszcza dzięki faktowi, że można zrobić:

var id = 3;

var page=1;

[G.....n]

@MirkoStats: No i jeszcze dochodzi fakt, że do obiektu nie trzeba pamiętać kolejności parametrów, do zwykłych argumentów - trzeba.
W przypadku tylko np. strony jest to intuicyjne, jeśli masz już 3 parametry to trochę gorzej. Czy sygnatura funkcji to function(entryId, commentId) czy function(commentId, entryId)?

[M.....s]

@Ginden: dzięki, poszperam i spróbuję zrobić oddzielny branch ze zmianami, które proponujesz. No i dokształcić się w ES6

[M.....s]

Moje SDK do API Wykopu takie piękne

ps. specjalnie tak napisałem, żeby was troche sprowokować ( ͡° ͜ʖ ͡°)

[m4kb0l]

var id = 3;

var page=1;

A potem

funkcja({id,page})

@Ginden: dzięki cumplu. Już jakiś czas programuje w JS a nie wiedziałem o tym.

[bonus]

@maciej-jantarski: Two spaces, not longer and no tabs: Google, npm, Node.js, Idiomatic, JS Style Guide