Discussion :

[tuba uba]

Bonjour Madame, Monsieur,

Pour un projet je dois créer un système de management de banque. Pour cela je veux créer un fichier où on peut enregistrer les noms. Mon programmation reste très basique. Dès que j'essaie d'enregistrer sur le fichier cela pose problème.Pouvez vous corriger le problème? Merci

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
47
48
49
50
51
52
53
54
55
56
57
 # include <stdlib.h>
# include <stdio.h>
# include <tgmath.h>
# define  NB_ELEM
 
 
void affichage(int n , float x[n]){  // initialisation de tableau 
  int i;
    for(i = 0;i<n; i++){
    x[i]=0;
    printf(" la valeurs de x[%d] = %f\n",i,x[i]);
    };
    return;
};
 
 
void changer_fichier(int N2, float x1[NB_ELEM]){  // création de fichier 
  FILE *fp;
  int i = 0;
 
  fp = fopen("/media/Tonny/TOSHIBA EXT/programmation_c/exercice1/test1.txt","w");
  if (fp = NULL){
    printf("le fichier n'existe pas ");
  };
 
  for(i = 0; i<N2; i++){
  fprintf(fp,"%f", x1[i]);
  };
  return;
};
 
 
 
int main (void){
  int n,i;
  n  = 0;
  i = 0;
  printf ("saisir une valeur");
  scanf("%d",&n);
  printf("valeur saisi est égale : %d", n);
  float x1[n];
  float x2[n];
 
 
  for(i = 0; i<n;i++){
    x2[i]=0;
  };
 
  printf("%f",x2[n]);
  affichage(n,x1);  
  changer_fichier(n,x2); // enregistrer dans le fichier 
 
 
 
  exit(EXIT_SUCCESS);
 
};

erreur :
valeur saisi est égale : 10 la valeurs de x[0] = 0.000000
la valeurs de x[1] = 0.000000
la valeurs de x[2] = 0.000000
la valeurs de x[3] = 0.000000
la valeurs de x[4] = 0.000000
la valeurs de x[5] = 0.000000
la valeurs de x[6] = 0.000000
la valeurs de x[7] = 0.000000
la valeurs de x[8] = 0.000000
la valeurs de x[9] = 0.000000

Erreur de segmentation (core dumped)

[Xelland]

bonjour

Code :

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

# define  NB_ELEM

=> tu ne définies pas la valeur
alors quand il arrive à

Code :

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

float x1[NB_ELEM]

les "return" dans les void ne sont pas nécessaires


je n'ai pas regardé plus loin s'il y avait d'autres erreurs

[dalfab]

Bonjour,

ligne 49, tu écris printf("%f",x2[n]);. A cet instant combien vaut n selon toi? Le programme pourtant ne plante pas ici.
mais le problème est ligne 22. Le compilateur as-tu d'avertir. Tu écris if (fp = NULL), c'est un test il faut mettre un double égal ==.

[Sve@r]

Bonjour

Pouvez vous corriger le problème? Merci

On peut te dire où sont les problèmes, à toi de les corriger.
Donc float x[n] dans les paramètres des fonctions (valeur de n ???). Ecrire x[i]=0 juste avant de l'afficher... Ta fonction "affichage" a pour commentaire "initialisation" ??? Et tu testes si fp vaut NULL (en fait c'est pas un test comme le dit dalfab mais une affectaion) mais s'il vaut NULL tu continues quand-même à remplir un fichier qui n'a pas été ouvert.

[tuba uba]

Merci pour aide rapide. Vous avez raison c'est affectation qui était responsable du message erreur. Même si je n'arrive pas remplir comme je le souhaite le fichier texte. Au moins les erreurs ont été corrigé. Je vais continuer à améliorer mon programme. Pour ce qu'est la valeur de n, j'ai toujours pris n=10. Ce qui m'a dérouter un peu c'est quand je compilait,il n'y avait pas erreur. C'est au moment exécution j'ai eu cette erreur.

[Sve@r]

Pour ce qu'est la valeur de n, j'ai toujours pris n=10.

Il te faut revoir les notions de visibilité des variables. Une variable n située dans un bloc A ne sera pas la même chose qu'une variable n située dans un bloc B. De plus un tableau float x[n] en tant que paramètre (alors que n n'existe pas) je n'arrive même pas à comprendre comment ça compile et ce que le compilateur peut y comprendre (peut-être y voit-il un VLA?). Idem pour le #define NB_ELEM sans valeur (ceci dit essayer d'afficher ensuite NB_ELEM d'une quelconque façon entraine une erreur de compilation).

Ce qui m'a dérouté un peu c'est quand je compilait,il n'y avait pas erreur. C'est au moment exécution j'ai eu cette erreur.

La compilation c'est juste une traduction de syntaxe. Elle ne remplacera pas la réflexion. Si tu écris dans une syntaxe correcte un algo incorrect, ça compilera. Et en plus le compilateur ne te préviendra même pas si tu dépasses les limites (exemple si tu tapes dans l'indice 15 d'un tableau de 10 éléments) car la philisophie du C c'est "pour aller le plus vite possible je ne vérifie rien, c'est au programmeur de vérifier ce qu'il fait" (enfin les compilos modernes avec les options adéquates peuvent détecter certaines incohérences mais quoi qu'il arrive ils ne sont pas là pour remplacer ton cerveau)

[WhiteCrow]

Il te faut revoir les notions de visibilité des variables. Une variable n située dans un bloc A ne sera pas la même chose qu'une variable n située dans un bloc B. De plus un tableau float x[n] en tant que paramètre (alors que n n'existe pas) je n'arrive même pas à comprendre comment ça compile et ce que le compilateur peut y comprendre (peut-être y voit-il un VLA?). Idem pour le #define NB_ELEM sans valeur (ceci dit essayer d'afficher ensuite NB_ELEM d'une quelconque façon entraine une erreur de compilation).
[...]

Dans le cas de void changer_fichier(int N2, float x1[NB_ELEM]) on obtient le prototype void changer_fichier(int N2, float x1[]) qui est tout à fait valide et équivalent à void changer_fichier(int N2, float *x1).

Dans le cas void affichage(int n , float x[n]) c'est la notation VLA et non un VLA qui est utilisé. n est bien défini auparavant en tant qu'int, il n'y a donc pas de soucis. Cette utilisation de la notation VLA sera d'ailleurs recommandée dans la prochaine norme C23 afin d'améliorer entre autre la lisibilité.
En l'état void affichage(int n , float x[n]) est strictement équivalent à void affichage(int n , float x[]).

[Sve@r]

En l'état void affichage(int n , float x[n]) est strictement équivalent à void affichage(int n , float x[]).

Joli

[gijy]

avec float X1[n] et float x2[n] tu veux sans doute vouloir créer dynamiquement tes tableaux; or avec cette ecriture c'est à la compilation que la zone memoire est allouée. n n'etant connu lors de la compile.....
Tu as donc toutes les chances d'avoir une taille de tableau farfelue et donc de taper dans une zone non allouée à ton prog d'où l'erreur de segmentation à l'execution.
Pour gérer dynamiquement des tailles de tableaux en C tu doit passer par les fonctions malloc and co et de travailler avec des pointeurs.

Pour le reste je suis d'accord avec les reponses NB_ELEM non definie, init du tableau dans la fonction d'affichage!!! etc...

[Sve@r]

avec float X1[n] et float x2[n] tu veux sans doute vouloir créer dynamiquement tes tableaux; or avec cette ecriture c'est à la compilation que la zone memoire est allouée. n n'etant connu lors de la compile.....

Ce n'est plus vrai depuis qu'existent les VLA (Variable Length Array). Avec ce nouvel outil, tu peux créer un tableau dynamique non connu au moment de la compilation
Exemple

Code c :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <stdio.h>
 
int main() {
	// Saisie taille
	int n;
	fputc('?', stdout);
	scanf("%d", &n);
	printf("n=%d\n", n);
 
	// VLA
	int tab[n];
	for (int i=0; i < n; i++) tab[i]=i*i;
	for (int i=0; i < n; i++) printf("tab[%d]=%d\n", i, tab[i]);
}

Ceci dit personnellement je n'aime pas car si le tableau ne peut pas être alloué, tu n'as aucun moyen de le détecter avant remplissage (et là... ). Je préfère donc en rester au bon vieux malloc dans lequel tu peux tester NULL pour vérifier la réussite de l'allocation avant de continuer le travail.