Win a copy of Mesos in Action this week in the Cloud/Virtualizaton forum!
  • Post Reply
  • Bookmark Topic Watch Topic
  • New Topic

My program has a bug. Need help

 
Jere Johnson
Greenhorn
Posts: 28
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
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.

 
Ernest Friedman-Hill
author and iconoclast
Marshal
Pie
Posts: 24211
35
Chrome Eclipse IDE Mac OS X
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
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.
 
Wayne L Johnson
Ranch Hand
Posts: 399
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
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.
 
  • Post Reply
  • Bookmark Topic Watch Topic
  • New Topic