Vektor der Struktur gibt nur den letzten Wert zurück

Question

Leider kann ich den vollständigen Code nicht einfügen, aber ich werde mein Bestes versuchen, um das Problem zu erklären. Ich habe einen Vektor der Struktur. Ich übergebe den Vektor an eine Funktion, die den Vektor mit Struktur füllt und den Vektor zurückgibt. Aber wenn ich versuche, auf die Vektorelemente zuzugreifen, bekomme ich nur das letzte Element eingefügt. Ich denke, dass das Problem irgendwo ist, dass ich die Adresse der Struktur speichere, und deshalb behält Vektor nur den letzten Wert bei, aber ich bin nicht sicher, wie man das korrigiert.

Hier meine Struktur ist:

struct NA 
{ element1; 
    element2; 
}; 

Hier ist, wie ich meine Vektor übergeben, nachdem sie erklärt:

Vector<NA> del; 
func(del); 

Hier ist meine Funktion ist: (q ist eine Variable ergibt sich aus einer gespeicherten Prozedur mit

func(Vector<NA> &dels) 
{ 
    NA& del(*new NA); 
    while(q.nextquery()) 
    { 
    while(q.nextrow()) 
    { 
     q.bind(del.element1); 
     q.bind(del.element2); 
     dels.insert(&del); 
    } 
    return dels.entries() 
} 

Answer 1

NA& del(*new NA); 
while(q.nextquery()) 
    dels.insert(&del); 

Offensichtlich erstellen Sie nur ein Objekt del, aber Sie fügen es mehrmals in demselben Vektor ein. Am Ende zeigen alle Einträge des Vektors auf das gleiche Objekt.

Was Sie wollen, ist wahrscheinlich ein neues del Objekt für jeden Eintrag in Ihrem Vektor zu erstellen. Setzen Sie daher die creation-Anweisung in die Schleife.

func(Vector<NA> &dels) 
{ 
    while(q.nextquery()) 
    { 
    while(q.nextrow()) 
    { 
     NA& del(*new NA); // <-- Here 
     q.bind(del.element1); 
     q.bind(del.element2); 
     dels.insert(&del); 
    } 
    return dels.entries() 
} 

Answer 2

func() wird, um mehrere Kopien eines einzelnen Zeigers auf ein einziges dynamisch zugewiesenen Objekt in den Vektor schieben. Jede Schleifeniteration ändert dasselbe Objekt, sodass jeder Eintrag im Vektor auf die Daten des letzten gelesenen Abfragedatensatzes zeigt.

Probieren Sie etwas Ähnliches statt:

func(Vector<NA> &dels) { 
    while(q.nextquery()) { 
     while(q.nextrow()) { 
      NA *del = new NA; 
      q.bind(del->element1); 
      q.bind(del->element2); 
      dels.insert(del); 
     } 
    } 
    return dels.entries(); 
} 

Auf der anderen Seite, warum Sie Zeiger im Vektor sind Speichern? Und mit einer nicht standardmäßigen Vektorklasse? Es wäre sicherer, tatsächliche Objekte in eine Standard-std::vector Klasse zu speichern statt:

void func(std::vector<NA> &dels) { 
    while(q.nextquery()) { 
     while(q.nextrow()) { 
      NA del; 
      q.bind(del.element1); 
      q.bind(del.element2); 
      dels.push_back(del); 
     } 
    } 
}