It's not a secret anymore!
The moose likes Beginning Java and the fly likes My program has a bug. Need help Big Moose Saloon
  Search | Java FAQ | Recent Topics | Flagged Topics | Hot Topics | Zero Replies
Register / Login
JavaRanch » Java Forums » Java » Beginning Java
Bookmark "My program has a bug. Need help" Watch "My program has a bug. Need help" New topic

My program has a bug. Need help

Jere Johnson

Joined: Mar 29, 2003
Posts: 28
Im trying to get rid of bug. When i ask the user to enter miles or kilometers choice, It sort of stalls. I have to hit enter twice.

Diapers are the best invention
Ernest Friedman-Hill
author and iconoclast

Joined: Jul 08, 2003
Posts: 24199

The problem in a nutshell: BufferedReader is buffered, but so is, usually, standard input. Therefore, if you use a BufferedReader on standard input, there are two different buffers of different sizes (one in Java, one in your OS's console driver), and filling up one won't necessarily fill up the other.
Anyway, you're using BufferedReader just to get the readLine() method; you can turn buffering off. Just use

That "1" says use a buffer size of one character -- i.e., no buffer. You should generally do this any time you use BufferedReader with standard input.

[Jess in Action][AskingGoodQuestions]
Wayne L Johnson
Ranch Hand

Joined: Sep 03, 2003
Posts: 399
I'm pretty sure that the program you've provided here won't even compile, let alone run. Are you sure this is the latest version?
A few thoughts. First, since this program is command-line only, you could just as easily extend "Object" (remove the "extends Frame" clause) and get rid of the awt imports.
Second, you define m, k, h, min and sec as "int" variables at the top of the program, then you use them (in Integer.parseInt(...) expressions) before they've been set. Then you try and shadow (re-define) them as "String" variables inside the if statements. It would probably be best to define them as "String" variables at the top, and do the "Integer.parseInt()" expressions AFTER reading the values in.
The if(miles_units)/else statements are mismatched. And you are trying to use a boolean value (miles_units) to handle three cases (miles, kilometers, invalid argument). A boolean value can handle only two cases.
Overall I think you have the necessary steps to do what you want, you just need to massage it a bit.
I agree. Here's the link:
subject: My program has a bug. Need help
jQuery in Action, 3rd edition