在循环内定义变量是否很好？[关闭]

我的老师曾经告诉过我，我不应该在循环内定义变量，但是老实说，我仍然不明白为什么。

那有什么缺点？

有人能向我解释吗？

在循环中定义变量不是问题。实际上，这是一个好习惯，因为标识符应限制在最小范围内。

不好的是，如果您可以在循环运行前分配一次变量，则可以在循环内分配变量。根据分配右侧的复杂程度，这可能会变得相当昂贵，甚至可能支配循环的运行时间。如果编写的循环在所有迭代中使用相同的计算值，则绝对应在循环上方对其进行计算-这比最小化其作用域更重要。

澄清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;
       }
   }