0

大家好，谁能告诉我我哪里出错了？

这个类的基本目的是定义一个最喜欢的项目数组列表，在这种情况下是关于汽车的。汽车对象具有汽车名称和汽车 1-5 的评级。

您如何查看字符串是否等于汽车对象评级。我弄乱了将字符串或 int 与数组列表中的汽车对象进行比较的部分。我的 equals() 方法有什么问题？contains() 方法可以以同样的方式工作吗？

numberOfItemsOfRating 方法允许用户指定评级，因此该方法返回没有带有评级的汽车。searchForItems 方法检查指定的字符串描述是否与数组列表中的汽车名称匹配，因此返回数组列表中的汽车。

这是我使用构造函数和变量的两种方法的一瞥：

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;
            }
        }  
      }
}
4

5 回答 5

1

您正在根据对象本身进行相等性检查，而您应该针对对象的属性进行检查。在您的特定情况下，您应该查看rating集合中每个 Car/Item 的属性。您的代码将如下所示：

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 类称为“项目”。您可能需要调和这一点，因为它可能会使阅读您的代码的其他人感到困惑。

不要忘记修复您的searchForItem方法，目前您正在测试数组列表与字符串的相等性，它永远不会返回 true。以与上述相同的方式更正它，但使用description汽车的属性，而不是rating属性。

于 2012-04-09T15:30:09.957 回答
0 
cars.get(i)

返回一个项目，而不是一个字符串。所以

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

是不正确的。

于 2012-04-09T15:24:34.647 回答
0 
if(cars.equals(description))

您的 ArrayList cars（在这种情况下是整个列表）永远不会等于单个字符串。

如果您想搜索汽车，您需要检查列表中的所有项目，并查看它们的名称（或您存储在Item-class 中的任何信息）是否与给定的description.

于 2012-04-09T15:26:16.053 回答
0

这不是您应该使用 equals 方法的方式，而是建议您使用或实现Item#getRating()and Item#getDescription()。用于cars.get(i).getDescription().equals(description)检查说明。要检查评级，请使用cars.get(i).getRating() == rating.

您不应使用 equals 将 Item 与字符串进行比较，因为这会违反 equals contract。

于 2012-04-09T15:26:29.503 回答
0

if(cars.get(i).equals(al)) 在这里，您将字符串与对象进行比较，因此 在使用时是错误的，您可以尝试以下编码

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() 可能是汽车类中名为“description”的变量之一，键入为“string”并将其放入 getter 和 setter。

于 2013-05-16T09:51:01.770 回答