Lisibilité des appels de longue durée

Question

J'ai écrit du code qui utilise des appels de fonction qui sont vraiment longs et dépassent souvent 80 caractères. Normalement, je dissociais ces appels de fonction comme ceci:

LongFunctionName(first_argument, 
       another_argument, 
       finally_last_argument); 

Cependant, il semble bizarre lorsque je tente de mettre cela en une instruction if, principalement parce qu'il est pas très clair quelle valeur il compare à:

if(LongFunctionName(first_argument, 
        another_argument, 
        finally_last_argument) != value_compared_to) 
{ 
    // stuff to be called 
} 

Comment formateriez-vous cette instruction pour qu'elle soit plus lisible et qu'elle contienne 80 caractères?

Answer 1

j'envisager de mettre l'appel de fonction sur sa propre ligne:

const auto value = LongFunctionName(first_argument, 
           another_argument, 
           finally_last_argument); 
if (value != value_compared_to) { 
    // ... 
} 

Vous pouvez même donner la variable value un joli nom descriptif qui aide à comprendre le code.

Answer 2

Vous avez deux choix:

1)

Accepter qu'il semble ainsi

OU

2)

Évaluer la valeur de retour de la fonction d'appel en une déclaration séparée (pseudo-code)

retval = LongFunctioName(first_argument, second_argument, third_argument); 
if(retval != desired_val) { ... } 

Answer 3

Ne nous commençons à celui-là, je le ferais

if 
(  LongFunctionName 
     (  first_argument 
     ,  another_argument 
     ,  finally_last_argument 
     ) != value_compared_to 
) 
{ 
     // stuff to be called 
} 

Answer 4

Stockage de la valeur de retour dans une variable est la meilleure solution imo. Mais vous pouvez faire autre chose:

if (value_compared_to != LongFunctionName(first_argument, 
              another_argument, 
              finally_last_argument))