Souci sur une fonction :)

Et voilà je perds ma virginité de se forum en faisant mon premier post :p

J'ai un petit soucis avec ma fonction qui me sert pour un pendu qui devrais me camoufler une chaine de caractère avec en entrée deux chaines de caractères
je m'attends a ce qu'elle me retourne par exemple en motSecret "bonjour" comme entrée

"*******"
au lieu de cela elle me renvoi
"*"


la voici la voila la coquine: je vous préviens indulgence je commence tout juste =) c'est d'ailleurs pour un tp d'un tutoriel C

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34

char* camouflageMot(char* motSecret,int longueurMot,char* lettresEssayees,int nombresEssais) { int* motCamoufle = NULL; motCamoufle = malloc(longueurMot * sizeof(int)); if (motCamoufle == NULL) { exit(0); }   int i=0; for(i=0;i<longueurMot;i++) { int j=0; for(j=0;j<nombresEssais;j++) { if(motSecret[i]==lettresEssayees[j]) { motCamoufle[i]=motSecret[i]; } else { if (motCamoufle[i]==motSecret[i]){} else {motCamoufle[i]='*';}   }   }       }   return motCamoufle; }

---

si tu remplaces :

Code:

---

1 2	int* motCamoufle = NULL; motCamoufle = malloc(longueurMot * sizeof(int));

---

par

Code:

---

1 2	char* motCamoufle = NULL; motCamoufle = malloc(longueurMot * sizeof(char));

---

Je crois ce que ca aller bcp mieux.

Pourquoi stocker une chaine de caractère dans un int * ?

Attention aux pointeurs !!! C'est le piège des débutants.

Bonjour Lomithrani,

Moi aussi, je vais "perdre ma virginité" parce que pour la première fois je vais aider quelqu'un sur ce forum (en général c'est moi qui pose des questions :langue2: )


Pour commencer, quelques points de corrections (sans importance ...)

Code:

---

1 2 3

int* motCamoufle = NULL; motCamoufle = malloc(longueurMot * sizeof(int));

---

C'est plus un char* que tu utilises non ?

Code:

---

1 2 3

char* motCamoufle = NULL; motCamoufle = malloc(longueurMot * sizeof(char));

---

et pour tes variables de boucles, il n'est pas nécessaire de les initialiser avant ta boucle. C'est ta boucle "for" qui va leur attribuer des valeurs...


Passons à la partie plus intéressante maintenant :D

Pourquoi ne fais tu pas appel avant tes deux boucles à la fonction "memset" dans le bibliothèque "string.h" pour intialiser motCamoufle à la chaine "*******"
utilisation : (apres ton malloc)

Code:

---

1 2	memset(motCamoufle, (int) '*', longueurMot);

---

Et pour remplacer les lettres de ce mot camouflé au fur et à mesure que l'utilisateur essaie de nouvelles lettres, je te conseille de faire plutôt une fonction qui prend en entrée cette lettre et la remplace dans "motCamoufle" si elle est à la même position que dans le mot initial.
Tu comprendras très vite l'intérêt de séparer tes fonctions en fonctions plus petites et plus faciles à lire...

:lol: :lol: :lol:
Je viens de voir que je me suis fais doubler par fregolo52 pour l'histoire des pointeurs !!!

rolf ce que je peux être stupide :D merci les gars ^^