C++オーバーロード演算子<<静的ヘルパーメソッドを使用してアクセス違反エラーを作成する

Question

私はBillというクラスを持っています。クラスのプロパティの中には、intのcategoryと呼ばれるものがあります。 請求書を印刷するときには、番号ではなくカテゴリの名前が表示されます。だから私は、対応する文字列にcategory整数に変換するために、この静的なヘルパー関数を書きました：

// 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); 
    } 
} 

は今、私はこのようなBillクラス定義のoperator<<外をオーバーロードしようとした：

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; 
} 

bill型のオブジェクトを作成し、このコード行を実行すると、エラーが発生します。 cout << billObject;

これで十分ではない情報があれば、私はもっと喜んで追加します。 このエラーを解決するにはどうすればよいですか？

Answer 1

コンパイラは、getCategoryByNumber関数のコンテキストでC文字列からstd::stringオブジェクトを作成し、それへの参照を返し、関数が復帰すると即座に削除されます。

の代わりにconst char*またはstd::stringを返すことを検討してください。getCategoryByNumber

Answer 2

コンパイラは、 "ローカルオブジェクトへの参照の返却"または同様の警告を表示する必要があります（すべての警告をオンにした場合は、常にそうする必要があります）。あなたはまた、exit()を呼び出し避けなければならないが、その代わりにthrow例外は、アプリケーション・プログラムに

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()); 
}