Question

public void returnRental(Customer cust){
    Rental toDelete = null; //Rental to be removed from list.
    LinkedList<Video> toReturn = null; //List of videos to be added to inventory.

    //Find appropriate rental according to customer name. 
    for(int i = 0; i < rentals.size(); i++){
        if(cust.getName() == rentals.get(i).getRentee().getName()){
            toReturn = rentals.get(i).getRented();
            toDelete = rentals.get(i);
        }
    }

here is the snippet of code that is giving me problems. I’ve debugged it in eclipse quite a bit which ended up just confusing me more. It hits the if, and passes the condition. But once it gets to assigning values to “toReturn” it assigns it an empty list with size 0. Where as I check my rentals Linked list and the correct value are there, but for some reason it is not getting assigned to my variables correctly 🙁 The same happens to “toDelete” but this isn’t a list, it is one instance of my class Rental. (The linked list is a list of rentals, which contains a linked list of videos)

No errors are thrown…
Its a little difficult to explain, if you need more information please let me know and i’ll clarify.

I’m at a loss, possibly because I’m not iterating through my linked list correctly?

Answer 1

I’ve upvoted dystroy’s answer because incorrect string comparison is always wrong.

But because that would fail differently (customer names not matching rentee names), I’m wondering if your issue is really caused by either of the following:

1. a problem in getRented(); or
2. cust having a null name on call, which would match a Rentee with a null name.