Question

我有一个程序，它从几个输入文件中读取数据，每个文件都是逐行处理的。读取文件每一行的代码是相同的，但提取数据的格式和要求因文件而异。更具体地说，我有一个存储键/值对的配置文件和一个记录文件 - 每行存储一条记录。

我注意到读取文件所需的代码实际上是相同的，但每行“处理”代码是不同的。因此，像任何优秀的OOP学生一样，我想提取重复的代码并将其封装在抽象类中。这是抽象类的接口：

abstract class FileProcessor {
  public processFile() {};
  protected abstract processLine(aLine); // implemented by the subclass
  public abstract getData();
}

因此，processFile包含“样板文件”文件处理代码。 processFile方法调用抽象的processLine方法，该方法在子类中实现，并允许子类特定的文件处理操作。现在我有以下子类。

public ConfigFileProcessor extends FileProcessor {
  processLine(String line) {
    // do some file-specific processing
  }
}

public RecordFileProcessor extends FileProcessor {
  processLine(String line) {
    // do some file-specific processing
  }
}

问题是getData方法。正如我之前所说，每个文件中的数据都不同 - 在这种情况下，一个包含键/值对，另一个包含记录。如何以一般（非特定于特定的）方式定义getData的返回类型？我正在考虑返回一个Collection并使用通配符来指定类型，但是我没有足够的经验来理解这个选择的含义/权衡。

设计可以改进吗？在这种情况下，是否值得经历封装的麻烦？我想我可以创建两个不同的processFile方法来返回不同的数据类型，但是如果可能的话，我想把我的OOP教育付诸实践。

提前感谢您的时间和精力。

Answer 1

泛型是正确的方法。让抽象类使用泛型，每个子类定义泛型。

abstract class FileProcessor<T> {
  public void processFile() {};
  protected abstract T processLine(aLine); // implemented by the subclass
  public abstract T getData();
}

public ConfigFileProcessor extends FileProcessor<Configuration>{ ... }
public RecordFileProcessor extends FileProcessor<Record>{ .. }

抛开：你的抽象类不编译。没有指定返回类型。

不同的退货类型

1 个答案: