Lisibilité pour-boucles sans compteurs

Question

I ont récemment écrit une implémentation pour une simple liste liée, et à plusieurs reprises dans mon code, il ressemble à

Node* current_node = head; 
while (current_node != nullptr) { 
    if (current_node->data == query) { 
    // perform some action 
    break; 
    } 
current_node = current_node->next; 
} 

Et je viens récemment pensé que je pouvais à nouveau mettre en œuvre ce que

for (Node* current_node = head; current_node != nullptr; current_node = current_node->next) { 
    if (current_node->data == query) { 
    // perform some action 
    break; 
    } 
} 

Je sais que les deux sont syntaxiquement correct, et que les différences de rendement devraient être négligeables, mais je me demandais si avoir une condition d'égalité dans le contrôle est généralement mis en œuvre dans une boucle for? Auparavant, je n'avais utilisé que des inégalités (ex:>, <, etc.) dans les boucles for-loops. Quelle version est plus conventionnelle/lisible?

Answer 1

Ce n'est pas une mauvaise pratique en boucle à travers une liste chaînée par boucle for, mais vous pouvez l'améliorer à:

std::list<type> list; 
auto it = std::find(begin(list), end(list), query); 
if (it != end(list)) 
    // perform some action