Un de mes premiers programme : Quelques questions...

Tout d'abord bonsoir à tous,
voilà je viens de commencer le c en cours, j'ai un petit peu de mal mais j'ai réussi à faire le début de programme suivant sur lequel j'aurai quelques questions.

Ce programme permet d'enregistrer 4 score (mi temps, fin du match, prolongations et pénaltes)

Je souhaite garder ma structure utiliser pour le type Score.

Le programme est en pièce jointe.

Voici mes différentes questions :

Tout d'abord que puis je faire de mieu, niveau syntaxe ? propreté ?

Ensuite, je trouve cela bizarre de devoir faire 4 malloc dans lireScore(), peux t'on mieux faire ?

Suis je sur que cela va marcher à chaque fois ? Je m'explique, j'ai copié l'adresse que pointe s dans s1 afin de sauvegarder le pointeur sur le premier score, mais est-il possible que les malloc me remettent s sur une nouvelle adresse mémoire, du coup s1 ne pointerai plus sur rien de cohérent ?


Merci d'avance pour toutes vos réponses.

Salut,


Pourquoi faire 4 allocations quand tu peux gérer un tableau de score ?
De plus, tu alloues de la mémoire sans jamais la libérer...

En fait, comme je l'ai dit dans mon premier sujet je souhaite utiliser le type Score comme il est défini, je ne peux donc pas utiliser de tableau.

Code:

---

(s -> suivant) = (score*) malloc (sizeof(score));

---

Il est inutile (et même déconseillé) de caster un malloc : http://www.cprogramming.com/faq/cgi-...&id=1043284351

T'es vraiment obligé de gérer tes scores via une liste chainée ?

Oui je suis obligé, en fait là il n'y a qu'une ébauche de ce que je veux faire, je souhaite que l'on ne soit pas obligé ensuite de rentrer tous les scores, par exemple si le match se fini sans prolongations, je ne souhaite pas stocker les scores de la prolongations.

J'ai une autre question, j'ai entendu parler de realloc ? cette fonction me serait t'elle pas plus utile ?

merci

Un realloc permet de reallouer de la mémoire déja allouée à une taille différente. Ça ne correspond pas à ton problème puisque tu n'as pas besoin de changer la taille de ta structure : c'est toujours 2 entiers.
Il y a un grave problème qu'il faut que tu règles dans ton programme c'est que tu ne libères pas la mémoire après tes allocations -> fuite mémoire.
Pour bien faire, un malloc / calloc / realloc devrait toujours être suivi d'un free (après exploitation de la mémoire allouée évidemment), un peu comme le mécanisme des parenthèses ouvrantes et fermantes.

Merci de ta réponse,

à quel moment dois je libérer la mémoire parce que je ne comprends pas si le la libère, mes données vont être perdues non ?

merci

Citation:

---

Envoyé par allezlolo

Merci de ta réponse,

à quel moment dois je libérer la mémoire parce que je ne comprends pas si le la libère, mes données vont être perdues non ?

merci

---

Tu libères ta mémoire dès que tu n'en as plus utilité ou au plus tard avant la fin du programme ou avant que le pointeur dans lequel est stocké l'adresse du bloque alloué soit hors de portée.

Thierry

D'accord merci pour vos réponses;)

Je galère sur une autre question, j'ai le type suivant :

Code:

---

1 2 3 4

typedef struct equipe { char *nom; char* tab[24]; }equipe;

---

Je souhaite enregistrer le nom de mon équipe dans nom et le nom du coach dans tab[0] que dois mettre ? J'ai essayé de nombreuses solutions sans m'en sortir :(

Code:

---

1 2 3 4 5 6 7

void lireEquipe(equipe *e) { printf ("Entrez le nom de l'équipe : "); // (e -> nom) = (char*) malloc (sizeof(char)*50); scanf ("%s",(e->(&nom))); // printf ("Entrez le nom du coach : "); // scanf ("%s",&(e -> tab[0])); }

---

Je retrouve toujours une segmentation fault (core dumped)

merci d'avance

C'est:

Code:

---

scanf ("%s", e->nom);

---

Mais e->nom n'est qu'un pointeur. Il faut allouer la mémoire avant de réaliser cette saisie. Pourquoi as-tu commenté l'allocation? Par ailleurs, scanf() n'est pas vraiment la fonction la plus adaptée pour réaliser proprement une saisie utilisateur de chaîne de caractères.

Thierry

J'ai déjà essayé ce bout de code, en remplaçant j'ai donc :

Code:

---

1 2 3 4 5 6 7

void lireEquipe(equipe *e) { printf ("Entrez le nom de l'équipe : "); (e -> nom) = (char*) malloc (sizeof(char)*50); scanf ("%s",(e->nom)); // printf ("Entrez le nom du coach : "); // scanf ("%s",&(e -> tab[0])); }

---

Et toujours la même erreur...

Que me conseille tu à part scanf ?

Merci beaucoup

petite précision, voici mon main :

Code:

---

1 2 3 4 5

int main() { equipe *e; e = (equipe*) malloc (sizeof(equipe)); lireEquipe(e); }

---

c'est bon c'est résolu ;) merci à tous

Citation:

---

Envoyé par allezlolo

J'ai déjà essayé ce bout de code, en remplaçant j'ai donc :

Code:

---

1 2 3 4 5 6 7

void lireEquipe(equipe *e) { printf ("Entrez le nom de l'équipe : "); (e -> nom) = (char*) malloc (sizeof(char)*50); scanf ("%s",(e->nom)); // printf ("Entrez le nom du coach : "); // scanf ("%s",&(e -> tab[0])); }

---

Et toujours la même erreur...

Que me conseille tu à part scanf ?

Merci beaucoup

---

Pour la saisie de chaines, remplace scanf() par fgets() ou au minimum, indique le nombre de caractères max à saisir dans la chaine de format:

Code:

---

scanf("%49s", e->nom);

---

Par ailleur, attention également à la façon dont tu utilises malloc(). Il faut toujours vérifier que malloc() retourne une adresse différente de NULL. Ainsi:

Code:

---

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

void lireEquipe(equipe *e) { printf ("Entrez le nom de l'équipe : "); /* -tc- Purge le tampon de stdout et force l'affichage de l'invite */ fflush(stdout); /* -tc- Pourquoi utilises-tu toujours des parentheses autour de e->nom.   Par ailleurs, il est idiomatique de ne pas caster la valeur de retour de   malloc() en C. */ /* -tc- Finalement, sizeof (char) vaut 1 par definition. Il n'est pas utile ici */ e -> nom = malloc (sizeof *e->nom *50); /* -tc- On verifie donc la validite de la valeur retournee par malloc() */ if (e->nom != NULL) { int c;   scanf ("%49s", (e->nom)); /* -tc- A ce stade, il faut encore purger le tampon du flux d'entree   standard qui contient au moins le caractere '\n' */ while ((c = getchar()) != '\n' && c != EOF) { } } }

---

Thierry