Discussion :

[kayenne77]

bonjour,

Quelqu'un pourrait m'expliquer cette boucle qui contient un super pointeur svp?
car lorsque je désalloue un chainon via la fonction oteN, le compilateur me mets un message d'erreur à la fin du programme.

"Exception non gérée à 0x003514f7 dans SKI.exe*: 0xC0000005: Violation d'accès lors de la lecture de l'emplacement 0x00000014."

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
for ( ; (*spN); spN=&(*spN)->psN)
	{
		if ((*spN)->nbi==0)
		{
			oteN(spN);
		}
		else
		{
			printf ("%-50s : %d inscrits\n", (*spN)->nat, (*spN)->nbi);
		}
 	}
}
	void oteN(NAT **spN)
	{
		NAT *n;
		n = *spN;
		(*spN)=n->psN;
		free (n->nat);
		free (n);
	}

[Invité(e)]

Bonjour,

Tu passes au chainon suivant de deux manières différentes dans ton code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
for (; (*spN); spN = &(*spN)->psN)
  {
    if ((*spN)->nbi == 0)
      {
        oteN (spN);
      }
    else
      {
        printf ("%-50s : %d inscrits\n", (*spN)->nat, (*spN)->nbi);
      }
  }

Code :

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

1
2
3
4
5
6
7
8
9
void oteN (NAT ** spN)
{
  NAT *n;
  n = *spN;
  /* manque un & ici */
  (*spN) = n->psN;
  free (n->nat);
  free (n);
}

Code :

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

1
2
3
4
5
6
7
8
void oteN (NAT ** spN)
{
  NAT *n;
  n = *spN;
  (*spN) = &n->psN;
  free (n->nat);
  free (n);
}

Cela dit, si tu passes au chainon suivant à la fois dans l'instruction for et dans la fonction oteN, tu risques d'avoir des surprises...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
for (; (*spN); )
  {
    if ((*spN)->nbi == 0)
      {
        oteN (spN);
        /* oteN est passé au chainon suivant */
      }
    else
      {
        printf ("%-50s : %d inscrits\n", (*spN)->nat, (*spN)->nbi);
        /* on passe au chainon suivant */
        spN = &(*spN)->psN;
      }
  }