Multiple удалить в деструкторе

Question

Если у меня есть два динамических массивов в качестве частных элементов данных:

std::string* first_array; 
std::string* second_array; 

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

myClass::~myClass() 
{ 
    delete[] first_array; 
    delete[] second_array; 
} 

Обеспокоенность у меня есть это, не все время у меня есть данные внутри обоих массивов, так что-то вроде этого считается лучшей практикой?

myClass::~myClass() 
{ 
    if(first_array) 
     delete[] first_array; 
    if(second_array) 
     delete[] second_array; 
} 

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

Answer 1

Стандарт C++ указывает, что вызов delete по значению нулевого указателя приведет к NOP (No-Operation). Это совершенно допустимо, приемлемо и, как правило, предпочтительнее.

Answer 2

Удаление указателя, который может быть NULL, в порядке. Как уже было сказано другими, вызов delete по NULL-указателю приводит к NOP.

Однако, после удаления любого указателя, вне зависимости от того, где, почему, как и когда, вы должны сразу назначить указатель на NULL. Несоблюдение этого может привести к случайному двойному удалению указателя, что катастрофически плохо для вашей программы (https://isocpp.org/wiki/faq/freestore-mgmt#double-delete-disaster).

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