public class KV<K, V> {
public K key;
public V value;
public KV(K key, V value) {
this.key = key;
this.value = value;
}
}
我正在尝试重构类变量value
,它发生在适当的位置。这意味着没有弹出对话框;我按回车键,它会尝试对整个项目进行重构,包括注释和其他内容,包括:
<%--<link href="<c:url value="../core/core.css" />" />--%>
在一个。jsp文件。这太“聪明”了,无法尝试重构与整个项目相匹配的注释。这通常会导致很多bug风险,并且在Java环境中重构不再安全。
同样的事情也发生在Intellij 12中。说真的,我不需要Intellij来推荐任何被认为不安全的东西,或者当它不确定是同一件事的时候!
我可以排除重构,但我没有时间每次都评估五个“建议”。这只会增加人为错误的机会:大多数时候我只是按回车键,哇,事情被重构了。
当Java环境有时试图替换中的内容时,重构也是一个主要问题。js文件。说真的,这必须停止。
没有弹出对话框,我无法取消选中“搜索字符串”。即使勾选了该选项,Intellij在默认情况下也不应包含建议,尤其是当建议位于当前文件之外时。它也可以建议重构它们,但默认情况下应该将它们排除在外。也就是说,它应该是一种选择加入功能,而不是默认情况下销毁所有内容。
这是最近所谓的“智能”Intellij重构的一个严重的用户体验问题。重构JS文件时,我不想在Java文件中搜索注释或字符串!句号!反之亦然!
安全第一!如果需要,知道自己在做什么的开发人员将自己搜索字符串。在动态语言环境中,它使Intellij无法使用,因为通常情况下,如果没有任何清晰的模式,有时会进行重构,有时会在整个项目中更改内容,等等。
应该有一个选项说,“仅相对于此文件或100%推断时重构!”,尤其是对于动态语言!对于静态语言,它甚至不应该尝试在文件之外查找注释和字符串。
我并不是有意将其公开,但我在两年多前的bugtracker中提出了这个问题,但没有人注意到。
编辑
对于那些认为我可能会走得更远的人,我刚刚尝试了一下:
对于此类:
public class KV<K, V> {
public K key;
public V val;
public KV(K key, V val) {
this.key = key;
this.val = val;
}
}
并将其添加到任何Java类中,例如:
public class CoreConfig {
String abc = "kv.val";
String def = "somethingElse.val";
}
重构时。val像以前一样,我得到了以下建议,一个来自灾难的输入,以及一些我必须一次评估和排除的东西。这需要努力,而且很烦人,也很危险。就像有人大喊:住手!然后,哦,一分钟后什么都没有,沮丧和一篇1000字的文章(这篇)。
说真的,有没有办法阻止这种危险行为!?有什么原因可以解释为什么默认情况下会启用此选项??
编辑20200706
2020年大便仍在继续:https://youtrack.jetbrains.com/issue/IDEA-245370
当您按Shift F6(重构重命名)
两次时,它会打开对话框,您可以禁用“在注释和字符串中搜索”
有什么原因可以解释为什么默认情况下会启用此选项??
是的,有。现在人们倾向于使用太多的DSL、模板和语言注入。在普通的旧Java中,所有这些东西主要以字符串的形式存在。反射API还将方法/类名表示为字符串。任何IDE都不可能支持并识别它们,所以最好为您提供一个安全网,以防可能的重构错误。但是,如果您具有出色的单元测试覆盖率,那么您在这里可能是安全的。
至于注释,它们通常包含代码示例。这些代码块在重命名过程中很快就会过时,所以当IDE提醒您它们的存在时,这会很有帮助。
当然,这种检查有时会导致误报。但是,如果您为变量提供描述性的不言自明的名称(我的意思不是“var”或“val”),这不太可能发生。因此,IDE正在以某种方式推动您采用更好的代码风格。
如果你仍然不相信,那么按照@Meo的建议,禁用字符串和注释中的搜索。