如何命名只调用其他方法的类/方法？

Question

假设我遵循单一职责原则，并且我有以下课程。

public class Extractor {

   public Container extract(List<Container> list) {

       ... some extraction
   }
}

public class Converter {

   public String convert(Container container) {

       ... some conversion
   }
}

正如您所看到的，它遵循原则，所有类/方法的名称都说明了它们的作用。现在我有另一个类，它有这样的方法。

public class SomeClass {
   private Extractor extractor = new Extractor();
   private Converter converter = new Converter();
   private Queue queue = new Queue();

   public void someMethod(List<Container> list) {
       Container tmp = extractor.extract(list);
       String result = converter.convert(tmp);

       queue.add(result);
   }
}

正如您所看到的“someMethod”方法确实调用了提取、转换和添加。我现在的问题是，如何调用这样的类/方法？它实际上并不是提取、转换或添加，而是调用这些？ 如果您按照该方法的职责来命名该方法，那会是什么？

Answer 1

如果您希望名称真正通用，我会使用 addToQueue() 或 populateQueue() ，因为将某些内容放入该对象似乎是该方法的重点。

但实际上在这个级别上，我会根据它试图完成的业务逻辑来命名它，在这种情况下，名称实际上取决于它的用途。

---

如果您无法想出一个好名字，则表明您的程序抽象相当任意/人为，并且可能暗示可能有更好的方法来做到这一点。或者也许不是。

Answer 2

听起来像是某种构建器类。您获取一种格式的数据，对其进行转换，然后创建某种输出格式。那么“SomethingSomethingBuilder”怎么样？

我假设有人对我投了反对票，因为我忘记为该方法提供一个好名字。对此感到抱歉。

因此，此方法将增量数据添加到您的构建器类中。我会称其为“Add”、“AddData”或“Push”（我可能会选择“push”，因为这在许多标准类中具有非常相似的含义）。

“Builder”的替代方案可能是“SomeKindOfCreator”。显然，您可以根据您的班级实际创建的内容来命名它。

Answer 3

好吧，既然您似乎添加到队列中并且没有返回任何内容，我将其称为 addToQueue。您转换+提取的事实是我认为不需要公开的实现细节。

Answer 4

processAndQueueMessage 怎么样？

另外（不相关），您不应该在 SomeClass 中创建（使用 new）Extractor 和 Converter ，您应该注入它们（在构造时或在设置器中），并使用它们的接口。这将使测试变得更容易，并减少实现之间的耦合。

// Assuming Converter and Extractor are interfaces to the actual implementations
public class SomeClass {
   private final Extractor extractor ;
   private final Converter converter;
   private Queue queue = new Queue();

   public SomeClass(Extractor extractor, Converter converter) {
       this.converter = converter;
       this.extractor = extractor;
   }

   public void someMethod(List<Container> list) {
       Container tmp = extractor.extract(list);
       String result = converter.convert(tmp);

       queue.add(result);
   } 
}

您可以使用以下方法创建它：（

final SomeClass myProcessor = new SomeClass(new MyExtractorImplementation(), new MyConverterImplementation());

或者使用 DI 容器，例如 Spring 或 Pico）

Answer 5

您要做的就是考虑方法调用序列的复合含义，将其转换为简洁的动词或动词短语，并将其用作名称。如果您无法想出一个简洁的名称，那么您可以使用通用/中性名称（例如“process”）或使用完全虚假的名称（例如“sploddify”）。