Có bao giờ là một ý tưởng tốt để sử dụng tên mẫu thiết kế trong các lớp thực hiện? [đóng cửa]

Gần đây tôi tình cờ gặp một codebase trăn khá lớn với rất nhiều MyClassAbstractFactory, MyClassManager, MyClassProxy, MyClassAdapter, vv lớp.

Mặc dù một mặt, những cái tên đó đã chỉ cho tôi nghiên cứu và tìm hiểu các mẫu tương ứng, nhưng chúng không được mô tả nhiều về những gì lớp học làm .

Ngoài ra, họ dường như nằm trong danh sách cấm của các từ trong lập trình: variable, process_available_information, data, amount, compute: tên quá rộng, mà không cho chúng tôi bất cứ điều gì về chức năng khi được sử dụng bởi bản thân .

Vậy nên có CommunicationManagerhay không PortListener? Hoặc có lẽ tôi không hiểu vấn đề gì cả ...?

• AbstractFactorythực sự là một sự lựa chọn nghèo nàn cho một cái tên Không có cách nào để biết những gì được tạo ra bởi nhà máy này và khi bạn tìm kiếm một thực thể tạo ra Animals, bạn sẽ không bao giờ tìm thấy nhà máy tương ứng theo tên.

• AnimalAbstractFactorycũng không phải là một lựa chọn khôn ngoan, vì trong hầu hết các ngôn ngữ, nó sẽ là dư thừa với abstracttừ khóa trong chữ ký.

  Điều này đang được nói, có một số lý do chính đáng, được đánh dấu bằng các bình luận, để thực sự bao gồm Abstracttrong tên: không chỉ có một số bối cảnh mà bạn không có chữ ký đầy đủ, mà chỉ là tên, mà còn, giữ AnimalFactorycho một giao diện có thể là một lựa chọn khôn ngoan (trừ khi, không may, quy ước của ngôn ngữ / khung là giao diện tiền tố với I).

• AnimalCreationUtilitycũng sẽ là một lựa chọn tồi: nếu đó là một nhà máy , hãy làm mọi thứ dễ dàng hơn cho những người sẽ đọc mã và gọi nó là một nhà máy .

• abstract AnimalFactoryđược rồi Nó không có sự dư thừa, và rõ ràng đó là một nhà máy trừu tượng , ủy thác việc tạo ra động vật cho trẻ em của nó.

Vì vậy, có, bao gồm tên của mẫu thiết kế là một ý tưởng tốt, nhưng nó chỉ nên là một phần của tên, và không nên thừa với các phần khác của chữ ký.

Phụ thuộc vào ví dụ cụ thể. Mẫu Builder hầu như luôn được phục vụ tốt nhất bằng cách đặt tên lớp * Builder của bạn, trong khi Singleton thường không cần phải được đặt tên như vậy.

Nếu bạn không đặt tên mẫu trong tên lớp của mình và thậm chí nếu bạn làm như vậy, bạn thường nên đặt một nhận xét trong lớp giải thích rằng nó thực hiện một mẫu cụ thể.

Toàn bộ quan điểm của việc sử dụng tên mẫu trong các lớp là để dễ hiểu những gì lớp làm. Nếu bạn đặt tên cho lớp AnimalFactory thì rõ ràng là lớp tạo ra các thể hiện Animal. Nếu tên của lớp của bạn bao gồm tên của một mẫu và nó không mô tả những gì nó đã chọn một mẫu sai hoặc thực hiện nó không chính xác.

Tôi nghĩ rằng nó có thể làm việc thực sự tốt. Ví dụ:

// Command for retrying card entry with CVN.
public class RetryCardEntryWithCVNCommand { ... }

// Query for getting expired accounts
public class GetExpiredAccountsQuery { ... }

// Decorator for logging exception. Implies that it's an additional 
//mechanism for logging exceptions.
public class LogExceptionToDbDecorator { ... }

// Factory for creating account filters
public class AccountFilterFactory { ... }