Exception ArrayIndexOutOfBound

[e9926044]

Hi,


Hat jemand eine Idee, was ich gegen diese Fehlermeldung machen kann? tritt bei einer Zeile einer Tabelle auf wenn ich eine Zeile wegnehme:

Exception occurred during event dispatching:
java.lang.ArrayIndexOutOfBoundsException: 5 >= 5

 

[P3AC3MAK3R]

Poste am besten mal ein wenig Code.

 

[e9926044]

private void removeRow(int reihe, int Seite, DefaultTableModel model)
        {
            int selRow = tableList[Seite].getSelectedRow();              
            model.removeRow(reihe);
            model.fireTableDataChanged();
            tableList[Seite].repaint();   
            obj = null;
            model = null;   
            row[Seite]--;            // hier bekomme ich beim Debuggen diesen Fehler, keine Ahnug wieso              
            if (selRow != -1)
                tableList[Seite].changeSelection(selRow-1, 0, false, false);                  
        }

[/code]

 

[Ariol]

versuchs mal mit

row[Seite-1]--;

EDIT: Ich vermute mal, dass du ein int-Array mit der Anzahl der Seiten(5 Stück so angelegt hast)

int rows[] =int[5]

Ansprechen tust du aber Seite1 mit rows[0], Seite 2 mit row[1], usw...

Du musst das dann vermutlich überall ändern.

 

[e9926044]

Das passta uf jeden Fall aber ich ruf die obere Methode so auf:

 if (tableModelListen[x].getValueAt(i, 0) != null && tableModelListen[x].getValueAt(i, 2) != null)
                                                    {
 /* hier liegt der Fehler, i = Reihe, x = Seite */  removeRow(i, x, (DefaultTableModel)tableList[x].getModel());                                                                                                    
                                                    }

Die FehlerMeldung scahut so aus:

Exception in thread "AWT-EventQueue-0" java.lang.ArrayIndexOutOfBoundsException: 1 >= 1
        at java.util.Vector.elementAt(Vector.java:427)
        at javax.swing.table.DefaultTableModel.getValueAt(DefaultTableModel.java:632)
        at javax.swing.JTable.getValueAt(JTable.java:2652)
        at javax.swing.JTable.prepareRenderer(JTable.java:5666)
...

Sieht da jemand einen Zusammenhang, ich komm einfach nicht drauf,

Es hat auf jeden Fall irgendwas mit der JTable zu tun, irgendwie so, dass ich eine Zeile von der JTable wegnehme, denn wenn ich eine Zeile wegnehme und eine leere Zeile hinzufüge, dann funktionierts,


lg

 

[e9926044]

Diese Methode in der API Klasse Vector löst meine Exception aus, Was kann ich da machen bzw. was mache ich da falsch??

lg

public synchronized E elementAt(int index) {
	if (index >= elementCount) {
	    throw new ArrayIndexOutOfBoundsException(index + " >= " + elementCount);
	}

        return (E)elementData[index];
    }

Ich möchte ja einfach nur, dass, wenn ich von meinem Programm etwas bekomme, dass sich eine Zeile zur JTable hinzufügt und wenn die Zeile voll ist, dass die Zeile gelöscht wird,

 

[Ariol]

Das hier

 Exception in thread "AWT-EventQueue-0" java.lang.ArrayIndexOutOfBoundsException: 1 >= 1

bedeutet, das du ein Array der Länge 1 hast (also ein Element auf Position 0) aber du versuchst auf Position 1 zuzugreifen.

Du fängst bei 0 an zu Zählen, also kannst du bei einem Array der Länge x auf x Elemente (array[0] bis array[x-1]) zugreifen.

An der Position x gibt es kein Element und genau deshalb wird deine Exception ausgeworfen

 

[e9926044]

Ja das ist mir klar aber ich habe keine Ahnung wo das bei mir in Code ist, die Exception wirft der Compiler irgendwo, was soll ich da machen??

 

[Ariol]

Zeig mal wie row[] initialisiert wird

 

[ms]

Ja das ist mir klar aber ich habe keine Ahnung wo das bei mir in Code ist, die Exception wirft der Compiler irgendwo, was soll ich da machen??

Der Compiler wirft keine Exception.
Im Stacktrace wird dir die Zeile im Code angezeigt, wo der Fehler auftritt.
Genau dort passiert das.
Und was der Grund ist wurde ja schon erwähnt.

ms

 

[e9926044]

int[] row = new int[xSeiten];


der Entscheidende Code sieht so aus:


case 1:         try {  
                                    for (int x = 0; x < alarmListenCnt; x++)                                               
                                        {
                                        for (int i = 0; i < row[x]; i++)
                                            {
                                            if (alarmNr.equals(tableModelListen[x].getValueAt(i, 4)) && tableModelListen[x].getValueAt(i, 1) == null)                      // welche Zeile ist die gesuchte
                                                {
                                                //addQuitButton(i, x);   
                                                tableList[x].setValueAt(gegangen.toString(), i, 1);
                                                tableList[x].repaint();
                                                time = tableModelListen[x].getValueAt(i, 0).toString();
                                                alarmNr = tableModelListen[x].getValueAt(i, 4).toString();                            
                                                if (tableModelListen[x].getValueAt(i, 0) != null && tableModelListen[x].getValueAt(i, 2) != null)                                                                                                        
 /* hier liegt der Fehler, i = Reihe, x = Seite */  removeRow(i, x, (DefaultTableModel)tableList[x].getModel());                                                                                                    
                                                }
                                            }
                                        }  
                                                                                
                                      }
                                catch (Exception exce){}
                                break;


                                     // i,            x,              das wird oben übergeben 
private void removeRow(int reihe, int xSeite, DefaultTableModel model)
        {
        int selRow = tableList[xSeite].getSelectedRow();   
        //tableList[xSeite].setValueAt(null, reihe, 6);
        model.removeRow(reihe);         
        model.fireTableDataChanged();
        tableList[xSeite].repaint();  
        addReihe(model, xSeite);
        model = null;   
        row[xSeite]--;
        if (selRow != -1)
            tableList[xSeite].changeSelection(selRow-1, 0, false, false);                  
        }

 

[e9926044]

Ja das ist mir klar aber ich habe keine Ahnung wo das bei mir in Code ist, die Exception wirft der Compiler irgendwo, was soll ich da machen??

Der Compiler wirft keine Exception.
Im Stacktrace wird dir die Zeile im Code angezeigt, wo der Fehler auftritt.
Genau dort passiert das.
Und was der Grund ist wurde ja schon erwähnt.

ms

ja mir ist aber überhaupt nicht klar, wo ich da in meinem Code etwas falsch mache,
Vielleicht hat ja jemand eine Idee,

vielen dank

 

[Ariol]

Das Problem liegt darin das du die die Variable in row[x] kleiner machst:

Angenommen row[x] ist 5, dann zählst du hier von 0 bis 4

for (int i = 0; i < row[x]; i++)

wenn jetzt eine zeile die Löschbedingung erfüllt wird setzt du den Wert in row[x] auf 4

row[xSeite]--;

das heisst dein höchstes "i" darf höchstens 3 sein, aber es zählt in der Schleife immer noch bis 4:

for (int i = 0; i < row[x]; i++)

wenn du jetzt ein z.B. die letzte Zeile löschen willst (index = 4) versuchts du mit

row[xSeite]--;

auf den 5. Wert zuzugreifen, obwohl es nur 3 gibt.

 

[e9926044]

Ja, in die Richtung habe ich auch schon gedacht aber ich weiß nicht, wie ich das lösen kann, ich dachte mir, wenn ich

row[xSeite]--;

schreibe, dann wird das row in der for- Schleife ja auch dekrementiert:

for (int i = 0; i < row[x]; i++)

da ja row[xSeite] und row[x] die gleiche Zahl beinhalten sollen und wenn ich unten row[xSeite]-- schreibe, dann läuft ja die for- Schleife auch nur bis row[xSeite]--, oder liege ich da falsch,

Wie könnte ich das ganze lösen, momentan fällt mir dazu leider nicht sehr viel ein


herzlichen Dank für die Hilfe

lg

 

[e9926044]

Wie könnte ich das am Besten lösen von diesem Stan djetzt aus, steh momentan ziemlich an,

lg

 

[e9926044]

Vielleicht hat ja noch irgendwer eine Idee dazu, wie ich das machen könnte, komm nicht weiter bei diesem Problem,

lg

 

[SlaterB]

was ist denn aktuell überhaupt der Fehler?
anfangs ging es um ein int-Array, später um das DefaultTableModel,

tritt der Fehler bei einem deiner Aufrufe (wie etwa removeRow()) auf oder beim repaint()?
ein vollständiges Programm würde das ganze zu einer Sache von ein paar Minuten machen,
so ist es nur sinnloses Rumgerate

 

[e9926044]

Die ganze Klasse ist ziemlich groß aber ich habe den Fehler eh schon lokalisiert, das Problem ist, dass ich die Zeilen einer JTable durchlaufe und in der Spalte 1 etwas einfüge, wenn jetzt in einer Zeile in der Spalte 0 und 2 etwas drinnensteht und ich eben in Spalte 1 etwas hineinschreibe, dann ist die Zeile vollständig und ich lösche die Zeile

-> daher ist meine Zeilenanzahl nach dem Löschen um eins kleiner (row[x]--) aber ich bekomme trotzdem ganz am Schluss nach dem durchlaufen folgende Exception:

Exception in thread "AWT-EventQueue-0" java.lang.ArrayIndexOutOfBoundsException: 1 >= 1

case 1:                   for (int x = 0; x < listenCnt; x++)                                                
                                        {
                                        for (int i = 0; i < row[x]; i++)
                                            {
                                            if (alNr.equals(tableModelListen[x].getValueAt(i, 4)) && tableModelListen[x].getValueAt(i, 1) == null)                      
                                                {
                                                tableList[x].setValueAt(gegangen.toString(), i, 1);
                                                tableList[x].repaint();                            
                                                if (tableModelListen[x].getValueAt(i, 0) != null && tableModelListen[x].getValueAt(i, 2) != null) 
                                                    {
 /* hier liegt der Fehler, i = Reihe, x = Seite */  removeReihe(i, x, (DefaultTableModel)tableList[x].getModel()); 
                                                    row[x]--;
                                                    }
                                                }
                                            }
                                        }  
                                break;



private void removeReihe(int reihe, int xSeite, DefaultTableModel model)
        {
        model.removeRow(reihe);         
        model.fireTableDataChanged();
        tableList[xSeite].repaint();  
       // addReihe(model, xSeite);  // wenn diese Zeile nicht auskommentiert ist, dann funktionierts, aber ich habe halt in der JTable ein paar Zeilen, wo gar nichts ausgefüllt ist aber ich möchte, das nur die Zeilen angezeit werden, wo irgendetwas drinnensteht, also ich möchte keine leeren Zeilen in meiner JTable
        model = null;            
        }

private void addReihe(DefaultTableModel model, int xSeite)
        {
        Object[] obj = {};
        model.addRow(obj);
        obj = null;
        }

Ich hab schon einiges versucht aber ich weiß nicht, wo ich da übers Array drüberlaufe,

 

[SlaterB]

mit halbgaren Ausreden kommst du nicht weiter,
es ist immer noch nicht klar, wann und wo die Exception auftritt,
da muss es doch 30 Zeilen StackTrace geben?
poste die doch einfach mal..

---------

was soll
Object[] obj = {};
model.addRow(obj);
bringen?

wenn du in eine JTable mit mehr als 0 Spalten eine Zeile mit 0 Spalten einfügst,
dann muss es ja später irgendwo krachen,
allerdings scheint deine Fehlermeldung eine andere zu sein..
außerdem sagst du ja, dass es gerade dann funktioniert, wenn die Zeile drin ist

 

[e9926044]

naja, die Zeile die ich mit

 private void addReihe(DefaultTableModel model, int xSeite)
        {
        Object[] obj = {};
        model.addRow(obj);
        obj = null;
        }

hinzufüge, hat aber genausoviel Spalten wie die anderen!?


so schaut der StackTrace aus:

 Exception in thread "AWT-EventQueue-0" java.lang.ArrayIndexOutOfBoundsException: 4 >= 4
        at java.util.Vector.elementAt(Vector.java:427)
        at javax.swing.table.DefaultTableModel.getValueAt(DefaultTableModel.java:632)
        at javax.swing.JTable.getValueAt(JTable.java:2652)
        at net.component.table.getCellRenderer(table.java:47)
        at javax.swing.plaf.basic.BasicTableUI.paintCell(BasicTableUI.java:2066)
        at javax.swing.plaf.basic.BasicTableUI.paintCells(BasicTableUI.java:1969)
        at javax.swing.plaf.basic.BasicTableUI.paint(BasicTableUI.java:1765)
        at javax.swing.plaf.ComponentUI.update(ComponentUI.java:143)
        at javax.swing.JComponent.paintComponent(JComponent.java:758)
        at javax.swing.JComponent.paint(JComponent.java:1022)
        at javax.swing.JComponent.paintToOffscreen(JComponent.java:5104)
        at javax.swing.BufferStrategyPaintManager.paint(BufferStrategyPaintManager.java:285)
        at javax.swing.RepaintManager.paint(RepaintManager.java:1132)
        at javax.swing.JComponent._paintImmediately(JComponent.java:5052)
        at javax.swing.JComponent.paintImmediately(JComponent.java:4862)
        at javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:727)
        at javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:683)
        at javax.swing.RepaintManager.seqPaintDirtyRegions(RepaintManager.java:663)
        at javax.swing.SystemEventQueueUtilities$ComponentWorkRequest.run(SystemEventQueueUtilities.java:128)
        at java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:209)
        at java.awt.EventQueue.dispatchEvent(EventQueue.java:597)
        at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:273)
        at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:183)
        at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:173)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:168)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:160)
        at java.awt.EventDispatchThread.run(EventDispatchThread.java:121)
Exception in thread "AWT-EventQueue-0" java.lang.ArrayIndexOutOfBoundsException: 2 >= 2
        at java.util.Vector.elementAt(Vector.java:427)
        at javax.swing.table.DefaultTableModel.getValueAt(DefaultTableModel.java:632)
        at javax.swing.JTable.getValueAt(JTable.java:2652)
        at net.component.JankTable.getCellRenderer(table.java:47)
        at javax.swing.plaf.basic.BasicTableUI.paintCell(BasicTableUI.java:2066)
        at javax.swing.plaf.basic.BasicTableUI.paintCells(BasicTableUI.java:1969)
        at javax.swing.plaf.basic.BasicTableUI.paint(BasicTableUI.java:1765)

usw.

 

[SlaterB]

also das mit addRow klappt tatsächlich, da war ich zu vorlaut

ich habe soviel wie möglich von deinem Code bei mir eingebaut und alles läuft hervorragend,
daher kann ich persönlich dir nicht weiterhelfen bis du nicht ein komplettes Programm oder andere Hinweise lieferst

Tipp:
deine vielen Arrays scheinen unnötig und fehleranfällig, wenn ich auch hier keine direkte Probleme sehe,
verrückt ist z.B.
if (tableModelListen[x].getValueAt(i, 0) != null && tableModelListen[x].getValueAt(i, 2) != null) {
removeReihe(i, x, (DefaultTableModel)tableList[x].getModel());

hier ist (DefaultTableModel)tableList[x].getModel() doch sicherlich == tableModelListen[x]?
wieso diese unterschiedlichen Zugriffe, verwende nur eines von beiden

und in der Operation removeReihe() dann übergibst du das Model als Parameter,
während du die zugehörige JTable aus einem Array holst,
wer sagt, dass der Parameter das Model zu der Tabelle im Array ist?
wenn schon, dann beide übergeben oder nur die JTable udn das Model aus der JTable holen,
oder beides aus den Arrays holen oder am besten nur ein Array der JTables und das Model immer direkt als der Table holen

wozu das row-Array? jedes Model weiß doch seine Zeilenanzahl?
usw



mein fehlerfreies VOLLSTÄNDIGES Programm:

public class TestGUI
    extends JFrame
{
    JTable[] tableList = new JTable[1];
    DefaultTableModel[] tableModelListen = new DefaultTableModel[1];
    int[] row = new int[1];

    public TestGUI()
        throws Exception
    {

        tableModelListen[0] = new DefaultTableModel(10, 1);
        row[0] =tableModelListen[0].getRowCount();
        for (int i = 0; i < row[0]; i++)
        {
            tableModelListen[0].setValueAt("" + i, i, 0);
        }
        tableList[0] = new JTable(tableModelListen[0]);

        JButton b = new JButton("test");
        b.addActionListener(new ActionListener()
            {
                public void actionPerformed(ActionEvent e)
                {
                    action();
                }
            });

        getContentPane().add(tableList[0], BorderLayout.CENTER);
        getContentPane().add(b, BorderLayout.SOUTH);

        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setSize(800, 400);
        setVisible(true);
    }

    public static void main(String[] args)
        throws Exception
    {
        TestGUI g = new TestGUI();
        for (int i = 0; i < 1000; i++)
        {
            Thread.sleep(30);
            g.repaint();
        }
    }

    private void action()
    {
        int listenCnt = tableList.length;
        for (int x = 0; x < listenCnt; x++)
        {
            for (int i = 0; i < row[x]; i++)
            {
                tableList[x].setValueAt("test", i, 0);
                tableList[x].repaint();
                boolean random = Math.random() > 0.5;
                if (random)
                {
                    System.out.println("entferne Reihe i: " + i);
                    removeReihe(i, x, (DefaultTableModel)tableList[x].getModel());
                    row[x]--;
                }
            }
        }
    }

    private void removeReihe(int reihe, int xSeite, DefaultTableModel model)
    {
        model.removeRow(reihe);
        model.fireTableDataChanged();
        tableList[xSeite].repaint();
        // addReihe(model, xSeite); 
        model = null;
    }

    private void addReihe(DefaultTableModel model, int xSeite)
    {
        Object[] obj =
            {};
        model.addRow(obj);
        obj = null;
    }

}

 

[SlaterB]

du führst die ganze Aktion nicht zufällig von einem anderen Thread als einem normalen ActionListener aus aus?
das könnte Probleme machen, siehe
http://www.java-forum.org/de/viewtopic.php?t=54763

 

[e9926044]

Da hast du recht, da ich den Fehler nicht gefunden habe, habe ich das ganze ein bisschen ungeschrieben und da den Überblick verloren,
Bin jetzt wieder im Lot aber ich weiß einfach nicht wo der Fehler liegt,
DankeSchön für Dein Beispiel, funktioniert super, keine Ahnung was es da bei mir hat,

ich werde jetzt auch versuchen, jede Änderung meiner Models in einen eigenen Thread zu geben und zu diesem Thema hätte ich noch eine Frage:

for (int i = 0; i < 10; i++)
{
SwingUtilities.invokeLater(new Runnable() {
         public void run() { 

// wie kann ich hier auf i zugreifen, ich glaub, es gibt 2 möglichkeiten, entweder die Variable final machen oder irgendwo anders deklarieren,
Kann ich das anders auch noch irgendwie machen?

}});
}

 

[SlaterB]

genau,
i als Exemplarvariable zu definieren wäre aber öde,

dann lieber
final int iFinal = i;

vielleicht auch die ganze Schleife in den Thread legen, mehrere Thread anzulegen wäre etwas heftig,

aber in der Richtung ist eh alles experimentell,
wichtig ist die Frage, wo denn nun der Code ursprünglich durchgeführt wurde?
wenn in einem ActionListener (so wie in meinem Code-Beispiel) dann dürfte das eh überflüsslig sein,
ein Versuch schadet aber nie, allein schon die Verzögerung wirkt manchmal Wunder (was dann aber auch nur ein Hinweis auf kaputtes Design ist..)

 

[tuxedo]

Wenn du einen Vektor (oder sonst irgendwas in der Art), Element für Element durchgehst, und zwischendrin die Größe veränderst (ein element löschen, eind hinzunehmen), dann führt das zu deinem Problem.

Vorschlag: Du löschst nicht während dem Durchlauf Elemente, sondern merkst dir wo gelöscht werden soll... Und nach dem Durchlaufen löschst du dann... BEi Vektor aber aufgepasst: Wenn du ein Element löschst, rückt der Index nach...

- Alex