Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

Java Java Objects (Retired) Creating the MVP Comparing Characters

Ema Vargová
Ema Vargová
4,185 Points
Posted by Ema Vargová
Ema Vargová
4,185 Points

Where is a mistake?

I do not know where I made a mistake. Please, help me to fix it.

ConferenceRegistrationAssistant.java 
import java.io.Console;

public class ConferenceRegistrationAssistant {

  public int getLineFor(String lastName) {
    /* If the last name is between A thru M send them to line 1
       Otherwise send them to line 2 */
    Console console = System.console();
    lastName = console.readLine("Enter your last name:  ");
    int line;
    if (lastName.charAt(0) < "M") {
      return line = 1;
    } else {
      return line = 2;
    }
  }

}

1 Answer

Grigorij Schleifer
Grigorij Schleifer
10,365 Points
Grigorij Schleifer
Grigorij Schleifer
10,365 Points

Hi Ema,

lets see the code:

I deleted the console object and the lastName declaration because lastName is already defined for you and you don´t need to ask the User for the lastName.

import java.io.Console;

public class ConferenceRegistrationAssistant {

  public int getLineFor(String lastName) {
    int line = 0;
// don´t forget to initialize the int to 0
    if (lastName.charAt(0) <= 'M') {
      line = 1;
    } else {
      line = 2;
    }
    return line;
// I wrote the return statement after the if loop
// I think it looks more cleaner like this
  }
}

The code below is also OK with two return statements:

import java.io.Console;

public class ConferenceRegistrationAssistant {

  public int getLineFor(String lastName) {
    int line = 0;

    if (lastName.charAt(0) <= 'M') {
      return line = 1;
    } else {
      return line = 2;
    }
  }
}

Choose one that appears to be more logic for you.

Let us know if you need more help

Grigorij