Discussion :

[souduko]

Bonsoir,
je suis débutant en programmation C. J'ai un problème avec le passage d'un pointeur sur une structure en paramètre d'une fonction de saisie au clavier. Le programme se plante à l'endroit mentionné dans le code ci-dessous.Pouvez-vous m'aider?

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
34
#include<stdio.h>
#include<conio.h>
 
struct Personne{
    char nom[20];
    char prenom[20];
    int age;
};
typedef struct Personne PERSONNE;
 
void Saisie(PERSONNE *pers)
{
     printf("****Saisie d'une personne****\n");
     printf("Donner le nom \n");
     scanf("%s",&(pers->nom));  //le programme se plante ici
     printf("Donner le prenom \n");
     scanf("%s",&(pers->prenom));
     do{
        printf("donner l'age");
        scanf("%d",&(pers->age));
        printf("lecture faite");
        }
     while( pers->age <=0);
}
 
//la fonction main
int main()
{
    PERSONNE *pers;
    Saisie(pers); 
    printf("\n Appuie sur n'importe quelle touche pour fermer");
    getch();  
    return 0;
}

[Invité]

Bonsoir,

Ton problème est que tu n'initialises pas ton pointeur PERSONNE *pers;. De ce fait, il pointe vers une zone complètement aléatoire.
Il te faut allouer une zone mémoire avec malloc() de la taille sizeof(PERSONNE). Seulement par la suite tu pourras l'utiliser comme tu le souhaites, puis une fois fait, libérer cette zone mémoire avec la fonction free().

Notes :

• Pour des tableaux de char, le & n'a pas à être utilisé dans scanf(), et on préfèrera utiliser à la place la fonction fgets() plus sécurisante.
• On préfèrera utiliser des noms de types non tout en majuscules, qui ont pour habitude de désigner des constantes #define
• Préférer utiliser getchar() (déjà inclu dans <stdio.h>) plutôt que getch() qui n'est pas portable.

[gerald3d]

Comme l'a dit Winjerome il faut initialiser pers dans ton main();. Comme cette variable est déclarée dans main(); tu peux éviter une allocation dans le tas. Il suffit simplement de déclarer pers comme une variable et non comme un pointeur : PERSONNE pers;.
Une fois fait il te suffit de transmettre l'adresse de cette variable à la fonction Saisie(&pers);.