Problem mit Gui.

[Java_Gui_Fan]

Hi,
ich habe ein Problem, und zwar bekomm ich eine IndexOutOfBoundsException. Ich muss in eine Combobox alle Benutzer ausgeben können, oder nur einen bestimmten Teil. Die Auswahl, welche angezeigt werden erfolgt über RadioButtons.

Hinweis:
-------

e <---- Objekt der Klasse EmployeeManager. e ist eine Liste, von Employees (und bei den Employees gibts wieder eine Teilung für die Speziellen). Und je nachdem, was der User auswählt, will ich nur bestimmte Employees von e oder alle anzeigen.

Hier der Code von der Klasse EmployeeManager.java (Wo die Exception kommt).

public String toStringVorname(int i){
        return String.format(employeeList.get(i).getFirstName());
    }

public String toStringManager(int i){
        if(employeeList.get(i) instanceof ContractEmployee){
            ContractEmployee e = new ContractEmployee(employeeList.get(i).getFirstName(), employeeList.get(i).getLastName(), employeeList.get(i).calculateHourlyRate());
            return String.format("Contract Employee: " + e.toStringFormatContractGui().substring(1));
        }
        else if(employeeList.get(i) instanceof PermanentEmployee) {
            PermanentEmployee e = new PermanentEmployee(employeeList.get(i).getFirstName(), employeeList.get(i).getLastName(), employeeList.get(i).calculateHourlyRate());
            return String.format("Permanent Employee: " + e.toStringFormatPermanentGui().substring(1));
        }
        else if (employeeList.get(i) instanceof TemporaryEmployee) {
                TemporaryEmployee e = new TemporaryEmployee(employeeList.get(i).getFirstName(), employeeList.get(i).getLastName(), employeeList.get(i).calculateHourlyRate());
                return String.format("Temporary Employee: " + e.toStringFormatTemporaryGui().substring(1));
        }
        else{
            return String.format("Keine Employees gespeichert");
        }
    }

und hier der Code von der Klasse Main.java (GUI):

private void jRadioButtonAlleActionPerformed(java.awt.event.ActionEvent evt) {                                                 
        if (jRadioButtonAlle.isSelected()) {
            String[] str = null;
            for (int i = 0; i < e.size(); i++) {
                String s = e.toStringManager(i);
                if (s.equals("Keine Employees gespeichert")) {
                    JOptionPane.showMessageDialog(this, "Es sind keine Employees gespeichert!", "Error", JOptionPane.ERROR_MESSAGE);
                    break;
                } else {
                    jComboBoxAuswahl = new JComboBox();
                    jComboBoxAuswahl.addItem((Object) s);
                }
            }
        }
    }

So, hoff ihr könnt mir helfen. =)

Danke schonmal im Voraus. *gg*

 

[nrg]

schwer zu sagen. vorallem ohne stacktrace. also an den gettern der liste liegt es vermutlich nicht, weil diese ja in abhängigkeit von dem zähler i sind. sicher, dass es keine StringIndexOutOfBoundsException ist? weil das könnte ich mir durch das substring(1) schon eher vorstellen. mal davon abgesehen würd ich dir raten nicht 1000x das gleiche objekt aus der liste zu holen. ist viel performanter das nur einmal zu tun und dann mit einer referenzvariable weiterzuarbeiten.

 

[Antoras]

Die Exception bedeutet, dass du auf ein Element zugreifst, das nicht existiert. Fliegt die Exception schon ganz am Anfang oder erst am Ende der Schleife? Was gibt denn size() von der Klasse EmployeeManager zurück?

Sonst noch ein paar Tipps:

Das:

        if(employeeList.get(i) instanceof ContractEmployee){
            ContractEmployee e = new ContractEmployee(employeeList.get(i).getFirstName(), employeeList.get(i).getLastName(), employeeList.get(i).calculateHourlyRate());
            return String.format("Contract Employee: " + e.toStringFormatContractGui().substring(1));
        }

kann man auch so schreiben:

 Employee em = employeeList.get(i);
        if(em instanceof ContractEmployee){
            ContractEmployee e = new ContractEmployee(em.getFirstName(), em.getLastName(), em.calculateHourlyRate());
            return String.format("Contract Employee: " + e.toStringFormatContractGui().substring(1));
        }

Hier würde ich eine Konstante erstellen:

if (s.equals("Keine Employees gespeichert")) {
-->
static final String NO_EMPLOYEES = "Keine Employees gespeichert";
if (s.equals(NO_EMPLOYEES)) {

Das verhindert Tippfehler.

Willst du hier wirklich jedes Mal eine neue ComboBox erstellen, oder willst du die alte nicht viel lieber nur leeren und dann neue Objekte hinzufügen?

                } else {
                    jComboBoxAuswahl = new JComboBox();
                    jComboBoxAuswahl.addItem((Object) s);
                }

 

[Java_Gui_Fan]

Es ist egal, ob ich eine neue Combobox erstellte, oder mit .removeAllItems() lösche. Es bleibt bei der Exception... =( es ist ganz sicher keine StringIndexOut...Exception.

 

[nrg]

ok. dann poste doch mal den stacktrace

 

[Michael...]

        if(employeeList.get(i) instanceof ContractEmployee){
            ContractEmployee e = new ContractEmployee(employeeList.get(i).getFirstName(), employeeList.get(i).getLastName(), employeeList.get(i).calculateHourlyRate());
            return String.format("Contract Employee: " + e.toStringFormatContractGui().substring(1));

Ist es hier Absicht, dass Du immer ein neues Objekt von ContractEmployee erstellst (genauso bei Permanent und Temporary...)?
Es würde vermutlich ein einfacher class cast ausreichen, in etwa so:

        if(employeeList.get(i) instanceof ContractEmployee){
            ContractEmployee e = (ContractEmployee) employeeList.get(i);
            return String.format("Contract Employee: " + e.toStringFormatContractGui().substring(1));

Grundsätzlich sollte man sich das Konzept noch mal anschauen. Eventuell sollte es in der Superklasse Employee eine Methode toStringFormatGui() geben, die von den einzelnen Subklassen nach deren Bedürfnisse überschrieben wird, dann muss man in obigen Code nicht nach den genauen Ausprägungen unterscheiden.