[pepz]

Tegin valmis siis oma esimese Node.js mooduli mis võtab Steami marketist eseme hindu või siis Backpack.tf kaudu kõik esemed ja nende hinnad.
Soovitused teretulnud!

Github: https://github.com/Netifriik/node-steam-market-manager
Npmjs: https://www.npmjs.com/package/steam-market-manager

[lehm2]

Tere, ei oska küll mooduli sisu ja kasutamist hinnata aga vaatasin koodi. Hea kui projekti juures oleks mõni "lintimise" fail soovitavalt kasuta .eslinti, mille abil näed ära kus vead võivad tekkida.

Mõned kohad mille kallal norida, "error handling", https://github.com/Netifriik/node-steam-market-manager/blob/master/lib/index.js#L97 - puudub lõplik error handling. Ole ettevaatlik ```throw new Error``` kasutamisega, kuna ühtegi try ... catch statementi pole, siis ühe sellise erori tekkimisel app sureb. Kasuta pigem callbacke, tundub, et pole sellised errorid mille peale pillid kotti panna. Mitte küll nii oluline aga mainin ära "this context"-i kasutamine, ```var self = this;``` - see on küll lahendus aga pigem soovitaks ```.bind(this)-i```, seda selleks et kui tulevikus midagi juurde arendad siis segadust vähem oleks this contextiga.

Kuna package.json-s puudub konkreetne Node versioon, siis eeldan et kasutasid viimast versiooni, võid kasutada osa ES6 võimalusi. Näiteks asynci moodulit pole vaja võid kasutada Promiseid, mis vaikevalikus olemas on .

Lühidalt minu poolt, aitäh lugemast .

[pepz]

Suured tänud, üritan järgmise commitiga tolled asjad ära parandada.
Proovisin [i].bind(this) kasutada _requestItem funktsioonis aga mingi errori viskas, et ei ole funktsioon.[/i]
Edit: Ikka minu viga oli, panin valesse kohta bind(this), nüüd töötab, super!

https://github.com/Netifriik/node-steam-market-manager/blob/master/lib/index.js#L97
Seal on selline probleem, et kui async parallel jooksutada ja populeerida kuskil callbackis err siis teised parallel read mis ei ole veel täidetud katkestatakse. Mingi aeg peaks asyncil tulema uuendus mis lubab edasi minna kui tekib error.
Hetkel, et visata mingi err seal, peaksin läbi käima kõik tulemused ja ma ei tea kas see on worth it.