Pourquoi ce code plante (segmentation fault)

Version imprimable

Bonjour à tous,

Qqun peut-il m'expliquer pourquoi ce code donne un segmentation fault à l'exécution ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
#include <stdio.h>
#include <stdlib.h>
#include "structure.h"
 
int main(int argc, char *argv[]) {
	Coordonnees *pointeur = NULL;
	Volume musique = MOYEN;
 
	pointeur->x = 7;
	pointeur->y = 35;
 
	printf("pointeur : x=%d , y=%d\n",pointeur->x,pointeur->y);
 
	if (musique == MOYEN) {
		printf("Le volume de la musique est moyen\n");
	}
 
	return 0;
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
#ifndef STRUCTURE_H_
#define STRUCTURE_H_
 
typedef struct Coordonnees Coordonnees;
struct Coordonnees {
    int x;
    int y;
};
 
typedef enum Volume Volume;
enum Volume {
    FAIBLE, MOYEN, FORT
};
 
#endif

Merci ;)

19/01/2011, 00h33
quetzacoatl

Bonjour,
Dans le main() vous faites:

Citation:

pointeur->x = 7;
pointeur->y = 35;

Alors que vous avez précédemment fait:

Citation:

Coordonnees *pointeur = NULL;

Le pointeur "pointeur" ne pointe donc sur "rien", ainsi il ne pointe pas sur une zone capable d'acceuillir les champs x et y.
Le mieux dans ce type de cas simples est de ne surtout pas utiliser les pointeurs qui ici ne sont d'aucune utilité, mais d'utiliser la structure "normale":

Code:

Coordonnees pointeur = NULL;

L'accès aux champs se fait alors par "."
19/01/2011, 10h28
diogene

Code:

Coordonnees pointeur = NULL;

L'initialisation est alors incorrecte, pointeur n'est plus un pointeur :

Code:

Coordonnees pointeur = {0,0};
19/01/2011, 12h05
quetzacoatl

Euh oui,
J'ai fait un copié collé et j'ai oublié de modifier cela, je pense qu'il n'est même pas la peine d'initialiser à 0, dans le cas où l'on est, on peut sans doute directement faire:

Code:

Coordonnees pointeur = {7,35};