Salut lijasu,
Déjà ton code fonctionne donc c'est déjà bien!
La seule remarque importante que je peux te faire, c'est à propos de la factorisation de ton code.
C'est un des premiers premiers principes que l'on apprend en programmation et qui consiste dans le fait de ne pas dupliquer du code à plusieurs endroits. Une des principales raisons est que si un code dupliqué possède un bug, ce dernier existe alors en plusieurs exemplaires dans l'application. En général, on ne corrige qu'un seul endroit et c'est le drame car le problème revient forcément plusieurs fois (ce qui est généralement très mal vu par les utilisateurs et autres chefs de projet fonctionnel).
On peut ainsi voir dans ton code, certaines lignes qui se répètent. Par exemple
console.log(quantity + unit + " vaut "+ result + " octets.");.
Voici une manière de factoriser:
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17
| var result = false;
if (unit === "K") {
result = quantity * 1024;
} else if (unit === "M") {
result = quantity * 1024 * 1024;
} else if (unit === "G") {
result = quantity * 1024 * 1024 * 1024;
} else if (unit === "T") {
result = quantity * 1024 * 1024 * 1024 * 1024;
}
if (result !== false) {
console.log(quantity + unit + " vaut " + result + " octets.");
} else {
console.log("ce n'est pas une valeur.");
} |
Le reste de mes commentaires vont plus être sur la forme que sur le fond:
- attention à la précision de tes indentations
- ton formatage doit être cohérent: e.g. si tu mets un espace avant une accolade, il faut que ce soit le cas tout le temps
La rigueur de codage est primordiale!
Au final, l'exercice est plutôt bien réussi donc bravo.
0 |
0 |