dtaskoff / hackbulgaria-javascript Goto Github PK

Евалата, най-доброто решение до сега. Няма бъгове. Супер код 👍

Наред:

Count Down чупи timer-a

1. Въведи някакви секунди
2. Натисни Count Down

Ефектът е:

Когато са въведени и минути и секунди си брои OK 👍

Супер е написано, браво Дани.

Обръщам ти внимание, че има един пакет, който се нарича hashtable и е идеален за такива случаи - https://www.npmjs.org/package/hashtable 💃

Тук - https://github.com/dtaskoff/hackbulgaria-javascript/blob/master/week1/students-system/client/app.js#L61 - можеш да напишеш forEach, защото така var student си отива пак горе, заради function scope-a

Тук - https://github.com/dtaskoff/hackbulgaria-javascript/blob/master/week1/students-system/client/app.js#L82 - може да пробвам с някой от workarounds - https://developer.mozilla.org/en/docs/Web/API/NodeList#Why_can't_I_use_forEach_or_map_on_a_NodeList.3F - или просто да ползваш jQuery, с което ще започнем в събота ;)

groupBy и searchFor може да ги метнеш горе в кода, вместо долу, тъй като те се ползват преди тяхното деклариране.

Супер е написано, браво за Promise-ите.

Имам само една забележка за код, който се повтаря:

// line 41
makeGetRequestTo('http://localhost:8080/names

// line 57
makePostRequestTo('http://localhost:8080/name',

// line 62
makeGetRequestTo('http://localhost:8080/names',

Изнеси си всички URLs в един речник и просто взимай стойностите от там.

Така при ссмяна на сървъра ще смениш само на 1, вместо на 3 места.