Comment optimiser/refactoriser un tel code?

Question

Dans le projet actuel, il y a beaucoup de méthode GetData(), qui obtiennent différents types de données à partir de base de données manuscrite à l'exécution, et les définir dans différents champs d'une classe. Les projets ont alors de telles méthodes.

void GetData(Datatype type, int& value); 
void GetData(Datatype type, double& value); 
void GetData(Datatype type, long& value); 
void GetData(Datatype type, longlong& value); 
.... 

Il y a beaucoup de types de données, donc, ces méthodes sont souvent appelées avec un interrupteur avec beaucoup de branches.

void GetData(Datatype type, int& value) 
{ 
    switch(type) 
    { 
     Type1: 
     value = GetDataFromDB1(TYPE1); 
     Type2: 
       value = .. //get from different source 
     ... 

    } 

}

void GetData(Datatype type, double&　value) 
.... 

Comme vous le voyez, les s du GetData() sont classés selon le deuxième param. Et dans chaque GetData(), il y a beaucoup de branches. Est-ce une méthode raisonnable pour obtenir des données?

Answer 1

Pour répondre à "la meilleure façon de refactoriser cela" prendrait plus de contexte. Par exemple, il est peut-être approprié de changer la façon dont les données sont stockées en plus de la façon dont elles sont récupérées comme vous le montrez. Je doute que cette structure de code ait besoin d'optimisation.

Answer 2

Désolé, programmeur Java à venir :)
J'ai 2 solutions ici:

1. Si vous voulez modifier le code pas beaucoup, vous pouvez mettre Datatypes et leurs opérations pertinentes dans une Carte. Si long switch(type) déclaration pourrait être une déclaration map.get(type).

2. Si plus de façon décente, utilisez Polymorphisme. Vous pouvez définir une interface pour DBOperations, chaque DBOperation spécifique pouvant être définie dans sa classe implémentée. Donc GetData juste besoin d'invoquer DBOperation interface. C'est une idée approximative .

Answer 3

Je ne sais pas, si je comprends les choses, mais cela pourrait aider

typedef int Datatype; 

template <Datatype d, class T> struct Wrap{ 
    Wrap(T (*p)(Datatype)) : mp(p){} 
    template<class T> void GetData(T &value){ 
     value = (*mp)(d); 
    }; 
    T (*mp)(Datatype); 
}; 

int GetDataFromDB(Datatype){ 
    return 0; 
} 

int main(){ 
    Wrap<0, int> wi0(GetDataFromDB); 
    int val; 
    wi0.GetData(val); 
} 

Answer 4

On dirait que vous êtes une interface outgrowing SQL personnalisée. Regardez dans les bibliothèques SQL C++ existantes. Malheureusement, je ne peux pas en recommander (la dernière fois que j'en avais besoin, j'en ai écrit une nouvelle à des frais déraisonnables), mais voir la question connexe C++ SQL database library comparison.

En outre, libpqxx, le client C++ PostgreSQL officiel, semble très agréable. Hmm, en regardant autour, je n'arrive pas à trouver une bibliothèque avec un support générique pour struct s comme ce que j'ai écrit. Il doit y en avoir quelques-uns ... vous devriez pouvoir brancher vos tables et vos en-têtes (avec une petite modification, peut-être) et obtenir un code d'interface automatique.

Answer 5

Vous pourriez vouloir le refactoriser pour le rendre plus maintenable.

Le refactoring pour l'optimisation n'aura probablement aucun avantage, ou négatif, sauf si vous avez montré qu'il s'agit d'un problème de performance, par profilage.