Apakah baik untuk mendefinisikan variabel di dalam satu loop? [Tutup]

Instruktur saya pernah mengatakan kepada saya bahwa saya tidak boleh mendefinisikan variabel di dalam satu lingkaran , tetapi saya sejujurnya masih tidak mengerti mengapa.

Apa kerugiannya?

Bisakah ada yang menjelaskan hal itu kepada saya?

Bukan masalah untuk mendefinisikan variabel dalam satu lingkaran. Sebenarnya, ini praktik yang baik, karena pengidentifikasi harus dibatasi pada ruang lingkup sekecil mungkin.

Yang buruk adalah untuk menetapkan variabel dalam satu loop jika Anda bisa menetapkannya sekali saja sebelum loop berjalan. Bergantung pada seberapa kompleks sisi kanan penugasan ini, ini bisa menjadi agak mahal dan bahkan mungkin mendominasi waktu run dari loop. Jika Anda menulis loop yang menggunakan nilai perhitungan yang sama di semua iterasi, Anda harus menghitungnya di atas loop - yang lebih penting daripada meminimalkan cakupannya.

Untuk memperjelas: selama compute()selalu mengembalikan nilai yang sama, ini

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

lebih pintar dari ini:

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

Tipe kompleks memiliki konstruktor dan destruktor non-sepele.

Itu akan dipanggil pada awal dan akhir dari loop body (seperti yang diinisialisasi dan keluar dari ruang lingkup). Jika inisialisasi mahal seperti itu perlu mengalokasikan sebagian memori maka itu harus dihindari.

Namun untuk tipe sepele itu tidak masalah. Alokasi dan deallokasi itu sendiri hanya menambah dan mengurangi nilai dari stack pointer. (yang akan dioptimalkan)

Nah, nasihatnya agak terlalu sederhana (itu pernyataan yang meremehkan).
Mengikuti itu berkisar sepanjang jalan dari ide yang baik atas yang peduli dan ide yang buruk untuk mustahil .

1. Anda harus mengikutinya setiap kali menggunakan kembali lebih murah daripada menghancurkan yang lama dan membuat yang baru.

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

2. Anda harus menghindari itu sebagai masalah gaya ketika itu tidak masalah untuk kinerja.

   #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. Anda benar - benar harus menghindari itu ketika memiliki kinerja yang lebih buruk atau semantik yang salah.

   #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. Anda tidak dapat mengikutinya ketika jenis yang digunakan tidak memungkinkan swapping, atau memindahkan-tugas atau menyalin-tugas.

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