java - 查找具有正确参数的对象的代码出错

Question

我有拍卖任务的代码。有 4 个类别:Item、Bid、Auction、Person。项目包含:项目的名称、项目的描述、项目的minPrice、的LinkedList >所有出价拍卖包含:所有元素的LinkedList，投标人的LinkedListBid 包含:Bid 的价格、Person 类的对象person人员包含:投标人的姓名。

因此，在这次简短的汇报之后，我想总结一下我的问题。如果您有其他类型的问题，我会提供我的类图。 https://drive.google.com/open?id=19mjayMIWFRNygvzP2xIGEWVzZcKNXIZD

有一个addBid(String itemName，String nameOfBidder，长价格)Auction 类中的方法，应该从 bidder LinkedList 中找到投标人(如果不存在，则创建它)，然后根据项目名称找到正确的投标人，然后使用Item 类中的 addBid 方法在项目对象中添加新的出价。

我的代码中有一个错误，当我尝试根据其 itemName 查找项目时，如果不存在具有此类名称的项目对象，它应该返回 NoSuchElementException 。但每次我检查失败时，其实我都不明白为什么。

我试图通过使用不同类型的循环(例如 foreach)来解决我的问题。但几天都无法解决。

这是我的 addBid 方法的 Auction 类中的方法

public void addBid(String ItemName, String nameOfBidder, long price) {

        if(ItemName==null||nameOfBidder==null){
            throw new NullPointerException("Name of the bidder cannot be null");
        }

        if(ItemName==""||nameOfBidder==""||price==0||price<0){
            throw new IllegalArgumentException("Name of the bidder cannot be empty");
        }

        for(Person p:bidders) {
            if (bidders.contains(p.getName()==nameOfBidder)) {
                for (Item i:allItems ) {
                    if(!(allItems.contains(i.getName()))){
                        throw new NoSuchElementException("There is no such Item in the Auction");
                    }
                    if(allItems.contains(i.getName()==ItemName)){
                        i.addBid(p,price);
                    }
                }
            }
            else {
                Person person = new Person(nameOfBidder);
                bidders.add(person);
                for (Item i:allItems ) {
                    if(!(allItems.contains(i.getName()))){
                        throw new NoSuchElementException("There is no such Item in the Auction");
                    }
                    if(allItems.contains(i.getName()==ItemName)){
                        i.addBid(person,price);
                    }
                }
            }
        }

    }

我的 Junit 测试中最后一次检查失败了(NoSuchElementException )

   public void testAddBidIllegalArgument() {
        a.registerItem(new Item("Clock", "Ancient clock", 1000));
        try {
            a.addBid("", "Max", 5);
            fail("Auction.addBid() should throw an IllegalArgumentException if the itemName argument is empty!");
        } catch (IllegalArgumentException e) {
        }

        try {
            a.addBid("Clock", "", 5);
            fail("Auction.addBid() should throw an IllegalArgumentException if the nameOfBidder argument is empty!");
        } catch (IllegalArgumentException e) {
        }

        try {
            a.addBid("Clock", "Max", 0);
            fail("Auction.addBid() should throw an IllegalArgumentException if the price argument is zero!");
        } catch (IllegalArgumentException e) {
        }

        try {
            a.addBid("Clock", "Max", -1);
            fail("Auction.addBid() should throw an IllegalArgumentException if the price argument is negative!");
        } catch (IllegalArgumentException e) {
        }

        try {
            a.addBid("New", "Max", 5);
            fail("Auction.addBid() should throw a NoSuchElementException if no item in the auction has the given name!");
        } catch (NoSuchElementException e) {
        }
    }

请帮我解决我的错误!并帮助我通过最后的检查!

Answer 1

以下条件看起来不正确:

for(Person p:bidders) {
    // bidders holds Person objects and you are checking for boolean. p.getName() == nameOfBidder will evaluate to true. Perhaps you want to check for name equality first and then contains.
    if (bidders.contains(p.getName()==nameOfBidder)) {
   }
}

for (Item i:allItems ) {
    // allItems holds Item objects and you are checking by name string
    if(!(allItems.contains(i.getName()))){ 

    }
}

此外，还可以简化初始 null 和检查条件。

给你，简化得多的代码:

public void addBid(String itemName, String nameOfBidder, double price) {
    if (itemName == null || nameOfBidder == null) {
        throw new NullPointerException("Name of the bidder cannot be null");
    }
    if (itemName.equals("") || nameOfBidder.equals("") || price <= 0) {
        throw new IllegalArgumentException("Name of the bidder cannot be empty");
    }
    Optional<Person> person = bidders.stream().filter(e -> e.getName().equals(nameOfBidder)).findAny();
    Optional<Item> item = items.stream().filter(e -> e.getName().equals(itemName)).findAny();
    if (person.isPresent()) {
        checkItemAndAddBid(item, person.get(), price);
    } else {
        Person newPerson = new Person(nameOfBidder);
        bidders.add(newPerson);
        System.out.println("Creating a new bidder: "+newPerson.getName());
        checkItemAndAddBid(item, newPerson, price);
    }
}

public void checkItemAndAddBid(Optional<Item> item, Person person, double price) {
    if (!item.isPresent()) {
        throw new NoSuchElementException("There is no such Item in the Auction");
    } else {
        item.get().addBid(person, price);
    }
}

完整的运行示例代码可用:at github