Döngü içinde bir değişken tanımlamak iyi midir? [kapalı]

Eğitmenim bir keresinde bana bir döngü içinde bir değişken tanımlamamam gerektiğini söyledi , ama dürüstçe hala nedenini anlamıyorum.

Bunun dezavantajları nelerdir?

Bunu herhangi bir beden açıklayabilir mi?

Döngü içindeki bir değişkeni tanımlamak sorun değil . Aslında, iyi bir uygulamadır, çünkü tanımlayıcılar mümkün olan en küçük kapsamla sınırlandırılmalıdır.

Ne kötü etmektir atamak sadece de bir kez atamak eğer bir döngü içinde bir değişken önce döngü çalışır. Ödevin sağ tarafının ne kadar karmaşık olduğuna bağlı olarak, bu oldukça pahalı hale gelebilir ve hatta döngünün çalışma süresine hakim olabilir. Tüm tekrarlamalar aynı hesaplanan değeri kullanan bir döngü yazarsanız, kesinlikle hesaplayabilir gerektiğini yukarıda kapsamını minimize daha önemli olduğunu - döngü.

Açıklığa kavuşturmak için: compute()her zaman aynı değeri döndürdüğü sürece ,

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

bundan daha akıllı:

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

Karmaşık tipler önemsiz olmayan yapıcılara ve yıkıcılara sahiptir.

Bunlar döngü gövdesinin başlangıcında ve sonunda çağrılır (başlatıldığında ve kapsam dışına çıktığında). Başlatma, biraz bellek ayırması gerektiği gibi pahalıysa, bundan kaçınılmalıdır.

Ancak önemsiz türler için sorun değil. Tahsis ve dağıtmanın kendisi sadece yığın işaretçisinden bir değer ekler ve çıkarır. (optimize edilecek)

Onun tavsiyesi biraz fazla basit (bu bir eksiklik).
O tüm yol kadar uzanmaktadır ardından iyi bir fikir üzerinde kimin umurunda ve kötü bir fikir için imkansız .

1. Yeniden kullanım eskisini yok etmek ve yeni bir tane oluşturmaktan daha ucuz olduğunda takip etmelisiniz.

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

2. Performans açısından önemli olmadığında bir stil meselesi olarak kaçınmalısınız.

   #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. Daha kötü performansa veya yanlış semantiğe sahip olduğunda gerçekten kaçınmalısınız.

   #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. Sen olamaz kullanılan türü ne takas izin verdiğinde onu takip ne de hareket atama ne de kopya atama.

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