Avertissement du compilateur - suggérer des parenthèses autour de l'affectation utilisée comme valeur de vérité

Question

Lorsque j'essaie de compiler le morceau de code ci-dessous, j'obtiens cet avertissement :

warning: suggest parentheses around assignment used as truth value

Pourquoi cela se produit-il ? Il s'agit d'une expression idiomatique assez courante, je crois. J'ai même utilisé quelque chose comme cela plus tôt dans mon code.

struct PIDList* 
getRecordForPID(struct PIDList* list, pid_t pid) {
    while(list = list->next)
        if (list->pid == pid)
            return list;

    return NULL;
}

Answer 1

Soyez explicite - le compilateur ne vous avertira pas que vous avez peut-être fait une erreur.

while ( (list = list->next) != NULL )

ou

while ( (list = list->next) )

Un jour, vous serez heureux que le compilateur vous l'ait dit, les gens faire faire cette erreur ;)

Answer 2

Si cette expression idiomatique est courante, il est encore plus courant d'utiliser = alors qu'ils veulent dire == . La convention quand il s'agit vraiment de la = est d'utiliser une couche supplémentaire de parenthèses :

while ((list = list->next)) { // yes, it's an assignment

Answer 3

Il s'agit simplement d'un avertissement de sécurité. Il s'agit d'une expression idiomatique relativement courante, mais aussi d'une erreur relativement fréquente lorsque l'on veut dire "avoir". == là-dedans. Vous pouvez faire disparaître l'avertissement en ajoutant une autre série de parenthèses :

while ((list = list->next))