Logique imbriqué && opérateur permissibles?

Question

J'ai écrit une fonction .oct à utiliser dans Octave qui ne donne pas le résultat que je m'attendais et je pense que le problème pourrait résider dans l'extrait de code donné ci-dessous - est-il correct d'imbriquer l'opérateur logique & & comme je l'ai fait ou doit-il être séparé en utilisant d'autres boucles internes?

if (((final_side_PDF > final_uwr_PDF && final_side_PDF > final_unr_PDF) && final_side_PDF > final_dwr_PDF) && final_side_PDF > final_dnr_PDF) 
    { 
    market_model(ii) = 0.0; 
    } 
else if (((final_uwr_PDF > final_side_PDF && final_uwr_PDF > final_unr_PDF) && final_uwr_PDF > final_dwr_PDF) && final_uwr_PDF > final_dnr_PDF) 
    { 
    market_model(ii) = 1.0; 
    } 
else if (((final_unr_PDF > final_side_PDF && final_unr_PDF > final_uwr_PDF) && final_unr_PDF > final_dwr_PDF) && final_unr_PDF > final_dnr_PDF) 
    { 
    market_model(ii) = 2.0; 
    } 
else if (((final_dwr_PDF > final_side_PDF && final_dwr_PDF > final_uwr_PDF) && final_dwr_PDF > final_unr_PDF) && final_dwr_PDF > final_dnr_PDF) 
    { 
    market_model(ii) = -1.0; 
    } 
else if (((final_dnr_PDF > final_side_PDF && final_dnr_PDF > final_uwr_PDF) && final_dnr_PDF > final_unr_PDF) && final_dnr_PDF > final_dwr_PDF) 
    { 
    market_model(ii) = -2.0; 
    } 
else 
    { 
    market_model(ii) = market_model(ii-1); 
    } 

Modifier en réponse aux commentaires

Je souhaite vérifier que l'un des cinq variables; final_side_PDF, final_uwr_PDF, final_unr_PDF, final_dwr_PDF et final_dnr_PDF; a la plus grande valeur et renvoie un numéro d'identification unique; 0,1,2, -1, -2; en fonction de laquelle est le max. Le code ci-dessus est contenu dans une boucle principale (ii).

Answer 1

Suggestion:

typedef int final_t; 

// From your code I assume that "final_xx" can have the same values, otherwise a std::map would be simpler. 
std::multimap<final_t, double> finals; 

finals.insert(std::make_pair(final_side_PDF, 0.0)); 
finals.insert(std::make_pair(final_uwr_PDF, 1.0)); 
finals.insert(std::make_pair(final_unr_PDF, 2.0)); 
finals.insert(std::make_pair(final_dwr_PDF, -1.0)); 
finals.insert(std::make_pair(final_dnr_PDF, -2.0)); 

auto max_final = finals.rbegin(); 

if(finals.count(max_final->first) == 1) 
    masket_model(ii) = max_final->second; 
else 
    market_model(ii) = market_model(ii-1); 

Si "final_xx" tous ont des valeurs uniques que vous pouvez le rendre encore plus simple:

typedef int final_t; 

std::map<final_t, double> finals; 

finals[final_side_PDF] = 0.0; 
finals[final_uwr_PDF] = 1.0; 
finals[final_unr_PDF] = 2.0; 
finals[final_dwr_PDF] = -1.0; 
finals[final_dnr_PDF] = -2.0; 

auto max_final = finals.rbegin(); 

market_model(ii) = max_final->second;