3

Можно ли использовать new [] вместо new? К примеру:

int main()
{
    int q = 1;
    int *mass = new int [q];
    ...
    delete [] mass;
}

Спрашиваю потому, что хочется сделать код более универсальным, чтобы каждый раз не делать проверки на значения переменной q, где она может быть больше единицы или равной ей. Будет ли такой код корректно функционировать? И не будет ли такой стиль считаться дурным тоном?

6
  • Э... а что мешает так делать? 14 янв 2018 в 13:24
  • Собственно говоря, поэтому и спрашиваю: нет ли никаких подводных камней при таком оформлении. 14 янв 2018 в 13:25
  • Хотите сделать код более универсальным - используйте стандартные контейнеры (std::vector<>,...), а не извращайтесь с использованием сырых new/delete. + прочитайте ответ к похожему вопросу - ru.stackoverflow.com/questions/678966/… .
    – user227465
    14 янв 2018 в 13:25
  • Да, это корректно. Нет, ничего дурного нет, подразумевая, что в теле есть какое-то изменение указателя mass или q, а это не просто написано из прихоти. Если код буквально так начинается, то можно добавить комментарий, дабы избежать разночтения. С другой стороны, в 9 случаях из 10 стоит использовать полноценный контейнер).
    – Fat-Zer
    14 янв 2018 в 13:27
  • Спасибо за ответ. Я мог бы использовать контейнеры, но задание не позволяет. 14 янв 2018 в 13:28

2 ответа 2

5

Такой способ создания одиночных объектов непригоден в случаях, когда необходимо полиморфное удаление полиморфных объектов. Полиморфное удаление возможно только через new/delete, но не через new[]/delete[].

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

6
  • А что не так с полиморфным удалением через delete[]?
    – αλεχολυτ
    22 янв 2018 в 12:30
  • @alexolut если размер статического типа не равен размеру динамического, то при проходе такого массива не будем попадать в элементы уже начиная со второго. То есть если в памяти лежат объекты размером 8 байт, а указатель на тип размером 4 байта, то шагать delete[] будет по 4 байта, а не по 8 со всеми вытекающими. 22 янв 2018 в 12:56
  • @Croessmah ах, в этом плане, ну да ... сам же где-то недавно писал об этом :)
    – αλεχολυτ
    22 янв 2018 в 12:58
  • @alexolut: "Не так" в первую очередь то, что стандарт открытым текстом говорит, что попытка такого удаления приводит к неопределенному поведению. Размер массива значения не имеет. Обычный компилятор С++ не станет даже пытаться реализовывать полиморфное удаление в delete [] - он просто вызовет впрямую деструктор базового класса и все. 22 янв 2018 в 16:21
  • @AnT обычный, получается, только clang. GCC и MSVC удалили по иерархии.
    – αλεχολυτ
    22 янв 2018 в 18:21
4

Никто не мешает писать new T[n], где n == 1.

Однако само по себе использование new T[n] и new T считается дурным тоном, т.к. есть std::vector<T> и std::make_unique<T>.

В частности С++ Core Guidelines не рекомендуют использование "naked new":

ES.60: Avoid new and delete outside resource management functions

Reason
Direct resource management in application code is error-prone and tedious.

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.