É bom definir uma variável dentro de um loop? [fechadas]

Meu instrutor me disse uma vez que eu não deveria definir uma variável dentro de um loop , mas sinceramente ainda não entendo o porquê.

Quais são as desvantagens disso?

Alguém poderia me explicar isso?

Não é um problema definir uma variável dentro de um loop. De fato, é uma boa prática, pois os identificadores devem ser confinados ao menor escopo possível.

O que é ruim é atribuir uma variável a um loop se você puder atribuí-lo uma vez antes de o loop ser executado. Dependendo da complexidade do lado direito da tarefa, isso pode se tornar bastante caro e até dominar o tempo de execução do loop. Se você escreve um loop que usa o mesmo valor calculado em todas as iterações, definitivamente deve computá-lo acima do loop - isso é mais importante do que minimizar seu escopo.

Para esclarecer: desde que compute()sempre retorne o mesmo valor, isso

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

é mais inteligente que isso:

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

Tipos complexos têm construtores e destruidores não triviais.

Esses serão chamados no início e no final do corpo do loop (como é inicializado e sai do escopo). Se a inicialização for cara, como ela precisa alocar memória, isso deve ser evitado.

No entanto, para tipos triviais, isso não é problema. A própria alocação e desalocação está apenas adicionando e subtraindo um valor do ponteiro da pilha. (que será otimizado)

Bem, o conselho dele é um pouco simples demais (isso é um eufemismo).
A sequência abrange desde uma boa ideia sobre quem se importa e uma má ideia até impossível .

1. Você deve segui-lo sempre que a reutilização for mais barata do que destruir o antigo e criar um novo.

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

2. Você deve evitá-lo por uma questão de estilo quando não importa para o desempenho.

   #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. Você realmente deve evitá-lo quando tiver um desempenho pior ou a semântica errada.

   #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. Você não pode segui-lo quando o tipo usado não permite troca, nem atribuição de movimento nem atribuição de cópia.

   #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;
       }
   }