J'ai pris 2 fichiers, comme ça, au hasard, et… j'ai trouvé le code limpide, ça m'a surpris.
Les fonctions documentent leurs pré-requis, les "if() d'une ligne" utilisent des accolades (donc pas de goto fail probable), le code est aéré…
Vraiment surpris!
Posté par nico4nicolas .
Évalué à 3.
Dernière modification le 04 février 2019 à 18:08.
Y a aussi quelques switch/case imbriqués (et je n'aime pas ça), un switch d'environ 600 lignes. J'ai aussi vu des "if() d'une ligne" sans accolade. Par contre, je suis d'accord sur le fait que le code est aéré, décemment commenté et relativement facile à lire malgré le camel case utilisé pour les noms de variables (ah, la religion !).
En même temps on parle quand même de la boîte qui développe et maintient plusieurs systèmes d'exploitation. Lorsqu'elle les vend, elle vend également la maintenance. Pour les maintenir correctement il faut un code propre.
C'est vrai, et surtout, les softs pour leur système ont une compatibilité ascendante qui peut dépasser les 20 ans, sans recompilation. Ça n'empêche, winfile, c'est une application finale, pas une lib, donc la nécessité d'être carré est «moins importante».
Et ça date de leurs débuts dans le monde des OS graphiques, ils auraient pu avoir du code plus cavalier.
# Étonnament propre...
Posté par freem . Évalué à 6.
J'ai pris 2 fichiers, comme ça, au hasard, et… j'ai trouvé le code limpide, ça m'a surpris.
Les fonctions documentent leurs pré-requis, les "if() d'une ligne" utilisent des accolades (donc pas de goto fail probable), le code est aéré…
Vraiment surpris!
[^] # Re: Étonnament propre...
Posté par nico4nicolas . Évalué à 3. Dernière modification le 04 février 2019 à 18:08.
Y a aussi quelques switch/case imbriqués (et je n'aime pas ça), un switch d'environ 600 lignes. J'ai aussi vu des "if() d'une ligne" sans accolade. Par contre, je suis d'accord sur le fait que le code est aéré, décemment commenté et relativement facile à lire malgré le camel case utilisé pour les noms de variables (ah, la religion !).
[^] # Re: Étonnament propre...
Posté par Marco . Évalué à 3.
En même temps on parle quand même de la boîte qui développe et maintient plusieurs systèmes d'exploitation. Lorsqu'elle les vend, elle vend également la maintenance. Pour les maintenir correctement il faut un code propre.
[^] # Re: Étonnament propre...
Posté par freem . Évalué à 2.
C'est vrai, et surtout, les softs pour leur système ont une compatibilité ascendante qui peut dépasser les 20 ans, sans recompilation. Ça n'empêche, winfile, c'est une application finale, pas une lib, donc la nécessité d'être carré est «moins importante».
Et ça date de leurs débuts dans le monde des OS graphiques, ils auraient pu avoir du code plus cavalier.
Suivre le flux des commentaires
Note : les commentaires appartiennent à celles et ceux qui les ont postés. Nous n’en sommes pas responsables.