为什么我的 getIndex 方法不起作用？

Question

我编写了一个名为 getIndex 的方法，当我运行测试器时它不会返回数字。

public int getIndex(String s){
    int index = -1;
    for(int i = 0; (i < names.length && index == 1); i++)
    {
        if(names[i].equals(s)) {
            index = i;
        }        
    }
    return index;
}

如果不清楚，我试图让方法返回给定字符串在数组中的哪个位置。

编辑—— [这是我在 ideone 上的代码] (http://ideone.com/CM9xID)

Answer 1

如果比较 index == 1，则循环永远不会开始，因为 index 是用 -1 声明的。如果你用 1 声明你的索引，循环会开始，但如果 names[1] 不等于 s，则循环将无限运行，所以只需删除它就可以了。

public int getIndex(String s){
    for(int i = 0; (i < names.length; i++)
    {
        if(names[i].equals(s)) { //use equalsIgnoreCase if comparing should be case insensitive
            return i; //found
        }        
    }
    return -1; //not found
}

执行循环的另一个版本是（如果您只想要索引，您可以直接使用 indexOf()，如上一个示例所示）：

List<String> names2 = new ArrayList<String>(Arrays.asList(names));
for(String name : names2){
    if(name.equals(s))
        return names2.indexOf(name);
}

这是对loop 的增强。

也可以快捷方式直接获取索引：

 List<String> names2 = new ArrayList<String>(Arrays.asList(names));
 return names2.indexOf(s);

Ideone Fiddle

请注意，您始终只能获得第一次出现的索引。如果 s 在数组中不止一次，则必须对其进行增强以存储找到的索引并返回所有索引！

public List<Integer> getIndexes(String s){
    List<Integer> found = new ArrayList<Integer>();
    for(int i = 0; (i < names.length; i++)
    {
        if(names[i].equals(s)) { //use equalsIgnoreCase if comparing should be case insensitive
            found.add(i);
        }        
    }
    return found;
}

Answer 2

您的循环条件立即为假。

你的代码说：

同时 0 < LENGTH_OF_NAMES AND -1 is equal to 1

-1 从不等于 1，所以你的循环永远不会发生

变化：

for(int i = 0; (i < names.length && index == 1); i++)

收件人：

for(int i = 0; i < names.length; i++)