2016-12-03 64 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;