Normal
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:[code=Java] 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] ); }}[/code]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.
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:
[code=Java]
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] );
}
[/code]
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.
