Discussion :

[NeMo_O]

bonjour je suis débutant total je programme depuis quelques semaines

mon problème est simple : je désire demander à l'utilisateur un nom de fichier que je stocke dans une variable de type char par l'intermédiaire de scanf.

- si le nom de fichier est <= à 4 caractères il n'y a aucun problème et je peux accéder au fichier par fopen()
- par contre pour plus de 4 caractère une erreur de segmentation se produit

J'aimerais donc savoir comment résoudre ce problème pour n'être pas limité en nombre de caractères.

Merci

[coca25]

Le type char correspond à un caractère, si tu veux récupérer un nom de fichier, c'est une chaine de caractère qu'il te faut. En C ça correspond à un tableau de char:

Code :

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

1
2
 
char nom_fichier[TAILLE];

Le fait que ca te fasse pas d'erreur quand c'est inférieur à 4 caractères, c'est une chance.

[NeMo_O]

salut coca25 merci à toi j'ai compris.

mais le problème se pose quand je désire avoir accés au fichier :
je ne peux pas taper directement :

fopen(nom_fichier, " ... " );

enfin je pense car les caractères sont stockés un par un dans des cases différentes du tableau.

merci en tout cas

[coca25]

j'ai pas compris, quel problème se pose???
http://c.developpez.com/faq/c/?page=...NGS_definition

chaine de caractere = tableau de caractere + caractere de terminaison

[Emmanuel Delahaye]

mon problème est simple : je désire demander à l'utilisateur un nom de fichier que je stocke dans une variable de type char par l'intermédiaire de scanf.

- si le nom de fichier est <= à 4 caractères il n'y a aucun problème et je peux accéder au fichier par fopen()
- par contre pour plus de 4 caractère une erreur de segmentation se produit

J'aimerais donc savoir comment résoudre ce problème pour n'être pas limité en nombre de caractères.

Montre ton code. J'ai une boule de cristal, mais je l'ai eu d'occase sur ebay et elle est fêlée...

[NeMo_O]

Ok déja j'explique le but de ma fonction : je désire juste ouvrir un fichier dont l'utilisateur rentre son nom au clavier ( pour ensuite le lire, y faire différentes modifications,...)

Voici le code qui me semble contenir pas mal d'erreurs lol :

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
 
#include <stdio.h>
 
int main() {
  char pn;
  FILE *fp;
 
     printf("entrez le nom du fichier à ouvrir :\n");
     scanf("%s",&pn);
 
   fp=fopen(pn,"r");
   // code
 
   fclose(fp);
 
return 0;
}

Merci pour votre aide

[granquet]

dans pn, tu as la place pour 1 (et 1 SEUL, pas plus) caractére.
il ne faut utiliser un TABLEAU de caractere.

relis ton cours//livre de C.

[kamouminator]

tu as declaré:

Code :

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

char pn;

alors qu'il faut declarer une chaine de caractere:

Code :

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

char pn[MAX + 1];

MAX etant la taille maximum de ta chaine
+1 pour le caractére de fin de chaine.

ensuite tu utilise scanf()
c'est deconceillé, parce que si l'utilisateur tape plus de MAX caractére le comportement du programme deviens indefini car il y a debordement
il faut utiliser fgets()

http://man.developpez.com/man3/fgets.3.php


sinon tu peux allouer dinamiquement ton tableau (fonctions malloc/realloc) et utiliser alors getchar() pour la saisieet remplacer le '\n' par '\0' (caractere de fin de chaine)

[Emmanuel Delahaye]

tu as declaré:

Terminologie : défini.

http://emmanuel-delahaye.developpez....tm#definitions

alors qu'il faut declarer une chaine de caractere:

Code :

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

char pn[MAX + 1];

Bzzt ! Ceci est un tableau de char non initialisé. Ce n'est pas une chaine de caractères.

sinon tu peux allouer dinamiquement ton tableau (fonctions malloc/realloc) et utiliser alors getchar() pour la saisieet remplacer le '\n' par '\0' (caractere de fin de chaine)

L'embrouille pas, le pauvre...

[zooro]

Terminologie : défini.
...
Bzzt ! Ceci est un tableau de char non initialisé. Ce n'est pas une chaine de caractères.
...
L'embrouille pas, le pauvre...

kamouminator, vous êtes le maillon faible. Au revoir !

[NeMo_O]

Salut à tous merci pour vos réponses je ne m'attendais pas à en avoir autant en si peu de temps lol.

Oui kamouminator, le problème est que l'on peut dépasser max mais je verrais plus tard pour malloc déja j'essaye de faire fonctionner ceci :

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
 
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
 
int main() {
 
char pn[10];
FILE *fp;
 
  printf("Entrez le nom du fichier:\n");
  scanf("%s",pn);
 
  *fp=fopen(pn,"w");
 
  fclose(fp);
 
return 0;
}

donc la chaine de caractère entrée est bien stockée dans le tableau pn mais lors de la compilation l'erreur se produit lors de l'appel a fopen : " types incompatibles ".
Si je met des guillemets autour de "pn" il prendra ca non plus comme une variable mais si je ne met pas de guillemet il y a une erreur :s

j'aimerais bien savoir comment faire lol .

Merci

( ensuite une fois que je saurais faire ça j'essaierai avec malloc pour minimiser la place prise en mémoire )

[coca25]

je crois qu'il faut revoir les bases du C.
fopen retourne un pointeur qu'il faut d'ailleurs tester:
FILE *fopen(const char *path, const char *mode);

donc:

Code :

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

1
2
 
  fp=fopen(pn,"w");

[NeMo_O]

Ouf merci beaucoup coca25 lol t'a raison je devrais mieux lire les syntaxes des fonctions avant de poster des messages lol ( enfin c'est surtout que les pointeurs c'est encpre un petit peu confus ).

Merci en tout cas ça marche je vais pouvoir enfin travailler dans mon fichier texte maintenant

[Emmanuel Delahaye]

Code :

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

1
2
3
 
FILE *fp;
  *fp=fopen(pn,"w");

Horrible !

Code :

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

1
2
3
4
5
6
7
8
9
 
   FILE *fp;
 
   fp = fopen(pn,"w");
   if (fp != NULL)
   {
      /* tu fais tes petites affaires avec le fichier... */
      fclose (fp);
   }

ou (je préfère)

Code :

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

1
2
3
4
5
6
7
 
   FILE *fp = fopen(pn,"w");
   if (fp != NULL)
   {
      /* ... */
      fclose (fp);
   }

Il faut cesser de programmer au hasard, et commencer à se former sérieusement. Le langage C n'est pas un jouet. Il demande discipline et respect.

J'ai dit.

[Médinoc]

Un code "de base" pour demander le nom d'un fichier et ouvrir ledit fichier:

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
#include <stdio.h>
#include <string.h>
 
/*
 * Fonction pour retirer le \n d'une ligne saisie et vider la ligne du flux.
 */
void PurgeLineAndStream(char *lineBuf, FILE *stream)
{
	/* Chercher le \n. */
	char *pLF = strchr(lineBuf, '\n');
	if(pLF != NULL)
	{
		/* Le \n est dans la ligne, on le supprime. */
		*pLF = '\0';
	}
	else
	{
		/* Le \n n'est pas dans la ligne, il est donc encore dans le flux.
		   On vide donc le flux jusqu'à trouver le \n. */
		int c;
		while((c=fgetc(stream)) != '\n' && c != EOF)
		{}
	}
}
 
 
int main(void)
{
	char pn[10];
	FILE *fp = NULL;
 
	printf("Entrez le nom du fichier:\n");
	fgets(pn, sizeof(pn), stdin);
	PurgeLineAndStream(pn, stdin);
 
	printf("Ouverture de \"%s\"...\n", pn);
	fp = fopen(pn,"w");
	if(fp != NULL)
	{
		/* ... */
		fclose (fp);
	}
 
	return 0;
}

Ce code empêche une saisie trop longue de mettre en péril le programme, puis ouvre le fichier en vérifiant le retour...

[kamouminator]

kamouminator, vous êtes le maillon faible. Au revoir !

ok, je
^^

[NeMo_O]

merci à tous j'ai fait un pti mix de vos réponses et ca marche nickel maintenant je peux passer à l'étape suivante

PS : merci pour le script Médinoc

[Emmanuel Delahaye]

merci pour le script Médinoc

Pas de script en C. Du code source...