Fehler im Programm

[riddick18]

package Schulprojekte;
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;

public class test extends JFrame implements ActionListener
{
//Atribute
	private JRadioButton[] Option = new JRadioButton[5];
	private JCheckBox[] Wahl = new JCheckBox[3];
	private String[] Text = 
	{"Durchschnittsnote 1", "Durchschnittsnote 2","Durchschnittsnote 3","Durchschnittsnote 4","Durchschnittsnote 4<"}; // Auswahlmöglichkeiten
		
    private String[] Extra=
    {"Grundschule","S1","S2"};  //Weitere Auswahlmöglichkeiten
    
    
// Konstruktor
    public test ()
    {
    	super("Übung für die Arbeit"); // Überschrift des Fensters
    	
    	ButtonGroup Gruppe = new ButtonGroup();
    	
    	JPanel Platte = new JPanel();
    	
    	Box links = Box.createVerticalBox();
    	Box rechts = Box.createVerticalBox();
    	Platte.setLayout (new FlowLayout());
    	
 // Radiobuttons erzeugen, mit Ereignissen verknüpfen und einsetzten
    	
    	for (int i =0;i<6;i++)
    	
    	{
    	Option[i] = new JRadioButton(Text[i]);
    	Option[i].addActionListener(this);
    	Gruppe.add (Option[i]);
        links.add(Option[i]); // Buttons werden der Gruppe und linksoben zugeordnet
        }
    	
    	
    	Option[0].setSelected (true);

// CheckBoxen erzeugen, mit Ereignissen verknüfen und einsetzen
    	for (int i =0;i<3;i++)
    	
    	{
    	Wahl[i]= new JCheckBox(Extra[i]);
    	Wahl[i].addActionListener(this);
    	rechts.add(Wahl[i]);
    	}
    	
    	Platte.add(links);
    	Platte.add(rechts);
    	setContentPane(Platte);
    	
        }

public void actionPerformed (ActionEvent Ereignis)
{
	
	Object Quelle =  Ereignis.getSource();
	String Titel = "Übung für die Arbeit";
	for (int i =0;i<3;i++)
		if (Wahl[i].isSelected())
			Titel= Titel+"("+Extra[i]+")";
	setTitle(Titel);
    for (int i=0;i<6;i++)
    	if (Quelle == Option[i])
    		JOptionPane.showMessageDialog(null,null);
    if (Wahl[1].isSelected()|Wahl[2].isSelected()|Wahl[3].isSelected()|Wahl[0].isSelected()&& Option[2].isSelected()|Option[1].isSelected())
    	JOptionPane.showMessageDialog(null,"Gut gemacht");
}

public static void main (String[] args)
{
	test Rahmen = new test();
	Rahmen.setSize(1200,500);
	Rahmen.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
	Rahmen.setVisible(true);
	
}
}

Und als Fehlermeldung bekomme ich :
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 5
at Schulprojekte.Aufgabe.<init>(Aufgabe.java:36)
at Schulprojekte.Aufgabe.main(Aufgabe.java:78)


Ich schreibe morgen eine Arbeit und es wäre nett wenn mir schnell jemand sagen könnte wo mein Fehler ist da ich ein ähnliches Programm hab und das funktioniert perfekt.

Ich weiß nicht ob das wichtig ist aber ich arbeite mit Eclipse helios 3.6

Danke schonmal

 

[Volvagia]

Warum ignorieren alle eigendlich immer die Exceptions und den StackTrace? Der versucht einen doch zu sagen, wo der Fehler liegt.

In Zeile 36 versucht er auf Feld 5 eines Arrays zuzugreifen, welches nicht existiert. Steht doch alles da.


Ach ja, Klassen schreibt man groß und Variablen klein. Viel scheint ihr ja nicht gerade gelernt zu haben.

 

[VfL_Freak]

Moin,

Warum ignorieren alle eigendlich immer die Exceptions und den StackTrace? Der versucht einen doch zu sagen, wo der Fehler liegt.
In Zeile 36 versucht er auf Feld 5 eines Arrays zuzugreifen, welches nicht existiert. Steht doch alles da.

genau ... außerdem sind die Java-Tags nicht nur als Zierrahmen des Editor hier gedacht

Schau Dir mal Deine entsprechende Deklaration und den Zugriff an, dann solltest Du den Fehler sofort sehen !!

Gruß
Klaus

 

[riddick18]

erstmal danke

ich hätte nicht gedacht das ich so schnell eine Antwort bekomme.

Angesichtsdessen das mein Lehrer auch nicht so viel weiß glaub ich .. können wir auch leider nicht viel, deswegen auch meine nächste Frage was ist Arrays.

Ich ignorier die Fehlermeldung nicht, aber ich weiß einfach bisher noch nichts damit anzufangen.

 

[Volvagia]

Eine mehrfeldrige Variable.

private String[] Text = 
    {"Durchschnittsnote 1", "Durchschnittsnote 2","Durchschnittsnote 3","Durchschnittsnote 4","Durchschnittsnote 4<"}; // Auswahlmöglichkeiten

 

[VfL_Freak]

Moin,

Eine mehrfeldrige Variable.

private String[] Text = 
    {"Durchschnittsnote 1", "Durchschnittsnote 2","Durchschnittsnote 3","Durchschnittsnote 4","Durchschnittsnote 4<"}; // Auswahlmöglichkeiten

zu einen dies, aber auch :

private JRadioButton[] Option = new JRadioButton[5]; // Wertebreich von 0 ... 4

// ...

for( int i = 0; i < 6; i++ )
{
    Option[i] = new JRadioButton( Text[i] );   // aber Zugriff von 0 ... 5
    // ...
}

Jetzt klarer ?

Gruß
Klaus

 

[riddick18]

Ahhh okay

und nochmal vielen Dank für die schnelle Hilfe

 

[Andi_CH]

Ich würde die Zahl durch eine Konstante ersetzen - du machst einerseits weniger Programmierfehler und andererseits wird dein Program änderungsfreundlicher.

final int size=5;

private JRadioButton[] Option = new JRadioButton[size];
 
// ...
 
for( int i = 0; i < size; i++ )
{
    Option[i] = new JRadioButton( Text[i] );
    // ...
}

 

[Murray]

Ich würde die Zahl durch eine Konstante ersetzen - du machst einerseits weniger Programmierfehler und andererseits wird dein Program änderungsfreundlicher.

Damit wird die Redundanz schon etwas reduziert - und damit sowohl eine Fehlerquelle eliminiert als auch die Wartbarkeit verbessert.
Trotzdem steckt da immer noch vermeidbare Redundanz, weil ja sowohl das Array der Buttons als auch das der Texte gleich lang sein müssen.

Deshalb würde ich das lieber so machen:

    private String[] text = {  // Auswahlmöglichkeiten
    		"Durchschnittsnote 1", 
    		"Durchschnittsnote 2",
    		"Durchschnittsnote 3",
    		"Durchschnittsnote 4",
    		"Durchschnittsnote 5"
    		};
    private JRadioButton[] option = new JRadioButton[text.length];

    /* ... */

    for( int i = 0; i < text.length; i++ ) {
        option[i] = new JRadioButton( text[i] );
    }
}

So passen die Längen der beiden Arrays automatisch zueinander. Wenn man eine neue Auswahl ergänzen will, dann muss man nur noch die Liste für die Initialisierung des text-Arrays erweitern.