Notes du devoir à la maison

Notes du devoir à la maison

by Jean-Cédric Chappelier -
Number of replies: 0

(re)Bonjour à toutes et à tous !

Je viens de publier les notes du devoir à la maison
et vais vous envoyer sous peu par email personnel votre programme et les points obtenus.

Vos correcteurs ont PEUT ETRE ajouté des commentaires dans votre code
(qui va vous être retourné), mais sinon, les principales erreurs sont
détaillées ci-dessous.
Veuillez donc bien lire les commentaires ci-dessous avant toute autre question :

L'exercice est très bien réussi en moyenne (voir la distribution de points en fin de message ;
le total maximum hors bonus était de 58 points) ;
les rendus sont globalement très bons.

Les principales erreurs ont été :

- les oublis d'unsigned lorsque pertinent
- non respect des types : comparaison d'int avec size_t, oubli d'utilisation des types définis (à quoi servent-ils alors ?)
- beaucoup d'élèves géraient mal le cas de l'abandon dans la fonction `validate(Move)`.
- trop de « magic numbers » (p.ex. 4 au lieu de .size())
- erreurs d'indexation (index out-of-range, off-by-one)
- copiés-collés dans learn() (ou dans query: 2 appels à validate())
- plusieurs étudiant(e)s ne semblent pas savoir qu’il est possible de retourner directement une valeur d’un type struct ; par exemple retourner directement {0,0} sans avoir à créer de variable intermédiaire.
 
À améliorer :

- Une partie non négligeable des étudiant(e)s fait encore des boucles for avec une variable incrémentée plutôt que des boucles for(auto ...) lorsque l'index n’est pas utilisé.

- Respectez les types (utilisez ceux qui font sens ; ayez le même type de part et d'autres d'opérateurs comme = ou == ou <=, etc.)

- Assurez vous **à chaque fois** que vos index ne débordent pas

- Des codes un peu compliqués sans raison

- Le style : l'indentation/espaces n'est pas forcément toujours bien faite, pas de convention de nommage des variables (mélange de PascalCase, snake_case et camelCase). Bien écrire et indenter son code est avant tout un avantage **pour vous** (y voir clair ; et plus tard, celles et ceux qui travailleront avec vous).


Distribution des points :