Problème dans le code

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

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, 16h17

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, 16h20

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, 16h21

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, 16h23

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, 16h26

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, 16h28

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, 16h32

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, 16h36

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, 16h38

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.

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

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, 10h05

Que de compléments

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

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

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?!?!

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

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, 07h56

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 !

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

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;
}

Problème dans le code

C

Vue hybride

Discussions similaires

Partager

Partager