-1

I would like my array to from index to index like thisfor example: (0,0), (1,0), (2,0) etc. I've tried what seems like it should be the right way, but my loops stop after the first column and I get an index out of bounds exception.

Here's what I did:

int[][] array2d = 
            {
                {4,5, 3,8},
                {8,3,99,6},
                {5,7, 9,1}
                
            }; 

int currentRow = 0;
        for (int currentColumn = 0; currentColumn < (array2d[currentRow].length); currentColumn++) 
        { 
            for(currentRow = 0; currentRow < array2d.length; currentRow++) 
            {
                System.out.println(array2d[currentRow][currentColumn]);
            }
        }
Axel K.
  • 1
  • 1
  • The question header says the array is ragged, but your example does not show a ragged array. To get a good answer, you should tell us what you want to happen in cases where some rows have elements in a column but other rows do not have elements in that same column. – Old Dog Programmer Dec 03 '22 at 03:30
  • 1
    Oh thanks for pointing that out! I did originally want to figure out how to do it with a ragged array. I decided to figure things out on a regular array array so I have an idea of what to do with a ragged array, but I completely forgot to reword things – Axel K. Dec 03 '22 at 19:39

3 Answers3

1

First, let's address a bug in your code. Consider this loop:

int i;
for (i = 0; i < 10; i++) { /* loop body */ }

The loop runs until i < 10 evaluates to false. In this example, when the loop exits, i will have a value of 10.

Consider the inner loop in your code:

 for (currentRow = 0; currentRow < array2d.length; currentRow++)

When this loop exits, currentRow will be equal to array2d.length. Then, control will return to the outer loop. When it does, in the control expression currentColumn < (array2d[currentRow].length), this part array2d[currentRow] will throw an IndexOutOfBoundsException.

As you already know, here is the standard way of going through a 2D array in row by column order is like this:

  for (int row = 0; row < array2d.length; row++) { 
      for (int column = 0; column < array2d[row].length; column++ { 
        // things to do in inner loop
      }

If you wanted to process a rectangular 2D array in column order, the code might look like this:

 for (int column = 0; column < array2d[0].length; column++) { 
    for (int row = 0; row < array2d.length; row++) {
       // loop body
    }
 }

But, what if the array is ragged?

Start by finding the row with the maximum number of columns at the start:

  int maxColumns = 0;
  for (int i = 0; i < array.length; i++) { 
     maxColumns = Math.max (maxColumns, array2d[i].length);
  }

Then, your code might look something like this:

  for (int column = 0; column < maxColumns; column++) { 
     for (int row = 0; row < array2d.length; row++) { 
        if (column < array2d[row].length) {
           // do something with array2d [row][column]
        } else { 
           // do something for the case where 
           // array2d [row] doesn't have a value in "column" 
        }
     }
  }

An alternative might be to try to trap an ArrayIndexOutOfBoundsException. However, that might be regarded as a poor programming practice.

Old Dog Programmer
  • 901
  • 1
  • 7
  • 9
  • Spectacular demontration, you pretty much explained clearly everything that I was struggling to understand or figure out. It works just fine now, thanks! – Axel K. Dec 03 '22 at 20:26
0
        int[][] array2d =
                {
                    {1,2,3,4},
                    {5,6,7,8},
                    { 9,10,11,12}
                };
        int r=array2d.length,c=array2d[0].length;
        for (int i=0;i<c;i++)
        {
            for(int j=0;j<r;j++)
            {
                System.out.print(array2d[j][i]+" ");
            }
            System.out.println();
        }

You've ran the columns before the rows and because its a 2d array of length 3x4 , Its giving a indexOutOfBoundsException whilest iterating through the array.

ProgrammingGeek11
  • 68
  • 8
0

You need to get the maximum number of columns before printing.

public static void main(String[] args) {
    int[][] array2d = {
        {4, 5, 3, 8},
        {},
        {8, 6},
        {2},
        {5, 7, 9, 1, 0}
    };
    int maxRow = array2d.length;
    int maxColumn = array2d[0].length;
    for (int r = 1; r < maxRow; ++r)
        maxColumn = Math.max(maxColumn, array2d[r].length);
    for (int c = 0; c < maxColumn; ++c) {
        for (int r = 0; r < maxRow; ++r)
            if (c < array2d[r].length)
                System.out.print(array2d[r][c] + " ");
            else
                System.out.print("* ");
        System.out.println();
    }
}

output:

4 * 8 2 5 
5 * 6 * 7 
3 * * * 9 
8 * * * 1 
* * * * 0

Or you can get away with it by scanning the row one more time.

public static void main(String[] args) {
    int[][] array2d = {
        {4, 5, 3, 8},
        {},
        {8, 6},
        {2},
        {5, 7, 9, 1, 0}
    };
    int rowSie = array2d.length;
    for (int c = 0; true; ++c) {
        StringBuilder sb = new StringBuilder();
        boolean out = false;
        for (int r = 0; r < rowSie; ++r)
            if (c < array2d[r].length) {
                sb.append(array2d[r][c]).append(" ");
                out = true;
            } else
                sb.append("* ");
        if (!out)
            break;
        System.out.println(sb);
    }
}

output:

4 * 8 2 5 
5 * 6 * 7 
3 * * * 9 
8 * * * 1 
* * * * 0
ﾓｷｬﾃﾞ
  • 385
  • 1
  • 4