Array Index out of Bound Exception while running the program to find largest of an array

Question

Here is the code:Getting Array index out of Bound exception

class Max {
    public static void main(String args[]) {
        int a[][];
        Scanner src = new Scanner(System. in );
        System.out.println("Enter the no of rows");
        int rows = src.nextInt();
        a = new int[rows][5];
        for (int i = 0; i < rows; i++) {
            for (int j = 0; j < 5; j++) {
                a[i][j] = src.nextInt();
            }
        }
        System.out.println("Array is");
        for (int i = 0; i < rows; i++) {
            for (int j = 0; j < 5; j++) {
                System.out.print("  " + a[i][j]);
            }
            System.out.println();
        }
        int l = a[0][0];
        int i;
        for (i = 0; i < rows; i++)
            l = a[i][0];
        for (int j = 0; j < 5; j++)
            if (l < a[i][j])
                l = a[i][j];
        System.out.println("Max" + l);
    }
}

on run time it gives the following: Exception in thread "main" java.lang.Array index out of Bound Exception:3 at Max.main Max.java:33 Can Anyone suggest what is wrong in the code????

Your code is unreadable. Format it correctly. All Java IDEs can do that with a single keyboard shortcut. And tell us which line is 33. — JB Nizet
– JB Nizet, Commented Mar 2, 2014 at 9:08
I would step through your code in a debugger to debug you code and understand why it is not doing what you expect. — Peter Lawrey
– Peter Lawrey, Commented Mar 2, 2014 at 9:10

Nicolas Defranoux · Accepted Answer · 2014-03-02 09:11:05Z

2

You miss a { after the for (i = 0; i < rows; i++) so that the for (int j = 0; j < 5; j++) loop is inside the first one.

Without that, this gives:

for (i = 0; i < rows; i++)
  l = a[i][0];
// End of the for i loop, now i = rows. 
for (int j = 0; j < 5; j++)
  if (l < a[i][j])  // i = rows: bang.
    l = a[i][j];

answered Mar 2, 2014 at 9:11

Nicolas Defranoux

2,6761 gold badge13 silver badges13 bronze badges

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

Comments

Mohsen Kamrani · Accepted Answer · 2014-03-02 09:37:59Z

Try this:

public static void main(String args[]) {
    int a[][];
    Scanner src = new Scanner(System. in );
    System.out.println("Enter the no of rows");
    int rows = src.nextInt();
    a = new int[rows][5];
    for (int i = 0; i < rows; i++) {
        for (int j = 0; j < 5; j++) {
            a[i][j] = src.nextInt();
        }
    }

    System.out.println("Array is");
    for (int i = 0; i < rows; i++) {
        for (int j = 0; j < 5; j++) {
            System.out.print("  " + a[i][j]);
        }
        System.out.println();
    }
    int l = a[0][0];
    int i;
    for (i = 0; i < rows; i++) {
        l = a[i][0];
        for (int j = 0; j < 5; j++)
            if (l < a[i][j])
                l = a[i][j];
    }
    System.out.println("Max" + l);
}

Collectives™ on Stack Overflow

Array Index out of Bound Exception while running the program to find largest of an array

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related