1

I have a method that returns all the names of people in a LinkedList for a plane.

However even though there is a return statement in the method, I'm still getting told there is a missing return statement.

How can I work around this without putting another return statement in? Why isn't it considered valid? Does putting another return statement in change what is returned?

Any feedback is greatly appreciated.

public String check() {

        for (Person person: passengers)
            {
                return person.getName();
            }    
    }
Improve this question
3
  • Every code path must return somethingy, i.e. if there is no passenger in passengers, even then the block must return` some value. Do the needful, such that the method is able to take care of this scenario as well :-) Commented May 29, 2015 at 0:22
  • This will not return "all the names of people", only the first person's name. A function is exited after a return statement. Commented May 29, 2015 at 0:22
  • @SU3 What would be the correct way to cycle through all the passengers names then if the return exits the loop? Commented May 29, 2015 at 0:26

3 Answers 3

Reset to default
6

Because if passengers is empty, the loop will never be entered.

If the loop is never entered, assuming the only return statement is in it, we have a serious problem, don't you think ? It's like if there were no return at all.

You need to add another return statement outside of the loop.

Also note that the return will automatically exit the method, so I don't think this is exactly what you wanted as per this sentence in your question :

I have a method that returns all the names of people in a LinkedList for a plane.

Edit

As per your edit, here how you can return a list containing all names :

return passengers.
          .stream()
          .map(Person::getName)
          .collect(Collectors.toList());

Note that you will need to change the signature of your method to

public List<String> check()
Improve this answer
Sign up to request clarification or add additional context in comments.

4 Comments

But what would I return? would it be return null;? and does placing it outside the loop mean that after running through the passenger list it would just jump to that return?
What would be the correct way to cycle through all the passengers names then if the return exits the loop?
@Mark You would return what you want to return in the case where your list is empty. And no, that doesn't mean that you would jump to that return as if you already returned in the loop, you will exit the loop.
@Mark I edited to give you an example of what I think you want.
4

In answer to your question in the comments. You can only return a single object from a function. You could take another container and populate it with the names and return that. For example,

public LinkedList<String> check() {

  LinkedList<String> names = new LinkedList<String>();

  for (Person person: passengers) {
    names.add( person.getName() );
  }

  return names;
}
Improve this answer

Comments

1

What exactly are you trying to accomplish, here?

Currently, check will only ever return the name of the first passenger. Think about how your program flows and what you want it to do.

To answer your question, you need to have an 'escape' for every possible path in your code. Even if a certain block should always catch and return (not by definition, but just by how you think the code should flow), you need to handle the case such that that block doesn't catch and return. This can be done by either fixing the first block so that it really is a catch-all, or by simply returning or throwing an error if the first block doesn't catch.

i.e. 

public boolean check() {
   ... 
   if (shouldAlwaysBeTrue) return false;
}

doesn't work because shouldAlwaysBeTrue is not true by definition.

public boolean check() {
    ... 
    if (shouldAlwaysBeTrue) return false;
    return true;
}
Improve this answer

Comments

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.