#4 JS18(rest/graphql/jwt): 9

Aperto
aperto 2 anni fa da gitgod · 0 commenti
gitgod ha commentato 2 anni fa

http://gitlab.a-level.com.ua/GennadySht/HomeWork/src/master/js/18Rest_Graph/hw_18_01_flash.html#L10-L14 - нехватает removeEventListener

http://gitlab.a-level.com.ua/GennadySht/HomeWork/src/master/js/18Rest_Graph/hw_18_02_PedestrianTrafficLight.html#L19 - тут его хватает, но domEventPromise потерял универсальность;

везде вижу колдунство по поводу цветов (то массив lights, то red-неred). прикольно для практики, но неявно;

http://gitlab.a-level.com.ua/GennadySht/HomeWork/src/master/js/18Rest_Graph/hw_18_03_speedtest.html#L24-L27 - нахиба? почему бы в массив не добавлять результаты speedTestOne?

http://gitlab.a-level.com.ua/GennadySht/HomeWork/src/master/js/18Rest_Graph/hw_18_03_speedtest.html#L8-L11 - performance.now() более уместен для замера времени;

http://gitlab.a-level.com.ua/GennadySht/HomeWork/src/master/js/18Rest_Graph/hw_18_05_jwt.html#L7 - достаточно было просто написать try-catch. если бы в нем что-то ломалось, то автоматом возвращался бы undefined (если не писать return нигде кроме блока try)

http://gitlab.a-level.com.ua/GennadySht/HomeWork/src/master/js/18Rest_Graph/hw_18_01_flash.html#L10-L14 - нехватает `removeEventListener` http://gitlab.a-level.com.ua/GennadySht/HomeWork/src/master/js/18Rest_Graph/hw_18_02_PedestrianTrafficLight.html#L19 - тут его хватает, но `domEventPromise` потерял универсальность; везде вижу колдунство по поводу цветов (то массив lights, то red-неred). прикольно для практики, но неявно; http://gitlab.a-level.com.ua/GennadySht/HomeWork/src/master/js/18Rest_Graph/hw_18_03_speedtest.html#L24-L27 - нахиба? почему бы в массив не добавлять результаты `speedTestOne`? http://gitlab.a-level.com.ua/GennadySht/HomeWork/src/master/js/18Rest_Graph/hw_18_03_speedtest.html#L8-L11 - `performance.now()` более уместен для замера времени; http://gitlab.a-level.com.ua/GennadySht/HomeWork/src/master/js/18Rest_Graph/hw_18_05_jwt.html#L7 - достаточно было просто написать `try-catch`. если бы в нем что-то ломалось, то автоматом возвращался бы `undefined` (если не писать `return` нигде кроме блока `try`)
Sign in to join this conversation.
Nessuna etichetta
Nessuna milestone
Nessun assegnatario
1 Partecipanti
Caricamento...
Annulla
Salva
Non ci sono ancora contenuti.