[JAVA] Modificare elemento ArrayList

[LuLyLuNa]

Effettuo la ricerca di un elemento nell'arraylist.
Una volta trovato, modifico un dato dall'interfaccia utente.
Cliccando su aggiorna non avviene alcuna modifica.

    private void bottoneAggiornaActionPerformed(java.awt.event.ActionEvent evt) {                                                
        // TODO add your handling code here:
        
        e.getStudente(j).setMatricola(matricola.getText());
        e.getStudente(j).setNome(nome.getText());
        e.getStudente(j).setCognome(cognome.getText());
        e.getStudente(j).setAppello(String.valueOf(appello.getSelectedItem()));
        e.getStudente(j).setVoto(Integer.valueOf((String) voto.getSelectedItem()));
        
        matricola.setText("");
        cognome.setText("");
        nome.setText("");
        appello.setSelectedIndex(0);
        voto.setSelectedIndex(0);
        found = false;
    }

[cigiri18]

sei sicuro il metodo venga effettivamente invocato?
quanto vale j?

[LuLyLuNa]

Originariamente inviato da cigiri18
sei sicuro il metodo venga effettivamente invocato?
quanto vale j?

    private void bottoneModificaActionPerformed(java.awt.event.ActionEvent evt) {                                                
        // TODO add your handling code here:     
        String search = JOptionPane.showInputDialog(this, "Inserisci la matricola da cercare", "RICERCA", JOptionPane.DEFAULT_OPTION);
        for (i = 0; i < e.size(); i++)
            if(search.equals(e.getStudente(i).getMatricola())) {
                formAggiungi.setVisible(true);
                formAggiungi.pack();
                formAggiungi.setLocationRelativeTo(this);
                
                bottoneAggiungi.setEnabled(false);
                bottoneAggiorna.setEnabled(true);   
                found = true;
                j = i;
                matricola.setText(e.getStudente(j).getMatricola());
                cognome.setText(e.getStudente(j).getCognome());
                nome.setText(e.getStudente(j).getNome());
                appello.setSelectedItem(e.getStudente(j).getAppello());
                voto.setSelectedItem(e.getStudente(j).getVoto());
            }
        
        if (found == false)
            JOptionPane.showMessageDialog(null, "Matricola studente non trovata!", "Errore", JOptionPane.ERROR_MESSAGE);
    }

fino a questo momento avevo sempre usato i.. ho pensato di usare un'altra variabile (in questo caso j) ed uguagliarla a i in modo che non si verificasse una eventuale modifica in altri frammenti di codice

[cigiri18]

quando vengono invocati i due metodi?
il problema credo sia in bottoneModificaActionPerformed, quando tu, dentro il ciclo, fai j=i. In questo modo ad ogni ciclo imposti un nuovo valore su j e, alla fine del ciclo, avrai j=e.size()-1, che non so quanto possa valere, ma sono praticamente certo non sia il valore che vorresti.

una soluzione può essere il lasciar perdere il salvataggio di una variabile per conoscere la posizione che vuoi modificare(aggiungi che j, come nome di indice di un array, fa schifo, meglio id), e scorrere invece l'array quando devi aggiornarlo, trovando la posizione(matricola=matricola), e quindi modificando.

[LuLyLuNa]

L'istruzione j = i è dentro un if quindi avviene solo quando si verifica la condizione di uguaglianza tra stringhe...

[cigiri18]

giusto, avevo letto male.
prova a fare qualche stampa, quando viene invocato il metodo di modifica bottoneAggiornaActionPerformed, dello stato delle variabili.

[LuLyLuNa]

Lo stato delle variabili è giusto.. se voglio modificare il secondo studente, i e j valgono rispettivamente 1 quindi è corretto.
ciò che mi sembra di capire è che è come se non "vedesse" i vari metodi set per reimpostare il valore dei campi... è proprio quello che non funziona

[Alex'87]

Originariamente inviato da LuLyLuNa

    private void bottoneModificaActionPerformed(java.awt.event.ActionEvent evt) {                                                
        // TODO add your handling code here:     
        String search = JOptionPane.showInputDialog(this, "Inserisci la matricola da cercare", "RICERCA", JOptionPane.DEFAULT_OPTION);
        for (i = 0; i < e.size(); i++)
            if(search.equals(e.getStudente(i).getMatricola())) {
                formAggiungi.setVisible(true);
                formAggiungi.pack();
                formAggiungi.setLocationRelativeTo(this);
                
                bottoneAggiungi.setEnabled(false);
                bottoneAggiorna.setEnabled(true);   
                found = true;
                j = i;
                matricola.setText(e.getStudente(j).getMatricola());
                cognome.setText(e.getStudente(j).getCognome());
                nome.setText(e.getStudente(j).getNome());
                appello.setSelectedItem(e.getStudente(j).getAppello());
                voto.setSelectedItem(e.getStudente(j).getVoto());
            }
        
        if (found == false)
            JOptionPane.showMessageDialog(null, "Matricola studente non trovata!", "Errore", JOptionPane.ERROR_MESSAGE);
    }

fino a questo momento avevo sempre usato i.. ho pensato di usare un'altra variabile (in questo caso j) ed uguagliarla a i in modo che non si verificasse una eventuale modifica in altri frammenti di codice

Qualche suggerimento.

1) Non devi preoccuparti di modificare i per sbaglio perché quella i non deve esistere fuori da quel ciclo! E' una variabile che nasce nel ciclo e muore con esso. Ed è per questo motivo che tale variabile andrebbe dichiarata col ciclo:

for (int i = 0; i < e.size(); i++) {
    ...
}

2) Tutte quelle chiamate a getStudente sono brutte, inutili e appesantiscono la lettura del codice.

Studente studente = e.getStudente(i);
studente.getMatricola();
studente.getBlablabla();

3) Non stavi usando una mappa per memorizzare gli studenti? In quel caso non serve "iterare": l'accesso agli elementi di una mappa è tempo costante (O(1)), basta dargli la chiave...

4) if (found == false) { ... } non si può vedere: found è già un booleano, basta
if (!found) { ... }

[Alex'87]

Originariamente inviato da LuLyLuNa
Lo stato delle variabili è giusto.. se voglio modificare il secondo studente, i e j valgono rispettivamente 1 quindi è corretto.
ciò che mi sembra di capire è che è come se non "vedesse" i vari metodi set per reimpostare il valore dei campi... è proprio quello che non funziona

I campi dove sono dichiarati? Sicuro/a che siano proprio le istanze che ti aspetti?

[LuLyLuNa]

Originariamente inviato da Alex'87
Qualche suggerimento.

1) Non devi preoccuparti di modificare i per sbaglio perché quella i non deve esistere fuori da quel ciclo! E' una variabile che nasce nel ciclo e muore con esso. Ed è per questo motivo che tale variabile andrebbe dichiarata col ciclo:

for (int i = 0; i < e.size(); i++) {
    ...
}

2) Tutte quelle chiamate a getStudente sono brutte, inutili e appesantiscono la lettura del codice.

Studente studente = e.getStudente(i);
studente.getMatricola();
studente.getBlablabla();

3) Non stavi usando una mappa per memorizzare gli studenti? In quel caso non serve "iterare": l'accesso agli elementi di una mappa è tempo costante (O(1)), basta dargli la chiave...

4) if (found == false) { ... } non si può vedere: found è già un booleano, basta
if (!found) { ... }

1. Ho dichiarato i globale in modo da utilizzarla in qualsiasi parte del codice.

2. Questa è la mia classe ElencoStudenti ecco perché faccio tutte quelle chiamate. Concordo sul fatto che siano orribili. Posso creare un oggetto di tipo studente nel main e funziona per tutto???

public class ElencoStudenti extends ArrayList<Studente> {
    
    // metodo per l'inserimento degli studenti
    public void addStudente(String matricola, String cognome, String nome, Object appello, Object voto) {
        Studente st = new Studente(matricola, cognome, nome, appello, voto);
        this.add(st);
    }
    
    // metodo per la selezione di un singolo studente
    public Studente getStudente(int i){
        return this.get(i);
    }
     
}

3. Non sono riuscita ad utilizzare le hashmap quindi per finire il progetto in breve tempo ho lasciato la classe estesa ad arraylist. Sono consapevole che una hashmap sarebbe più adatta

4. Sorry sto fondendo!!!!!