Discussion :

[darkman19320]

Bonjour tout le monde!

J'ai un problème que je n'arrive pas à m'expliquer.

J'ai une classe A qui contient un pointeur sur une classe B. Cette classe B contient un pointeur sur la classe A...

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
 
class A
{
 B* m_bPtr;
public:
  A() {
    B(this);
  }
};
 
class B
{
  A* m_aPtr;
public:
  B(A* aPtr): m_aPtr(aPtr) {}
};

Mon problème est le suivant:

Au cours de l'exécution de mon code, pendant une opération qui ne touche ni à la classe A ni à la classe B, le pointeur de la classe A situé dans la classe B change sans aucune raison.

Sans parler d'un problème (évident?) de conception, je ne comprends ce genre de comportement.

Avez-vous des idées?

Merci d'avance.

[Rewpparo]

Code :

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

1
2
3
4
5
6
7
8
9
 
class A
{
 B* m_bPtr;
public:
  A() {
    B(this);
  }
};

Déja là il y a un problème, je ne sais pas si c'est lié. Tu mets en pointeur vers A (this) dans un pointeur vers B, alors que A n'hérite pas de B.
Sinon le reste du code ne permet pas de voir d'où vient le problème. Tu dois avoir quelque part un bout de code qui change ton pointeur et qui est appelé de manière intempestive.
Si vraiment tu es perdu, mets un breakpoint dans ton compilo à tous les endroits qui pourraient changer le pointeur.

[darkman19320]

Autant pour moi je me suis trompé quand j'ai recopié la classe A.

Voici la bonne version:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
class A
{
{
 B* m_bPtr;
public:
  A() {
    m_bPtr = new B(this);
  }
};

[gbdivers]

Déja là il y a un problème, je ne sais pas si c'est lié. Tu mets en pointeur vers A (this) dans un pointeur vers B, alors que A n'hérite pas de B.
Sinon le reste du code ne permet pas de voir d'où vient le problème. Tu dois avoir quelque part un bout de code qui change ton pointeur et qui est appelé de manière intempestive.
Si vraiment tu es perdu, mets un breakpoint dans ton compilo à tous les endroits qui pourraient changer le pointeur.

Attention, lis bien son code. Il appelle le constructeur de B qui prend un pointeur sur A comme paramètre. Donc pas de problème d'héritage.

@darkman19320
A priori, pas de problème normalement. Peux tu faire un code compilable qui reproduit le problème ?

[darkman19320]

Voici un exemple en pièce jointe.

Il contient les deux classes A et B, et un main.
Il y a également un script CMake.

[koala01]

Salut,

Bon, il y a bien quelques aberrations, comme

Code :

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

1
2
3
4
void A::setValue(int value)
{
  m_bPtr->m_aPtr->value = value; //completement absurde mais en gros c'est ce qui est fait....
}

ou

Code :

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

1
2
3
4
5
  for (int i= 0; i<1000000; ++i)
    {
      v.push_back(i+i);
      a.setValue(0);
    }

mais, a priori, il n'y a strictement aucune raison pour que les adresses de tes pointeurs ne soient modifiées de manière aléatoire

A l'instar de l'agent scully, je dirais que "la vérité est ailleurs"

Ceci dit, les problèmes liés aux changement impromptus d'adresse des pointeurs sont souvent liés à des accès en écriture "hors écart" sur des tableaux gérés à la main (exemple, tenter de modifier tab[N] alors que le tableau est prévu pour ne contenir que N éléments (car les indexes valides vont de 0 à N-1 ), voir à un accès en écriture sur un tableau non alloué

Si je peux te donner un conseil : commence déjà par remplacer toutes tes gestions de tableau dynamique C style (chaque fois que tu fais un Type * t = new Type[XXX] ) par des tableaux C++ : la classe vector, disponible dans l'espace de noms std par simple inclusion du fichier d'en-tete <vector> est faite pour cela

Dans le meme ordre d'idée, abandonne les chaines de caractères C style ( char * str) au profit des chaines de caractères C++ : la classe string, elle aussi disponible dans l'espace de noms std (mais par inclusion du fichier d'en-tête <string> ) est elle aussi faite pour cela

Si ces premières actions ne résolvent pas ton problème, vérifie que tous les accès aux différents tableaux se font dans les écarts admis (donc de l'index 0 à l'index N-1) et que tous tes pointeurs sont correctement initialisés.

Si le problème n'est toujours pas résolu avec tout cela, il nous faudra du code un peu plus précis et complet pour te venir en aide

[darkman19320]

Merci pour ces réponses détaillées.
Je vais me pencher pour remplacer l'ensemble des tableaux par des vecteurs.

Je vous tiens au courant.