Discussion :

[TheChicken]

Bonjour,
Je dois programmer un jeu de Yathzee pour un projet de fac en C, et j'ai un petit problème concernant une déclaration dynamique d'un tableau de structure. Je m'explique :
J'ai deux structure (dans deux fichier .C différent)

La première :

Code :

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

1
2
3
4
5
struct FeuilleScore {  
int partieSuperieure [9];  
int partieInferieure [7];  
int scoreFinal;  
};

(j'ai mis dans un fichier .h correspondant : struct feuilleScore

et la deuxieme :
(j'ai mis en début de fichier #include "ProjetB.h" (c'est le .h du fichier .c où j'ai déclaré ma structure feuilleScore))

Code :

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

1
2
3
4
5
6
struct Joueur {  
char* nom;  
int nblancerRestants;  
int Des [5]; //les dés du joueur (5 dés)  
struct FeuilleScore;  
};

Le but est de créer le bon nombre de structures Joueur (une pour chaque joueurs de la partie).

j'ai donc fait cette petite fonction :

Code :

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

1
2
3
4
5
6
7
8
 struct Joueur* nombreJoueurs(void){  
 int nombreJoueurs;  
 printf("Combien de joueurs vont participer a la partie ? (1/2/3/...)");  
 scanf("%d",&nombreJoueurs);  
    struct Joueur* tab;  
    tab = malloc(sizeof(tab) * nombreJoueurs);  
    return tab;  
}

jusque là pas de problèmes.
Puis j'ai fais une fonction qui permet d'associer a chaque structures Joueurs le nom du joueur

Code :

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

1
2
3
4
5
6
7
void nomJoueurs (struct Joueur * tab) {  
    int i = 0;  
    for(i;i<sizeof(tab);i++){  
        printf("Nom du joueur %d ? ",i+1);  
        scanf("%s",tab[i].nom);  
    }  
}

sauf qu'a partir de là ... ça ne marche plus du tout, peut importe le nombre de joueurs , mon nomJoueurs me propose toujours le nom de deux joueurs a mètre, et ensuite windows me dit que mon programme a cessé de fonctionner.

Je pense donc que le problème vient de mon allocation dynamique de ma structure. Dois-je faire la même chose pour la structure FeuilleScore qui est dans la structure Joueur ?

Merci d'avance

[Invité]

Il te faut un tableau à deux dimensions, le programme quitte car il fait une erreur de segmentation fault en essayant d'accéder à une zone qui ne lui a pas été allouée

[jeroman]

Salut

Code :

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

sizeof(tab)

donne la taille du pointeur et non la taille de l'objet pointé. Du coup, ça ne peut pas marcher correctement.

Je trouve le nom de la fonction nombreJoueurs assez moyen, vu que sa principale fonction est d'allouer de la mémoire. creerJoueurs ou quelque chose du genre conviendrait peut-être mieux.

Tu peux créer un tableau dynamique possédant un nombre d'éléments correspondant au nombre de joueurs, chacun de ces éléments étant un pointeur sur une structure Joueur. Et via une boucle, il suffit ensuite d'allouer de la mémoire pour chacun de ces pointeurs, d'une taille de la structure.

La fonction retourne la mémoire allouée, mais elle doit aussi faire connaitre à la fonction appelante le nombre de joueurs saisi (on peut le faire en passant un pointeur sur "int" en argument, pointant sur un objet "int" de la fonction appelante, où il suffira de modifier la valeur pointée).

Tu peux donc par exemple créer un prototype de ce genre :

Code :

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

struct Joueur ** creerJoueurs(int * nombre_joueurs)

Et modifier aussi le prototype de la fonction nomJoueurs :

Code :

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

int nomJoueurs(struct Joueur ** joueurs , int nombre_joueurs)

La fonction serait appelée comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
struct Joueur ** joueurs;
int nombre_joueurs;
 
joueurs = creerJoueurs(&nombre_joueurs);
if (joueurs == NULL)
{
 ... erreur ...
} else
{
...
}

ce qui permettrait ensuite :

Code :

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

1
2
3
4
5
6
7
8
9
...
if (nomJoueurs(joueurs , nombre_joueurs) == 0)
{
 ... erreur ...
} else
{
...
}
...

[TheChicken]

voila mes deux fonctions :

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
struct Joueur** creerJoueurs (int * nombre_joueurs){
    int i = 0;
	int nombreJoueurs;
	printf("Combien de joueurs vont participer a la partie ? (1/2/3/...) ");
	scanf("%d",&nombreJoueurs);
	*nombre_joueurs = nombreJoueurs;
    struct Joueur** tab;
    tab[nombreJoueurs];
    for(i;i<nombreJoueurs;i++){
        tab[i] =  malloc(sizeof(struct Joueur));
    }
    return tab;
}
 
void nomJoueurs (struct Joueur** tab,int nombre_joueurs) {
    int i = 0;
    printf("%d",nombre_joueurs);
    for(i;i<nombre_joueurs;i++){
        printf("Nom du joueur %d ? ",i+1);
        scanf("%s",(*tab[i]).nom);
    }
}

cependant , lorsque j'exécute mes fonction par :

Code :

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

1
2
3
4
int main (void) {
    int taille;
    nomJoueurs(creerJoueurs(&taille),taille);
}

je ne peu pas rentrer de nom pour les joueurs :/ qu'est-ce que j'ai loupé ?

[Invité]

Tu dois malloquer ton tableau avant de l'utiliser !

Cette ligne ne fait rien à part faire planter ton programme : tab[nombreJoueurs];

Il y a de nombreux sujets sur le malloc de tableaux à deux dimensions, fais une petite recherche.

[diogene]

Code :

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

1
2
3
4
5
6
7
8
9
10
struct Joueur** creerJoueurs (int * nombre_joueurs)
{
 ....
    struct Joueur** tab;
    tab[nombreJoueurs];
    for(i;i<nombreJoueurs;i++){
        tab[i] =  malloc(sizeof(struct Joueur));
    }
    return tab;
}

tab[nombreJoueurs] ne fait rien.
tab[i] =.... est faux, puisqu'on n'a pas alloué de mémoire pour tab[i]

Il faut également tester la réussite des allocations.
Par exemple :

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
struct Joueur** creerJoueurs (int * nombre_joueurs)
{
    int i ;
    int nombreJoueurs = 0;
    *nombre_joueurs  = 0;
     do
     {
          printf("Combien de joueurs vont participer a la partie ? (1/2/3/...)\n");
          scanf("%d",&nombreJoueurs);
     }while(nombreJoueurs <=0)
 
   // allocation de la table des pointeurs
    struct Joueur** tab = malloc(nombreJoueurs*sizeof *tab);
    if(tab != NULL)
    {
        // allocation des joueurs
        for(i=0;i<nombreJoueurs;i++)
        {
             tab[i] =  malloc(sizeof(struct Joueur));
             if(tab[i] == NULL) break;
        }
        if(i<nombreJoueurs) // erreur d'allocation dans tab[i]
        {
                 // récupérer la mémoire allouée pour les tab[k] k=0..i-1
                 // récupérer la mémoire allouée pour tab
                 // mettre tab à NULL
        }
    }
    if(tab != NULL)  *nombre_joueurs  = nombreJoueurs ;
    return tab;
}

Maintenant, pourquoi faire un tableau de pointeurs sur des structures joueurs plutôt qu'un tableau de structure joueurs ? Ce serait plus simple.

[jeroman]

voila mes deux fonctions :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
struct Joueur** creerJoueurs (int * nombre_joueurs){
    int i = 0;
	int nombreJoueurs;
	printf("Combien de joueurs vont participer a la partie ? (1/2/3/...) ");
	scanf("%d",&nombreJoueurs);
	*nombre_joueurs = nombreJoueurs;
    struct Joueur** tab;
    tab[nombreJoueurs];
    for(i;i<nombreJoueurs;i++){
        tab[i] =  malloc(sizeof(struct Joueur));
    }
    return tab;
}

Quelques remarques :

Code :

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

1
2
3
4
	int nombreJoueurs;
	printf("Combien de joueurs vont participer a la partie ? (1/2/3/...) ");
	scanf("%d",&nombreJoueurs);
	*nombre_joueurs = nombreJoueurs;

On peut écrire directement :

Code :

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

1
2
	printf("Combien de joueurs vont participer a la partie ? (1/2/3/...) ");
	scanf("%d",nombre_joueurs);

-------

Code :

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

tab[nombreJoueurs];

Ceci ne fait rien. En plus, ça déborde.
Il faut allouer de la mémoire :

Code :

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

1
2
struct Joueur** tab;
tab = malloc( sizeof *tab * *nombre_joueurs);

-------

Il n'y a nulle part aucun contrôle sur le retour des malloc. C'est "dangereux" car si la fonction échoue ça fait planter le programme.

-------

Le programme doit toujours libérer lui-même la mémoire dynamique qu'il a allouée (une fois qu'il n'en a plus besoin)

[jeroman]

Maintenant, pourquoi faire un tableau de pointeurs sur des structures joueurs plutôt qu'un tableau de structure joueurs ? Ce serait plus simple.

J'avoue que j'ai un peu compliqué l'affaire avec cette idée. C'était pas très utile pour ce genre de programme.
Au pire, à ce stade (y a pas bcp de code encore), on peut revenir à l'autre solution, il y a peu de choses à modifier.

[TheChicken]

J'ai complété le code

Code :

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

1
2
3
4
5
6
7
if(i<nombreJoueurs){ // erreur d'allocation dans tab[i]
            for(k=0;k<nombreJoueurs;k++){
                free(tab[k]);// récupérer la mémoire allouée pour les tab[k] k=0..i-1
            }
            free(tab);// récupérer la mémoire allouée pour tab
            tab = NULL;// mettre tab à NULL
        }

mais ma fonction nomJoueurs ne fonctionne pas (je passe une fois dans ma boucle for (puisque "Nom du joueur 1 ? " apparait ), mais après mon programme bug)

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
void nomJoueurs (struct Joueur** tab,int nombre_joueurs) {
    int i = 0;
    char* temporaire;
    for(i;i<nombre_joueurs;i++){
        printf("Nom du joueur %d ? ",i+1);
        scanf("%s",temporaire);
        (*tab[i]).nom = malloc(sizeof(char)*strlen(temporaire)+1);
        strcpy(temporaire,(*tab[i]).nom );
    }
}
int main (void) {
    int taille;
    nomJoueurs(creerJoueurs(&taille),taille);
    return 0;
}