Fehlermeldung

[Cheese]

Hallo zusammen,

ich habe Java erst vorkurzem angefangen. Bei meinem Programm bekomme ich immer wieder die Meldung: [JAVA=42] in thread "main" java.lang.ArrayIndexOutOfBoundsException: 6
at MeinFenster2.<init>(MeinFenster2.java:27)
at MeinFenster2.main(MeinFenster2.java:45) [/code]
Ich bin leider noch Java-Anfänger ich weiß was es genau heißen soll. ???:L

Hier der Quelltext:

import javax.swing.*;  

import java.awt.*;
import java.awt.event.*;


public class MeinFenster2 
extends JFrame implements ActionListener
{

//Attribute
JButton[] Knopf = new JButton[6];
private final String[] Text = {"Prima", "Gut", "Mäßig", "Schlecht", "Miserabel", "S*****e"};
private final String[] Diagnose = {"Das ist ja toll :D", "Das freut mich!","Das geht ja noch", "Das tut aber Leid!",
		                      "This is really bad", "Das macht man nur"};

//Konstuktor
public MeinFenster2 ()
{
	super ("Hallo, wie geht es dir?");
	int i;
	JPanel Cover = new JPanel ();
	Cover.setLayout (new GridLayout(6,1));
	for ( i = 0; i < 6; i++);
	{
		
		Knopf[i] = new JButton (Text[i]);
		Knopf[i].addActionListener (this);
		Cover.add (Knopf[i]);
 }
    setContentPane (Cover);}

public void actionPerformed (ActionEvent Ereignis)
{
	Object Quelle = Ereignis.getSource();
	for (int i = 0; i < 6; i++)
		if (Quelle == Knopf [i])
			JOptionPane.showMessageDialog
			(null, Diagnose[i]);
}
	
	 
	public static void main(String[] args) 
	{
		MeinFenster2 Rahmen = new MeinFenster2 ();
		Rahmen.setSize (500,500);
		Rahmen.setVisible(true);
			

	}

}

Ich hoffe ihr könnt mir helfen.

 

[Marcinek]

Normalerweise kommt das von einem Array. Du benutzt nun zwei und in den forschleifen hast du so wie ich das sehe auch die korrekten werte stehen.

Bei einem Array von 6 Elementen gehst du von 0...5 und iwo machst du noch eine 6 ;D

 

[ARadauer]

for ( i = 0; i < 6; i++);


die schleife stimmt ja grundsätzlich, warum kommt da bei i 6 raus? -> der ; ist zu viel.
die anweisung läuft 6 mal durch und geht dann mit i = 6 in die { }


Variblen schreibt man klein...

 

[Final_Striker]

for ( i = 0; i < 6; i++); <-- Das Semikolon muss weg

 

[Cheese]

for ( i = 0; i > 6; i++) Die [6] wird nicht abgefragt. [6] gibt es natürlich nicht!!.Das heißt ja i > 6 <-- i muss kleiner sein als 6 sonst macht er nicht weiter.

 

[Cheese]

Hallo,

ich hab die for-Schleife korrigiert (Semikolon entfernt)

Und jetzt bekomme ich diese Meldung


[JAVA=42] Exception in thread "main" java.lang.Error: Unresolved compilation problem:

at paket1.MeinFenster2.main(MeinFenster.java:43) [/code]

Ich glaube an meiner Main Teil ist irgendetwas falsch.

Ich hoffe ihr könnt mir vllt helfen

 

[TKausL]

Scheint ein Klammerfehler zu sein. Kannst du mal den aktuellen Code posten? Oben seh ich kein Klammerfehler.

 

[XHelp]

Und wie sieht der neue Code aus? Aber bevor du den postest: rück mal vernünftig ein und verwende

{

}

bei IFs und Schleifen.

 

[Cheese]

Hier ist der Quellcode:

 import javax.swing.*;  
import java.awt.*;
import java.awt.event.*;
 
public class MeinFenster2 
extends JFrame implements ActionListener
{
//Attribute
JButton[] Knopf = new JButton[6];
private final String[] Text = {"Prima", "Gut", "Mäßig", "Schlecht", "Miserabel", "S*****e"};
private final String[] Diagnose = {"Das ist ja toll :D", "Das freut mich!","Das geht ja noch", "Das tut aber Leid!",
                              "This is really bad", "Das macht man nur"};
//Konstuktor
public MeinFenster2 ()
{
    super ("Hallo, wie geht es dir?");
    int i;
    JPanel Cover = new JPanel ();
    Cover.setLayout (new GridLayout(6,1));
    for ( i = 0; i < 6; i++)
    {  
        Knopf[i] = new JButton (Text[i]);
        Knopf[i].addActionListener (this);
        Cover.add (Knopf[i]);
 }
    setContentPane (Cover);}
public void actionPerformed (ActionEvent Ereignis)
{
    Object Quelle = Ereignis.getSource();
    for (int i = 0; i < 6; i++)
{
        if (Quelle == Knopf [i])
            JOptionPane.showMessageDialog
            (null, Diagnose[i]); }
}
    public static void main(String[] args) 
    {
        MeinFenster2 Rahmen = new MeinFenster2();
        Rahmen.setSize (500,500);
        Rahmen.setVisible(true);     
    }
}

 

[TKausL]

Eingerückt ist immernoch nicht... Und in Zeile 43 steht nix.

 

[XHelp]

Der Code ist ok, aber wenn du weitere Fragen hast: rück den erstmal vernünftig ein!

 

[Tomate_Salat]

Der Code ist "ok". Die Einrückung ist nur fürchterlich und Konventionen stimmen nicht so richtig. Variablen schreibt man lowerCamelCase.

Schreib mal in die aller erste Zeile deiner MeinFenster2.java folgendes:

[c]package paket1;[/c]

dann sollte es gehen

 

[jfg]

Sowas hier solltest du vermeiden:

    int i;
    JPanel Cover = new JPanel ();
    Cover.setLayout (new GridLayout(6,1));
    for ( i = 0; i < 6; i++)

Den Scope (Gültigkeitsbereich) von Variablen so niedrig wie möglich halten. Hilfsvariablen immer da deklarieren wo sie auch verwendet werden. Also:

    for (int i = 0; i < 6; i++)

Davon abgesehen halte ich es für ziemlich schlechten Stil, beim Iterieren über Arrays mit fixen Längen zu arbeiten. Wenn du etwas an der Länge deiner String-Arrays änderst, fliegt dir dein ganzer Code auseinander. Also besser:

    for (int i = 0; i < Knopf.length; i++)