Discussion :

[Masmeta]

Bonjour

Je recherche des conseils pour refactoriser mon code C++.

J'ai réalisé plusieurs classes de type DefautFoo de la forme suivante

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
 
#ifndef DEFAULT1_H
#define DEFAULT1_H
[...]
 
class default1
{
public:
    DLLEXP void initialize(const char* filepath);
    DLLEXP void learn(object info, std::vector<object>& other, std::vector<object>& target,  int& ret );
    DLLEXP void recall( std::string Filename,   std::vector<std::string>& oldValue,   std::vector<std::string>& newValue);
    DLLEXP void getDefault( const cv::Mat& src,  std::string &res, unsigned int rstep, unsigned int dstep );
 
private   
    std::string defaultType = "default1";
};
 
#endif //DEFAULT1_H

Sur les classes défauts, j'ai des différences :

• le prototypes des fonctions est différents pour certain type de défaut
• la valeur de defaultType est différentes
• la fonction getDefaut peut avoir une implémentation différente avec le prototype de la fonction identique.

Je cherche à optimiser le code en POO, quelle peut être la meilleure stratégie ?
Merci de vos avis.

[Bktero]

Pour se lancer dans du refactoring, il faut identifier certains défauts (sans mauvais jeu de mots ) de ton code que tu souhaites corriger. Quelles limites rencontres-tu aujourd'hui ? Que souhaiterais-tu améliorer ?

De plus, ça serait bien de nous expliquer un peu le but de ton groupe de classe. Quand tu parles de "défaut", tu veux des "erreurs" sur un appareil, comme un défaut d'allumage dans une chaudière ou un défaut de pression dans une pompe ? Un code sert à résoudre un problème. Sans la compréhension du problème, il n'est pas possible de fournir un bon code à mon avis.

[Masmeta]

Le but est de trouver des défauts sur une pièces mécanique. Actuellement, il y a 9 défauts d'ou la réalisation de 9classes de traitements. Cependant il y a 5 classes qui ont le même code à des changements de valeurs prêts ou de prototype prêts.
La première chose est pour moi d'améliorer la maintenabilité du code.
Je pense partir sur le design pattern de la factory avec une classe abstraite de défaut.

[ternel]

Pour factoriser du code (OOP ou non) il faut repérer ce qui est commun.

Que veux-tu faire avec n'importe quel défaut?

La subtilité de la question, c'est que pour chaque traitement à effectuer, chaque défaut peut le faire à sa sauce, mais c'est un détail, du moment qu'il le fait.

Par exemple, considère des formes géométrique, comme des cercles, des carrés, des triangles, etc.
On peut décider que chaque forme doit pouvoir être dessinée dans une surface donnée.

Il y a plusieurs stratégies de code:

• OOP: classe de base et fonction virtuelle, mais l'affichage est-il la seule responsabilité de la forme (SRP)
• design pattern visiteur
• double dispatch (template?)
• d'autres choses plus exotiques