Khai báo các biến trả về trong các phương thức c # so với trả về giá trị trực tiếp

Trong một cuộc tranh luận liên quan đến các biến trả về, một số thành viên của nhóm thích một phương thức trả kết quả trực tiếp cho người gọi, trong khi những người khác thích khai báo một biến trả về sau đó được trả về cho người gọi (xem ví dụ mã bên dưới)

Đối số sau này là nó cho phép nhà phát triển gỡ lỗi mã tìm giá trị trả về của phương thức trước khi trả về cho người gọi, do đó làm cho mã dễ hiểu hơn: Điều này đặc biệt đúng khi các lệnh gọi phương thức được nối tiếp.

Có bất kỳ hướng dẫn nào là hiệu quả nhất và / hoặc có bất kỳ lý do nào khác tại sao chúng ta nên áp dụng một phong cách hơn một phong cách khác không?

Cảm ơn

    private bool Is2(int a)
    {
        return a == 2;
    }

    private bool Is3(int a)
    {
        var result = a == 3;
        return result;
    }

Bởi vì tôi sử dụng Resharper với Visual Studio, Ctrl-RV (Hoặc Ctrl-Alt-V, nếu bạn sử dụng các ràng buộc phím Resharper / IntelliJ) sẽ biến ví dụ đầu tiên của bạn thành ví dụ thứ hai. Vì vậy, khi tôi muốn gỡ lỗi, tôi có thể làm điều đó đủ dễ dàng. Và nếu tôi quên đặt lại thì tôi sẽ không cảm thấy tệ vì Ctrl-RI sẽ đặt nó trở lại một lần nữa để dễ đọc hơn.

Nghiêm túc, lãng phí thời gian của bạn để tranh luận về những điều quan trọng hơn. Giống như nơi để đặt dấu ngoặc nhọn hoặc khoảng trắng hàng đầu của bạn so với các tab.

Cá nhân tôi thấy ví dụ đầu tiên dễ đọc hơn. Bạn vẫn có thể gỡ lỗi nó, bằng cách đặt điểm dừng trong câu lệnh return và thêm a == 2vào cửa sổ đồng hồ hoặc bằng cách sử dụng đồng hồ nhanh.

Nhưng đây thực sự là một vấn đề sở thích cá nhân. Cả hai phiên bản đều ổn.

Khi mã dễ đọc như ví dụ của bạn, không có gì sai khi trả về kết quả của một hoạt động logic như return a == 2. Tuy nhiên, nếu giá trị trả về là một câu lệnh phức tạp hơn hoặc trông giống như

return a > 2? doOptionA().getResult() > makeDecision("greaterThan2") : doOptionB().getResult() == makeDecision("lessThan2");

sau đó bạn sẽ muốn sử dụng các biến để lưu trữ các phần đó trước và đơn giản hóa câu lệnh return, để dễ đọc.

Trong một ví dụ đơn giản như vậy, một trong hai là OK.

Đối với các ví dụ phức tạp hơn, tôi thích cách thứ hai. Đó chỉ là vì nó dễ đọc hơn và những người khác có thể sẽ phải duy trì mã.