Discussion :

[Mike888]

Salut !

A chaque fois que je tente ce genre d'écriture dans mon code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
...
for ( j = 0; j < strlen( chaine ); j++ )
...

j'hérite d'un beau warning :

"SAISON.c" : W8012 Comparaison de valeurs signées et non signées dans la fonction encodepoints en ligne 39

Qu'est-ce qu'il se passe au juste ? :

[DaZumba]

Comme indique dans la doc, strlen() renvoie un size_t (tres souvent un unsigned) que tu compares avec un (signed) int. D'ou cet avertissement.
Je te conseille de faire un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
int size = (int) strlen(chaine);
for (j = 0; j < size; j++)
{ 
   ...
}

cela evite en plus de re-evaluer strlen() a chaque iteration (a moins que la taille varie dans la boucle et que tu joues la-dessus).

[Mike888]

Non, la taille ne varie pas; j'ai fait ça uniquement comme raccourci (et en plus ça évite une variable supplémentaire)...
Mais bon, si c'est pas permis, alors, ... je me plierai

[bigquick]

Ce n'est pas le fait qu'il soit dans le for qui n'est pas permis, mais il faut simplement caster le retour, comme la montré DaZumba. Tu peux aussi bien faire

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
for (j=0; j<(int)strlen(chaine); j++)
{
}

que

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
int size = (int) strlen(j);
for (j=0; j<size; j++)
{
}

L'avantage du 2ème est de ne pas recalculer la taille de la chaine à chaque iteration, ce qui est beaucoup plus couteux qu'une variable en plus

[Emmanuel Delahaye]

Je te conseille de faire un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
int size = (int) strlen(chaine);
for (j = 0; j < size; j++)
{ 
   ...
}

Argh, un cast !

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
size_t len = strlen(chaine);
for (j = 0; j < len; j++)
{ 
   ...
}

[Mike888]

Mais enfin, qu'est-que vous avez tous contre les cast ??? Ca a pas été inventé pour rien quand meme ?.... :

[Emmanuel Delahaye]

Mais enfin, qu'est-que vous avez tous contre les cast ??? Ca a pas été inventé pour rien quand meme ?.... :

Ca peut avoir une utilité dans certains cas précis et extrèmement rare (fonctions variadics, par exemple), mais la regle de base c'est "pas de cast".

'Cast is evil'

En général, ça sert à masquer les problèmes sans les corriger.

[Médinoc]

En l'occurence, Emmanuel, ton code ne corrige pas l'erreur non plus.
Pour la corriger, il suffit de déclarer j lui aussi en size_t (Surtout que ça m'étonnerait qu'il puisse être négatif dans ton code)

[Trap D]

De toute façon, c'est simplement un warning, quand on en connaît la raison et qu'il n'est pas du à une erreur de programmation on peut passer dessus dans un premier temps. Il est vrai que pour la beauté du code...
De plus, avec les différents compilos du marché, des warnings de l'un ne sont pas signalés parl'autre.

[Emmanuel Delahaye]

En l'occurence, Emmanuel, ton code ne corrige pas l'erreur non plus.
Pour la corriger, il suffit de déclarer j lui aussi en size_t (Surtout que ça m'étonnerait qu'il puisse être négatif dans ton code)

Oui, et en plus, c'est à ça que je pensais...
Comme quoi, le code partiel, c'est pas bon... Comme on ne vérifie rien, on ne fait que des bétises...