Un "main" de 1400 lignes en C++... comment faire plus court?

[fiatlux]

Bonjour

J'ai besoin d'un conseil. J'ai écris un programme en C++ avec 4 classes (4 fichiers ".h", 4 fichiers correspondant ".cpp") et un main (2 fichiers: main.h et main.cpp) pour un projet.

Et mon "problème" c'est que mon fichier main contient à peu près 25 à 30 fonctions (prototypées dans le main.h et explicitées dans le main.cpp). Du coup mon main.cpp fait.... 1400 lignes... hum. Et si je donne ce code à la personne qui va m'évaluer, ça risque de mal passer. Donc ma question est : comment est-ce qu'un programmeur un peu plus avancé que moi ferait ça plus "proprement" ? Est-ce qu'il faut créer un autre fichier avec juste des fonctions dedans ?... ou autre chose ?

Merci d'avance pour vos conseils
-----

[DomiM]

Pour diminuer la taille d'un code c++ il faut créer des fonctions dés que plus de 3 lignes sont répété avec éventuellement des valeurs de variable différentes

par exemple pour chercher dans une liste

CListe* CListe::GetById(int _id)
{
    CListe *l=this;
    do
    {
		if (l->GetIdent()==_id) return(l);
    } while ((l=l->next));
    return NULL;
}

au lieu de répéter cette boucle à chaque fois que l'on cherche un élément de la liste

[fiatlux]

Merci pour la réponse.
Mais en réalité, je ne veux pas forcément diminuer la longueur de mon code, qui est très compact (malgré les 1400 lignes, mais c'est un gros projet, fait sur 6 mois). Ce que je voudrais c'est plutôt savoir comment arranger les fichiers, autrement dit : comment est-ce qu'un bon programmeur arrangerait ce code ? est-ce qu'il laisserait ces 25 fonctions dans le fichier main.cpp ? Ou est-ce qu'il les mettrait ailleurs (un fichier fonctions.cpp, qui lui ferait alors 1400 lignes à son tour) histoire que le fichier main.cpp se résume à la fonction "main" et point barre? C'est juste une histoire de "présentation", de "propreté du code".

[Jack]

il faut penser réutilisation. Tes 25 fonctions doivent pouvoir être regroupées par "affinité". Tu pourrais donc créer des "packages" regroupant les fonctions ayant une dépendance forte.
En passant, tu parles beaucoup de fonctions alors que tu programmes en C++. Comment peux-tu te retrouver avec des tas de fonctions ne dépendant d'aucune classe?

Si ces fonctions sont des méthodes de classe, alors le regroupement se fait d'office. Et tu peux éventuellement faire des packages de classes possédant là encore une forte dépendance.

A+

[A voir en vidéo sur Futura]

[bzh_nicolas]

Moi je crée les fichiers "utilitaires.h" et "utilitaires.cpp" (ou fonction_utilitaires...)
En fait toutes les méthodes et fonctions qui ne vont pas ailleurs vont la dedans...
Sinon si tu as plusieurs méthodes qui travaillent sur des string ou char* tu peux toutes les mettre ensemble dans un autre fichier ("traite_chaine.h" et "traite_chaine.cpp" par exemple), pareil si tu as plusieurs fonctions de tri ("tri.h" et "tri.cpp"), etc...
En gros soit tu fais un fichier fourre-tout avec un nom "qui passe".
Soit tu tries tes fonctions et méthodes selon ce qu'elles font où les types sur lesquels elles travaillent.
Ca passait bien durant mes études...

[fiatlux]

Merci à tous les deux.

Comment peux-tu te retrouver avec des tas de fonctions ne dépendant d'aucune classe?

Mes classes représentent plutôt des objets, qui ont des méthodes propres, mais mes fonctions font plutôt de l'algorithmie (je fais du traitement d'image en fait), qui ne travaillent pas vraiment sur les objets en questions.

En gros soit tu fais un fichier fourre-tout avec un nom "qui passe".
Soit tu tries tes fonctions et méthodes selon ce qu'elles font où les types sur lesquels elles travaillent.

La 2e solution me plaît bien

[whoami]

Bonjour,

Mais en réalité, je ne veux pas forcément diminuer la longueur de mon code

Ce qui est incompatible avec ton titre :

Un "main" de 1400 lignes en C++... comment faire plus court?

[TROLL]

(malgré les 1400 lignes, mais c'est un gros projet, fait sur 6 mois)

Un gros projet avec 1400 lignes ?

Et ce, en 6 mois. Tu travailles 2 minutes par jour ?

[/TROLL]

[Jack]

Mes classes représentent plutôt des objets

Je ne vois pas trop ce que ça peut être d'autre, les objets étant juste des instances de classe.

Un "main" de 1400 lignes en C++... comment faire plus court?

C'est vrai que ça semblait surprenant avant de voir qu'il s'agissait du nom du fichier et pas celui de la fonction.

Bon courage

A+

[piwi]

Comme Jack, je pense qu'une bonne méthode pour les fonctions algorithmiques est de les ranger dans des fichiers créés par regroupement logique de fonction (les fonctions qui calculent, les fonctions qui ordonnent, les fonctions de traitement de chaines, etc...)

C'est propre et en plus vous même vous savez ou retrouver vos fonctions et où ranger les nouvelles que vous créez.

Cordialement,
piwi

[fiatlux]

Merci pour vos réponses.
Oui, c'est vrai que le titre est un peu ambigu, je parlais effectivement du fichier "main.cpp" et non pas de la fonction main.

Un gros projet avec 1400 lignes ?
Et ce, en 6 mois. Tu travailles 2 minutes par jour ?

Héhé Non, mais je ne bosse pas que là-dessus. Et de plus, chaque ligne me prend beaucoup plus de temps car 1) c'est des algorithmes de traitement d'image pour brutes épaisses (ca m'est arrivé de passer 10 jours à écrire 7 lignes, mais y'a pas que du codage, y'a de la recherche à côté. L'implémentation est juste une des étapes du projet) et 2) j'ai pas encore l'habitude du C++, je code sous Matlab d'habitude (car bien adapté pour le traitement d'image).

Je ne vois pas trop ce que ça peut être d'autre, les objets étant juste des instances de classe.

Oui héhé : cf ci-dessus, je découvre un peu le C++, j'ai plus l'habitude de Java et nettement plus de Matlab.

Comme Jack, je pense qu'une bonne méthode pour les fonctions algorithmiques est de les ranger dans des fichiers créés par regroupement logique de fonction

C'est ce que j'ai choisi de faire

[Jack]

YAPUKA alors!