Problème dans le code

**wauant** · 03/04/2007, 17h14

Bonjour j'ai un problème dans mon code, lorsque je compile il y a aucun problème par contre quand je l'execute il me met "segmentation fault".

Voici mon code :

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
#include <stdio.h>
#include <string.h>
 
char *temp1,*temp2;
void modif(char *,char *);
 
int main()
{
 temp1="lol";
 temp2="test";
 modif(temp1,temp2);
 return 0;
}
void modif(char *ar1,char *ar2)
{
  /* Déclarations : */
  /* Noms des fichiers et pointeurs de référence */ 
  FILE *INFILE, *OUTFILE;
  /* Autres variables */
  char NOM_PERS[30];
  /* Ouverture de l'ancien fichier en lecture */
  do
    {
     INFILE = fopen("powerdale.xml", "r"); 
    }
  while (!INFILE);
 
  /* Ouverture du nouveau fichier en écriture */
  do
    {
     OUTFILE = fopen("powerdale.xml.tmp", "w"); 
	}
  while (!OUTFILE);
  /* Traitement */
  /* Copie de tous les enregistrements en   */
  /* remplaçant l'enregistrement à modifier */
  /* par sa nouvelle valeur. */
  while (!feof(INFILE))
    {
     fscanf(INFILE, "%s\n", NOM_PERS);
     if (strcmp(NOM_PERS,ar1) == 0)
          fprintf(OUTFILE, "%s\n", ar2);
     else
          fprintf(OUTFILE, "%s\n", NOM_PERS);
    }
  /* Fermeture des fichiers */
  fclose(OUTFILE);
  fclose(INFILE);
}

Si je met en commentaire la ligne "modif(temp1,temp2);" ca marche sans problème mais bon ca fait pas ce qu eje veux

J'ai chercher l'origine de cette erreur et visiblement c'est quand on essaye d'accéder à une partie de la mémoire ou on ne peut accéder. Donc c'ets sans doute un problème de pointeurs
C'est sans doute une bête erreur mais j'ai jamais été très doué avec les pointeurs.
Merci d'avance

**femtosa** · 03/04/2007, 17h17

Tu utilises des pointeurs sans les initialiser ...

Avant de faire

Code :

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

1
2
temp1="lol";
temp2="test";

Tu dois réserver de l'espace mémoire ... !

Une façon simple permettant de masquer tes pointeurs serait de déclarer tes variables comme ceci :

Code :

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

1
2
char temp1[] = "lol";
char temp2[] = "test";

De plus

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
/* Ouverture de l'ancien fichier en lecture */
do
{
INFILE = fopen("powerdale.xml", "r"); 
}
while (!INFILE);
 
/* Ouverture du nouveau fichier en écriture */
do
{
OUTFILE = fopen("powerdale.xml.tmp", "w"); 
}
while (!OUTFILE);

Ne me paraît pas génial ... ! Un petit message permettant se prévenir que l'ouverture ne s'est pas bien dérouler serait certainement plus utilise qu'un boucle essayant indéfiniment d'ouvrir un fichier qui aura peu de chance d'être ouvert si ça ne marche pas du premier coup ...

**souviron34** · 03/04/2007, 17h20

c'est surtout qu'en assignant tes variables comme ça (var = "chaine"), elles deviennent non modifiables...

Et qu'ensuite tu appelles une fonction modif, qui je suppose (

) va les modifier.....

Donc tu peux soit faire comme on vient de te le dire, soit les allouer dynamiquement, copier les chaines dedans, et passer les pointeurs à la fonction...

**stephl** · 03/04/2007, 17h21

Envoyé par femtosa

Tu utilises des pointeurs sans les initialiser ...

Avant de faire

Code :

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

1
2
temp1="lol";
temp2="test";

Tu dois réserver de l'espace mémoire ... !

Non. C'est correct. Les pointeurs pointent maintenant vers les chaînes constantes "lol" et "test".

**femtosa** · 03/04/2007, 17h23

Envoyé par stephl

Non. C'est correct. Les pointeurs pointent maintenant vers les chaînes constantes "lol" et "test".

Mea culpa !

Réflexion trop rapide ... !

**stephl** · 03/04/2007, 17h26

Envoyé par femtosa

Mea culpa !

Réflexion trop rapide ... !

Pas de problème, ça m'arrive aussi. En revanche, je suis tout à fait d'accord avec votre remarque sur l'ouverture des fichiers.

**wauant** · 03/04/2007, 17h28

Envoyé par femtosa

Tu utilises des pointeurs sans les initialiser ...

Avant de faire

Code :

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

1
2
temp1="lol";
temp2="test";

Tu dois réserver de l'espace mémoire ... !

Une façon simple permettant de masquer tes pointeurs serait de déclarer tes variables comme ceci :

Code :

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

1
2
char temp1[] = "lol";
char temp2[] = "test";

Oui mais si je fais ca comme ca je n'ai plus des pointeurs c'est juste des variables et moi il me faut des pointeurs plus loin. En tout cas si je remplace les 2 lignes par ce que tu me suggère il me met la même erreur

Pour ce qui est de l'ouverture là d'accord je pourrait faire un petit message mais je crois pas que le problème se situe à ce niveau là

**femtosa** · 03/04/2007, 17h32

Envoyé par wauant

Oui mais si je fais ca comme ca je n'ai plus des pointeurs c'est juste des variables et moi il me faut des pointeurs plus loin.

Tu travaille toujours avec des pointeurs !

Si tu utilise ceci :

Code :

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

temp1

Tu travailles avec un pointeur sur le premier caractère de ta chaîne !

C'est équivalent à

Code :

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

&temp[0]

De plus, il y a un petit côté illogique dans ta façon de déclacer tes variables 'temp1' et 'temp2', car elles sont global, et tu les passes en paramètre ... Soit tu les déplaces dans le 'main', soit le passage de paramètre n'est pas requis (mais certainement mieux pour l'execrice

)

**wauant** · 03/04/2007, 17h36

Ha d'accord mais j'ai tjs le même problème je remet mon code en entier après les modifications.

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
#include <stdio.h>
#include <string.h>
 
 
void modif(char *,char *);
 
int main()
{
 char temp1[]="lol";
 char temp2[]="test";
 modif(temp1,temp2);
 return 0;
}
void modif(char *ar1,char *ar2)
{
  /* Déclarations : */
  /* Noms des fichiers et pointeurs de référence */ 
  FILE *INFILE, *OUTFILE;
  /* Autres variables */
  char NOM_PERS[30];
  /* Ouverture de l'ancien fichier en lecture */
  do
    {
     INFILE = fopen("powerdale.xml", "r"); 
	 if (!INFILE) 
         printf("\aERREUR: Impossible d'ouvrir "
                "le fichier: %s.\n", ar1);
    }
  while (!INFILE);
 
  /* Ouverture du nouveau fichier en écriture */
  do
    {
     OUTFILE = fopen("powerdale.xml.tmp", "w"); 
	 if (!OUTFILE) 
         printf("\aERREUR: Impossible d'ouvrir "
                "le fichier: %s.\n", ar2);
	}
  while (!OUTFILE);
  /* Traitement */
  /* Copie de tous les enregistrements en   */
  /* remplaçant l'enregistrement à modifier */
  /* par sa nouvelle valeur. */
  while (!feof(INFILE))
    {
     fscanf(INFILE, "%s\n", NOM_PERS);
     if (strcmp(NOM_PERS,ar1) == 0)
          fprintf(OUTFILE, "%s\n", ar2);
     else
          fprintf(OUTFILE, "%s\n", NOM_PERS);
    }
  /* Fermeture des fichiers */
  fclose(OUTFILE);
  fclose(INFILE);
}

**stephl** · 03/04/2007, 17h38

Etes-vous bien sûr que les mots contenus dans le fichier source ne dépassent pas la taille de votre buffer NOM_PERS? Chez moi, cela fonctionne correctement.

**wauant** · 03/04/2007, 17h41

ha oui c'était simplement ca

Je sentais l'erreur toute bête mais comme je suis pas un habitué de la programmation C je l'ai pas vue.
Merxi pour cette aide aussi rapide

**Emmanuel Delahaye** · 03/04/2007, 18h20

Envoyé par wauant

Bonjour j'ai un problème dans mon code, lorsque je compile il y a aucun problème par contre quand je l'execute il me met "segmentation fault".

Il faut vraiment revoir les bases du C et l'utilisation des fichiers...

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
 
#include <stdio.h>
#include <string.h>
 
void modif (char const *ar1, char const *ar2)
{
/* Ouverture de l'ancien fichier en lecture */
   FILE *INFILE = fopen ("powerdale.xml", "r");
   if (INFILE != NULL)
   {
/* Ouverture du nouveau fichier en écriture */
      FILE *OUTFILE = fopen ("powerdale.xml.tmp", "w");
      if (OUTFILE != NULL)
      {
/* Traitement */
/* Copie de tous les enregistrements en */
/* remplaçant l'enregistrement à modifier */
/* par sa nouvelle valeur. */
         char NOM_PERS[30];
 
         while (fgets (NOM_PERS, sizeof NOM_PERS, INFILE) != NULL)
         {
            if (strcmp (NOM_PERS, ar1) == 0)
               fprintf (OUTFILE, "%s\n", ar2);
            else
               fputs (NOM_PERS, OUTFILE);
         }
/* Fermeture des fichiers */
         fclose (OUTFILE);
      }
      fclose (INFILE);
   }
}
 
int main (void)
{
   char const *temp1 = "lol";
   char const *temp2 = "test";
   modif (temp1, temp2);
   return 0;
}

**Emmanuel Delahaye** · 03/04/2007, 18h22

Envoyé par femtosa

Tu utilises des pointeurs sans les initialiser ...

Avant de faire

Code :

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

1
2
temp1="lol";
temp2="test";

Tu dois réserver de l'espace mémoire ... !

Pourquoi ? Dans l'usage qui en est fait, c'est correct. Simplement, il faut des pointeurs 'const'. Donc :

Code :

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

1
2
char const *temp1="lol";
char const *temp2="test";

**femtosa** · 04/04/2007, 08h56

Envoyé par Emmanuel Delahaye

Pourquoi ? Dans l'usage qui en est fait, c'est correct. Simplement, il faut des pointeurs 'const'. Donc :

Code :

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

1
2
char const *temp1="lol";
char const *temp2="test";

Ok ! J'ai appris quelque chose, merci ! Je ne connaissais pas cette syntaxe !

**Thierry Chappuis** · 04/04/2007, 10h53

Envoyé par femtosa

Tu travaille toujours avec des pointeurs !

Si tu utilise ceci :

Code :

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

temp1

Tu travailles avec un pointeur sur le premier caractère de ta chaîne !

C'est équivalent à

Code :

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

&temp[0]

Il y a tout de même des différences entre un pointeur et un tableau. En effet, lorsque temp1 se trouve dans une expression, il y a bien convertion implicite en un pointeur sur le premier élément du tableau. Toutefois, le type de temp1 n'est pas pointeur sur le premier élément du tableau, et temp1 n'est pas une lvalue. Cela signifie qu'il est impossible de modifer sa valeur par une affectaction, contrairement à un pointeur. En clair, tu ne peux pas te servir de temp1 comme d'un pointeur dont tu peux modifier la valeur à ta guise pour, par exemple, parcourir un tableau de caractères dans une boucle. Ainsi, tu peux faire:

Code :

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

1
2
3
4
5
6
7
8
char const *temp1 = "chaine1";
char *pc = NULL;
size_t len;
 
for (pc = temp1, len = 0; *pc != 0; pc++)
{
    len++;
}

mais pas

Code :

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

1
2
3
4
5
6
7
char const *temp1 = "chaine1";
size_t len = 0;
 
while (*temp1++ != 0)
{
    len++;
}

La conversion implicite d'un tableau en pointeur n'a par exemple pas lieu lorsque le tableau en question est passé à l'opérateur sizeof. En effet:

Code :

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

1
2
3
4
char s_chaine[] = "Bienvenue sur developpez!"
char *pc = s_chaine
printf("sizeof s_chaine vaut %u\n", sizeof s_chaine);
printf("sizeof pc vaut %u\n", sizeof pc);

En fait, l'équivalence entre un tableau et un pointeur a essentiellement lieu au niveau de l'arithmétique des pointeurs. Soit les définitions suivantes:

Code :

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

1
2
char s_chaine[] = "chaine1";
int i = 2;

On a

Code :

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

1
2
3
4
5
6
7
s_chaine[i] = 'A';
/* équivalent à */
*(s_chaine + i);
/* équivalent à */
*(i + s_chaine) = 'A';
/* équivalent à */
i[s_chaine] = 'A';        /* étrange, mais néamoins correct! */

Un tableau est également converti implicitement en pointeur lors du passage en argument d'une fonction. Ainsi, le code suivant est toujours valable:

Code :

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

1
2
3
4
5
6
7
8
size_t m_strlen(char const chaine[])
{
    while (*chaine++ != 0)
    {
        len++;
    }
    return len;
}

En résumé, il y a équivalence entre les pointeurs et les tableaux dans certaines situations. Il s'agit toutefois de rester conscient des différences. Pour en savoir plus:
http://c-faq.com/aryptr/aryptr2.html
http://c-faq.com/aryptr/aryptrequiv.html
http://c-faq.com/aryptr/aryptrparam.html

Thierry

**femtosa** · 04/04/2007, 11h05

Que de compléments

Merci beaucoup, y finalement encore du chemin pour connaître toutes les petites subtilités !

**stephl** · 04/04/2007, 11h08

Pourquoi ceci est-il correct?

Code :

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

1
2
3
4
char s_chaine[]="chaine1";
int i=2;
 
i[s_chaine]='A';

Ca compile mais je ne comprends pas du tout pourquoi?!?!

**DaZumba** · 04/04/2007, 11h11

Envoyé par stephl

Ca compile mais je ne comprends pas du tout pourquoi?!?!

Parce qu'addition est commutative.
Tu as

Code :

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

1
2
 
s_chaine[i] = *(s_chaine + i) = *(i + s_chaine) = i[s_chaine]

A part pour la frime, cela ne sert pas a grand'chose.

**stephl** · 04/04/2007, 11h20

Envoyé par DaZumba

Code :

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

1
2
 
*(s_chaine + i) = *(i + s_chaine)

Merci. OK pour cela, mais je ne pensais pas que mettre des [] après un entier était permis. Je ne crois pas que ce soit dans K&R. D'où cela vient-il?

EDIT
Je n'ai rien dit

...

Envoyé par K&R2

The array subscripting operation is defined so that E1[E2] is identical to *(E1+E2). Therefore,
despite its asymmetric appearance, subscripting is a commutative operation.

**DaZumba** · 04/04/2007, 11h47

Envoyé par stephl

Merci. OK pour cela, mais je ne pensais pas que mettre des [] après un entier était permis. Je ne crois pas que ce soit dans K&R. D'où cela vient-il?

La notation

Code :

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

1
2
 
array[i]

est du "sucre syntaxique". Pour le compilateur, l'acces a un element du tableau se fait toujours par l'arithmetique des pointeurs

Code :

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

1
2
 
*(array + i) /* *(array + i *sizeof *array) en arithmetique normale */

Le compilateur voit donc toujours

Code :

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

1
2
 
i[array]

comme

Code :

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

1
2
 
*(i + array)

C'est donc syntaxiquement correct et la norme n'a aucune raison d'interdire cette notation.

Problème dans le code

C

Discussions similaires

Partager

Partager