Welcome to the Treehouse Community

The Treehouse Community is a meeting place for developers, designers, and programmers of all backgrounds and skill levels to get support. Collaborate here on code errors or bugs that you need feedback on, or asking for an extra set of eyes on your latest project. Join thousands of Treehouse students and alumni in the community today. (Note: Only Treehouse students can comment or ask questions, but non-students are welcome to browse our conversations.)

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and a supportive community. Start your free trial today.

Java Java Objects (Retired) Creating the MVP Comparing Characters

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,363 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