owlps/owlps-positioning/TODO

61 lines
2.1 KiB
Plaintext

- Tests unitaires
° Remplacer TS_ASSERT_DIFFERS(a, b) par TS_ASSERT(a != b) pour
tester operator!=() au lieu de !operator==().
° Finir le test de Input.
° Finir le test de Output.
- PosUtil
° Déplacer les #define des canaux Wi-Fi dans le .cc
- Revoir le diagramme UML
° Associations : devraient êtres représentées par des attributs
pointeurs.
° Compositions : devraient être représentées par des attributs
normaux.
- Building
Dans le destructeur, vérifier qu'il faut bien supprimer les Area et
les Waypoint. Si oui, faut-il aussi les enlever des listes dans
Stock ? (Pour l'instant ils ne sont pas dans Stock.)
- AccessPoint
Attribut float friis_index ?
- ReferencePoint
° La liste des requêtes devrait être un unordered_set (et pas un
vector), pour garantir l'unicité des entrées.
- Mobile
Attributs Viterbi ? (Cf. l'ancien clientinfo.hh.)
- Doxygen
° Supprimer les commentaires inutiles et autoriser les membres non
documentés dans Doxyfile.
° Pourquoi une description détaillée placée dans un .cc ne s'affiche
pas pour operator<<() ?
- « C++ en action »
° Espaces de noms ? 109
° Réserver l'espace mémoire des vector avec reserve(). 217
° Utiliser hash_map (maintenant unordered_map) plutôt que map s'il
n'y a pas besoin de trier. 232
° Copie de conteneur vers un flux (cas de certains operator<<). 275
- « Coder proprement »
° Revoir toutes les classes pour respecter si possible les
principes exposés dans le chapitre 6 (p. 103) : implanter
systématiquement des accesseurs pour tous les attributs d'une
classe expose l'implantation de la classe et rend ses attributs
publics. La classe Direction (et maintenant Timestamp) tend à
respecter ce principe en utilisant mieux les opérateurs et en ne
proposant pas d'accesseur direct.
- Divers
° Passage de pointeurs en argument : const * const <type>
- Timestamp
Dans les tests, il y a une probabilité de 1/1000000 que la valeur
en ns et la valeur arrondie en ms soient identiques. Dans ce cas,
certains tests peuvent échouer.