มันเป็นการปฏิบัติที่ไม่ถูกต้องหรือไม่ที่จะสร้างบล็อคของรหัส?

ใน C ++ มันเป็นวิธีปฏิบัติที่ไม่ถูกต้องหรือไม่ในการสร้างบล็อกของโค้ดภายในฟังก์ชั่นบางอย่างเช่นต่อไปนี้:

    bool f()
    {
       {
           double test = 0;

           test = // some other variable outside this function, for example.

           if (test == // some value)
             return true;
       }

       {
           double test = 0;

           test = // some variable outside this function, different from the last one.

           if (test == // some value)
             return true;
       }

       return false;

    }

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

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

บล็อกมีความสมเหตุสมผลอย่างสมบูรณ์แบบถ้าคุณใช้มันเพื่อกำหนดขอบเขตของทรัพยากร ไฟล์, การเชื่อมต่อเครือข่าย, การจัดสรรหน่วยความจำ, ธุรกรรมฐานข้อมูล, อะไรก็ตาม ในกรณีเหล่านั้นการบล็อกนั้นเป็นส่วนหนึ่งของโครงสร้างแบบลอจิคัลของรหัส: คุณวางไข่ทรัพยากรมันมีอยู่ในช่วงระยะเวลาหนึ่งและจากนั้นจะหายไปตามเวลาที่กำหนด

แต่ถ้าสิ่งที่คุณกำลังทำคือการกำหนดชื่อฉันก็จะบอกว่าพวกมันเป็นการฝึกฝนที่ไม่ดี โดยทั่วไปการพูดแน่นอน; สถานการณ์พิเศษสามารถนำไปใช้

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

หากมนุษย์กำลังเขียนรหัสที่พวกเขาจำเป็นต้องใช้ชื่อซ้ำในฟังก์ชั่นเดียวกันฉันจะบอกว่าบล็อกเหล่านั้นอาจจะต้องแยกฟังก์ชั่น โดยเฉพาะอย่างยิ่งหากมีการใช้ชื่อเหล่านั้นกับประเภทและ / หรือความหมายต่าง ๆ ในบล็อกเหล่านั้น

การสร้างบล็อกเป็นแบบนี้ไม่เลว มันเป็นขอบเขตการทำงาน

โดยปกติจะทำเมื่อใช้ RAII (การได้รับทรัพยากรคือการเริ่มต้น) และคุณต้องการควบคุมเมื่อมีการเรียก destructors

หากเวลานานฉันจะลองย้ายรหัสนั้นไปเป็นฟังก์ชั่นของตัวเอง

ในความคิดของฉันเพียงใช้เพื่อรีไซเคิลชื่อตัวแปรไม่ใช่ความคิดที่ดี แต่ฉันเห็นว่ามันมีประโยชน์ในกรณีที่มีหน่วยความจำต่ำ