Чи добре визначити змінну всередині циклу? [зачинено]

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

Які недоліки в цьому?

Чи може мені це пояснити будь-яке тіло?

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

Що погано, це призначити змінну в циклі, якщо ви можете так само добре призначити її один раз до циклу. Залежно від того, наскільки складна права частина завдання, це може стати досить дорогим і навіть може домінувати в часі циклу. Якщо ви пишете цикл, який використовує однакове обчислене значення у всіх ітераціях, вам обов'язково слід обчислити його вище циклу - це важливіше, ніж мінімізувати його область.

Для уточнення: якщо 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;
       }
   }