为什么eclipse说我的方法没有返回有效结果？

Question

我在 Java 中为数独游戏编写这段代码已有一段时间了，我不知道出了什么问题，也许是“if”或 de “For”，但 IDE 说我的方法没有返回一个布尔类型。

// check if the number has already been used in the columns
private boolean checkColumns(int x, int y, int value) {
    for (int j = 0; j < 9; j++) {
        if (this.gridPlayer[j][y].getValue() == value) return false;
        else return true;
    }

    }
// Check if the number has already been used in the lines
private boolean checkLines(int x, int y, int value) {
    for (int i = 0; i <= 9; i++) {
        if (this.gridPlayer[x][i].getValue() == value) return false;
         else return true;
    }
    }

// Check if the number has already been used and the subGrid
private boolean checkSubGrid(int x, int y) {
    for (int i = 0; i <= 9; i++) {
        for (int j = 0; j <= 9; j++) {
            if (this.gridPlayer[x][y].getValueOfSubGrid(x, y) == this.gridPlayer[i][j].getValueOfSubGrid(i, j)) {
                if (this.gridPlayer[x][y].getValue() == this.gridPlayer[i][j].getValue()) {
                    return false;
                } else {
                    return true;
                }
            } else if (this.gridPlayer[x][y].getValueOfSubGrid(x, y) != this.gridPlayer[i][j].getValueOfSubGrid(i,
                    j)) {
                return true;
            }
        }
    }
}

Answer 1

欢迎，
在您的checkSubGrid() 方法中，如果运行时未输入最后一个else if，则需要返回一个值： else if (this.gridPlayer[x][y]...) {

如果方法不是void，需要放一个return。

 if(a > 1) {
   return a;
 } else {
   return b;
 }

在上面这种情况下，我们有一个if - else 语句，该方法将总是返回真或假（或有异常）。

 if(a > 1) {
   return a;
 } else if(a == 0) {
   return b;
 }

另一方面，方法可以或不能输入第二个if，他们没有回报。您不确定编译器是否会返回。

您可以通过设置默认返回或设置 else 语句来解决此问题。

 if(a > 1) {
   return a;
 } else if(a == 0) {
   return b;
 } else {
   return null;
 }

或者

 if(a > 1) {
   return a;
 } else if(a == 0) {
   return b;
 }
 return null;

Answer 2

编译器假设它不是 100% 确定来自内部的 return 语句 您的“for”循环将被调用，因此它会看到您的方法不返回任何值的路径，即使它们声明它们也是如此。

您需要在循环之外有一些返回值，即使您确定这永远不会发生，即

private boolean checkLines(int x, int y, int value) {
  for (int i = 0; i <= 9; i++) {
    if (this.gridPlayer[x][i].getValue() == value) return false;
     else return true;
  }
 return false; //even if you think it will never be run it is necessary 
}