[JAVA] Problema utilizzo hashmap

[LuLyLuNa]

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

Provo a scrivere tutto su file txt

    public void scrivi(ElencoStudenti e, int i) throws IOException{      
        FileWriter f = new FileWriter(doc, true);
        f.write((String)e.matricola+" - "+e.getStudente(i).getCognome()+" - "+e.getStudente(i).getNome()+" - "+e.getStudente(i).getAppello()+" - "+e.getStudente(i).getVoto()+"\r\n");  
        f.close();
    }

E questo è poi il main

    private void bottoneAggiungiActionPerformed(java.awt.event.ActionEvent evt) {                                                
        // TODO add your handling code here:
        
        e.matricola = matricola.getText();
        e.addStudente(cognome.getText(), nome.getText(), appello.getSelectedItem(), voto.getSelectedItem());        
        if (d.exists() != true) {
            d.newDoc();
            try {
                d.scrivi(e, i);
            } catch (IOException ex) {
                Logger.getLogger(mainFrame.class.getName()).log(Level.SEVERE, null, ex);
            }
        } else {            
                
            try {          
                d.scrivi(e, i);
            } catch (IOException ex) {
                Logger.getLogger(mainFrame.class.getName()).log(Level.SEVERE, null, ex);
            }
        }
       i++;
       matricola.setText("");
       cognome.setText("");
       nome.setText("");
       appello.setSelectedIndex(0);
       voto.setSelectedIndex(0);
    }

Provo ad avviarlo e ricevo questo errore:
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at studenti.Documento.scrivi(Documento.java:41)
at studenti.mainFrame.bottoneAggiungiActionPerformed( mainFrame.java:323)

Credo sia un problema dell'hashmap perché avevo scritto tutto con arraylist e funzionava.. Poi ho cambiato idea credendo fosse più corretto utilizzare l'hashmap e avere così la matricola come chiave

[schumy2000]

E questo è poi il main

    private void bottoneAggiungiActionPerformed(java.awt.event.ActionEvent evt) {                                                
        // TODO add your handling code here:
        
        e.matricola = matricola.getText();
        e.addStudente(cognome.getText(), nome.getText(), appello.getSelectedItem(), voto.getSelectedItem());        
        if (d.exists() != true) {
            d.newDoc();
            try {
                d.scrivi(e, i);
            } catch (IOException ex) {
                Logger.getLogger(mainFrame.class.getName()).log(Level.SEVERE, null, ex);
            }
        } else {            
                
            try {          
                d.scrivi(e, i);
            } catch (IOException ex) {
                Logger.getLogger(mainFrame.class.getName()).log(Level.SEVERE, null, ex);
            }
        }
       i++;
       matricola.setText("");
       cognome.setText("");
       nome.setText("");
       appello.setSelectedIndex(0);
       voto.setSelectedIndex(0);
    }

Questo non mi pare proprio il main...più che altro sono le azioni che vengono fatte quando clicchi sul bottone.

Hai inizializzato ElencoSutudenti nei passi precedenti?
Perché l'errore è qui

public void scrivi(ElencoStudenti e, int i) throws IOException{      
        FileWriter f = new FileWriter(doc, true);
    ==>    f.write((String)e.matricola+" - "+e.getStudente(i).getCognome()+" - "+e.getStudente(i).getNome()+" - "+e.getStudente(i).getAppello()+" - "+e.getStudente(i).getVoto()+"\r\n");  
        f.close();
    }

[LuLyLuNa]

Intendevo dire che è il bottone all'interno del main -.-

Cmq ho solo creato l'elenco nel main

public class mainFrame extends javax.swing.JFrame {
    ElencoStudenti e = new ElencoStudenti();

[schumy2000]

così col codice a pezzi è un po complicato capire l'errore...

[Alex'87]

Sei sicuro di cosa viene restituito da e.getStudente(i)?
Cosa fa quel metodo? Ti restituisce l'elemento che ha come chiave i. E' quello che vuoi?

La matricola gestita in quel modo fa un po' acqua... Dovresti passarla con lo studente (e dovrebbe essere un attributo di Studente...)

[LuLyLuNa]

Originariamente inviato da Alex'87
Sei sicuro di cosa viene restituito da e.getStudente(i)?
Cosa fa quel metodo? Ti restituisce l'elemento che ha come chiave i. E' quello che vuoi?

La matricola gestita in quel modo fa un po' acqua... Dovresti passarla con lo studente (e dovrebbe essere un attributo di Studente...)

ho pensato che gestire la matricola con lo studente non ha senso.. la matricola deve essere la chiave, quindi può essere la chiave dell'hashmap relativa all'oggetto

[Alex'87]

Originariamente inviato da LuLyLuNa
ho pensato che gestire la matricola con lo studente non ha senso.. la matricola deve essere la chiave, quindi può essere la chiave dell'hashmap relativa all'oggetto

La matricola è una proprietà dello Studente, ha perfettamente senso usarla come chiave... Altrimenti dove pensi di tenerla la matricola? In giro per il codice? Questo non ha senso...

Che c'è di male nel fare questo?

Studente studente = new Studente(....);

mappa_studenti.put(studente.getMatricola(), studente);

[LuLyLuNa]

Originariamente inviato da Alex'87
La matricola è una proprietà dello Studente, ha perfettamente senso usarla come chiave... Altrimenti dove pensi di tenerla la matricola? In giro per il codice? Questo non ha senso...

Che c'è di male nel fare questo?

Studente studente = new Studente(....);

mappa_studenti.put(studente.getMatricola(), studente);

Era quello che intendevo non ci siamo capiti!

[Alex'87]

Originariamente inviato da LuLyLuNa
Era quello che intendevo non ci siamo capiti!

Ah ok scusa, ho letto male il tuo messaggio