allocation dynamique d'un tableau de struct C Fermé

Question

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

le père · Answer

Bonjour

Si tu veux allouer une structure par joueur, il faut te baser sur la taille de la structure, pas celle du pointeur !
    tab = malloc(sizeof(tab) * nombreJoueurs); à remplacer par
    tab = malloc(sizeof(struct Joueur) * nombreJoueurs); 

Pour ta boucle sur les joueurs, ton for(i;i<sizeof(tab);i++) ne risque pas de fonctionner.sizeof(tab), c'est la taille de la variable tab en octets, qi n'a aucun rapport avec le nombre de joueurs. Il faut rendre la globale la variable nombreJoueurs et faire for(i;i<nombreJoueurs;i++)

TheChicken · Answer

Merci, cependant, pour mon projet je n'ai pas le droit d'utiliser de variables globale. Une fois le tableau de structure crée il n'y a pas moyen de connaitre la taille du tableau (le nombre de structures Joueurs) sans passer par une variable globale ?

le père · Answer

Tu peux toujours créer une variable que tu passes par référence à ta fonction nombreJoueurs, et ainsi récupérer le nombre de joueurs, pour le passer ensuite aux fonctions qui en ont besoin. C'est un moyen complètement artificiel de respecter la contrainte.
Sinon, tu peux aussi allouer une structure de plus que nécessaire, et mettre dedans une donnée impossible (par exemple un nom vide) et faire ton test de fin de boucle sur ce nom plutôt que sur le nombre de joueurs.

TheChicken · Answer

Voila a quoi ressemble ça ressemble maintenant 

struct Joueur* nombreJoueurs(int taille){
	int nombreJoueurs;
	printf("Combien de joueurs vont participer a la partie ? (1/2/3/...) ");
	scanf("%d",&nombreJoueurs);
	taille = nombreJoueurs;
    struct Joueur* tab;
    tab =  malloc(sizeof(struct Joueur) * nombreJoueurs);
    return tab;
}

void nomJoueurs (struct Joueur* tab,int taille) {
    int i = 0;
    printf("%d",taille);
    for(i;i<taille;i++){
        printf("Nom du joueur %d ? ",i+1);
        scanf("%s",tab[i].nom);
    }
}

int main (void) {
    int taille = 0;
	nomJoueurs(nombreJoueurs(taille),taille);
}

sauf que la taille reste toujours a 0, et donc je ne peut pas passer dans la boucle for de nomJoueurs.

le père · Answer

Dans cette partie:
void nomJoueurs (struct Joueur* tab,int taille) {
    int i = 0;
    printf("%d",taille);
    for(i;i<taille;i++){
        printf("Nom du joueur %d ? ",i+1);
        scanf("%s",tab[i].nom);
    }
}

Il faut que tu lises le nom dans une variable provisoire, puis que tu alloues à tab[i].nom la mémoire nécessaire (strlen(nomprovisoire)+1 octets) . Ensuite, tu copies (strcpy) le nom provisoire vers tab[i].nom

Allocation dynamique d'un tableau de struct C

5 réponses

Discussions similaires