ループ内で変数を定義するのは良いですか？[閉まっている]

私のインストラクターはかつて、ループ内で変数を定義するべきではないと教えてくれましたが、正直なところまだ理由がわかりません。

その欠点は何ですか？

体はそれを私に説明できますか？

ループ内で変数を定義することは問題ではありません。実際、識別子は可能な限り最小のスコープに限定する必要があるため、それは良い習慣です。

ループが実行される前に一度だけ変数を割り当てることができれば、ループ内で変数を割り当てることは悪いことです。割り当ての右側がどれだけ複雑かによっては、これはかなり高価になり、ループの実行時間を支配することさえあります。すべての反復で同じ計算値を使用するループを作成する場合は、ループの上で確実に計算する必要があります。これは、スコープを最小化するよりも重要です。

明確にするために：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;
       }
   }