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

Avoinna
2 vuotta sitten avasi gitgod · 0 kommenttia
gitgod kommentoitu 2 vuotta sitten

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`)
Kirjaudu sisään osallistuaksesi tähän keskusteluun.
Ei tunnistetta
Ei merkkipaalua
Ei osoitettua
1 osallistujaa
Ladataan...
Peruuta
Tallenna
Sisältöä ei vielä ole.