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 Creating the MVP Conference Registration Assistant

I am trying to return the number 1 if the first letter of a name is between A and M and return the 2 otherwise.

Does anyone mind pointing out what is wrong with my code in doing this?

ConferenceRegistrationAssistant.java
public class ConferenceRegistrationAssistant {

  public int getLineNumberFor(String lastName) {
    int lineNumber = 0;
    char letter = lastName.charAt(0);
    if( letter <= 'A' && letter >= 'M' ){
      lineNumber = 1;
    }else {    
      lineNumber = 2;
     }
    return lineNumber;
    }
}
Example.java
public class Example {

  public static void main(String[] args) {
    /*
      IMPORTANT:  You can compare characters using <, >. <=, >= and == just like numbers
     */
    if ('C' < 'D') {
      System.out.println("C comes before D");
    }

    if ('B' > 'A') {
      System.out.println("B comes after A");
    }

    if ('E' >= 'E') {
      System.out.println("E is equal to or comes after E");
    }

    // This code is here for demonstration purposes only...
    ConferenceRegistrationAssistant assistant = new ConferenceRegistrationAssistant();
    /*
      Remember that there are 2 lines.
      Line #1 is for A-M
      Line #2 is for N-Z
     */
    int lineNumber = 0;
    /*
      This should set lineNumber to 2 because
      The last name is Zimmerman which starts with a Z.
      Therefore it is between N-Z
     */
    lineNumber = assistant.getLineNumberFor("Zimmerman");

    /*
      This method call should set lineNumber to 1, because 'A' from "Anderson" is between A-M.
     */
    lineNumber = assistant.getLineNumberFor("Anderson");

    /*
      Likewise Charlie Brown's 'B' is between 'A' and 'M', so lineNumber should be set to 1
     */
    lineNumber = assistant.getLineNumberFor("Brown");
  }

}

1 Answer

andren
andren
28,558 Points

There is a logical fault in your if statement, you check if the letter is less than or equal to A and greather than or equal to M. You should be checking for the opposite like this:

public int getLineNumberFor(String lastName) {
    int lineNumber = 0;
    char letter = lastName.charAt(0);
    if( letter >= 'A' && letter <= 'M' ){
        lineNumber = 1;
    } else {    
        lineNumber = 2;
     }
    return lineNumber;
}

The letter should be greater than or equal to A and lesser or equal to M.