My program has a bug. Need help

[Jere Johnson]

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.

 
package myprojects.program0;
import java.awt.*;
import java.awt.event.*;
import java.io.*;
class Program0 extends Frame {

public static void main(String args[]) 
{
int hours, minutes, seconds;
double distance, speed, miles, kilometers;
boolean miles_units; // boolean variable for miles or kilometers
int input, m,k,h,min,sec;

InputStreamReader istream = new InputStreamReader (System.in);
BufferedReader bufRead = new BufferedReader (istream);

System.out.println ("*****************************************************************");
System.out.println ("*                         Program 0                             *");   
System.out.println ("*       Calculate Speed given Input of Distance and Time        *");
System.out.println ("*                                                               *");
System.out.println ("*               Written by Jeremiah Johnson                     *");
System.out.println ("*****************************************************************");
System.out.println (" ");


try {

//distance = Integer.parseInt (d);
miles = Integer.parseInt (m);
kilometers = Integer.parseInt (k);
hours = Integer.parseInt (h);
minutes = Integer.parseInt (min);
seconds = Integer.parseInt (sec);


System.out.println ("Units you are using? (1 = miles or 2 = kilometers)");
String z = bufRead.readLine();

input = Integer.parseInt (z);

// Kilometers vs Miles
if (input == 1)
miles_units = true;

if (input ==2)
miles_units = false;


if (miles_units){
System.out.println ("Enter the miles traveled: ");
String m = bufRead.readLine();

System.out.println ("Enter number of hours it took: ");
String h = bufRead.readLine();

System.out.println ("Enter number of minutes it took: ");
String min = bufRead.readLine();

System.out.println ("Enter number of seconds it took: ");
String sec = bufRead.readLine();
}

System.out.println ("Enter the kilometers traveled: ");
String k = bufRead.readLine();

System.out.println ("Enter number of hours it took: ");
String h = bufRead.readLine();

System.out.println ("Enter number of minutes it took: ");
String min = bufRead.readLine();

System.out.println ("Enter number of seconds it took: ");
String sec = bufRead.readLine();

}

else 
throw new IllegalArgumentException
 ("Invalid Option, Must type in 1 = miles or 2 = kilometers !!");




// Exception for input values
if (miles  < 0)
throw new IllegalArgumentException
("Distance must be positive!");
if (kilometers < 0)
throw new IllegalArgumentException
("Distance must be positive!");
if (minutes > 59 || minutes < 0)
throw new IllegalArgumentException
("Minutes are not valid!");
if (seconds > 59 || seconds < 0)
throw new IllegalArgumentException
("Seconds are not valid!");

double time = hours + (minutes*.01) + (seconds*.001);

//speed = distance/time;
speed = miles/time;
speed = kilometers/time;

System.out.println ("--------------------------");
System.out.println ("                          ");
System.out.println ("Your Speed is " + speed + " mph");
}

catch (IOException err) {
System.out.println ("Error reading line");}

 

[Ernest Friedman-Hill]

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

    BufferedReader bufRead = new BufferedReader (istream, 1);

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]

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.