Transponieren von 2 Arrays - IndexOutOfBoundsException

[Student]

editiert, aber immer noch nicht am ziel .. :-(
----------


Hi,
also ich bin anscheinend schon im Wochenende. :roll:

Ich habe eine Tabelle, die ich folgendermaßen mit Inhalt fülle.

String[] columnNames = { "Spalte1",
                         "Spalte2",
                         "Spalte3" };

int spaltenzahl = columnNames.length;
int zeilenzahl  = 5;

Object[][] rowData = new Object[zeilenzahl][spaltenzahl];

for (int i = 0; i < zeilenzahl; i++ )
{
         for ( int j = 0; j < spaltenzahl; j++ )
         {
                  rowData[i][j] = "Zelle " + i + "," + j;

         }
}

Die beiden Arrays werden dann an im Konstruktor übergeben:

MyTableModel dataModel  = new MyTableModel( columnNames, rowData );

So. Ich will jetzt die Tabelle transponieren.
Dazu habe ich folgende Methode geschrieben bzw. versucht zu schreiben:

public void transponiere()
{
      int lengthRowData   = m_dataModel.getRowCount();
      int lengthColumnTop = m_dataModel.getColumnCount();

      Object[]   transpColumnTop = new Object[lengthRowData + 1];
      Object[][] transpRowData   = new Object[lengthColumnTop - 1][lengthRowData + 1];

      transpColumnTop[0] = m_dataModel.getColumnName(0);

      // transponiertes columnTop-Array füllen
      for( int i = 0; i < lengthColumnTop; i++ )
      {
            transpColumnTop[i+1] = m_dataModel.getValueAt( i, 0 );
      }

      // transponiertes rowData-Array füllen
      for (int i = 0; i < (lengthColumnTop-1); i++ )
      {
            System.out.println( "i:" + i + "\n --- ");

            for ( int j = 0; j < lengthRowData; j++ )
            {
                  System.out.println( "j:" + j + "\n --- ");

                  if( i == 0 )
                  {
                    System.out.println( "transpRowData[i][j]: (j,i)/("+j+","+i+")" + transpRowData[i][j]);
                    transpRowData[i][j] = m_dataModel.getColumnName( j );                                    
                  }
                  else
                  {
                    System.out.println( "(j,i)/("+j+","+i+")" + m_dataModel.getValueAt( j, i ) );
                    transpRowData[i][j] = m_dataModel.getValueAt( j, i );
                  }

                 System.out.println( "---");
            }
       }

//*
      // Testausgabe 
      for (int i = 0; i < lengthRowData; i++ )
      {
            for ( int j = 0; j < lengthColumnTop; j++ )
            {
                   System.out.println(transpRowData[i][j] );
            }
      } //*/

}

Wenn ich Testausgabe nun laufen lasse erhalte ich folgende Ausgabe:

i:0
---
j:0
---
transpRowData[j]: (j,i)/(0,0)Spalte1
---
j:1
---
transpRowData[j]: (j,i)/(1,0)Spalte2
---
i:1
---
j:0
---
(j,i)/(0,1)Zelle 0,1
---
j:1
---
(j,i)/(1,1)Zelle 1,1
---
i:2
---
j:0
---
(j,i)/(0,2)Zelle 0,2

und folgende Fehlermeldung:

ArrayIndexOutOfBoundsException: 3

Ich erhalte jetzt einen Fehler in Zeile 79:

transpRowData[i][j] = m_dataModel.getColumnName( j );

Das 'j' ist falsch. Aber ich weiß nicht, wie ich das machen kann, dass in der ersten Spalte eben die ehemaligen Spaltennamen stehen.


ich habe nun noch eine kleine Änderung hinzugefügt.
Die Abbruchbedigungen innerhalb der beiden verschachtelten FOR-Schleifen haben sich geändert, wobei die innere nicht sicher ist .. also anders ausgedrückt: Fehlerpotential ist vorhanden.



Eventuell hilft das hier noch zur Verdeutlichung:

Vorher:

Spalte1 Spalte2 Spalte3
 (0,0)     (0,1)    (0,2)
 (1,0)     (1,1)    (1,2)
 (2,0)     (2,1)    (2,2)
 (3,0)     (3,1)    (3,2)
 (4,0)     (4,1)    (4,2)

Nachher:

Spalte1 (0,0) (1,0) (2,0) (3,0) (4,0)
Spalte2 (0,1) (1,1) (2,1) (3,1) (4,1)
Spalte3 (0,2) (1,2) (2,2) (3,2) (4,2)

Ich hoffe, dass das eventuell noch etwas weiterhilft.

Danke ..

 

[Gast]

Hallo

etwas schwierig ohne den ganzen Code. Schau aber mal

Object[][] transpRowData = new Object[lengthColumnTop - 1][lengthRowData + 1];

Wenn mich nicht alles täuscht müßte es heißen

Object[][] transpRowData = new Object[lengthRowData + 1][lengthColumnTop - 1];

Warum bei "lengthColumnTop - 1" alledings eine 1 abgezogen wird ist mir nicht klar geworden

Guß

 

[abollm]

Object[][] transpRowData = new Object[lengthColumnTop - 1][lengthRowData + 1];

Wenn mich nicht alles täuscht müßte es heißen

Object[][] transpRowData = new Object[lengthRowData + 1][lengthColumnTop - 1];

...

... und lass dir doch einmal in der Testausgabe zunächst nur die Längenwerte "lengthColumnTop" und "lengthRowData" ausgeben.

 

[Student]

... und lass dir doch einmal in der Testausgabe zunächst nur die Längenwerte "lengthColumnTop" und "lengthRowData" ausgeben.

erst mal das hier:

System.out.println( "lengthColumnTop: " + lengthColumnTop );
System.out.println( "lengthRowData: " + lengthRowData );

das ergebnis:

lengthColumnTop: 3
lengthRowData: 5

wenn ich das mache, was der gast ;-) sagt, also das hier:

transpColumnTop = new Object[lengthRowData + 1];
transpRowData   = new Object[lengthRowData + 1][lengthColumnTop - 1];

// [..]

for (int i = 0; i < (lengthColumnTop); i++ )
{
   [..]

erhalte ich eine ArrayIndexOutOfBoundsException.

 

[abollm]

wenn ich das mache, was der gast ;-) sagt, also das hier:

transpColumnTop = new Object[lengthRowData + 1];
transpRowData   = new Object[lengthRowData + 1][lengthColumnTop - 1];

// [..]

for (int i = 0; i < (lengthColumnTop); i++ )
{
   [..]

erhalte ich eine ArrayIndexOutOfBoundsException.

Aber wenn du dir deine Tabellen ansiehst, dann hat Gast grundsätzlich den Punkt schon getroffen. Du hast in der Ausgangsmatrix fünf reine Datenzeilen und drei reine Datenspalten.

Nach der Transposition (Transponierung?) musst du also den ursprünglichen Spaltenkopf in der JTable entsprechend berücksichtigen. Ich hab' mich jetzt aber nicht in deinen geposteten Code eingelesen.

 

[Student]

Nach der Transposition (Transponierung?)

:arrow: http://de.wikipedia.org/wiki/Transposition
:arrow: http://de.wikipedia.org/wiki/Matrix_(Mathematik)#Die_transponierte_Matrix

:arrow: :wink:

gut. danke. ich fahr jetzt erstmal nach hause und werde das dort nochmal austesten.
wenn jemand noch einen groben schnitzer entdeckt wäre es klasse, wenn das hier noch geschrieben würde.

zum thema: zu wenig code.
ich kann ja schlecht das forum vollspammen .. ;-)


ok. ich werde noch mal auf diesen thread zurückkommen ..


ciao ben.

 

[Student]

hi. also ...
ich komme einfach nicht dahinter .. denkblockade hoch 5.

also nochmal.
die derzeitige methode 'transponiere'

   public void transponiere()
    {
        int lengthColumnTop = m_dataModel.getColumnCount();        
        int lengthRowData   = m_dataModel.getRowCount();
        
        transpColumnTop = new Object[lengthRowData + 1];
        transpRowData   = new Object[lengthColumnTop - 1][lengthRowData + 1];

        lengthColumnTop = transpColumnTop.length;
        lengthRowData   = transpRowData.length;
        
        System.out.println( "maximaler Index transpColumnTop: " + (transpColumnTop.length - 1) );
        System.out.println( "maximaler Index transpRowData: " + (transpRowData.length - 1) );
               
        transpColumnTop[0] = m_dataModel.getColumnName(0);

        
// RICHTIG         
        // transponiertes columnTop-Array füllen
        for( int i = 0; i < (lengthColumnTop-1); i++ )
        {            
            transpColumnTop[i+1] = m_dataModel.getValueAt( i, 0 );
        }


        
        // transponiertes rowData-Array füllen
        for (int i = 0; i < (lengthColumnTop-1); i++ )
	{
            System.out.println( "i:" + i + "\n --- ");
	    for ( int j = 0; j < (lengthRowData); j++ )
	    {
                System.out.println( "j:" + j + "\n --- ");
                if( i == 0 )
                {
                    transpRowData[i][j] = m_dataModel.getColumnName( j );
                    //System.out.println( "(j,i)/("+j+","+i+")" + m_dataModel.getValueAt( j, i ) );
                    System.out.println( "transpRowData[i][j]: (j,i)/("+j+","+i+")" + transpRowData[i][j]);
                }
                else
                {
                    System.out.println( "(j,i)/("+j+","+i+")" + m_dataModel.getValueAt( j, i ) );
                    transpRowData[i][j] = m_dataModel.getValueAt( j, i );
                }

                System.out.println( "---");
			}
		}
          //System.out.println(transpRowData[0][4] );
  /*
        for (int i = 0; i < lengthColumnTop; i++ )
		{
			for ( int j = 0; j < lengthRowData; j++ )
			{
                System.out.print( "(i,j)/("+i+","+j+") --> ");
                System.out.println(transpRowData[i][j] );
			}
		}             //*/

    }

die klasse 'MyTableModel':

package SHD;

import java.util.*;
import javax.swing.*;
import javax.swing.table.*;
import java.awt.*;

class MyTableModel extends AbstractTableModel
{

	private Object[] m_columnName;
	private Object[][] m_rowData;

	public MyTableModel( Object[] columnName, Object[][] rowData )
	{
        m_columnName = columnName;
		m_rowData = rowData;
	}

	public String getColumnName(int columnIndex)
	{
		return m_columnName[columnIndex].toString();
	}

	public Object getValueAt( int x, int y )
	{
		String returnValue = "";

		returnValue = "x = " + x + ",y = " + y;
        //System.out.println( "(x,y)" + "("+x+","+y+")" );
		returnValue = m_rowData[x][y].toString();

		return returnValue;
	}

	public int getColumnCount()
	{
		return m_columnName.length;
	}

	public int getRowCount()
	{
		return m_rowData.length;
	}


    public Object[][] getRowData()
    {
        return m_rowData;
    }


    public void setTransponierteDaten( Object[] columnName, Object[][] rowData )
    {
        m_columnName = columnName;
        m_rowData    = rowData;
    }
}

die klasse 'TableDemo':

//package de.shd.basis.ui;
package SHD;

import javax.swing.*;

public class TableDemo
{
    public static void main( String[] args )
    {
        String[] columnNames = { "Spalte1",
                                 "Spalte2",
                                 "Spalte3" };

        int spaltenzahl = columnNames.length;
        int zeilenzahl  = 5;

        Object[][] rowData = new Object[zeilenzahl][spaltenzahl];

		for (int i = 0; i < zeilenzahl; i++ )
		{
			for ( int j = 0; j < spaltenzahl; j++ )
			{
				rowData[i][j] = "Zelle " + i + "," + j;
			}
		}

        MyTableModel dataModel  = new MyTableModel( columnNames, rowData );
        ScrollTable scrollTable = new ScrollTable( dataModel );

        JFrame frame = scrollTable.getFrame();
 /*       
        frame.dispose();
        frame = null;//*/

        scrollTable.transponiere();


/*
        dataModel.setTransponierteDaten( scrollTable.getTranspColumnTop(), scrollTable.getTranspRowData() );
        scrollTable = new ScrollTable( dataModel );
        frame = scrollTable.getFrame();   //*/
    }
}

bitte entschuldigt den ganzen code, aber ich weiß echt nicht mehr weiter.

ich bekomme in dieser form folgende ausgabe & fehlermeldung:

maximaler Index transpColumnTop: 5
maximaler Index transpRowData: 1
i:0
 --- 
j:0
 --- 
transpRowData[i][j]: (j,i)/(0,0)Spalte1
---
j:1
 --- 
transpRowData[i][j]: (j,i)/(1,0)Spalte2
---
i:1
 --- 
j:0
 --- 
(j,i)/(0,1)Zelle 0,1
---
j:1
 --- 
(j,i)/(1,1)Zelle 1,1
---
i:2
 --- 
j:0
 --- 
(j,i)/(0,2)Zelle 0,2
java.lang.ArrayIndexOutOfBoundsException
        at SHD.ScrollTable.transponiere(ScrollTable.java:97)
        at SHD.TableDemo.main(TableDemo.java:35)

die zeile 97 ist diese hier:

                else
                {
                    System.out.println( "(j,i)/("+j+","+i+")" + m_dataModel.getValueAt( j, i ) );

// -----------
// das hier ist zeile 97
                    transpRowData[i][j] = m_dataModel.getValueAt( j, i );
// -----------
                }

die fehlermeldung betrifft nicht den rechten teil .. also den zugewiesenen wert .. sondern

transpRowData[i][j]

wenn ich jetzt bei der arraydefinition eine änderung anbringe, z.b.

transpRowData   = new Object[lengthRowData + 1][lengthColumnTop-1];

dann erhalte ich den fehler in zeile 90

                if( i == 0 )
                {
// -----------
// das hier ist zeile 97
                    transpRowData[i][j] = m_dataModel.getColumnName( j );
// -----------
                    //System.out.println( "(j,i)/("+j+","+i+")" + m_dataModel.getValueAt( j, i ) );
                    System.out.println( "transpRowData[i][j]: (j,i)/("+j+","+i+")" + transpRowData[i][j]);
                }

der fehler ist also nur vom ELSE in den IF-zweig 'gewandert'.

kurz gesagt .. ich weiß einfach nicht mehr weiter.
danke für eure hilfe ..

 

[Student]

niemand?

 

[abollm]

niemand?

Vielleicht morgen, heute war ein langer Tag. Blicke zudem noch nicht so ganz durch deinen geposteten Code durch.

 

[Wildcard]

So müsste es gehen:

        int lengthColumnTop = m_dataModel.getColumnCount();        
        int lengthRowData   = m_dataModel.getRowCount(); 
        transpRowData   = new Object[lengthColumnTop][lengthRowData] ; 
        lengthRowData   = lengthColumnTop;
        lengthColumnTop = transpColumnTop.length; 

        for (int cols=0;cols<lengthColumnTop;cols++)
        {
            for (int rows = 0;rows<lengthRowData;rows++)
            {
                transpRowData[rows][cols] = m_dataModel.getValueAt(cols,rows);
            }
        }

sry, hab bei deinem Code nicht so ganz durchgeblickt ???:L

 

[Student]

Blicke zudem noch nicht so ganz durch deinen geposteten Code durch.

wenn es dazu fragen gibt. her damit ;-)

@wildcard:
etwas anspruchsvoller ist das schon. ich dachte eigentlich, dass ich das ganz gut beschrieben hätte.

wenn ich mich da täusche und da noch bedarf ist, dann sagt es bitte.
ich verschleppe das problem, aber das hilft mir ja auch nicht wirklich weiter, ???:L

grüße ben.

 

[Guest]

@Student:

Benötigst du eine allgemeine Lösung oder reicht dir auch eine für quadratische Matrizen? Wenn Letzteres dann habe ich eine Lösung für duch.

 

[abollm]

@Student:

Benötigst du eine allgemeine Lösung oder reicht dir auch eine für quadratische Matrizen? Wenn Letzteres dann habe ich eine Lösung für duch.

Das war ich, war nicht angemeldet.

 

[Student]

allgemein. ;-)

 

[Wildcard]

@Student

@wildcard:
etwas anspruchsvoller ist das schon. ich dachte eigentlich, dass ich das ganz gut beschrieben hätte.

Hab ich dich irgendwie falsch verstanden?
Fasse mal zusammen:
Du willst in deinem Model Spalten und Zeilen vertauschen.
Im Klartext soll das also später so aussehen wie in deinem Beispiel.

Wenn ich jetzt den Code benutze den ich dir gepostet habe stehen folgende Daten im Model:

(0,0) (1,0) (2,0) (3,0) (4,0) 
(0,1) (1,1) (2,1) (3,1) (4,1) 
(0,2) (1,2) (2,2) (3,2) (4,2)

Warum also anspruchsvoller?
Der Rest hat bei dir doch schon funktioniert?

Oder bin ich jetzt schon im Wochenende ???:L

 

[Student]

hey wildcard .. sorry, dass ich nicht mehr geantwortet habe. ich habe kurzfristig noch an einer anderen aufgabe arbeiten müssen.

lösung meines problems:

public class TransposedTable
{
    private Object[] colheads;
    private Object[][] data;


    public TransposedTable( Object[] colheads, Object[][] data )
    {
        this.colheads = colheads;
        this.data     = data;
    }

    // zum Test
    public Object[][] getTable()
    {
        return joinColheadsAndData();
    }

    private Object[][] joinColheadsAndData()
    {
        int rowcount  = data.length;
        int columncount = colheads.length;
        Object[][] joined = new Object[rowcount + 1][columncount];

        for( int col = 0; col < columncount; col++  )
        {
            joined[0][col] = colheads[col];
        }


        for( int row = 0; row < rowcount; row++ )
        {
            for( int col = 0; col < columncount; col++ )
            {
                joined[row+1][col] = data[row][col];
            }
        }

        return joined;
    }

    /**
     *
     * @return Object[][]
     */
    public Object[][] transposeCompleteTable()
    {
        Object[][] tableData = this.joinColheadsAndData();

        int rowcount  = tableData.length;
        int columncount = tableData[0].length;

        Object[][] transposedTableData = new Object[columncount][rowcount];

        for( int col = 0; col < columncount; col++ )
        {
            for( int row = 0; row < rowcount; row++ )
            {
                transposedTableData[col][row] = tableData[row][col];
            }
        }

        return transposedTableData;
    }
}

und die aufrufende klasse.

public class TableDemo implements TableData
{
    public static void main( String[] args )
    {
        TransposedTable transposed = new TransposedTable( COLHEADS, DATA );

        Object[][] tableData = transposed.getTable();

        //* Testausgabe
        for( int row = 0; row < tableData.length; row++ )
        {
            for( int col = 0; col < tableData[0].length; col++ )
            {
                 System.out.print( " " + tableData[row][col] );
            }
            System.out.println( "" );
        }

        System.out.println( "\n\n" );
        Object[][] transposedTableData = transposed.transposeCompleteTable();


        //* Testausgabe
        for( int row = 0; row < transposedTableData.length; row++ )
        {
            for( int col = 0; col < transposedTableData[0].length; col++ )
            {
                 System.out.print( " " + transposedTableData[row][col] );
            }
            System.out.println( "" );
        }
}

und das interface TableData

public interface TableData
{
  public static final Object[][] DATA = {
    {"(0,0)", "(0,1)", "(0,2)", "(0,3)", "(0,4)"},
	{"(1,0)", "(1,1)", "(1,2)", "(1,3)", "(1,4)"},
	{"(2,0)", "(2,1)", "(2,2)", "(2,3)", "(2,4)"}
  };

  public static final String[] COLHEADS = {
    "Spalte1", "Spalte2", "Spalte3", "Spalte4", "Spalte5"
  };

}

:arrow: ;-)
ich fasse also erst alles zu einer matrix zusammen und transponiere sie dann. ich habe mir einfach zu viele gedanken gemacht.

ich danke trotzdem allen, die sich gedanken gemacht haben.

grüße ben ;-)