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

Posted by Thomas Katalenas
Thomas Katalenas
11,033 Points

I have modeled an assistant for a tech conference. Since there are so many attendees, registration is broken up into two

ConferenceRegistrationAssistant.java 
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 */
    int line 1 = 0;
    int line 2 = 0;
    String ABC = "ABCDEFGHIJKLMNOPQRSTUVWXYZ";
    if(ABC.indexOf(lastName.charAt(0)) <= 12){
      line += 1;
      return line 1;
    }else{
      line2 += 1;
      return line 2;
    }

  }

}

Whats wrong with my code I trey to use 

lastName.charAt(0)

to find the first char of lastName

then I insert it into indexOf to see what that chars index is in ABC

and then comparison

<= 12

the problem I think is with returning lines...waahaa

1 Answer

jcorum
jcorum
71,830 Points
jcorum
jcorum
71,830 Points

Thomas, nice try, but you are doing way too much work:

  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 */
    int line = 0;
    if (lastName.charAt(0) < 'M') {
      line = 1;
    } else {
      line = 2;
    }
    return line;
  }

Just check the first letter of each name and if its 'M' or less it's line 1, otherwise line 2.