2016-12-03 47 views
-2

我需要创建一个find()函数来搜索电影标题的LinkedList,但我不能。也许我在代码中做错了什么......任何人都有任何指针?如何更好地优化我的查找功能?

@Override 
public Movie find(String name) { 
    if (name == null) { 
     System.out.println("Not found"); 
    } else if (name != null) { 
     System.out.println(name + " found!"); 
    } 
    return movies.getFirst(); 
} 
+3

那么你究竟在哪里搜索? –

+0

如果此代码正在工作,您应该询问如何在[代码评论](http://codereview.stackexchange.com/)上对其进行优化。 – byxor

+5

你的代码没有多大意义。如果你将任何**字符串(甚至是null)传递给该函数,它总是会给你第一部电影。如果传入'null',它会显示“未找到”,然后无论如何给你一个电影。 – byxor

回答

0

不知道电影类究竟是如何布局,但假设其电影的链接列表,这里是遍历列表,直到它找到的电影您正在搜索(返回null标题代码如果找不到这种匹配的电影)。

Movie movie = movies.getFirst(); 
while (movie != null) { 
    if (movie.getTitle().equals(name)) { 
    return movie; 
    } 
    movie = movie.getNext(); 
} 
return null;