Có phải thực tế xấu để tạo các khối mã?

Trong C ++, có phải thực tế xấu là tạo các khối mã bên trong một số chức năng, chẳng hạn như sau:

    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;

    }

Điểm của việc này sẽ là sử dụng cùng một tên biến "kiểm tra" nhiều lần, cho cùng một loại thủ tục. Trong thực tế dự án của tôi, tôi có nhiều biến và đang thực hiện nhiều thử nghiệm. Tôi thực sự không muốn tiếp tục tạo các biến mới với các tên khác nhau cho mỗi bài kiểm tra, xem xét cách các bài kiểm tra rất giống nhau.

Có phải là thực tế xấu khi chèn các khối mã để các biến đi ra khỏi phạm vi sau mỗi thử nghiệm, và sau đó tôi có thể sử dụng lại tên của chúng không? Hay tôi nên tìm kiếm một giải pháp khác? Cần lưu ý rằng tôi đã cân nhắc sử dụng cùng một bộ biến cho tất cả các thử nghiệm của mình (và chỉ đặt tất cả chúng thành 0 sau khi mỗi thử nghiệm kết thúc), nhưng tôi cảm thấy điều này có thể là một thực tiễn tồi.

Các khối là hoàn toàn hợp lý nếu bạn đang sử dụng chúng để phạm vi một số tài nguyên. Tập tin, kết nối mạng, phân bổ bộ nhớ, giao dịch cơ sở dữ liệu, bất cứ điều gì. Trong những trường hợp đó, khối thực sự là một phần của cấu trúc logic của mã: bạn sinh ra một tài nguyên, nó tồn tại trong một khoảng thời gian và sau đó nó biến mất vào một thời điểm được chỉ định.

Nhưng nếu tất cả những gì bạn đang làm là tìm kiếm một cái tên , thì tôi sẽ nói rằng chúng là thực tiễn tồi. Nói chung, tất nhiên; trường hợp đặc biệt có thể áp dụng.

Ví dụ: nếu chức năng này được tạo bởi một số hệ thống tạo mã, khung thử nghiệm hoặc tương tự, thì các khối vì mục đích của phạm vi tên là một điều hợp lý. Nhưng bạn sẽ nói về mã được viết cho mục đích của một cỗ máy chứ không phải con người.

Nếu một người đang viết mã trong đó họ cần sử dụng lại các tên trong cùng một hàm, tôi sẽ nói rằng các khối đó có thể cần phải là các hàm riêng biệt. Đặc biệt là nếu những tên đó đang được sử dụng với các loại và / hoặc ý nghĩa khác nhau trong các khối đó.

Nó không phải là thực hành xấu để tạo ra các khối như thế này. Đó là cách phạm vi hoạt động.

Thông thường, điều này được thực hiện khi sử dụng RAII (Thu nhận tài nguyên là khởi tạo) và bạn muốn kiểm soát khi các hàm hủy được gọi.

Nếu nó dài ra, tôi sẽ xem xét chuyển mã đó sang chức năng riêng của nó.

Theo tôi chỉ sử dụng nó để tái chế các tên biến không phải là một ý tưởng tốt. Nhưng tôi có thể thấy nó hữu ích trong trường hợp bộ nhớ thấp