Discussion :

[Plomeg]

Bonjour,

Je voudrais avoir un avis sur des fonctions membres d'une classe.

Faut-il créer (et garder) des fonctions dont on ne se sert jamais à coup sûr dans les conditions du programme créé mais qui pourrait éventuellement (mais peu probable) servir un jour, dans une amélioration du programme en question? Je veux dire qu'on peut facilement écrire ou réécrire une fonction auparavant inutile pour des besoins nouveaux (le principe de l'objet il me semble) mais est-ce une bonne chose pour le programme au final ou est-ce également une mauvaise habitude pour des programmes plus gros et compliqués (pour l'instant je suis plutôt mauvais^^)?

Dans le même ordre d'idée, vaut-il mieux ajouter des données en apparence inutiles dans une classe ou vaut-il mieux les enlever?

Merci d'avance pour vos réponses.

[Spout]

Ta question n'est pas propre qu'au C++, et je te conseille de la poser dans la rubrique conception.

De mon avis, ça dépend fortement de ta politique de gestion des sources, des exigences de ta boîte si c'est pour le milieu professionnel,... Je ne pense pas qu'il y ait de bonne ou mauvaise réponse à ta question

[koala01]

Salut,

Tant que ton programme n'a pas été distributé, tu fais strictement ce que tu veux, mais, l'idéal reste quand même de fournir les fonctions qui sont nécessaires, et de ne pas surcharger avec des fonctions inutiles...

Par contre, une fois qu'un programme a été distribué (et c'est encore plus vrai avec les bibliothèques), toute fonction utilisée dans une version antérieure distribuée devrait rester présente dans la nouvelle version, même si elle est devenue "inutile"

[Mat007]

Salut,

A mon avis il faut retirer tout ce qui est inutile dans la mesure du possible, et c'est en general toujours possible bien qu'en effet moins facile dans le cas d'une diffusion (a un client, au public, etc..).
C'est encore mieux de ne pas ajouter quelque chose qui ne sert a rien (YAGNI).

MAT.

[koala01]

Mais il est vrai qu'au fur et à mesure de l'évolution, on peut se rendre compte que certaines méthodes deviennent obsolètes...

Dans le cadre d'une application complète, on peut envisager de les supprimer, mais, dans le cadre d'une bibliothèque, c'est une question de compatibilité que de les laisser, si, du moins, elle a déjà été diffusée...

[DavidDeharbe]

Je conseillerais de t'en tenir au strict nécessaire, et de consacrer tes ressources à vérifier que ce que tu développes est correct, efficace et bien écrit.

David.
--

[zais_ethael]

Heu,... c'est bien joli de donner des avis comme ça, mais faudrait recadrer un peu de quoi on parle vous ne pensez pas?
J'ai beau lire tout le thread, je ne vois toujours pas si on parle à l'échelle d'une classe ou d'un programme.
Je m'explique:
- à l'échelle de la classe: méthode proposant une fonctionnalité déja disponible dans une autre méthode ou par combinaison de plusieurs méthodes (exemple: la méthode at() et l'opérateur [] de std::vector sont redondants, il est toujours possible d'utiliser les itérateurs)
- à l'échelle du programme: méthode qui est indispensable pour pouvoir utiliser la classe de manière exhaustive mais qui, dans un programme donné, n'est pas utilisée (exemple: fstream qui fournit la possibilité d'ouvrir un fichier en lecture, on pourrait très bien concevoir un programme qui n'utilise que des fichiers en écriture, ou pas du tout évidemment)
Oui, je me rends bien compte que pratiquement tout le monde avait compris la première possibilité Mais je suis plus enclin à penser que Plomeg parlait de la deuxième...