Discussion :

[mathieumadrid]

Bonjour,

Je veux récupérer un texte passé par la commande en le stockant dans une variable titre (avec char* titre; déclaré avant) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
printf("Titre : ");
	scanf("%s",titre);

Ensuite, je veux récupérer un autre texte de la commande que je mets dans numero. Par la suite, je veux travailler sur numero. Voici le code complet de ma fonction :

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
35
36
37
38
39
40
41
42
43
44
45
46
MEDIA* ecrire()
{
	MEDIA* media;
	char* titre;
	char* numero;
	int support;
	int numeroGlobal[3];
	int type;
	int numeroType[2];
	int jour[2];
	int mois[2];
	int annee[2];
	DATE date;
 
	printf("Titre : ");
	scanf("%s",titre);
	printf("Media : ");
	scanf("%s",numero);
 
	if(strlen(numero) != 13)
		return NULL;
 
	sscanf(numero++,"%1i",&support);
	sscanf(numero++,"%1i",numeroGlobal);
	sscanf(numero++,"%1i",&numeroGlobal[1]);
	sscanf(numero++,"%1i",&numeroGlobal[2]);
	sscanf(numero++,"%1i",&type);
	sscanf(numero++,"%1i",numeroType);
	sscanf(numero++,"%1i",&numeroType[1]);
	sscanf(numero++,"%1i",jour);
	sscanf(numero++,"%1i",&jour[1]);
	sscanf(numero++,"%1i",mois);
	sscanf(numero++,"%1i",&mois[1]);
	sscanf(numero++,"%1i",annee);
	sscanf(numero,"%1i",&annee[1]);
 
	date.jour[0] = jour[0];
	date.jour[1] = jour[1];
	date.mois[0] = mois[0];
	date.mois[1] = mois[1];
	date.annee[0] = annee[0];
	date.annee[1] = annee[1];
	media = creation(titre,support,numeroGlobal,type,numeroType,date);
 
	return media;
}

Le problème, c'est que quand j'appelle la fonction j'ai une erreur tandis que si j'enlève la lecture de titre alors ça marche !

Merci de m'éclairer !

[DaZumba]

Je veux récupérer un texte passé par la commande en le stockant dans une variable titre (avec char* titre; déclaré avant) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
printf("Titre : ");
	scanf("%s",titre);

Tu n'as pas assez de place pour stocker une chaine de caractere dans une variable de type pointeur vers integer (ton char *titre). Du coup, ca plante. Pour resoudre le probleme, on fait de la place:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
char titre[100];

qui peut contenir un titre d'au plus 99 caracteres + le '\0' final.

[Gruik]

Héhé l'erreur courante des gens qui viennent des langages evolués et croient que "char *" = "variable de type chaine"

[Thierry Chappuis]

Comme te l'a mentionné DaZumba, il te faut réserver l'espace destiné à recevoir la chaine titre. L'expression char *titre; définit un pointeur sur caractère et non un tableau de caractère. Il te faut réserver cet espace soit sur la pile en définissant titre comme un tableau de caractère, soit sur le tas en utilisant l'allocation dynamique de mémoire (qui n'est probablement pas nécessaire ici). Tu dois donc définir comme l'a dit DaZumba:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

char titre[100]; /*Peut contenir une chaine de 99 caractères + '\0' terminal*/

Dans le contexte de cette saisie utilisateur, je trouve que fgets est plus adaptée que scanf:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

fgets(titre, sizeof titre, stdin);

En effet, dans ce cas, quelque soit la chaine de caractère entrée par l'utilisateur, fgets ne stocke que les sizeof titre - 1 = 99 caractères dans titre et ajoute le caractère nul. C'est plus sécurisé.

Thierry

[mathieumadrid]

Merci pour vos réponses.