Exception Handling not working

[collwill]

This code has to handle exception errors gracefully. I am trying to write code in which a user inputs an index of array and it will display the element of that index. If the user inputs an index higher than 100 or non interger an exception error has to display in an text field for the user. I know I am missing something but I cannot see it. I have been working on this code for over 2 weeks and the class is almost done.

/*this code if they input correct index they get a element, 
but bad index & non interger dispalys error in compiler 
not in the text field. 
*/

public void actionPerformed(ActionEvent e){
	String inputString;
	int num;
	inputString=indexfield.getText();
	num=Integer.parseInt(inputString);

	try {
	if(num>99 ||num<0);
			}
	catch (IndexOutOfBoundsException ex)
		{
			valuefield.setText("Out of Bounds");
		}

	catch (NumberFormatException ex)
		{
			valuefield.setText("Not a number.");
		}
		valuefield.setText(number[num]+"");

	}

/*This code dispalys correct element in textfield, throws index 
out of bounds error in compiler & in text texfield, 
but does not handle the non interger at all. 
*/ 


public void actionPerformed(ActionEvent e){
	String inputString;
	int num;
	inputString=indexfield.getText();
	num=Integer.parseInt(inputString);

	try {
		if(num>99 ||num<0)
			throw new IndexOutOfBoundsException("");
		}

	catch (IndexOutOfBoundsException ex)
		{
			valuefield.setText(" Out of bounds");
		}

	catch (NumberFormatException ex)
		{
			valuefield.setText("Not a number.");
		}

		valuefield.setText(number[num]+"");
	}

[sunde887]

to handle a non ingteger you should be trying to parse the value.

try{
  int num = Integer.parseInt("ABC");
} catch(NumberFormatException nfe){
  //set text/do other stuff.
}

The problem is, if you enter the incorrect number(i.e: not a number) the number format exception is thrown, it is a runtime exception so it will just display the exception and exit.

[collwill]

I am a totally confused. The original input is a string, which I turn around and change it to a int so it can find it in the array. If I try using what you gave me to find a string it is getting an error of ("num is already defined in actionPerformed(java.awt.event.ActionEvent) which has defined - num=Integer.parseInt(inputString);.

public void actionPerformed(ActionEvent e){
	String inputString;
	int num;
	inputString=indexfield.getText();
	num=Integer.parseInt(inputString);

	try {
		if(num>99 ||num<0)
			throw new IndexOutOfBoundsException("");
		}

	catch (IndexOutOfBoundsException ex)
		{
			valuefield.setText(" Out of bounds");
		}

	try{
		int num = Integer.parseInt ("ABC");
	}
	catch (NumberFormatException ex)
		{
			valuefield.setText("Not a number.");
		}

		valuefield.setText(number[num]+"");
	}

[sunde887]

You don't have to copy mine exactly, it was more of to point out that Integer.parseInt is a dangerous method(can throw a numberformatexception)

What happens if you type in 1bc. Does it spit a stack trace out? It should because number format exception is a run-time exception, if you don't catch the attempt to parse it an error can be thrown.

The line

num = Integer.parseInt(inputString)

should be

try{
  num = Integer.parseInt(inputString);
} catch(NumberFormatException nfe){
  //do stuff here
}

After this you can test the indexing stuff.

[collwill]

Ok, finally got the applett to work, thank you for your help

[sunde887]

You are welcome :) If you ever need any help don't hesitate to come ask questions. I am on this forum about 15-16 hours a day(I know it's sad, but I don't have much of a life atm), and I love to help people out.