มันเป็นการดีที่จะกำหนดตัวแปรภายในวงหรือไม่? [ปิด]

ผู้สอนของฉันเคยบอกฉันว่าฉันไม่ควรกำหนดตัวแปรภายในลูปแต่ฉันก็ยังคงไม่เข้าใจว่าทำไม

อะไรคือข้อเสียของสิ่งนั้น?

มีร่างกายใดสามารถอธิบายสิ่งนั้นให้ฉันได้บ้าง

ไม่ใช่ปัญหาในการกำหนดตัวแปรภายในลูป ในความเป็นจริงมันเป็นแนวปฏิบัติที่ดีเนื่องจากตัวระบุควรถูก จำกัด ขอบเขตที่เล็กที่สุดเท่าที่จะเป็นไปได้

มีอะไรที่ไม่ดีคือการกำหนดตัวแปรภายในลูปถ้าคุณทำได้เช่นกันก่อนที่ลูปจะรัน ขึ้นอยู่กับความซับซ้อนของด้านขวามือของการมอบหมายสิ่งนี้อาจมีราคาค่อนข้างแพงและอาจควบคุมเวลาการทำงานของลูป หากคุณเขียนลูปที่ใช้ค่าที่คำนวณได้เหมือนกันในการวนซ้ำทั้งหมดคุณควรคำนวณลูปนั้นแน่นอนเหนือลูปซึ่งสำคัญกว่าการย่อขอบเขตให้เล็กที่สุด

เพื่อชี้แจง: ตราบใดที่compute()คืนค่าเดิมเสมอสิ่งนี้

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

ฉลาดกว่านี้:

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

ประเภทที่ซับซ้อนมีตัวสร้างที่ไม่สำคัญและ destructors

สิ่งเหล่านั้นจะถูกเรียกที่จุดเริ่มต้นและจุดสิ้นสุดของลูปเนื้อความ (เนื่องจากเป็นการเริ่มต้นและออกนอกขอบเขต) หากการเตรียมใช้งานมีราคาแพงเช่นนั้นจำเป็นต้องจัดสรรหน่วยความจำบางส่วนแล้วควรหลีกเลี่ยง

อย่างไรก็ตามสำหรับประเภทเล็กน้อยที่ไม่มีปัญหา การจัดสรรและการจัดสรรคืนตัวเองเป็นเพียงการเพิ่มและลบค่าจากตัวชี้สแต็ก (ซึ่งจะได้รับการเพิ่มประสิทธิภาพออก)

คำแนะนำของเขาง่ายเกินไปเล็กน้อย (นั่นเป็นการพูดน้อย)
ต่อไปนี้มันช่วงตลอดทางจากความคิดที่ดีกว่าที่ใส่ใจและความคิดที่ดีที่จะเป็นไปไม่ได้

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