C++: avertissement d'affectation de pointeur sur strchr()

Question

Je reçois l'erreur assignment makes pointer from integer without a cast sur le code suivant, qu'est-ce que cela signifie?

char * splitter; 
if(splitter = strchr(key, ':') != NULL && *(splitter + 1) == ' ') 

Answer 1

L'opérateur != a une priorité plus élevée que l'opérateur =. Cela signifie que votre expression splitter = strchr(key, ':') != NULL est réellement interprétée comme splitter = (strchr(key, ':') != NULL).

Mettez l'affectation en parenthèses pour augmenter la priorité de cette partie:

(splitter = strchr(key, ':')) != NULL 

Answer 2

L'opérateur sans égal = a une priorité plus élevée que l'opérateur d'affectation =, de sorte que votre ligne originale se lit comme splitter = (strchr(key, ':') != NULL) plutôt que votre (splitter = strchr(key, ':)) != NULL prévu, de sorte que le compilateur essaie d'attribuer au séparateur le résultat de la comparaison entre strchr() et NULL.

Answer 3

Le != a une plus haute précision que le =. Vous devez entièrement mettre entre parenthèses:

if (((splitter = strchr(key, ':')) != NULL) && (*(splitter + 1) == ' ')) 

Answer 4

C'est à cause de la priorité des opérations. Vous avez besoin de mettre un jeu supplémentaire de parens pour vous assurer qu'il se passe dans l'ordre:

char * splitter; 
if((splitter = strchr(key, ':')) != NULL && *(splitter + 1) == ' ') 

sinon il évaluera comme ceci:

splitter = (strchr(key, ':') != NULL) 

Depuis strchr(key, ':') != NULL sera évaluée à un 1 ou 0, dans votre exemple, vous affectez un entier à un type de pointeur, d'où l'avertissement.

Cependant, je voudrais juste l'écrire comme dans votre deuxième exemple, car il est plus simple et moins sujette aux erreurs. Le raccourcir d'une ligne n'ajoute rien sauf la complexité.