arraylist：比較對象與字符串

Question

嗨，大家好，誰能告訴我哪裏會出錯？

這個類的基本目的是定義一個喜歡的物品arraylist在這種情況下是關於汽車。汽車物品有汽車名稱和汽車1-5的評級。

如何查看字符串是否等於汽車物體等級。 即時通訊組件將你比較一個字符串或整數的數組列表中的汽車對象。我的equals（）方法有什麼問題？可以包含（）方法的工作方式相同嗎？

numberOfItemsOfRating方法允許用戶指定評級，因此該方法返回評級爲no的汽車。 searchForItems方法檢查指定的字符串描述是否與數組列表中的汽車名稱相匹配，並因此以arraylist方式返回汽車。

這裏是我的兩個方法與構造函數和變量一瞥：

public class FavouriteItems 
{ 
    private ArrayList<Item> cars; 

    /** 
    * Constructor for objects of class FavouriteItems 
    */ 
    public FavouriteItems() 
    { 
     cars= new ArrayList<Item>(); 

    } 

    /** 
    * Add a new Item to your collection 
    * @param newItem The Item object to be added to the collection. 
    */ 
    public void addToFavourites(Item newItem) 
    { 
     cars.add(newItem); 

    } 
    /** 
    * Count the number of Items with a given rating 
    * @return The number of Items (Item objects) 
    *   whose rating is rating (could be 0). 
    *   If the rating parameter is outside the valid 
    *   range 1..5 then print an error message and return 0. 
    */ 
    public int numberOfItemsOfRating(int rating) 
    { 
     int counter = 0; 
     if(rating >= 1 && rating <=5) 
     { 
      for (int i =0; i < cars.size(); i++) 
      { 
       int num = rating; 
       String al = Integer.toString(rating); 
       if(cars.get(i).equals(al)) 
       { 
        counter++; 
       } 
      } 
     } 
     else 
     { 
      System.out.println("No cars match your ratings"); 
      counter = 0; 
     } 
     return counter; 
    } 

    /** 
    * Find the details of a Item given its description 
    * @return Item object if its description is in the collection 
    * or null if there is no item with that description 
    */ 
    public Item searchForItem(String description) 
    { 
     for(int i=0; i<cars.size(); i++) 
     { 
      if(cars.equals(description)) 
      { 
       return cars.get(i); 
      } 
      else 
      { 
       return null; 
      } 
     } 
     } 
} 

Answer 1

這不是您應該使用equals方法的方式，而是我建議您使用或實現Item#getRating()和Item#getDescription()。使用cars.get(i).getDescription().equals(description)檢查描述。要檢查評分使用cars.get(i).getRating() == rating。

您不應該使用equals來比較Item與字符串，因爲這會違反等於contract。

Answer 2

cars.get(i) 

返回一個項目，而不是一個字符串。所以

if(cars.get(i).equals(al)) 

是不正確的。

Answer 3

if(cars.equals(description)) 

你的ArrayList cars（全名單在這種情況下）永遠不會等於一個字符串。

如果你想搜索一輛汽車，你需要檢查你的列表中的所有物品，看看它們的名字（或者你存儲在Item-class中的任何信息）是否與給定的description匹配。

Answer 4

您正在做基於對象本身的平等檢查，而應該針對對象的屬性進行檢查。在您的具體情況下，您應該查看收集中每個車輛/物品的rating屬性。您的代碼會是這個樣子：

final String ratingStr = Integer.toString(rating); 

int counter = 0; 
for (for final Item car: cars) { 
    if(ratingStr.equals(car.getRating()) { 
     ++counter; 
} 

System.out.println("Number of 'cars' with the rating is: " + counter); 

兩個快速評論，你應該實現您Item類平等的方法。但在這種情況下，這不是問題的實際來源。另外，你在代碼中提到了很多汽車，但是你的bean類被稱爲'Item'。您可能想要協調一下，因爲這可能會讓閱讀代碼的其他人感到困惑。

不要忘記修復你的searchForItem方法，目前你正在測試數組列表是否等於一個字符串，它永遠不會返回true。按照上述相同的方式對其進行修正，但使用您的汽車的description屬性，而不是rating屬性。

Answer 5

if(cars.get(i).equals(al)) 在這裏，你是比較對象的字符串，它是錯的 櫃面使用u可以嘗試下面的編碼

if(cars.get(i).getItem().equals(al)) 

是可能的，其中的getItem（）汽車類變量之一命名爲「item」，鍵入爲「string」並將其設置爲getter和setter。

lly if（cars.equals（description））是錯誤的。在這裏你企圖用繩子比較列表名稱 所以最好使用下面的編碼

if(cars.get(i).getDescription().equals(description)) 
    return cars.get(i); 

是可能的，其中getDescription（）命名爲「說明」中汽車類的變量之一，類型爲「字符串」並把它吸收和安裝。