Cách tránh các lớp Trình trợ giúp hoặc Trình quản lý của Khác

Tôi có khá nhiều lớp Helper trong dự án của mình. Tôi đã đọc rằng đây là một điều xấu, nhưng tôi nghi ngờ rằng "Người trợ giúp" là hậu tố sai cho họ. Tôi sẽ đưa ra một ví dụ.

Đầu tiên, tôi có một Userlớp học. Tôi cần một phương pháp GetSuggestedFriends()cho người dùng. Tôi muốn giữ logic để xác định danh sách bạn bè được đề xuất ra khỏi Userlớp, vì vậy nó không bị phồng lên. Ngay bây giờ, tôi có một FriendshipHelpercái nhận được Usertrong hàm tạo của nó. Nó chứa logic để nhận được những người bạn được đề xuất và bây giờ tôi có thể gọi myUser.FriendshipHelper.GetSuggestedFriends().

Ban đầu chỉ FriendshipHelpercó các phương thức tĩnh và một Userđối tượng được truyền vào từng cái. Nếu bây giờ tôi đang viết lớp từ đầu, có lẽ tôi sẽ gọi nó FriendshipManager- nó cũng làm những việc như thêm và xóa bạn bè.

Tôi cũng đã đọc rằng ...Managercác lớp học là xấu, mặc dù. Tôi nên gọi lớp này là gì? Hoặc, đây là "mã xấu"? Logic ở đâu để có được những người bạn được đề xuất, những người bạn hiện tại và thêm và xóa những người bạn đang sống? Chắc chắn không phải tất cả trong một Userlớp học khổng lồ ?

Cách tránh các lớp Trình trợ giúp hoặc Trình quản lý của «

Nói chung: bởi thiết kế tốt

Đầu tiên, tôi có một lớp Người dùng. Tôi cần một phương thức GetSuggestedFriends () cho người dùng.

Đúng. A user có mối quan hệ với người khác users. Và mối quan hệ có thể được thể hiện như một phương pháp trên user, ví dụ user.isFriend(user2). Đây là trách nhiệm của user-object. Bên cạnh đó, bạn hỏi một đối tượng cho sự giúp đỡ việc tìm kiếm những người bạn khác . Bạn ủy nhiệm cho việc tìm kiếm bạn bè để đối tượng khác và đó là khá tốt .

Ngay bây giờ, tôi có FriendshipHelper nhận Người dùng trong hàm tạo của nó. Nó chứa logic để nhận được những người bạn được đề xuất và bây giờ tôi có thể gọi myUser.FriendshipHelper.GetSuggestedFriends ()

Đó không phải là cho mỗi gia nhập xấu, nhưng có một nhược điểm: khởi tạo "Helper" với một usergiới hạn các khả năng để một mà user.

Những gì bạn cần là một đối tượng , giúp tìm kiếm bạn bè cho bất kỳ người dùng nào . Vì vậy, một phương pháp chung sẽ có ý nghĩa: userMatcher.findFriendsFor(user)đó trong trở lại cung cấp một bộ sưu tập của bạn bè có thể ( user).

Nếu bây giờ tôi đang viết lớp từ đầu, có lẽ tôi sẽ gọi nó là FriendshipManager

Vấn đề của bạn không phải là viết "các lớp trợ giúp", mà là tìm đúng tên . ;)

nó cũng làm những việc như thêm và xóa bạn bè.

Đó là một thiết kế sai . Lấy bản thân bạn làm ví dụ: Mẹ của bạn có thêm bạn bè vào cuộc sống của bạn hay bạn tự thêm họ?

Tất nhiên các bộ sưu tập của bạn bè là một tài sản của userbản thân và như vậy là phương pháp user.addFriend(user)hayuser.removeFriend(user)

Tôi nên gọi lớp này là gì?

Như đã nói trước đây: bạn chỉ có một vấn đề đặt tên và "người trợ giúp" của bạn vẫn ổn . Nhưng bạn phải suy nghĩ kỹ hơn về trách nhiệm của từng đối tượng.

Logic ở đâu để có được những người bạn được đề xuất, những người bạn hiện tại và thêm và xóa những người bạn đang sống? Chắc chắn không phải tất cả trong một lớp Người dùng khổng lồ

Không. Đây là hai công việc mà bạn cần một đối tượng riêng biệt , như trong cuộc sống thực, nơi bạn có người và công ty hẹn hò .

Tôi muốn đề nghị bạn có một FriendshipServicelớp có GetSuggestedFriends(User)phương thức (không tĩnh) . Tránh các phương thức tĩnh vì bạn không thể thực hiện giao diện khiến việc kiểm tra khó khăn hơn. Tránh thêm đối tượng người dùng vào hàm tạo vì bạn có thể muốn mở rộng FriendshipService của mình bằng các phương thức không liên quan cụ thể đến một người dùng. (Ví dụ: bạn có thể muốn đề xuất bạn bè cho một nhóm người dùng hoặc đề xuất bạn bè dựa trên thứ khác)

Người dùng rất có thể không nhận thức được FriendshipService(do Mẫu Trách nhiệm duy nhất)