Problème avec fonction

**Goundy** · 01/10/2005, 15h53

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 :

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

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

**Pouic** · 01/10/2005, 15h59

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

**Goundy** · 01/10/2005, 16h16

Envoyé par Pouic

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

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

**diogene** · 01/10/2005, 16h17

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.

**Goundy** · 01/10/2005, 16h19

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

Code :

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

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 :

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

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

**Goundy** · 01/10/2005, 16h21

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

**diogene** · 01/10/2005, 16h27

Goundy :

non. parceque c'est limiter de 0 à 53

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

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

Justement cNum == 12!!!

**Goundy** · 01/10/2005, 16h34

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

**Goundy** · 01/10/2005, 17h00

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

Code :

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

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 :

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

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

**diogene** · 01/10/2005, 17h03

Code :

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

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 :

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

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

**Goundy** · 01/10/2005, 17h05

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

**diogene** · 01/10/2005, 17h10

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

de quel genre ?
qu'est ce que size ?

**diogene** · 01/10/2005, 17h12

Code :

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

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 :

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

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

**Goundy** · 01/10/2005, 17h13

lol bon, il doit m'afficher par exemple:

Code :

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

1
2
>2 R
>A C

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

Code :

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

>sharedptr->size == sizeof(W32_EH_SHARED)

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

**Goundy** · 01/10/2005, 17h15

Envoyé par diogene

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

cela ne change rien

**diogene** · 01/10/2005, 17h17

cDeck[cNum].Type

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

cela ne change rien

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

**Goundy** · 01/10/2005, 17h23

Code :

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

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

**Goundy** · 01/10/2005, 17h28

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

**diogene** · 01/10/2005, 17h34

Code :

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

char Type;

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

Code :

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

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

**diogene** · 01/10/2005, 17h49

Il faut envisager qq chose du genre :

Code :

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

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

Problème avec fonction

C

Discussions similaires

Partager

Partager