problème avec strchr

Question

Je ne peux pas pourquoi le bit suivant du code C ne fonctionne pas:

int obtainStringLength(char* str, char c1, char c2) { 
    char* firstOcurrence = strchr(str, c1); 
    char* endOcurrence = strchr(str, c2); 
    return 2+(endOcurrence - firstOcurrence)/sizeof(char*); 
} 

L'idée est de trouver le nombre de caractères entre c1 et c2:

printf("%d\n", obtainStringLength("abc def ghi", 'a', 'i')); //should yield 11 

Malheureusement, cette impression est toujours 1. Quel est le problème? Est-ce que strchr ne devrait pas fonctionner comme string.IndexOf() de C#?

Answer 1

Division par sizeof(char*)? C'est incorrect - le résultat de la soustraction de deux pointeurs est une valeur numérique (ptrdiff_t) correspondant au nombre de valeurs, pas un pointeur ou une différence d'adresses.

Il y a aussi l'erreur «un par un» dans le calcul de la longueur. Alors que la dernière ligne devrait ressembler à:

return 1 + (endOcurrence - firstOcurrence); 

Answer 2

Non, strchr() renvoie un pointeur (l'adresse) le caractère recherché, ou NULL si le caractère n'a pas été trouvé.

C'est très différent de IndexOf().

Answer 3

Parce que chaque caractère occupe exactement sizeof (char) octets; pas sizeof (char*) octets.

Et sizeof (char) est, par définition 1, vous pouvez l'omettre:

return 1 + (endOcurrence - firstOcurrence); 

Answer 4

Votre déclaration de retour a plusieurs problèmes, en raison de ne pas comprendre l'arithmétique des pointeurs.

La soustraction du pointeur se divise déjà par la taille de l'élément, et char* était de toute façon le type incorrect.

Et vous devriez ajouterez 1, et non 2.