Local variable not initialized in a catch function

Question

I'm getting an error saying local variable may have not been initialized, or cannot be resolved to a variable at where the catch is. How should I fix this? Basically, I want my program to accept a few numbers, then stop and display some answers.

import java.util.Scanner;

public class math2{
public static void main(String[] args) {
    Scanner input = new Scanner(System.in);
    System.out.print("Enter Integer Values\nEnter a Non-Integer When Finished\n");

      int x = input.nextInt();
      int max = x;
      int min = x;
      int sum = x;
      double average = 0;

    try
    {
    int amount = 1;
    while(true)
    {
        x = Integer.parseInt(input.next());
        sum = sum + x;
        amount++;
        average = (double)sum/amount;

            if (x > max) {
                max = x;
            } else if (x < min) {
                min = x;
            }
    }
    }catch(NumberFormatException e){
    System.out.println("The Maximum Value is " + max);
    System.out.println("The Minimum Value Is " + min);
    System.out.println("The Sum Is " + sum);
    System.out.println("The Average Is " + average);}
    }

}

Now everything else works except for when the user inputs a non integer as their first number. — Yoon Joon So
– Yoon Joon So, Commented Jul 30, 2013 at 7:22
I have updated my solution to resolve the issue please have look into — Prabhaker A
– Prabhaker A, Commented Jul 30, 2013 at 8:14

Prabhaker A · Accepted Answer · 2013-09-24 05:57:07Z

6

declare below variable before try block,So that these are available in catach block also.
and also initialize double average to some default value.

  String x = input.next();
  int y = Integer.parseInt(x);*emphasized text*
  int max = y;
  int min = y;
  int sum = 0;
  double average = 0;

Update

After your edit in question I noticed that you are getting InputMismatchException,
if you give non-integer as first input.
For this you can catch that Exception to exit normally from the program.
Replace this code in place of int x = input.nextInt(); statement of your code.

    int x = 0;
    try{
        x = input.nextInt();
    }catch(InputMismatchException ime){
       //you cam use either of one statemnet.I used return statement
        return ;
        //System.exit(0);
    }

edited Sep 24, 2013 at 5:57

answered Jul 30, 2013 at 7:07

Prabhaker A

8,5131 gold badge20 silver badges24 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

AllTooSir · Accepted Answer · 2013-07-30 07:07:05Z

2

Firstly , define the variables out side the try block. If you define it within try{}, it will be scoped inside the try{...} and invisible for catch(){} block.

Secondly , You need to provide some initial value to average . Local variables are never initialized with default values.

double average = Double.NaN;

What if the variable was never initialized inside the while() loop due to an Exception ?

answered Jul 30, 2013 at 7:07

AllTooSir

49.5k17 gold badges132 silver badges164 bronze badges

2 Comments

kiheru Over a year ago

A fast answer +1. I'd prefer NaN for cases where the value is really undefined though.

Loki Over a year ago

Well he should do but thats not the problem... The problem is, he has to declare the variables before the try

Raunak Agarwal · Accepted Answer · 2013-07-30 07:07:36Z

1

Because you haven't initialized the variable 'average'. Also, I would suggest you to declare the variables outside the try catch block

answered Jul 30, 2013 at 7:07

Raunak Agarwal

7,2486 gold badges41 silver badges62 bronze badges

Comments

Surya Prakash Tumma · Accepted Answer · 2013-07-30 07:12:00Z

0

Declare max, min, average,sum variables before try block. Those variables are local to try block.. if you want to use at catch block make that variables as class members.

answered Jul 30, 2013 at 7:12

Surya Prakash Tumma

2,2234 gold badges29 silver badges50 bronze badges

Comments

ManMohan Vyas · Accepted Answer · 2013-07-30 07:12:32Z

import java.util.Scanner;

public class math2{
static double average;
public static void main(String[] args) {
    Scanner input = new Scanner(System.in);
    System.out.print("Enter Integer Values\nEnter a Non-Integer When Finished\n");

    try
    {

    String x = input.next();
    int y = Integer.parseInt(x);
    int max = y;
    int min = y;
    int sum = 0;
    int amount = 0;


    while(true)
    {
        sum = sum + y;
        y = Integer.parseInt(input.next());

        amount++;
        average = (double)sum/amount;

            if (y > max) {
                max = y;
            } else if (y < min) {
                min = y;
            }
    }
    }catch(NumberFormatException e){
    System.out.println("The Maximum Value is " + max);
    System.out.println("The Minimum Value Is " + min);
    System.out.println("The Sum Is " + sum);
    System.out.println("The Average Is " + average);}
    }

}

see you declare the variable at class level and compiler will assign it a default value for you ...but for local variables you need to assign it , as mention in other posts

Collectives™ on Stack Overflow

Local variable not initialized in a catch function

5 Answers 5

Comments

2 Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

Comments

2 Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related