C++ initialisation classe tableau instance

Question

J'ai une classe A comme suit:

class A 
{ 
public:  
    A() 
    {   
     printf("A constructed\n");   
    } 
    ~A(); 
    //no other constructors/assignment operators  
} 

Je les suivantes ailleurs

A * _a; 

J'initialiser avec:

int count = ... 
... 
_a = new A[count]; 

et je avoir accès avec

int key = .... 
... 
A *a_inst = &(_a[key]); 
.... 

Il s'exécute normalement, et le printf dans le constructeur est exécuté, et tous les champs dans A sont bien.

j'ai couru Valgrind avec les args suivants:

valgrind --leak-check=full --show-reachable=yes --track-origins=yes -v ./A_app 

et Valgrind ne cesse de crier à propos

Conditional jump or move depends on uninitialised value(s) 

puis la trace de la pile aux états de accesseurs.

Quelqu'un peut-il expliquer pourquoi cela se produit? Spécifiquement si ce que Valgrind dit est vrai, pourquoi le constructeur est-il exécuté?

Answer 1

Cela peut signifier que key ou count contient une valeur non initialisée. Même si vous l'initialisez dans la déclaration, par ex. int key = foo + bar;, il se peut que foo ou bar ne soit pas initialisé et que valgrind l'applique à key.

Answer 2

Edit: Essayez de régler A *a = 0;

Exécution de votre code dans un scénario simplifié ne produit pas d'avertissement de Valgrind. Considérez le code suivant:

#include <iostream> 

class A 
{ 
    public: 
     A() 
     {  
      std::cout << "A" << std::endl; 
     }  
}; 

int main() 
{ 
    A *a; 
    int count = 10; 
    a = new A[count]; 

    int key = 1; 
    A *inst = &(a[key]); 

    return 0; 
} 

Compilé avec:

$ g++ -g main.cc -o main 

et courir avec:

$ valgrind --leak-check=full --show-reachable=yes --track-origins=yes ./main 

Donc, je pense que plus d'information est nécessaire. Vous faites probablement quelque chose entre la définition de _a et l'allocation réelle de la mémoire sur le tas. Pourrais-je simplement suggérer que vous fusionniez la définition et l'allocation en une seule ligne?

int count = 10; 
    A *a = new A[count];