Là nó tốt để xác định một biến trong một vòng lặp? [đóng cửa]

Người hướng dẫn của tôi đã từng nói với tôi rằng tôi không nên xác định một biến trong vòng lặp , nhưng thật lòng tôi vẫn không hiểu tại sao.

Những bất lợi của điều đó là gì?

Có ai có thể giải thích điều đó cho tôi không?

Nó không phải là một vấn đề để xác định một biến trong một vòng lặp. Trên thực tế, đó là một thực tiễn tốt, vì các định danh nên được giới hạn trong phạm vi nhỏ nhất có thể.

Điều tồi tệ là chỉ định một biến trong vòng lặp nếu bạn cũng có thể gán nó một lần trước khi vòng lặp chạy. Tùy thuộc vào mức độ phức tạp của phía bên phải của bài tập, điều này có thể trở nên khá tốn kém và thậm chí có thể chi phối thời gian chạy của vòng lặp. Nếu bạn viết một vòng lặp sử dụng cùng một giá trị được tính toán trong tất cả các lần lặp, bạn chắc chắn nên tính toán nó trên vòng lặp - điều đó quan trọng hơn việc giảm thiểu phạm vi của nó.

Để làm rõ: miễn là compute()luôn trả về cùng một giá trị, điều này

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

thông minh hơn thế này:

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

Các loại phức tạp có các hàm tạo và hàm hủy không tầm thường.

Chúng sẽ được gọi ở đầu và cuối của thân vòng lặp (khi nó được khởi tạo và đi ra khỏi phạm vi). Nếu việc khởi tạo là tốn kém như nó cần phải phân bổ một số bộ nhớ thì điều đó nên tránh.

Tuy nhiên đối với các loại tầm thường mà không có vấn đề. Việc phân bổ và giải quyết chính nó chỉ là thêm và trừ một giá trị từ con trỏ ngăn xếp. (sẽ được tối ưu hóa)

Chà, lời khuyên của anh ta hơi quá đơn giản (đó là một cách nói nhẹ nhàng ).
Theo sau nó bao gồm tất cả các cách từ một ý tưởng tốt về người quan tâm và ý tưởng xấu đến không thể .

1. Bạn nên theo dõi nó bất cứ khi nào sử dụng lại rẻ hơn so với phá hủy cái cũ và tạo cái mới.

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

2. Bạn nên tránh xa nó như một vấn đề về phong cách khi nó không quan trọng đối với hiệu suất.

   #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. Bạn thực sự nên tránh xa nó khi nó có hiệu suất kém hơn hoặc ngữ nghĩa sai.

   #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. Bạn không thể theo dõi nó khi loại được sử dụng cho phép không hoán đổi, cũng không chuyển nhượng cũng không gán sao chép.

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