SonarQube抱怨要更改此条件,以便它不会始终评估为" true"

时间:2018-04-24 09:29:47

标签: c# sonarqube code-analysis

我在第2行遇到这个问题,而在解决声纳问题时,请建议替代解决它。

Exception exe = Server.GetLastError();
  if (exe != null) {
   Exception errorInfo = exe.GetBaseException();
   var error = errorInfo as HttpException;
   if (error != null)
    isNotFound = error.GetHttpCode() == (int) System.Net.HttpStatusCode.NotFound;
  }

1 个答案:

答案 0 :(得分:0)

您在评论中附加的图片有很多帮助。

有一行包含NLogManager.Error(... + exe.StackTrace);如果执行正确,则exe不是null,因此if语句是多余的。

分析器不会在exe.StackTrace上引发问题,因为此时它不确定exe是否可以为空(之前没有检查)。

我建议在NLogManager语句中使用if移动该行。