java - 这是一个有效的方法吗？ Eclipse 似乎暗示我的代码存在问题..我只是没有看到它

Question

我不知道为什么以下代码被 eclpise 认为是错误的。我一直在“hasPassed()”下方看到一条红色下划线......我已经盯着代码看了几个小时......

 public String hasPassed()
{
    if (totalMark ==100 && totalMark > passMark)
        return "Pass";

        else if (totalMark != 100)
            return "There is an error with the mark allocation";
}

Answer 1

您需要添加一个额外的返回，如果这 2 个条件不成立会怎样？

 public String hasPassed()
{
    if (totalMark ==100 && totalMark > passMark)
        return "Pass";

        else if (totalMark != 100)
            return "There is an error with the mark allocation";

     return "What i have to return"?
}

在这种情况下，IMO 重构您的方法更具可读性，但这是个人的。像这样。

public String hasPassed(){

  String toReturn = null;

  if (totalMark ==100 && totalMark > passMark){
          toReturn="Pass";
  }else if (totalMark != 100){
          toReturn= "There is an error with the mark allocation";
  }
     return toReturn;
}