C++ Überladen des Operators << Verwenden der statischen Hilfsmethoden Erstellen eines Zugriffsverletzungsfehlers

Question

Ich habe eine Klasse mit dem Namen Bill. Unter den Eigenschaften der Klasse befindet sich eine mit der Bezeichnung category vom Typ int. Beim Drucken der Rechnung sollte anstelle der Nummer der Name der Kategorie angezeigt werden. Also schrieb ich diese statische Hilfsfunktion die category Ganzzahl in den entsprechenden Zeichenfolge zu konvertieren:

// Takes an integer representing the category and returns its corresponding name 
static const std::string& getCategoryByNumber(int category) 
{ 
    switch (category) 
    { 
    case 1: 
     return "Food"; 
    case 2: 
     return "Gift"; 
    case 3: 
     return "Fuel"; 
    case 4: 
     return "Electricity"; 
    case 5: 
     return "Clothes"; 
    case 6: 
     return "Holidays"; 
    case 7: 
     return "Water"; 
    case 8: 
     return "Fees"; 
    default: 
     exit(1); 
    } 
} 

Jetzt habe ich versucht, die operator<< außerhalb der Bill Klassendefinition so zu überlasten:

std::ostream& operator<<(std::ostream& os, Bill& bill) { 
    int category = bill.getCategory(); 
    const std::string& nameOfCategory = getCategoryByNumber(category); 
    std::cout << nameOfCategory.c_str(); 
    return os; 
} 

Der Fehler tritt beim Erstellen eines Objekts vom Typ Rechnung und Ausführen dieser Codezeile auf: cout << billObject;

Wenn dies nicht genug Informationen sind, werde ich gerne mehr hinzufügen. Wie kann ich diesen Fehler beheben und was verursacht es?

Answer 1

Meine Vermutung, was wirklich hier passiert, ist der Compiler erstellt ein std::string Objekt aus einer C-Zeichenfolge im Kontext getCategoryByNumber Funktion, gibt einen Verweis darauf zurück, und das Objekt, das es verweist, wird sofort entfernt, sobald die Funktion zurückgibt.

Bitte berücksichtigen Sie die Rückgabe oder std::string statt std::string& von getCategoryByNumber.

Answer 2

Der Compiler sollte Sie gewarnt haben, "Verweis auf lokales Objekt zurückzugeben" oder ähnliches (wenn Sie alle Warnungen einschalten, was Sie immer tun sollten). Sie sollten auch vermeiden exit() Aufruf, sondern throw eine Ausnahme, das Anwendungsprogramm eine Chance zu geben

struct Bill 
{ 
    /* ... */ 
    int GetCategory() const; 
    static string GetCategoryName(int category) 
    { 
    switch (category) { 
    default: throw std::runtime_error("Bill: category '"+ 
             std::to_string(category)+"' unknown"); 
    case 1: return "Food"; 
    case 2: return "Gift"; 
    case 3: return "Fuel"; 
    case 4: return "Electricity"; 
    case 5: return "Clothes"; 
    case 6: return "Holidays"; 
    case 7: return "Water"; 
    case 8: return "Fees"; 
    } 
    } 
}; 

inline 
std::ostream& operator<<(std::ostream&os, Bill const&bill) 
{ 
    return os << Bill::GetCategoryName(bill.GetCategory()); 
}