2.1.6.9.9 Gli indici degli array devono essere numerici.
JavaScript permette l'utilizzo di stringhe come indici per array, ma indici numerici rendono il
codice più chiaro e comprensibile.
evitare più di una istruzione per riga ---> togliere; ✔
linee troppo lunghe dovranno essere spezzate in più parti ---> togliere. ✔
le funzioni di callback dovranno essere scritte in una solo linea esclusivamente se contengono
una sola istruzione. In tal caso, dovranno essere scritte secondo la notazione a freccia (arrow
function) descritta in ECMAScript 2015 ---> non è sempre così (Sime dice che in alcuni casi non è possibile farlo, perchè vengono gestite in un certo modo);
l'annidamento di funzioni di callback deve essere evitato e sostituito seguendo la notazione
Promise descritta in ECMAScript 2015. ---> non è la notazione Promise, ma costrutto ✔
2.1.6.9.15 ma se faccio un while? vale solo per cicli for ✔
2.1.6.9.16 togliere, è ovvio ✔
2.1.6.9.17 inizia con la 'l' minuscola (unire con 2.1.6.9.18, 2.1.6.9.19)
e manca una norma per dire quando usare let e var ✔
Critiche:
1) aggiungere un commento prima dei metodi ✔
2) il diario delle modifiche è già su git. Il nome del commit deve essere descrittivo (togliere il versionamento) ✔
3) aggiungere un regola per i ';' a fine istruzione ✔
4) Abbiamo usato tutti 2 spazi per l'indentazione (Two spaces, not longer and no tabs: Google, npm, Node.js, Idiomatic, Felix) ✔
2.1.6.9.9 Gli indici degli array devono essere numerici. JavaScript permette l'utilizzo di stringhe come indici per array, ma indici numerici rendono il codice più chiaro e comprensibile.
evitare più di una istruzione per riga ---> togliere; ✔
linee troppo lunghe dovranno essere spezzate in più parti ---> togliere. ✔
le funzioni di callback dovranno essere scritte in una solo linea esclusivamente se contengono una sola istruzione. In tal caso, dovranno essere scritte secondo la notazione a freccia (arrow function) descritta in ECMAScript 2015 ---> non è sempre così (Sime dice che in alcuni casi non è possibile farlo, perchè vengono gestite in un certo modo);
l'annidamento di funzioni di callback deve essere evitato e sostituito seguendo la notazione Promise descritta in ECMAScript 2015. ---> non è la notazione Promise, ma costrutto ✔
1 collection_one . forEach (( item ) => { console . log ( item )}); 2 collection_two . forEach ( function ( item ) 3 { 4 item = Math . random (); 5 console .log( item ); 6 } 7 );
dovrebbe essere:
1 collection_one . forEach (( item ) => { console . log ( item )}); 2 collection_two . forEach ( function ( item ) 3 { 4 item = Math . random (); 5 console .log( item ); 6 }); ✔
2.1.6.9.10 non so quanto serva ✔
2.1.6.9.11 non sono a cascata ma sono annidati ✔
2.1.6.9.12 schifo ✔
2.1.6.9.14 da togliere ✔
2.1.6.9.15 ma se faccio un while? vale solo per cicli for ✔
2.1.6.9.16 togliere, è ovvio ✔
2.1.6.9.17 inizia con la 'l' minuscola (unire con 2.1.6.9.18, 2.1.6.9.19) e manca una norma per dire quando usare let e var ✔
Critiche: 1) aggiungere un commento prima dei metodi ✔ 2) il diario delle modifiche è già su git. Il nome del commit deve essere descrittivo (togliere il versionamento) ✔ 3) aggiungere un regola per i ';' a fine istruzione ✔ 4) Abbiamo usato tutti 2 spazi per l'indentazione (Two spaces, not longer and no tabs: Google, npm, Node.js, Idiomatic, Felix) ✔