Discussion :

[zecreator]

Bonjour à tous,

Est-ce que l'on pourrait cesser d'écrire des conditions en JS de cette façon :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if (Projector.start === null) Projector.start = timestamp;

C'est franchement illisible lorsqu'il y a 20 000 lignes de codes à inspecter. Merci d'écrire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
if (Projector.start === null){
    Projector.start = timestamp;
}

En respectant l'indentation qui va avec. Merci.
C'est tout de même pas compliqué, BORDEL DE M...

Les accolades, faut pas hésiter à les utiliser, juste parce que "T'ES PAS TOUT SEUL AVEC TON CODE..."

[ProgElecT]

Salut

Oui on pourrait cesser.

[NoSmoking]

Bonjour,
je ne comprends pas trop le but de ton post, coup de gueule ?

Oui c'est une mauvaise façon de faire, entre bien d'autres.

Pourquoi ne pas commencer à le dire au(x) principal(aux) concerné(s) que tu as rencontré(s).

[zecreator]

je ne comprends pas trop le but de ton post, coup de gueule ?

Oui, je râle Je dois me farcir 37 000 lignes de codes de ce genre, avec des commentaires quand le ou les mecs ont eu le temps d'en mettre, et de l'indentation merdique. Et pour l'indentation, merci d'utiliser la touche TAB, pas 2 ou 3 espaces (le nombre varie selon la générosité du codeur). Si ce n'était que des cas isolés, ça irait, mais j'ai l'impression que ce genre d'écriture se généralise...

ça va me faire une bonne semaine je crois lol

[SpaceFrog]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 (Projector.start === null) && (Projector.start = timestamp);

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Projector.start= (Projector.start === null)? timestamp : Projector.start ;

[Doksuri]

3615 MAVI

[zecreator]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 (Projector.start === null) && (Projector.start = timestamp);

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Projector.start= (Projector.start === null)? timestamp : Projector.start ;

Ou comment économiser 3 lignes pour rendre son code illisible. Même pas sûr que ce genre de code puisse être correctement tracé par le débuggeur...

[Marco46]

Oui, je râle Je dois me farcir 37 000 lignes de codes de ce genre

Tu as une option --fix sur ESLint qui te fixe automatiquement une grande partie de ces problèmes. Tu devrais peut être jeter un oeil à cet outil !

[psychadelic]

C'est vrai, il faut mieux écrire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Projector.start = Projector.start || Math.round(+new Date() / 1000);

[Marco46]

ESLint est ton ami.