避免代码重复

Question

考虑以下代码：

if (matcher1.find()) {
   String str = line.substring(matcher1.start()+7,matcher1.end()-1);
   /*+7 and -1 indicate the prefix and suffix of the matcher... */    
   method1(str);
}
if (matcher2.find()) {
   String str = line.substring(matcher2.start()+8,matcher2.end()-1);
   method2(str);
}
...

我有 n 个匹配器，所有匹配器都是独立的（如果一个匹配器为真，则它对其他匹配器一无所知......），对于每个为真的匹配器 - 我对它匹配的内容调用不同的方法。
问题：我不喜欢这里的代码重复和“幻数”，但我想知道是否有更好的方法来做到这一点......？ （也许是访客模式？）有什么建议吗？

Answer 1

创建一个抽象类，并在子类中添加偏移量（也可以进行字符串处理......根据您的要求）。

然后将它们填充到列表中并处理该列表。

这是一个示例抽象处理器：

public abstract class AbsractProcessor {

    public void find(Pattern pattern, String line) {
        Matcher matcher = p.matcher(line);
        if (matcher.find()) {
            process(line.substring(matcher.start() + getStartOffset(), matcher.end() - getEndOffset()));
        }
    }

    protected abstract int getStartOffset();

    protected abstract int getEndOffset();

    protected abstract void process(String str);

}

Answer 2

只需使用捕获组标记要传递给方法的正则表达式部分。

例如，如果您的正则表达式是 foo.*bar 并且您对 foo 或 bar 不感兴趣，请将正则表达式设置为 foo(. *)栏。然后始终从 Matcher 中获取组 1。

然后，您的代码将如下所示：

method1(matcher1.group(1));
method2(matcher2.group(2));
...

进一步的一步是将您的方法替换为实现如下所示的类：

public interface MatchingMethod {
  String getRegex();
  void apply(String result);
}

然后您可以轻松地自动化该任务：

for (MatchingMethod mm : getAllMatchingMethods()) {
  Pattern p = Pattern.compile(mm.getRegex());
  Matcher m = p.matcher(input);
  while (m.find()) {
    mm.apply(m.group(1));
}

请注意，如果性能很重要，则预编译模式 可以提高运行时间。

Answer 3

你可以把它缩短一点，但我的问题是，这真的值得付出努力吗：

private String getStringFromMatcher(Matcher matcher, int magicNumber) {
   return line.subString(matcher.start() + magicNumber, matcher.end() - 1 )
}

if (matcher1.find()) {
method1(getStringFromMatcher(matcher1, 7);
}

if (matcher2.find()) {
method2.(getStringFromMatcher(mather2, 8);
}

Answer 4

将 Cochard 的解决方案与工厂（switch 语句）与所有 methodX 方法结合使用。所以你可以这样称呼它：

Factory.CallMethodX(myEnum.MethodX, str)

你可以在 Cochard 解决方案的填充步骤中分配 myEnum.MethodX