Bonne pratique avec des égalités pour les enums hors classe (rvalues ​​à gauche?)

Question

Je travaille dans un environnement qui met lentement à jour les anciens c vers C++. Les enums non-classe sont souvent utilisés dans les déclarations d'égalité. Maintenant, je suis sûr que quiconque a suivi un cours de programmation ou a lu un livre qui couvre les bonnes pratiques sait que les égalités entre les valeurs et les valeurs doivent être structurées avec la valeur à gauche pour éviter des modifications accidentelles des données 'est substitué par' = '. Je n'ai jamais vu cela avec des énumérations. La réponse facile est d'utiliser la classe enum mais cela ne peut pas toujours être fait immédiatement avec de grands corps de code plus ancien. Puisqu'une énumération est fondamentalement juste un nombre entier, est-ce que cela serait considéré comme une bonne pratique de garder également les valeurs à gauche pour les énumérations?

j'ai écrit un petit exemple pour mieux illustrer ci-dessous

#include <iostream> 
using namespace std; 
    int number; 
    enum Color { RED, GREEN, BLUE }; 
int main() { 
    number = 1234; 
    Color color = BLUE; 

    // This is done 
    if (555 == number) { /*...*/ } 

    // So that this never accidently happens 
    if (number = 555) { /*...*/ } 

    // Resulting in this (number changed to 555) 
    cout << number <<endl; 

    // I have never seen this done 
    if (RED == color ) { /*...*/ } 

    // should it be to, prevent this 
    if (color = RED) { /*...*/ } 

    //Resulting in this (color changed to 0 or RED) 
    cout << color <<endl; 

    return 0; 
} 

Answer 1

serait-il considéré comme une bonne pratique de ne garder rvalues ​​à gauche pour les énumérations aussi?

Ce n'est pas précisément une "bonne pratique" même avec des littéraux, en fait. Ce concept est apparu dans le passé, mais pour la plupart des gens ayant un littéral à droite est beaucoup plus lisible (et plus cohérent avec la notion mathématique que les variables, spécialement celles qui changent le plus, viennent en premier).

La suppression de la prévention des fautes de frappe pour des raisons de lisibilité n'est pas toujours souhaitable. Le code lisible est en fait un vieux concept qui n'a jamais changé et ne le sera probablement jamais. Donc, à la fin, s'en tenir à ce qui a plus de sens et est plus facile à maintenir. Empêcher ce genre d'erreurs devrait être laissé au compilateur, et tout compilateur décent déclenchera un avertissement dans ces cas.