Simple code. Could you help pls..?

[Iliyas]

Hi, Could you pls. look at the code. As I run the code the message:

"Do you have a coupon? Exception in thread "main" java.lang.NullPointerException
at TicketPricewithDiscount.main(TicketPricewithDiscou nt.java:15)"

is showing off :confused: . I have copied the code from the book and there it works fine(at least the author writes it works:) . Would appreciate your help. Thanks. (See the full code below).

import java.util.Scanner;

public class TicketPricewithDiscount {

public static void main(String args[]){
Scanner myScanner = new Scanner(System.in);
int age;
double price=0.00;
char reply;

System.out.print("How old are you?");
age = myScanner.nextInt();

System.out.print("Do you have a coupon? ");
reply = myScanner.findInLine(".").charAt(0);

if (age >= 12 && age< 65){
price=9.25;
}
if (age < 12 || age> 65){
price= 5.25;
}
if (reply == 'Y'|| reply=='y'){
price -=2.00;
}
if (reply != 'N' && reply!='n'&& reply!= 'Y' && reply!='y'){
System.out.print ("Huh?");
}

System.out.print ("Please pay $ ");
System.out.print(price);
System.out.print(". ");
System.out.println("Enjoyr the show!");
}
}

[Fubarable]

You should probably call myScanner.nextLine(); immediately after getting the age via myScanner.nextInt(). The reason for this is that nextInt doesn't process the end of line character, and so without nextLine() the scanner object will be immediately called after the nextInt() call and your NPE will be thrown.

For example:

    System.out.print("How old are you?");
    age = myScanner.nextInt();
    myScanner.nextLine();  // add this to swallow the end of line character

Question, why the call to findInLine(".")? What is the purpose of using this and not nextLine()?

[Eranga]

What you are going to do here?

reply = myScanner.findInLine(".").charAt(0);

[Iliyas]

Amazing. Thanks dude, it worked. the purpose of findInLine(".") call was to get a single character which had to be "Y" or "N" or "y" "n".

"System.out.print("Do you have a coupon? Y/N");
reply = myScanner.findInLine(".").charAt(0);"

Option between Y or N inputs into compiler.

[Eranga]

Ok, did you read more about findInLine() method on the Java doc? You have a patter like ".", and what happen if your input doesn't have that.

[Fubarable]

Again, what's wrong with just using nextLine() there? Why use findInLine(...)?

System.out.print("Do you have a coupon? Y/N ");
reply = myScanner.nextLine().charAt(0);"

[Iliyas]

Yeah. I see "nextLine()" resolves another problem. Had some discrepancy using "findInLine(".")" call.
thanks,

[tghn2b]

Just a minor point:

I think maybe when someone is at the age of 65 they will get in for free.

if they have a coupon, might even collect 2.00 for saying 'Y'

8-)

[Eranga]

Yeah. I see "nextLine()" resolves another problem. Had some discrepancy using "findInLine(".")" call.
thanks,

Yes, as I said in the lase post what happen if your input doesn't contain that pattern you are looking. Null...