如何在eclipse中提取将字段或常量值转换为参数的方法

Question

情况：我正在研究遗留代码并尝试提高可读性。 以下示例应该可视化意图：

private static final String CONSTANT_1 = "anyValue";
private static final String CONSTANT_2 = "anyValue";
private static final String CONSTANT_3 = "anyValue";
private static final String CONSTANT_4 = "anyValue";
private static final String CONSTANT_5 = "anyValue";

private final SomeType someField = new SomeType();

private void contentOfSomeMethods(){
    someMethod(someField, CONSTANT_1, true);
    someMethod(someField, CONSTANT_2, true);
    someMethod(someField, CONSTANT_3, true);
    someMethod(someField, CONSTANT_4, false);
    someMethod(someField, CONSTANT_5, false);
}

private void someMethod(SomeType type, String value, boolean someFlag) { }

想象一下，使用大约50个常量大约有50个someMethod调用。 我想对该代码进行安全的自动重构，以便contentOfSomeMethods方法更改为

private void contentOfSomeMethods(){
    doItWith(CONSTANT_1);
    doItWith(CONSTANT_2);
    doItWith(CONSTANT_3);
    doItNotWith(CONSTANT_4);
    doItNotWith(CONSTANT_5);
}

并生成两个额外的方法：

private void doItWith(String value) {
    someMethod(someField, value, true);
}

private void doItNotWith(String value) {
    someMethod(someField, value, false);
}

天真的方法是提取局部变量中的contentOfSomeMethods所有常量，然后使用提取方法重构来创建所需的方法。 然后内联返回局部变量。 但是这种解决方案并没有扩大规模 。

另一种方法是使用搜索和替换正则表达式，但这不是一个安全的重构 ，所以我可以打破代码而不注意它。

你有什么更好的建议吗？ 你知道Eclipse的一些插件吗？

Answer 1

我不知道任何可以直接执行此操作的实用程序。

我认为使用正则表达式是唯一可行的。 首先，您需要创建两个目标方法doItWith和doItNotWith 。 然后，您可以突出显示方法contentOfSomeMethods的内容， 按Ctrl + F ，并使用以下正则表达式：

查找： someMethod\\(someField, (\\w*), true\\);
替换为： doItWith(\\1);

接着

查找： someMethod\\(someField, (\\w*), false\\);
替换为： doItNotWith(\\1);

请务必选中“正则表达式”和“选定行”。 这是它的图片：

正则表达式将函数调用中使用的常量与(\\w*)匹配，然后在替换为\\1使用它。 仅在选定行上使用此正则表达式可以最大限度地减少破坏不相关代码的可能性。

Answer 2

使用正则表达式进行验证并验证它。

我假设每次调用someMethod只跨越一行。 如果不是这种方法仍然有用但速度较慢。

复制原始文件。

使用ctrl + alt + h显示someMethod并获取它们的计数。

正则表达式搜索和替换限制在适当的区域：
查找：someMethod（someField，（[] * CONSTANT_ [0-9] +）[] *，[] * true [] *）[] *;
替换：doItWith（“$ 1”）;
查找：someMethod（someField，（[] * CONSTANT_ [0-9] +）[] *，[] * false [] *）[] *;
替换：doItNotWith（“$ 1”）;

对原始文件和新文件进行差异显示仅显示已更改的原始文件的行。

diff --changed-group-format='%<' --unchanged-group-format='' original.java refactored.java | wc

您应该获得与someMethod的调用者相同的行数。

如果对someMethod的调用是多行的，或者如果您想要更多的验证，只需删除| wc | wc查看原始文件中修改的行，以确保只修改了正确的行。

Answer 3

唉，我在Eclipse中一无所知，今天就可以做到这一点。 这是我想在AutoRefactor中实现的一天： https ： //github.com/JnRouvignac/AutoRefactor/issues/8

然而，到达那里的道路很长。

我今天知道的唯一方法是提取局部变量然后提取方法（如你所建议的）或使用正则表达式（如其他人建议的那样）。