const Methode in einer Klasse Rückkehr Vektor von Zeigern

Question

Sagen, ich habe diese Klasse:

#include <vector> 
using namespace std; 

class Bag 
{ 
    vector<int*> items; 

public: 
    void addItem(int* i) 
    { 
     items.push_back(i); 
    } 
    const vector<int*> getItems() const 
    { 
     return items; 
    } 
}; 

Das Problem ist, ich durch die Zeiger in Parameterpunkten wiesen Änderungen der Werte verhindern will. Aber mit dem von getItems zurückkehrenden Vektor darf ich die angegebenen Werte ändern.

Jetzt, um das zu beheben, kann ich die Parameter-Elemente als deklarieren, aber dann kann ich auch die Werte in meiner Klasse nicht ändern. Gibt es eine andere Möglichkeit, die Werte zu schützen, aber in den Elementdeklarationen immer noch nicht const zu verwenden?

Answer 1

Wie @KerrekSB wies darauf hin, müssen Sie die Struktur rekonstruieren, die Art der das Element zu einem const ein Wechsel:

std::vector<const int*> getItems() const { 
    return std::vector<const int*>(items.begin(), items.end()); 
} 

Hier ist der Konstruktor von vector<const int*> nutzt die implizite Konvertierung von int* bis const int*. Sie können nicht einfach items hier zurückgeben, weil vector<T> ist invariant auf T. Dies bedeutet direkt, dass es keine Möglichkeit gibt, von vector<int*> zu vector<const int*> nur von einem Typ zu konvertieren.

---

Ihren Kommentar zu adressieren:

Ist dies der einzige Weg? Ich habe das Gefühl, eine Kopie zu erstellen ist ineffizient.

Wie für die Kopie kopiert es nur die Zeigerdaten, also normalerweise 8 Bytes pro Element. Für einen Vektor von Zeigern zu größeren Strukturen ist es typischerweise vernachlässigbar, aber für einen Vektor von int ist es tatsächlich ein ziemlich großer Overhead.

Was bringt uns zu der Frage: Warum speichern Sie Zeiger in erster Linie? Speichern Sie einfach vector<int>, und dann können Sie einfach einen const& zurückgeben.

Answer 2

Wenn Sie Kopien vermeiden wollen und lassen Sie nur lesenden Zugriff auf Ihre std::vector Elemente, könnten Sie stattdessen Funktionen bieten eine const_iterator in den Behälter zu bekommen:

class Bag 
{ 
    vector<int*> items; 

public: 
    void addItem(int* i) 
    { 
     items.push_back(i); 
    } 

    //could just be called begin if that gives the correct idea 
    vector<int*>::const_iterator itemsBegin() const 
    { 
     return std::cbegin(items); 
    } 

    vector<int*>::const_iterator itemsEnd() const 
    { 
     return std::cend(items); 
    } 
}; 

Sie dann würde Schleife über die Bag Inhalte mit dieser Iterator, anstatt eine ganze vector von Elementen. Wenn Sie die Funktionen begin und end zur Verfügung stellen, können Sie sogar bereichsbasierte For-Schleifen verwenden.