Code does not work

Hi,
tried running the following code but after entering integer input,the code gives an errror message..
pls enlighten me why..
thkx
Here is the code:

import java.util.Scanner;
import static java.lang.System.in;
import static java.lang.System.out;
class Guyton
{
public static void main(String args[])
{
Scanner myScanner = new Scanner(in);
double price = 0.00;
int age;
char reply;
out.println("Welcome to the McTwisp Movie Theatre.What is your age?");

age = myScanner.nextInt();
if (age >= 13 && age < 65)
{
price = 5.38;
}
if (age < 13 || age >= 65)
{
price = 3.00;
}
out.println("Do you have a coupon? Y/N");
reply = myScanner.findInLine(".").charAt(0);
if ( reply == 'Y' && reply == 'y')
{
price -= 2.00;
}
if (reply != 'Y' || reply != 'y' || reply != 'N' || reply != 'n')
{
out.println("Huh?");
}
out.println("Please pay $ ");
out.println("Have a great time");
}
}

Quote:

---

Originally Posted by scotts

Code:

---

  if (reply != 'Y' || reply != 'y' || reply != 'N' || reply != 'n')
  {
  out.println("Huh?");
  }

---

---

Here your logic is incorrect: suppose reply == 'Y; then most certainly it isn't equal to a 'y' or an 'N' or an 'n'. Change the || operator to the && operator.

kind regards,

Jos

"the code gives an errror message"

Why do people not actually give us the error message?
It's my wife that's the mindreader, not me...

Quote:

---

Originally Posted by Tolls

"the code gives an errror message"

Why do people not actually give us the error message?
It's my wife that's the mindreader, not me...

---

Your wife too? My conjecture is that all women are psychic in a scary way; us poor men just have to work hard, put the trash out, keep our mouths shut and do the debugging ;-)

kind regards,

Jos

Apart form Jos comments,one more thing here I want to add is you are trying to get the int value,but you are trying to use this below String method which obviously throw NullPointerException.
reply = myScanner.findInLine(".").charAt(0);

Another thing is put try catch block with printStackTrace() to debug..

Hi,
thks for all the solutions ..
after tweaking a part of the code it reads as:

import java.util.Scanner;
import static java.lang.System.in;
import static java.lang.System.out;
class Guyton
{
public static void main(String args[])
{
Scanner myScanner = new Scanner(in);
double price = 0.00;
int age;
char reply;
out.println("Welcome to the McTwisp Movie Theatre.What is your age?");

age = myScanner.nextInt();
if (age >= 13 && age < 65)
{
price = 5.38;
}
if (age < 13 || age >= 65)
{
price = 3.00;
}
out.println("Do you have a coupon? Y/N");
reply = myScanner.findInLine(".").charAt(0);
if ( reply == 'Y' && reply == 'y')
{
price -= 2.00;
}
if (reply != 'Y' && reply != 'y' && reply != 'N' && reply != 'n')
{
out.println("Huh");
}
out.println("Please pay $ ");
out.println("Have a great time");
}
}

and i don't quite understand the part about the int value being below the String value..pls give a clear explanation cos i'm still a noob in java
or help me recorrect the code
thx

Quote:

---

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

---

Break this statement up into two parts and test the results.
Test what findInLine() returns before using charAt().

Hi Norm,
when i seperate the code as you mentioned it gives this error msg when compiled:
reply = myScanner.findInLine();
^
any solutions?
thkx

Well, you didn't give us the error, but I'm guessing it's because you are assigning a String (which is what is returned by findInLine()) to a char (which is what reply is).

You should assign the result of findInLine() to a String, and then assign the result of charAt() on that String to "reply".

you have to be strong in writing the comments as well as clear in ur logic.
below are my highlighted change.

two changes I made and also u should include jos changes.

1.after reciving an int using scanner u should move the ciursor to next line using nextLine() method.

2.I have taken the input string and extracted the char instead of findInline()...


if (age < 13 || age >= 65)
{
price = 3.00;
}
myScanner.nextLine();//moving the cursor to next lineout.println("Do you have a coupon? Y/N");

reply = myScanner.nextLine().trim().charAt(0);

Quote:

---

reply = myScanner.nextLine().trim().charAt(0);

---

I think this is poor technique to chain one methods results to another method call when its possible for an earlier method to fail.
Especially when debugging you need to see the results of all the methods and not assume that the data is how you expect it to be.

Hi Norm,
As per yours myScanner.nextLine() is having the possiblity of throwing nullpointer exception.But,it will wait for the user to input y/n.


I identified the error and given the above solution.

Thanks Troll, Ramya and Norm for helping out..
i changed two of the lines to:

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

Ramya's way worked too..
glad its finally solved..