Existe-t-il un moyen de raccourcir cette condition ?

Question

while (temp->left->oper == '+' || 
       temp->left->oper == '-' || 
       temp->left->oper == '*' || 
       temp->left->oper == '/' || 
       temp->right->oper == '+' || 
       temp->right->oper == '-' || 
       temp->right->oper == '*' || 
       temp->right->oper == '/')
{
    // do something
}

Pour plus de clarté : temp est un pointeur qui pointe vers le suivant node structure :

struct node
{
    int num;
    char oper;
    node* left;
    node* right;
};

Answer 1

Lambda & std::string_view

string_view offre de nombreuses fonctionnalités de std::string et peut opérer sur des littéraux et il ne possède pas l'attribut string .

Utilisez Lambda au lieu de fonction pour un code très local qui n'est pas utile pour le reste du fichier. De même, il n'est pas nécessaire de passer des variables lorsque lambda peut les capturer. Obtenez également inline sans le spécifier pour une fonction que vous auriez autrement créée.

• https://github.com/isocpp/CppCoreGuidelines/blob/master/CppCoreGuidelines.md#Rf-capture-vs-overload

• https://github.com/isocpp/CppCoreGuidelines/blob/master/CppCoreGuidelines.md#Rstr-view

Faire char const :

• https://github.com/isocpp/CppCoreGuidelines/blob/master/CppCoreGuidelines.md#Rconst-immutable

  auto is_arithm = [](const char c) { return std::string_view("+-/*").find_first_of(c) != std::string::npos; };

  while (is_arithm(temp->left->oper) || is_arithm(temp->right->oper)) { }

---

Vous pouvez modifier const char c à const node *t accéder à son oper à l'intérieur de la lambda. Mais ce n'est pas une bonne idée car les membres de left / right de temp peut être modifié.

auto is_arithm2 = [](const node *t) {
  return std::string_view("+-/*").find_first_of(t->oper) != std::string::npos;
};

while(is_arithm2(temp->left) || is_arithm2(temp->right)){
}