Tôi đã tạo một phương thức xác thực riêng cho một xác nhận nhất định xảy ra nhiều lần trong lớp của mình (Tôi không thể lưu trữ dữ liệu được xác thực vì nhiều lý do). Bây giờ, ReSharper gợi ý rằng chức năng có thể được thực hiện tĩnh. Tôi hơi miễn cưỡng khi làm như vậy do các vấn đề đã biết với các phương thức tĩnh. Nó sẽ là một phương thức tĩnh riêng . Câu hỏi của tôi là, các phương thức tĩnh riêng có thể gây ra các vấn đề khớp và kiểm tra tương tự như các phương thức tĩnh công khai không? Đó có phải là một thực hành xấu? Tôi đoán là không, nhưng tôi không chắc có cạm bẫy ở đây không.
10
"Các vấn đề đã biết" với các phương thức tĩnh là gì?
—
Robert Harvey
@Ed: Phải rồi. Các phương thức tĩnh được viết đúng không nên chạm vào trạng thái hoặc trạng thái của API bên ngoài. Thao tác trạng thái nội bộ được gói gọn trong một lớp có vẻ hoàn toàn ổn đối với tôi và phương thức sẽ không cần phải được kiểm tra đơn vị, vì kiểm thử đơn vị kiểm tra hành vi bên ngoài của lớp.
—
Robert Harvey
Các phương thức tĩnh có xu hướng sửa đổi trạng thái toàn cầu và chúng cũng giết chết tính kế thừa (bất cứ khi nào bạn muốn mở rộng chức năng, bạn sẽ cần sửa đổi mã cuộc gọi vì bạn không thể ghi đè phương thức trong lớp dẫn xuất). Của bạn được gắn với một thực hiện. Các phương thức tĩnh không thể bị chế giễu khiến chúng rất khó kiểm tra đơn vị. Họ che giấu sự phụ thuộc . Tôi chắc chắn có nhiều hơn nữa. Không chỉ mù quáng tránh họ, tôi đang yêu cầu đưa ra quyết định sáng suốt.
—
Tamás Szelei
@ Tamás Phương thức tĩnh chỉ sửa đổi trạng thái toàn cầu nếu bạn viết chúng theo cách đó, điều mà tôi không bao giờ làm. Nói chung, tôi chỉ sử dụng các phương thức tĩnh trong các lớp tiện ích, các phương thức lấy một hoặc nhiều đối tượng và trả về một đối tượng mà không có tác dụng phụ. Những loại phương pháp này không có vấn đề nào bạn mô tả.
—
Robert Harvey
@ TamásSzelei Họ có xu hướng sửa đổi trạng thái toàn cầu như thế nào? Họ thậm chí không thể tìm thấy trạng thái toàn cầu trừ khi bạn chuyển nó vào một tham số.
—
CodeInChaos