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 trialJohan Herö
562 PointsI cant figuer out why it dosent work. Its not beutiful but it I think it should work;/
Any Ideas guys?
public class ConferenceRegistrationAssistant {
/**
* Assists in guiding people to the proper line based on their last name.
*
* @param lastName The person's last name
* @return The line number based on the first letter of lastName
*/
public int getLineNumberFor(String lastName) {
int lineNumber = 0;
char firstLetter = lastName.charAt(0);
if(firstLetter == (('a') || ('b') || ('c') || ('d') || ('e') || ('f') || ('g') || ('h') || ('i') || ('j') || ('k') || ('l') || ('m'))) {
lineNumber = 1;
}
else {lineNumber=2;}
/*
lineNumber should be set based on the first character of the person's last name
Line 1 - A thru M
Line 2 - N thru Z
*/
return lineNumber;
}
}
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
Florian Stegemann
Full Stack JavaScript Techdegree Student 22,660 PointsHi, i can see two problems with your if-clause:
First, you can't compare one variable to multiple characters like you tried in your example. If you wanted to compare firstLetter
to each character individually, you would have to do it like this:
if(firstLetter == 'a' || firstLetter == 'b' || firstLetter == 'c' /* ... and so on */)
Remember though, that you can also use the greater then and less then operators to compare firstLetter
to a range of letters like this:
if(firstLetter >= 'a' && firstLetter <= 'z')
This example reads like if firstLetter is greater or equal than a AND if firstLetter is less or equal than z, which basically checks wether firstLetter
is between a and z.
Secondly, keep in mind that Java is case sensitive. That means that a and A are not the same character when doing comparisons.