Problème avec fonction

Bonjour,
j'ai une fonction qui me retourne un pointeur de chaîne de caracteres, mais lors de son appel ca bug, je ne sais ou je me suis gourer :s
voilà le code:

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

char *GetCardValue(CARD cDeck[],int cNum){ char *vPointer, buf[3]; cNum%=13;   if ((cNum<=9)&&(cNum>0)){ sprintf(buf, "%d", cNum); strcpy(vPointer,buf); strcpy(vPointer,(" "+cDeck[cNum].Type)); } else switch(cNum){ case 0: strcpy(vPointer,("J "+cDeck[cNum].Type)); break; case 10: strcpy(vPointer,("Q "+cDeck[cNum].Type)); break; case 11: strcpy(vPointer,("K "+cDeck[cNum].Type)); break;   } return(vPointer); }

---

Merçi d'avance

Et tu alloues quand la mémoire nécessaire pour contenir la chaine pointée par vPointer ? :roll:

Citation:

---

Envoyé par Pouic

Et tu alloues quand la mémoire nécessaire pour contenir la chaine pointée par vPointer ? :roll:

---

:oops:
mouarf merçi j'suis bête j'ai carrément larguer ca lol

La remarque de Pouic résoud ton problème.
Mais il te reste une chose à voir dans ton code :
(1)cNum peut être égal à 12 et (2) cNum sera négatif si le paramètre (int) de la fonction est négatif. un default serait peut-être le bienvenu.

aye encore un problème lorsque j'affiche le resultat de cette fonction :s

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

char *GetCardValue(CARD cDeck[],int cNum){ char *vPointer, buf[3]; cNum%=13; vPointer=malloc(48); if ((cNum<=9)&&(cNum>0)){ sprintf(buf, "%d", cNum); strcpy(vPointer,buf); strcpy(vPointer,(" "+cDeck[cNum].Type)); } else switch(cNum){ case 0: strcpy(vPointer,("J "+cDeck[cNum].Type)); break; case 10: strcpy(vPointer,("Q "+cDeck[cNum].Type)); break; case 11: strcpy(vPointer,("K "+cDeck[cNum].Type)); break;   }   return(vPointer); }   void ShowDeck(CARD cDeck[MAX_CARDS]){ int cpt;   for (cpt=0; cpt<MAX_CARDS; cpt++) printf(">%s \n",GetCardValue(cDeck,cpt)); }

---

quand je fais genre:

Code:

---

printf("%s",GetCardValue(Deck,12);

---

eh ben ca m'affiche des choses autre que ce que je veux, je crois que c'est tout bête mais j'suis bigloux lol

Citation:

---

Envoyé par diogene

La remarque de Pouic résoud ton problème.
Mais il te reste une chose à voir dans ton code :
(1)cNum peut être égal à 12 et (2) cNum sera négatif si le paramètre (int) de la fonction est négatif. un default serait peut-être le bienvenu.

---

non. parceque c'est limiter de 0 à 53 ;)

Goundy :

Citation:

---

non. parceque c'est limiter de 0 à 53

---

et si cnum = 12 ou 25 ou ... cnum %13 == 12 !!!

Citation:

---

printf("%s",GetCardValue(Deck,12);

---

Justement cNum == 12!!! :)

oui t'as raison, j'suis perdu rholala lol.
Je vais recalculer merci

j'ai corriger mon truc mais j'ai un probleme avec la fonction d'affichage:

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

char *GetCardValue(CARD cDeck[],int cNum){/*Avoir la valeur de la carte*/ char *vPointer, buf[3]; cNum%=13; vPointer=malloc(48); if (cNum<=9 && cNum>0){ sprintf(buf, "%d", cNum); strcpy(vPointer,buf); strcpy(vPointer,(" "+cDeck[cNum].Type)); } else switch(cNum){ case 0: strcpy(vPointer,("A "+cDeck[cNum].Type)); break; case 10: strcpy(vPointer,("J "+cDeck[cNum].Type)); break; case 11: strcpy(vPointer,("Q "+cDeck[cNum].Type)); break; case 12: strcpy(vPointer,("K "+cDeck[cNum].Type)); break; }   return(vPointer); }

---

pour l'affichage:

Code:

---

1 2 3 4 5 6 7 8 9

void ShowDeck(CARD cDeck[MAX_CARDS]){/*Affichage du Deck*/ int cpt; char* ptr;   for (cpt=0; cpt<MAX_CARDS; cpt++){ ptr=GetCardValue(cDeck,cpt); printf(">%s \n",*ptr); } }

---

Code:

---

1 2 3 4

for (cpt=0; cpt<MAX_CARDS; cpt++){ ptr=GetCardValue(cDeck,cpt); printf(">%s \n",*ptr); }

---

Erreur , tu imprimes une chaîne (%s) et tu passe un char (*ptr)

Code:

---

printf(">%s \n",ptr);

---

mouai mais ca m'affiche des chose bizzare avec size... j'y comprend rien :/

Citation:

---

ca m'affiche des chose bizzare avec size...

---

de quel genre ?
qu'est ce que size ?

Code:

---

1 2 3 4 5 6

if (cNum<=9 && cNum>0) { sprintf(buf, "%d", cNum); strcpy(vPointer,buf); strcpy(vPointer,(" "+cDeck[cNum].Type)); }

---

Je suppose que le deuxième strcpy devrait être strcat . Non ?

Code:

---

1 2 3 4 5

if (cNum<=9 && cNum>0) { sprintf(vPointer, "%d", cNum); strcat(vPointer,(" "+cDeck[cNum].Type)); }

---

lol bon, il doit m'afficher par exemple:

Code:

---

1 2

>2 R >A C

---

...etc
ben lui il m'affiche par exemple:

Code:

---

>sharedptr->size == sizeof(W32_EH_SHARED)

---

des trucs comme ça, c'est bizzare :/

Citation:

---

Envoyé par diogene

[/code] Je suppose que le deuxième strcpy devrait être strcat . Non ?

---

cela ne change rien

Citation:

---

cDeck[cNum].Type

---

Qu'est ce c'est ? peut-on voir la définition de CARD ?

Citation:

---

cela ne change rien

---

Ca c'est sur mais le message est différent

Code:

---

1 2 3 4

typedef struct {/*Structure représentant chaque carte*/ int Value; /*Valeur de la carte*/ char Type; /*Type de la carte*/ }CARD;

---

voilà pour CARD.
Il doit avoir un problème d'assignation dans la fonction GetCardValue je crois mais je ne sais ou est le probleme lol
merci

tiens regarde l'affichage:
http://img288.imageshack.us/my.php?i...rderror4ij.jpg

Code:

---

char Type;

---

Alors, ce type de construction est incorrect : "J "+cDeck[cNum].Type.

Code:

---

strcpy(vPointer,(" "+cDeck[cNum].Type));

---

la chaîne copiée va être a l'adresse de la chaîne " " + la valeur du char donc dans les décors. Ceci ne réalise pas la concaténation de la chaîne et du caractère

Il faut envisager qq chose du genre :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

char *GetCardValue(CARD cDeck[],int cNum){/*Avoir la valeur de la carte*/ char *vPointer; vPointer=malloc(48); if (cNum<=9 && cNum>0){ sprintf(vPointer, "%d %c", cNum ,cDeck[cNum].Type); } else { char car ; switch(cNum) { case 0 : car = 'A'; break; case 10: car = 'J'; break; case 11: car = 'Q'; break; case 12: car = 'K'; break; } sprintf(vPointer, "%c %c", car, cDeck[cNum].Type); } return(vPointer); }

---

ben justement, si je fais ainsi j'en aurai pour longtemps :/, j'ai 52valeur :s.
Bon merçi je vais trouver un autre moyen, merci encore ;)

Citation:

---

si je fais ainsi j'en aurai pour longtemps :/, j'ai 52valeur :s

---

(donc tu vas sans doute supprimer le %13) tu peux envisager de mettre ces car dans un tableau et écrire au lieu d'un switch

Code:

---

1 2 3 4 5 6 7

char *TabCar = "A---------JQK........" .... if (cNum<=9 && cNum>0 sprintf(vPointer, "%d %c", cNum ,cDeck[cNum].Type); else sprintf(vPointer, "%c %c", TabCar[cNum] , cDeck[cNum].Type);

---

ouais effectivement, je te remerçie infiniment pour ton aide et ton interêt ;)
Je suis satisfait :)

D'ailleurs, on peut réduire à

Code:

---

1 2 3

char *TabCar = "A123456789JQK........" .... printf(vPointer, "%c %c", TabCar[cNum] , cDeck[cNum].Type);

---

Citation:

---

Je suis satisfait

---

Alors tout est bien :D

ouais mais enfait je n'ai rien changer, juste j'ai remplacer les strcpy et les strcat par un seul sprintf et ca marche nickel.
Merci ;)