Хорошо ли определять переменную внутри цикла? [закрыто]

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

Каковы недостатки этого?

Может ли кто-нибудь объяснить это мне?

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

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

Уточнить: пока compute() всегда возвращает одно и то же значение, это

int value = compute();
while (something) {
    doSomething(value);
}

умнее, чем это:

while (something) {
    int value = compute();
    doSomething(value);
}

Сложные типы имеют нетривиальные конструкторы и деструкторы.

Они будут вызваны в начале и в конце тела цикла (так как он инициализируется и выходит из области видимости). Если инициализация дорогая, так как ей нужно выделить немного памяти, этого следует избегать.

Однако для тривиальных типов это не проблема. Само выделение и освобождение - это просто сложение и вычитание значения из указателя стека. (который будет оптимизирован)

Ну, его совет немного слишком прост (это преуменьшение).
Следование за ним простирается от хорошей идеи о том, кому небезразлично, а плохой до невозможной .

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

   #include <iostream>
   #include <string>
   
   int main() {
       std::string s; // Don't needlessly free the buffer
       while ((std::cin >> s))
           std::cout << s;
   }

2. Вы должны избегать этого как вопрос стиля, когда это не имеет значения для производительности.

   #include <stdio.h>
   #include <stdlib.h>
   int f(int, int);
   
   int main() {
       for (int i = 0; i < 100; ++i) {
           int x = rand(); // Declared here so you don't need to hunt it down.
           printf("%d => %d\n", x, f(x-1, x+i));
       }
   }

3. Вы действительно должны избегать его, когда он имеет худшую производительность или неправильную семантику.

   #include <iostream>
   #include <string>
   std::string generate(int);
   
   int main() {
       for(int i = 0; i < 100; ++i) {
           std::string s = generate(i); // Using copy-ellision here
           std::cout << s;
       }
   }

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

   #include <iostream>
   #include <puzzle>
   
   int main() {
       for (int i = 0; i < 100; ++i) {
           Puzzle x(i); // Puzzle is an immutable class. For whatever reasons.
           std::cout << x;
       }
   }