MISE EN ORDRE ALPHABETIQUE DE 2 CHAINES SANS UTILISER STRCMP ET STRNCMP

Signaler
Messages postés
360
Date d'inscription
samedi 27 septembre 2003
Statut
Membre
Dernière intervention
30 mai 2006
-
Messages postés
2
Date d'inscription
jeudi 29 novembre 2007
Statut
Membre
Dernière intervention
31 mars 2008
-
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.

https://codes-sources.commentcamarche.net/source/24094-mise-en-ordre-alphabetique-de-2-chaines-sans-utiliser-strcmp-et-strncmp

Messages postés
2
Date d'inscription
jeudi 29 novembre 2007
Statut
Membre
Dernière intervention
31 mars 2008

re..

j'ai oublié.. supprimes la ligne 7 de ton programme. Elle ne sert pas à grand chose ! ^^

hackersmick
Messages postés
2
Date d'inscription
jeudi 29 novembre 2007
Statut
Membre
Dernière intervention
31 mars 2008

Cc bethany...

Merci pour ton programme, il m'a aidé un peu pour un projet de c.
De plus, j'ai trouvé ce qui n'allait pas :)

Un point important à savoir sur les fonctions :
Ne JAMAIS appeler une fonction avant de la définir !!(c'est ce que tu as fait pourtant :s !)

Sinon quand tu définies ton main() écris " int main() ".
J'ai augmenté MAX à 4096 et j'ai mis un system("PAUSE"); et un return 0; ...
et c'est parti !! ^^

a+

hackersmick
Messages postés
16
Date d'inscription
lundi 9 août 2004
Statut
Membre
Dernière intervention
13 septembre 2004

Salut bethany!
Oublie immediatement ce que neo t'a dit a propos du warning, c'est TRES mal :)
Un warning, en soi, ce n'est pas genant puisque ca n'empeche pas de compiler. Alors bien sur, tu peux l'ignorer, meme si c'est vrai qu'en general on n'aime pas avoir des messages a la compilation. Alors le meilleur moyen pour faire disparaitre ce warning, c'est de le corriger...

Dans ton cas, le warning indique qu'effectivement, certains embranchements (de if ... else) n'aboutissent pas à un return, or ta fonction est censée retourner un int. Cela ne veut pas dire que ton programme ne fera pas ce qu'il a à faire. Mais quand le compilateur génère ce Warning, il n'analyse pas le code dans les moindres détails pour vérifier si dans toutes les situations, il y aura une valeur de retour...

C'est un peu difficile a expliquer mais je vais essayer de prendre un exemple. Dans la première boucle FOR de ta fonction... Il y a un bloc IF (sans else) à l'interieur duquel il y a un autre bloc IF/else, avec deux return. Le compilateur ne cherche pas a savoir si ton programme va forcement passer a un moment ou a un autre par le bloc avec les return. (Il le ferait peut-etre si la condition était if(1), mais ce n'est pas le cas). En tout cas, il constate que s'il passe toujours par le "else" du premier if ("else" qui n'existe pas), il n'aura pas de return, donc il fait la gueule :) Je ne sais pas si c'est vraiment clair.. J'en doute ;)

En tout cas, pour corriger ce warning, je te conseille de retourner une valeur d'erreur (typiquement : return -1) tout à la fin de ta fonction, avant l'accolade fermante, puisque normalement tu n'es pas censé passer par la.

Je te conseille vivement, plutot que de désactiver les warnings qui t'ennuient, au contraire, de tous les activer (avec un paramètre de compilation, tel que -Wall si tu utilises GCC/G++, mais des paramètres similaires existent avec tous les compilateurs, je suppose, suffit de chercher un peu). Car corriger les warnings t'evitera bien souvent des erreurs difficiles a tracer! Et cela t'assurera un code relativement propre :)

Dernieres petites choses, a propos de ton main : int main (int argc, char**argv). Ca t'evitera un warning si tu compiles avec -Wall ;)
Et utilise plutot un switch à la place de 3 if :
switch(ordre)
{
case 0: //blabla
break;
case 1://blabla
break;
case 2: //blabla
break;
default: //Erreur!!!!!
printf("Erreur !!!! Aie Aie Aie c'est pas normal!! Qu'est-ce qui se passe?\n");
break;
}

Voila, bonne continuation!!
++
Soilwork
Messages postés
26
Date d'inscription
vendredi 2 avril 2004
Statut
Membre
Dernière intervention
15 mars 2005

Ah ok pour le m!=n !!! Mais un == ou ca???
Je vais modifier ma source :)
Afficher les 7 commentaires