Problème avec fonction

Version imprimable

Voir 40 message(s) de cette discussion en une page

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

01/10/2005, 15h59
Pouic

Et tu alloues quand la mémoire nécessaire pour contenir la chaine pointée par vPointer ? :roll:
01/10/2005, 16h16
Goundy

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
01/10/2005, 16h17
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.

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

01/10/2005, 16h21
Goundy

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 ;)
01/10/2005, 16h27
diogene

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!!! :)
01/10/2005, 16h34
Goundy

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);

01/10/2005, 17h05
Goundy

mouai mais ca m'affiche des chose bizzare avec size... j'y comprend rien :/
01/10/2005, 17h10
diogene

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)); 
          }

01/10/2005, 17h13
Goundy
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 :/
01/10/2005, 17h15
Goundy

Citation:

Envoyé par diogene

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

cela ne change rien
01/10/2005, 17h17
diogene

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

01/10/2005, 17h28
Goundy

tiens regarde l'affichage:
http://img288.imageshack.us/my.php?i...rderror4ij.jpg
01/10/2005, 17h34
diogene

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);      
  }

Voir 40 message(s) de cette discussion en une page