Dringend! NullPointerException

[scheibl]

Hab wida mal eine dumme frage. Bekomme immer eine NullPointerException. Will mit aus einer Liste mit Prüfungen (prueflist) die Häufigkeiten der einzelnen Gegenstände für eine Statistik herausholen. Bekomme dabei aber immer eine NullPointerException.

Danke im vorhinein!!!!

Hier der Code:

public void berechnung_gegenstände(List<Pruefung> prueflist)
    {
        List<Statistik> stat_list = new ArrayList();             //Liste mit allen Gegenstandsnr
        Statistik help = null;                                            //Hilfsvariable
        
        boolean check=false;
        
        
        for(int i=0; i<prueflist.size(); i++)
        {   
            for(int j=0; j<stat_list.size(); j++)
            {
                if(prueflist.get(i).getGstandnr() == stat_list.get(j).getGstandsnr() && (check == false))
                {
                    stat_list.get(j).setAnz(stat_list.get(j).getAnz()+1);
                    check=true;
                }
            }    
            if(check=false)
            {
                help.setAnz(0);
                help.setGstandsnr(prueflist.get(i).getGstandnr());
                stat_list.add(help);
            }    
            check = false;
        }

 

[SlaterB]

Zeile der Exception?
help ist null und nie was anderes?

 

[Niki]

Mann, Slater du bist schnell! Sitzt du den ganzen Tag vorm Browser und drückst Refresh

Ausserdem statt check=false gehören zwei =
Sag einfach

if(check==false)
            {
                help = new Statistik();
                help.setAnz(0);
                help.setGstandsnr(prueflist.get(i).getGstandnr());
                stat_list.add(help);
            }

 

[SlaterB]

ich poste die Probleme selber unter anderem Namen um als erster zu antworten

-----

help = new Statistik();
in der Schleife ist ja kein guter Vorschlag

-----

> if(check=false)

->

if(check==false)

oder besser

if(!check)

 

[MASTERmind]

ich poste die Probleme selber unter anderem Namen um als erster zu antworten

...lol
-----

help = new Statistik();
in der Schleife ist ja kein guter Vorschlag

...weil pro schleifendurchlauf ein neues objekt angelegt wird....
leg/instanziiere das objekt vorher einmal, wenn das für deinen kontext richtig ist?!
-----

> if(check=false)

->

if(check==false)

oder besser

if(!check)

weil check = false...zuweisung und nicht überprüfung!!!