Discussion :

[totofe]

Bonjour,

Lors de mon programme suis heurté à un problème que je comprend pas.
Mon programme fonctionne bien sous windows et que je le lance sous linux je rencontre le probleme suivant.

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
 
int lgMax[16]={30,6,100,100,40,20,40,40,100,100,100,100,100,10,40,40};
 
for(i=0;i<zone;i++)
    {
      strcpy(champ,&ligne[position[i]]); 
      lgMax[0]=30; //remet la valeur a 30 sinon bug... je c'est pas pourquoi
      if(strlen(champ)>lgMax[i])
      {
        strcpy(champ," ");
      }
      if(i==2)
        printf("%d ",lgMax[0]);//affiche 30
      while(strlen(champ)<lgMax[i]+1)
        strcat(champ," ");
        //lgMax bug après la boucle quand i=2
      if(i==2)
      {
        printf("%d ",lgMax[0]);//affiche 0
        exit(0);
      }
      fwrite (champ , 1 , lgMax[i] , fDest );
    }

Je comprend pas pourquoi dans mon for quand i=2 ca remet ma valeur lgMax[0] à 0 (les autres valeurs du tableau ne change pas)
Quelqu'un pourrait m'orienter sur le pourquoi de ce problème?

merci

[3DArchi]

Intuitivement, je dirais que ton premier strcpy déborde et vient écraser le tableau lgMax.
Tu dois avoir quelque par un char champ[TAILLE_MAX]. Tu devrais donc utiliser quelque chose comme ça:

Code :

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

strncpy(champ,&ligne[position[i]],TAILLE_MAX);

[Médinoc]

Sans la déclaration de champ, ça ne va pas être facile...

Mais ça sent le débordement de buffer, donc surveille un peu mieux ce que tu fais avec champ, et regarde si tu mets une variable entre les deux, si c'est elle qui est modifiée à la place...

[Emmanuel Delahaye]

Lors de mon programme suis heurté à un problème que je comprend pas.
Mon programme fonctionne bien sous windows et que je le lance sous linux je rencontre le probleme suivant.

Poste un code compilable. Il est impossible d'analyser ton code. Trop d'inconnues.

[totofe]

Poste un code compilable. Il est impossible d'analyser ton code. Trop d'inconnues.

désolé, je voulais pas copier le code enterrement, car il est plutôt immense.
j'ajoute les différentes variables, ainsi qu'un bou de code réalisé avant

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
58
59
60
61
 
int zone;
  int i,t;
  char ligne[1024];
  int position[16];
  int lgMax[16]={30,6,100,100,40,20,40,40,100,100,100,100,100,10,40,40};
  int pos; 
  char champ[101];
 
 if ((fDest = fopen(nomdest, "w")) == NULL)//Créé le fichier fixe test si erreure
  {
    sprintf(txterr,"impossible d'ouvrir le fichier %s \n",nomdest); 
    Ecriture_err();
    return(-2);
  }
 
  nb_ligne=0;
 
  while(fgets(ligne,sizeof(ligne), fic_header)) //compte nombre de ligne sans le header et ajout des data dans le fichier fixe
  {
    nb_ligne++;
    trimCRLF(ligne);
    //Ecriture
 
    t=strlen(ligne);
    zone=1;
    position[0]=0;
    for(i=0;i<t;i++)//remplace les ; par des \0
    {
      if(ligne[i]==';')
      { 
        ligne[i]=0;       
        position[zone++]=i+1; //donne la position de debut de chaque champs
      }                       
    }
    if (zone>16)// + de 15 champs
    {
      log=1;
      sprintf(txterr,"Attention il existe au moins un champ en trop à la ligne %d \n",nb_ligne+1);
      Ecriture_err();
      continue;
    }
for(i=0;i<zone;i++)
    {
      strcpy(champ,&ligne[position[i]]); 
      lgMax[0]=30; //remet la valeur a 30 sinon bug... je c'est pas pourquoi
      if(strlen(champ)>lgMax[i])
      {
        strcpy(champ," ");
      }
      if(i==2)
        printf("%d ",lgMax[0]);
      while(strlen(champ)<lgMax[i]+1)
        strcat(champ," ");
      if(i==2)
      {
        printf("%d ",lgMax[0]);
        exit(0);
      }
        //lgMax bug après la boucle quand i=2
    }

désolé je peux pas poster de code compilable à moins de mettre tout mon programme.

[BainE]

j aime bien le titre, si c'est un bug c'est pas un problème ?

[Emmanuel Delahaye]

désolé, je voulais pas copier le code enterrement, car il est plutôt immense.
j'ajoute les différentes variables, ainsi qu'un bou de code réalisé avant
<...>
désolé je peux pas poster de code compilable à moins de mettre tout mon programme.

Alors il faut apprendre à réduire me code au minimum compilable qui montre le problème, car ce n'est pas à nous de le faire. C'est d'ailleurs une technique de mise au point bien connue et très efficace.

D'une manière plus générale, ton code devrait être organisé en modules indépendants et testables individuellement. Ca fait partie des règles de Bonne Conception.

[totofe]

J'ai trouvé.
C'était ma variable champ qui débordé étrange car elle ne pouvait pas dépassé 100 (valeur max dans lgMax). et ce problème ne se rencontré que sur linux...
j'ai augmenté la taille du tableau de 1 et ça fonctionne.

[Emmanuel Delahaye]

J'ai trouvé.
C'était ma variable champ qui débordé étrange car elle ne pouvait pas dépassé 100 (valeur max dans lgMax). et ce problème ne se rencontré que sur linux...
j'ai augmenté la taille du tableau de 1 et ça fonctionne.

Non. Le problème se rencontre partout. Simplement comme le dépassement de tableau entraine un comportement est indéfini, les conséquences sont imprévisibles. Il peut arriver n'importe quoi, y compris un comportement conforme aux attentes, ce qui fait que ce bug (comportement indéfini) est particulièrement dangereux et fait que le C n'est pas un langage de débutant.

http://emmanuel-delahaye.developpez.com/notes.htm#ub


.

[3DArchi]

J'ai trouvé.
C'était ma variable champ qui débordé étrange car elle ne pouvait pas dépassé 100 (valeur max dans lgMax). et ce problème ne se rencontré que sur linux...
j'ai augmenté la taille du tableau de 1 et ça fonctionne.

Donc, dès que tu vas lire une ligne de 101 caractères