Это утечка памяти? Как это сделать?

Question

У меня есть что-то вроде этого:

void Test(void) 
{ 
    char errorMessage[256]; 

    spintf(errorMessage,... blablabla); 
    throw new CustomException(errorMessage); 
} 

Будет ли это утечка памяти, потому что ErrorMessage будет не освобожден? Или это вызовет исключение при доступе к сообщению об исключении внутри try {} catch, потому что errorMessage был освобожден при выходе из функции?

Заранее спасибо.

Answer 1

Память errorMessage уже будет освобождена при обращении с помощью обработчика улова. Однако вы можете просто скопировать его в std::string в конструктор CustomException.

С другой стороны, утечка памяти может быть вызвана самим исключением, поскольку вы положили его в кучу. Это необязательно.

Answer 2

Это не вызовет утечку памяти, но это может привести к неопределенному поведению в зависимости от того, что вы делаете с этим исключением. Я полагаю, утечки памяти могут произойти только при распределении динамической памяти

Answer 3

Ответ да, очень вероятно. Вы никогда не должны бросать объекты, созданные с помощью new. Вместо этого бросьте значение:

throw CustomException(errorMessage); 

и поймать с помощью константных ссылок:

try { 
    ... 
} 
catch(const CustomException & e) { 
    ... 
} 

Метания значения означает, что компилятор обрабатывает жизнь брошенного объекта. Кроме того, в вашем коде может возникнуть другая проблема, если конструктор копирования для вашего класса исключений неверен - однако это не имеет ничего общего с обработкой исключений.