Java 我没有变白这项工作？

当我输出这个

album1.findSongInAlbum（）

方法时，无论我输入什么，我都会得到“不工作”或者如果我输入了正确的名称，我会得到

album song found

，同样也会得到

not working

public boolean findSongInAlbum(String songName) {
    for (int i = 0; i < this.albumContainingListOfSongs.size(); i++) {
        if (songName.equals(albumContainingListOfSongs.get(i).getTitle1())) {
            System.out.println("Album song  found: " + albumContainingListOfSongs.get(i).toString());
        } else if (!songName.equals(albumContainingListOfSongs.get(i).getTitle1())) {
            System.out.println("Not workin");
            return false;
        }

    }
    return true;
}

public boolean findSongInAlbum（字符串songName）{
for（int i=0；i

输出应该是一个或另一个，要么是找到的歌曲，要么是未找到的歌曲，但决不能两者兼而有之。

您的逻辑是错误的

如果找到匹配项，应立即返回true

如果没有找到匹配项，则应留在循环中，并且不要返回false。仅在循环完成后返回false

这应该起作用：

public boolean findSongInAlbum(String songName) {
    for (int i = 0; i < this.albumContainingListOfSongs.size(); i++) {
        if (songName.equals(albumContainingListOfSongs.get(i).getTitle1())) {
            System.out.println("Album song found: " + albumContainingListOfSongs.get(i).toString());
            return true;
        }
    }
    System.out.println("Album song not found");
    return false;
}

public boolean findSongInAlbum（字符串songName）{
for（int i=0；i

这可能是因为您对每个不匹配项都输出“not workin”。如果没有匹配项，你只需要在最后做一次。我会在你最喜欢的IDE中逐步浏览代码，看看发生了什么。是的，完美无缺，非常感谢你的回答！！否则，如果返回值为false完全不必要，则会破坏循环：/