Nhiều biến có cùng tên trong một phương thức

Question

Tôi thích các biến được đặt tên chính xác nhưng trong một số trường hợp, nó rất khó thực hiện.

Vì vậy, nếu đối tượng của tôi thực hiện IDisposable sau đó tôi có thể sử dụng:

using (var whatever = new Whatever()) { //... } 

Nhưng đó là trường hợp hiếm hoi, vì vậy tôi tìm thấy một cách khác để xử lý nó - khối vô danh (không chắc chắn làm thế nào nó được gọi một cách chính xác):

//... 
{ 
    var whatever = new Whatever(); 
    //... 
} 
//... 
if (condition) 
{ 
    var whatever = new Whatever(); 
} 

Đây có phải là cách tiếp cận tốt không? Có bất kỳ cạm bẫy hoặc niềm tin rộng rãi nào làm giảm khả năng đọc mã không?

Answer 1

Về cơ bản, nếu trình biên dịch không phàn nàn và mã của bạn có thể đọc được và dễ hiểu, không có gì sai với điều đó.

Ví dụ:

foreach (var foo in foos) 
{ 
    var task = new FooTask(foo); 
    task.Run(); 
} 
foreach (var bar in bars) 
{ 
    var task = new BarTask(bar); 
    task.Run(); 
} 

Đây là thực sự (theo ý kiến ​​của tôi) một chút dễ dàng hơn để đọc hơn

foreach (var foo in foos) 
{ 
    var task1 = new FooTask(foo); 
    task1.Run(); 
} 
foreach (var bar in bars) 
{ 
    var task2 = new BarTask(bar); 
    task2.Run(); 
} 

Answer 2

tôi chỉ kiểm tra mã của bạn vì thông thường, với VSCodeAnalysis và R # và StyleCop thở xuống cổ tôi chắc chắn sẽ có rất nhiều cảnh báo. Nhưng không. Tất cả đều giữ im lặng. Vì vậy, nó không có vẻ là chống lại bất kỳ hướng dẫn mã hóa của Microsoft.

Tuy nhiên: Nếu bạn cần tạo các khối ẩn danh thừa chỉ nhằm tránh lỗi trình biên dịch do tên biến, nó không còn dễ dàng và đơn giản nữa. Về cơ bản, bạn đang ẩn từ trình biên dịch và ẩn sẽ không bao giờ giải quyết vấn đề. Từ quan điểm của tôi, nó sẽ là cách tốt hơn mã nếu bạn đã chọn một tên chính xác cho mỗi biến (đặt tên cho loại này là không chính xác tuyệt vời anyway) và loại bỏ các khối ẩn danh.

Theo nguyên tắc KISS: "Hãy đơn giản, ngu ngốc".

Answer 3

Tôi sẽ thận trọng với cách tiếp cận này; nó có thể cản trở khả năng đọc và khả năng gỡ lỗi. Tôi đã có những tình huống trong quá khứ, nơi tôi đã có hai breakpoints xem hai biến khác nhau khi tôi nghĩ rằng tôi đã xem một.

Các quy tắc mà C# áp đặt để giúp đảm bảo rằng các phương pháp của bạn không sử dụng tên không nhất quán là phức tạp. Nếu bạn quan tâm tìm hiểu thêm về chúng, hãy xem:

http://blogs.msdn.com/b/ericlippert/archive/tags/simple+names/