Question

Debug the following java code so that it will have given out put at the bottom.  Make sure to fix all syntax and logical errors.

import java.util.Scanner;

public class Graphs
{
   // draws 5 histograms
   public void drawHistograms()

      Scanner input = new Scanner( System.in );
      
      int number1 = 0; // first number
      int number2 = 0; // second number
      int number3 = 0; // third number
      int number4 = 0; // fourth number
      int number5 = 0; // fifth number

      int inputNumber; // number entered by user
      int value = 0; // number of stars to print
      counter = 1; // counter for current number

      while ( int counter <= 5 )
      {
         System.out.print( "Enter number: " );
         inputNumber = input.nextInt();

         // define appropriate num if input is between 1-30
         if ( inputNumber >= 1 && inputNumber <= 30 )
         {
            switch ( inputNumber )
            {
               case 1:
                  number1 = inputNumber;
                  break; // done processing case

               case 2:
                  number2 = inputNumber;
                  break; // done processing case

               case 3:
                  number3 = inputNumber;

               4:
                  number4 = inputNumber;
                  break; // done processing case

               case 5:
                  number5 = inputNumber;
                  break; // done processing case

            counter++
         } // end if
         else
            System.out.println(
               "Invalid Input\nNumber should be between 1 and 30" );
      } // end while

      // print histograms
      for ( counter = 1, counter >= 5, counter++ )
      {
         switch ( counter == 1 )
         {
            case 1:
               value = number1;
               break; // done processing case

            case 2:
               value = number2;
               break; // done processing case

            case 3:
               value = number3;
               break; // done processing case

            case 4:
               value = number4;
               break; // done processing case

            case 5:
               value = number5;
               break; // done processing case
         }

         for ( int j = 1; int j <= value; int j++ )
            System.out.print( "*" );
         
         System.out.println();         
      } // end for loop
   } // end method drawHistograms
} // end class Graphs

Sample output

--------------------Configuration: <Default>--------------------
Enter number: 5
Enter number: 2
Enter number: 8
Enter number: 4
Enter number: 6
*****
**
********
****
******

Process completed.

Answer 1

//Program is well tested and producing the same output as above. The above code is buggy and not feasible. So i wrote the //code from scratch to fulfill the need.

class Main
{
public static void drawHistograms(){

//Defining Scanner class to take input
Scanner sc = new Scanner(System.in);

//Defining Variables
int array[]=new int[10]; //increase the array size according to need
int i,j,k,number,counter=0;
char c ='*';
//Loop to recurcively take 5 inputs for making histogram. Change the value in loop to increase number of inputs

while(counter<5){
System.out.print("Enter Number: ");
number = sc.nextInt();
array[counter] = number;
counter = counter+1;
}

//making histogram
  
for(i =0; i<array.length; i++){
  
for(j=0;j<array[i];j++){
  
System.out.print(c);
  
}
System.out.println("");
}
  }

//main Function
public static void main (String[] args) {

//calling method

drawHistograms();
}
}