2

Есть участок кода в программе:

while (true) {
  char value[len+1];
  //какие-то операции с переменной value
}

Есть ли тут утечка памяти, если не обнулять переменную value в конце операций цикла?

3 ответа 3

5

Нет.
Ваш массив value относится к классу автоматической памяти. Он будет уничтожен без вашего участия.

Теперь о том, когда утечки могут возникать. Утечки памяти могут появиться когда вы начинаете выделять память вручную:

char *value = new char[len+1];

В моем примере объявлена переменная value типа "указатель на char". Эта переменная указывает на участок памяти, за освобождение которого отвечаете вы.

Чтобы утечки не было, нужно удалить то, на что указывает value:

delete[] value;

Таким образом код примет такой вид:

void foo(int len){
    char *value = new char[len+1];
    //...
    delete[] value;
}

PS: Кстати, ваш код примерно так и должен выглядеть. Потому что в С++ размер массива должен быть известен на этапе компиляции. Если ваш компилятор прожевал char value[len+1], то это не значит что другой сделает то же самое.

6
  • Я понимаю, что нужно использовать new, но пока необходимо использовать такой кусок кода)
    – nerik
    14 авг 2017 в 7:30
  • "Он будет уничтожен без вашего участия." - Это когда будет сделано? После повторного объявления char той же самой переменой или завершению функции?
    – nerik
    14 авг 2017 в 7:31
  • 1
    @nerik, ну, если len где-то выше объявлена как const int len, то ничего криминального. Если нет, то вас учат ошибкам. А вообще вам нужно использовать std::vector. Вы, наверно, до этого еще не дошли, это так, на будущее. 14 авг 2017 в 7:35
  • @nerik, а вы попробуйте объявить ту же самую переменную в той-же области видимости:). Компилятор заказывает память под автоматические переменные при входе в функцию и освобождает при выходе. 14 авг 2017 в 7:37
  • @yrHeTaTeJlb, это не ошибка, строго говоря. Это просто расширение компилятора.
    – ixSci
    14 авг 2017 в 8:27
4

Здесь не происходит динамическое выделение памяти, поэтому ни о какой утечке не может быть и речи. Хотя char value[len+1] будет работать только на gcc.

1
  • 2
    Так как объявления len не представлено, говорить о VLA преждевременно. Это вполне может быть константа времени компиляции.
    – αλεχολυτ
    14 авг 2017 в 8:27
1

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

Во всех кодах которые пишут умны люди (большие корпорации) тоже видно что переменные объявляться в теле цикла.

Моя логика говорит: зачем каждый раз выделять область памяти, это же тоже занимает какое то время для машины, но делаю как все))

Возможно "умные" компиляторы, все новые переменные в цикле как то сами оптимизируют код, не знаю.

2
  • 2
    Память для локальных переменных "выделяется" при входе в функцию. Для всех разом, одной машинной командой; и не важно, в цикле они объявляются или нет.
    – mymedia
    14 авг 2017 в 6:37
  • 1
    Так-то ресурсы программы это не только память. Если у вас есть сложный объект, который в конструкторе подключается к бд, читает файл или устанавливает соединение по сети, а в деструкторе делает обратные операции, то лучше вынести его из цикла. Пример 14 авг 2017 в 7:29

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

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