Aide dans une fonction en langage C (débutant)

[invitef84b5b58]

Salut les amis,
je programme en c, et j'ai une structure Etudiant qui comtient 3 variables (int code, char *nom et float moy)
Je fais une fonction qui me permet de lire les étudiants que j'ai !
voila ce que j'ai fais:

void Lecture(Etudiant Et[] , int *n)
{
int i;
	printf("donner le nombre des etudiants");
	n=(int*)malloc(sizeof(int));
	scanf("%d",n);
	printf("%d",*n);
	printf("je suis la \n");
	for(i=0;i<(*n);i++);
	{
	    printf("entrer le code de l'etudiant");
	    scanf("%d",&Et[i].code);
	    printf("entrer le nom de l'etudiant");
            scanf("%s",Et[i].nom);
	    printf("entrer la moyenne de l'etudiant");
 	    scanf("%f",&Et[i].moy);
	}
}
et voila mon main():
main()
{
	Etudiant *Et;
	int *n;
	Et=(Etudiant*)malloc(sizeof(Etudiant));
	n=(int*)malloc(sizeof(int));
	Lecture(Et,n);
	return 0;
}

mon problème c'est que j'arrive pas à completer la saisie, y'a aucun erreur lors de la compilation mais le programme cesse de fonctionner dès que je termine la saisie du premier étudiant !!
Quelqu'un peut m'aider et Merci
-----

[Jack]

Il faut lire les règles du forum programmation:
http://forums.futura-sciences.com/pr...ves-forum.html

La présentation du code doit se faire avec les balises code. Merci d'y penser la prochaine fois.

[Jack]

Pourquoi cette ligne dans la fonction Lecture?

n=(int*)n=(int*)malloc(sizeof(int));

puisque n a déja été instanciée dans la fonction main.

De plus, ton tableau d'Etudiant n'a été instancié que pour un seul étudiant:

Et=(Etudiant*)malloc(sizeof(Etudiant));

Il faudrait l'instancier dans la fonction, lorsque tu connais le nombre d'étudiants.

[invitef84b5b58]

Salut Jack,
Je suis désolé J'ai rien compris
Comment je ferai donc pour que mon tableaux d'etudiant contient plus d'un seul etudiant ?

[A voir en vidéo sur Futura]

[invite0bbe92c0]

Je ne comprends pas.
Quel est l’intérêt de ça :

n=(int*)malloc(sizeof(int));

surtout avec ce test

for(i=0;i<(*n);i++);

Qui n'a aucune chance de sortir de manière déterministe, *n étant indéterminé.

Ca me fait penser à du "cargo cult programming".

[invitef84b5b58]

Bluedeep
J'efface cette ligne

n=(int*)malloc(sizeof(int));

de la fonction lecture
mais elle dois rester dans le main() puisque je connais pas encore la dimmention !

pourquoi j'arrive pas à déclarer le tableau Etudiant comme ça

Etudiant Et[];

dans le main()
et plutôt je la fais dans la fonction et ça marche
comment je peux déclarer mon tableau Etudiant au main() dont sa dimmention je la connais pas?

[invite0bbe92c0]

comment je peux déclarer mon tableau Etudiant au main() dont sa dimmention je la connais pas?

Si vous voulez travaillez avec un tableau dont vous ne connaissez pas la taille a priori il n'y a pas 36 solutions, il y en a en gros 3 :

1 - laisser le C et changer de langage pour un langage acceptant des tableaux à accroissement dynamique.
2 - en C, utiliser une liste chaînée au lieu d'un tableau.
3 - en C, utiliser un tableau de taille arbitraire et gérer une recopie en situation d'overflow; c'est à dire allouer n positionss dans le tableau (n=10, par exemple) et quand c'est atteint créer un tableau avec 20 (ou 12 ou 50 ou 42008) positions, recopier le premier dedans et travailler sur le deuxième, et ainsi de suite.

Comme cela semble être un exercice, on exclut le 1).
Le 3) est assez inélégant et sur le papier augmente la charge CPU à chaque overflow.
Le 2) est la solution normale dans ce cas de figure.

Néanmoins, je maintiens que vous avez fait du "cargo cult programming" : vous avez pompé des lignes de codes sur internet (ou sur un bouquin) sans s'interroger sur leur signification.

Sinon, chercher "listes chainées en C" sur internet pour la bonne solution.

[invitedc9da242]

Bonjour,

Il faut que tu déclare une variable tableau avant ton main. "int tab= [x, y].

tu as un cours de langage C dans le forum électronique vas y faire un tour il parle des tableaux.

bon courage.

[invite0bbe92c0]

,

Il faut que tu déclare une variable tableau avant ton main. "int tab= [x, y].
.

Non.
Il ne connait pas la taille initiale qu'il doit affecter au tableau.
Il vaut donc mieux utiliser une linked list

[invitedc9da242]

Exact si tu ne connais pas le nombre d'entrée.

Bon courage.

[invitef84b5b58]

Merci, je vais chercher un cours des liste chaînées

[invite0bbe92c0]

Vous avez ça qui est pas mal fait sur la question : http://www.cprogramming.com/tutorial/c/lesson15.html

[Jack]

Il faudrait connaitre les données exactes du problème avant de tout remettre en question.

Il est possible de travailler avec un tableau, mais si la taille doit être définie dans la fonction Lecture, il suffit de le déclarer dynamiquement une fois le nombre n d'étudiants connu. Pourquoi également ne pas laisser la fonction retourner ce nombre n? Ca éviterait de le passer par adresse dans la fonction.

Le prototype de la fonction deviendrait:

int Lecture(Etudiant* Et);

[invite43901482]

1 - laisser le C et changer de langage pour un langage acceptant des tableaux à accroissement dynamique.
2 - en C, utiliser une liste chaînée au lieu d'un tableau.
3 - en C, utiliser un tableau de taille arbitraire et gérer une recopie en situation d'overflow; c'est à dire allouer n positionss dans le tableau

Il y a une 4ème solution, tout déclarer en statique, en indiquant les valeurs maximales adéquates selon le problème. C'est à dire la capacité maximale de l'étalissement, et le nombre de caractères maximal pour un nom d'étudiant.

Quand on est débutant on ne s'amuse ni avec l'allocation dynamique, ni avec les listes chaînées.

[invitef84b5b58]

le problème est le suivant:
Ecirire un programme en c en utilisant les fonctions:
Lecture: qui permet de lire un tableau en passant en paramètre le nom du tableau et sa taille.
Les élèment du tableau sont les étudiant caractérisé par : Code Et, Nom et Moy Et
et voila ce que j'ai fais:

typedef struct{
    int code;
    char *nom;
    float moy;
              }Etudiant;
// fonction lecture
void Lecture(Etudiant Et[] , int *n)
{
	int i;
	printf("donner le nombre des etudiants");
	scanf("%d",n);
	for(i=0;i<(*n);i++);
	{		
		printf("entrer le code de l'etudiant");
	    scanf("%d",&Et[i].code);
	    printf("entrer la moyenne de l'etudiant");
 	    scanf("%f",&Et[i].moy);
 	    printf("entrer le nom de l'etudiant");
        scanf("%s",Et[i].nom);
	}
main()
{
	Etudiant *Et;
	int *n;
	n=(int*)malloc(sizeof(int));
	Lecture(Et,n);
	return 0;
}

[invitef84b5b58]

j'ai un problème dans cette ligne !!

scanf("%f",&Et[i].moy);

[invite0bbe92c0]

j'ai un problème dans cette ligne !!

scanf("%f",&Et[i].moy);

Quel problème ?

[invitef84b5b58]

le programme se bloque au niveau de cette ligne !
je pense que c'est à cause du char *nom: je connai pas la taille du nom !

[Deedee81]

Bonjour,

j'ai un problème dans cette ligne !!

scanf("%f",&Et[i].moy);

Dans la fonction main, tu as instancié Et avec une allocation sur la taille de la structure étudiant :

	Et=(Etudiant*)malloc(sizeof(Etudiant));

Donc, ton Et correspond à un tableau pouvant contenir un seul élément.

Or, dans ta boucle et ton scanf, tu balaies un nombre à priori quelconque de données. Si i est supérieur à 0, tu as un crash mémoire.

[invitef84b5b58]

Et donc comment je vais faire pour que mon tableau contiendra plus d'un seul élèment?

[invite0bbe92c0]

Et donc comment je vais faire pour que mon tableau contiendra plus d'un seul élèment?

Pour 1 étudiant tu alloue sizeof(Etudiant) bytes, ok ?
Donc pour n étudiants tu alloues combien ?

[Deedee81]

Pour 1 étudiant tu alloue sizeof(Etudiant) bytes, ok ?
Donc pour n étudiants tu alloues combien ?

Et question subsidiaire : à quel moment faire l'allocation ? (réponse devenue évidente)

[invitef84b5b58]

Pour 1 étudiant tu alloue sizeof(Etudiant) bytes, ok ?
Donc pour n étudiants tu alloues combien ?

Je pense comme ça:

Et=(Etudiant*)malloc(sizeof(Et udiant)*(*n));

[invitef84b5b58]

Et question subsidiaire : à quel moment faire l'allocation ? (réponse devenue évidente)

faire l'allocation de l'espace pour une variable lorsque qu'on connais pas sa taille ou pour une varriable qui n'a pas une valleure fixe !

[Jack]

    int *n;
    n=(int*)malloc(sizeof(int));
    Lecture(Et,n);

Pourquoi pas tout simplement

     int n;
     Lecture (Et, &n);

Pour le reste, tu n'as pas initialisé de le tableau pour accueillir les n étudiants.

Petite remarque: c'est volontaire de donner le même identificateur pour les variables appelantes et les variables appelées?

[invite43901482]

je pense que c'est à cause du char *nom: je connai pas la taille du nom !

Et pour cause, je t'avais dis de bosser en statique plutôt que dynamique. Dans le cas où tu travailles en dynamique, il faudrait faire un travail d'initialisation.

du style

#include <stdio.h>


typedef struct
{
    char *nom;
} Etudiant;

Etudiant *Init(char *s);

int main(void)
{
    Etudiant *et = Init("toto");

    printf("%s\n", et->nom);

    free(et->nom);
    free(et);

    return 0;
}

Etudiant *Init(char *s)
{
    Etudiant *e = malloc(sizeof(Etudiant));
    if (e == NULL) exit(-1);

    e->nom = strdup(s);

    return e;
}

[Deedee81]

Salut,

Et pour cause, je t'avais dis de bosser en statique plutôt que dynamique.

C'est peut-être une exigence du prof dans l'exercice proposé