Случайный вывод с char * operator

Question

У меня есть класс в C++, и я хотел бы иметь возможность печатать объект, поэтому я создал оператор char * cast. Дело в том, что по какой-то причине вывод операторной функции представляет собой случайные символы вместо возвращаемой строки. вот код:

operator const char*()const { 
    std::cout << (std::to_string(Nom) + '/' + std::to_string(Denom)).c_str() << std::endl; 
    return (std::to_string(Nom) + '/' + std::to_string(Denom)).c_str(); 
} 

и основная функция:

Rational r1(7, 15); 
std::cout << r1<<std::endl; 
return 0; 

Первый линейный выход нормально ("7/15"), но возвращаемое значение просто случайные буквы.

Любые подсказки?

Answer 1

Дополнение к другим ответам: обычно неявный оператор литья является плохим стилем. Особенно приведение к типу, который не связан с вашим классом.

Если вы хотите, чтобы бросить свой объект, лучший способ может быть явное приведение:

class Rational 
{ 
public: 
    ... 
    std::string toString() const 
    { 
     return std::to_string(Nom) + '/' + std::to_string(Denom); 
    } 
}; 

Вы можете перегрузить «из» функцию с ним:

std::ostream & operator <<(std::ostream &stream, const Rational &obj) 
{ 
    stream << obj.toString(); 
    return stream; 
} 

Если вам нужно только вы можете использовать перегрузку без функции литья:

class Rational 
{ 
public: 
    ... 
    friend std::ostream & operator <<(std::ostream &stream, const Rational &obj) 
    { 
     stream << std::to_string(obj.Nom) + '/' + std::to_string(obj.Denom); 
     return stream; 
    } 
}; 

Answer 2

(std::to_string(Nom) + '/' + std::to_string(Denom)) создает временную переменную. .c_str() получает адрес некоторой памяти, принадлежащей этой переменной.

Вскоре после этого эта переменная уничтожается (вы не копируете/не перемещаете ее в любом месте). Память освобождается.

Адрес, полученный из c_str(), сохраняется, но теперь он указывает на некоторую память, которая может быть или не быть частью вашего приложения. Попытка прочитать это Undefined Behavior (или UB для краткости на этом сайте).

Answer 3

Замените объект, который управляет памятью, вместо необработанного const char*.

operator std::string() const { 
    std::stringstream ss; 
    ss << Nom << '/' << Denom; 
    return ss.str(); 
}