标签:java sonarqube sonarlint static-code-analysis
所以我对SonarLint有一个问题,我不知道如何处理.
假设我有一个方法课
public class Class(RemoteContext context)
RemoteContext context = context;
public void String method(String data) {
if(data == null)
context.raiseException("data can't be null");
//do stuff with data like data.get();
}
当我用sonarLint(3.2.)分析这个类时,我得到一个Null指针不应该被解引用问题.
所以我的问题是.如何解决这个问题?
context.RaiseException将停止方法执行,所以我认为这是误报.
该应用程序有很多情况(类/方法)有这个问题.
所以我认为注释是一种矫枉过正(丑陋的代码)
我也可以在每次raiseException()调用后输入return,但我的印象不是“程序员方式”.
我猜写自己的规则是最好的.
我正在查看这些主题并让我谷歌搜索,但没有找到任何有用的这种情况,当我有点必须做声纳实际做的“相反”.
不是提出问题,而是对方法“开绿灯”?
希望我对这个问题很清楚.
解决方法:
如果RemoteContext是您控制的类,并且您真的不想使用通常的新ExceptionType(…)模式,我会更改RemoteContext以构建异常但不抛出它,然后
if (data == null) {
throw context.buildException("data can't be null");
}
…因此SonarLint,Java编译器以及稍后对代码执行工作的程序员都很清楚,该方法的执行在此时停止(因为“引发异常”可能意味着很多事情).
(是的,这意味着要改变你拥有它的很多地方,但相对简单的搜索和替换可以实现这一点.)
标签:java,sonarqube,sonarlint,static-code-analysis 来源: https://codeday.me/bug/20190727/1550735.html
本站声明: 1. iCode9 技术分享网(下文简称本站)提供的所有内容,仅供技术学习、探讨和分享; 2. 关于本站的所有留言、评论、转载及引用,纯属内容发起人的个人观点,与本站观点和立场无关; 3. 关于本站的所有言论和文字,纯属内容发起人的个人观点,与本站观点和立场无关; 4. 本站文章均是网友提供,不完全保证技术分享内容的完整性、准确性、时效性、风险性和版权归属;如您发现该文章侵犯了您的权益,可联系我们第一时间进行删除; 5. 本站为非盈利性的个人网站,所有内容不会用来进行牟利,也不会利用任何形式的广告来间接获益,纯粹是为了广大技术爱好者提供技术内容和技术思想的分享性交流网站。