Out of bounds exception in code

Hey yall, just wanted to post something up here real quick to see if I can get some more eyes on it. I have code that is throwing an out of bounds exception and it is for a program that accepts appointments into a ragged array. The ragged parts are the columns representing the days of the months. I digress. When I try to overwrite a pre-existing appointment I have it prompt me to overwrite and then bam!

I noted the line number where it is thrown.

public boolean setAppointment(String name, int myMonth, int myDay)
{
int reset = 1;
while (reset == 1)
{
try
{
    if (calendar[myMonth][myDay] == null)
    {
        calendar[myMonth][myDay] = name;
        reset = 0;
        return true;
}
    else
    {
    System.out.println("The date is already set, would like to overwrite (Y/N)? ");
        BufferedReader bRead = new BufferedReader(new InputStreamReader(System.in));
 
        if (bRead.read() == 'Y' || bRead.read() == 'y')
        {
            calendar[myMonth][myDay] = name;
            reset = 0;
            return true;
    }
    else
    {
            System.out.println("Enter a different appointment date in this format (mm/dd): ");
        date = bRead.readLine();
        while (isDateValid(date) == false)
    {
    System.out.println("Invalid date, pleae re-enter in this date format (mm/dd): ");
    date = bRead.readLine();
            }
            reset = 1;
       // bRead.close();
        }
    }
    } catch (IOException ioe)
{
        System.out.println("IO error trying to close the stream!");
    System.exit(1);
            }
}
return true;

Here is were it is called to overwrite and tossed in as a parameter:

public boolean isDateValid(String checkDate)
    {
for (int i = 0; i < checkDate.length(); i++)
{
            if (Character.isLetter(checkDate.charAt(i)))
    return false;
}

if (checkDate.charAt(1) == '/')   <----throws exception here
{
for (int i = 0; i < checkDate.length(); i++)
{
if (i == 1)
    i++;
if (!Character.isDigit(checkDate.charAt(i)) || checkDate.length() > 4)
   return false;
}
 
month = Integer.valueOf((String) checkDate.substring(0,1));
day = Integer.valueOf((String) checkDate.substring(2));
}
else
    {
    if (checkDate.charAt(2) == '/')
{
for (int i = 0; i < checkDate.length(); i++)
{
    if (i == 2)
    i++;
if (!Character.isDigit(checkDate.charAt(i)) || checkDate.length() > 5)
    return false;
    }
 
    month = Integer.valueOf((String) checkDate.substring(0,2));
    day = Integer.valueOf((String) checkDate.substring(3));
}
            else
            {
                return false;
}
}
System.out.println("month is " + month + " and day is " + day);  //Test
 
        //Check for month
if (month > 0 && month < 13)
{
if (month==1 || month==3 || month==5 || month==7 || month==8 || month==10  || month==12)
{
if (day > 0 && day < 32)
    return true;
else
    return false;
}
else
{
if (month==4 || month==6 || month==9 || month==11)
{
if(day > 0 && day < 31)
    return true;
else
    return false;
}
else
{
if (month == 2)
{
if (day > 0 && day < 29)
    return true;
else
    return false;
}
}
}
}
else
{
    System.out.println("month is " + month + " and day is " + day);  //Test
    return false;
}
return false;
 
    }

at raggedArray.isDateValid(raggedArray.java:134)
at raggedArray.setAppointment(raggedArray.java:238)

Thanks in advance

Gabe

You should check the length of the date before checkDate.charAt(1).
If checkDate is empty, it will throw an exception.

OK, well maybe I didn't find the problem.

Now its a new one.

When I hit 'Y' instead of 'y' it works so this must be part of the problem:

can you not do a bufferedReader or statement for chars? it is obviously just reading in the first char I enter and not even checking the if statement.

I even tried to read them in as strings. I do not want to use the console class for this.

Does anyone have any suggestions. I know this is totally easy thats why its in the beginner section

Thanks

[ April 10, 2006: Message edited by: Gabriel White ]
[ April 10, 2006: Message edited by: Gabriel White ]

When I hit 'Y' instead of 'y' it works so this must be part of the problem

You're not calling your buggy isDateValid() function, that's all.

About the BufferedReader, why don't you store the returned character ?

[/qb]<hr></blockquote>

This reads the first character and checks if it is 'Y' and then reads the next character before checking if it is 'y'. Store the first character as Satou suggested and check that instead.
[ April 10, 2006: Message edited by: Joanne Neal ]

Thanks.