在循环中间返回和/或中断。它可以被接受吗？

Question

假设我们有一个整数数组。我们编写了一个函数来获取数组中第一个指定值的索引，如果数组不包含该值，则获取 -1。

例如，如果 array = { 4, 5, 4, 4, 7 }，那么 getFirstIndexOf(4) 将返回 0，getFirstIndexOf(7) 将返回 4，getFirstIndexOf(8) 将返回代码> 会返回-1。

下面，我介绍了编写此函数的三种不同方法。这是一种广泛接受的编码标准，在函数中间返回，在循环中间中断都是不好的做法。在我看来，这对他们来说可能是可以接受的用途。

public int getFirstIndexOf(int specifiedNumber) {
  for (int i = 0; i < array.length; i++) {
    if (array[i] == specifiedNumber) {
      return i;
    }
  }

  return -1;
}

VS。

public int getFirstIndexOf(int specifiedNumber) {
  int result = -1;
  for (int i = 0; i < array.length; i++) {
    if (array[i] == specifiedNumber) {
      result = i;
      break;
    }
  }

  return result;
}

VS。

public int getFirstIndexOf(int specifiedNumber) {
  int result = -1;
  for (int i = 0; i < array.length; i++) {
    if (array[i] == specifiedNumber && result == -1) {
      result = i;
    }
  }

  return result;
}

你怎么认为？哪个最好？为什么？也许还有另一种方法可以做到这一点吗？

Answer 1

我认为当您已经找到结果时运行完整循环是很糟糕的做法...

如果您确实想避免使用从循环中间返回，我建议使用“哨兵”来停止循环。

public int getFirstIndexOf(int specifiedNumber, int[] array) {

    boolean found = false;
    boolean exit = false;
    int i = 0;
    int arraySize = array.length();

    while(!found && !exit) {
        if(array[i] == specifiedNumber) {
            found = true;
        } else {
            if(i++ > arraySize) {
            exit = true;
            }
        }

    if(found ==true) {
       return i;
    } else {
       return 99999;
    }
}

编辑：我讨厌在 StackOverflow 中使用空格来缩进代码...

Answer 2

这就是为什么 do...while & while循环被发明。

根据要求：

public int getFirstIndexOf(int specifiedNumber) {
    int i = array.Length;
    while(--i > -1 && array[i] != specifiedNumber);

    return i;       
}