Erste Schritte NullPointerException

[A_du]

Hallo Zusammen,
Ich habe den Auftrag einen Code zu schreiben der die Personalnummern der Mitarbeiter in einem Array vergleicht (Bei gleicher Pnr soll true ausgegeben werden, ansonsten false). Das Programm läuft einwandfrei wenn ich die ganze ArrayListe ausgefüllt ist, sobald ich jedoch nicht den ganzen Array ausgefüllt habe wirft das Programm eine NullPointerException.

Wie kann ich das vermeiden?

 public static boolean hasDupl(Mitarbeiter[] mArr){
		
		for (int j=0; j < mArr.length; j++){
			for(int a = 1; a < mArr.length; a++){

			if (mArr[j].getPnr() == mArr[a].getPnr() && mArr[j] != mArr[a] ){
				return true;
			}
			}
		}
		return false;
}

 

[Klösp]

indem auf null abfragst.

if(mArr[i]!=null){
//mach was
}

Wenn du mehrere Bedingungen wie bei dir in der if-Abfrage hast, sollte die Abfrage auf null ganz links stehen, weil die Bedingungen von links nach rechts geprüft werden (bis das Ergebnis des gesamten Ausdrucks eindeutig ist).

 

[A_du]

Vielen Dank für den Tipp. Wie meinst du ganz links stehen?
Hab da nochmals den Code, aber ich habe wohl die NULL Abfrage am falschen Ort

public static boolean hasDupl(Mitarbeiter[] mArr){
		
		for (int j=0; j < mArr.length; j++){
			for(int a = 1; a < mArr.length; a++){
				
			if(mArr[j]!=null){
				
				if (mArr[j].getPnr() == mArr[a].getPnr() && mArr[j] != mArr[a] ){
				return true;
			}
			}
			
			}
		}
		return false;	
	}

 

[Klösp]

Du brauchst ja nicht unbedingt einen neuen If-Block.

    public static boolean hasDupl(Mitarbeiter[] mArr){
           
            for (int j=0; j < mArr.length; j++){
                for(int a = 1; a < mArr.length; a++){

                    if (mArr[j]!=null && mArr[j].getPnr() == mArr[a].getPnr() && mArr[j] != mArr[a] ){
                    return true;
                
                }
               
                }
            }
            return false;  
        }

Mit ganz links-stehen meinte ist folgendes.
So solltest du es nicht machen.

 if (mArr[j].getPnr() == mArr[a].getPnr() && mArr[j] != mArr[a] && mArr[j]!=null )

Denn wie gesagt werden einzelnen mit && oder || verknüpften Bedingungen solange von links nach rechts geprüft, bis das Ergebnis des Gesamtausdrucks feststeht.
Wenn du aber nicht als ersten auf null abfragst bekommst du deshalb immer noch eine NullPointerException.

€dit: falls du im Moment immer noch eine Exception bekommst, dann könnte das daran liegen, dass du bisher nur mArr[j] überprüfst.

 

[A_du]

Ok. Danke das macht Sinn .
Kann es aber sein, dass an einer anderer Stelle im Programm eine NullPointerException gemacht wird?
Denn die Ausgabe war unverändert.

public class MitarbeiterTest {

	public static void main(String[] args) {
		// TODO Auto-generated method stub
		Mitarbeiter mArr[] = new Mitarbeiter[10];
		   mArr[0] = new Mitarbeiter(7, "Hans", 'm');
		   mArr[1] = new Mitarbeiter(2, "Anna", 'w');
		   mArr[2] = new Mitarbeiter(6, "Erik", 'm');
		   mArr[3] = new Mitarbeiter(3, "Fritz", 'm');
		   mArr[4] = new Mitarbeiter(1, "Hansi", 'm');
		   
		   
		 String m = searchPersNr( mArr, 1); //suche Ma mit persNr==3
		 System.out.println( m );
		 Boolean doppelt = hasDupl(mArr);
		 System.out.println(doppelt);
		 int anzahlMaenner = countM(mArr);
		 System.out.println("Maenner: "+anzahlMaenner);
		 int anzahlFrauen = countF(mArr);
		 System.out.println("Frauen: "+anzahlFrauen);
	}

	public static String searchPersNr(Mitarbeiter[] mArr, int pnr) {
		
		for (int j = 0; j < mArr.length ; j++){
			
			if ((mArr[j]!=null) && (mArr[j].getPnr()== pnr)){
				return mArr[j].getName();
				}
			}
		return null;
		}
	
	public static boolean hasDupl(Mitarbeiter[] mArr){
		
		for (int j=0; j < mArr.length; j++){
			for(int a = 1; a < mArr.length; a++){
				
			
				if ((mArr[j]!=null) && (mArr[j].getPnr() == mArr[a].getPnr() && mArr[j] != mArr[a] )){
				return true;
			}
			
			}
		}
		return false;	
	}
	public static int countM(Mitarbeiter[] mArr){
		int counterM = 0;
		for (int j = 0; j <mArr.length; j++){
			
			if((mArr[j]!=null) && (mArr[j].getSex()=='m')){
				counterM++;
			}
				
		}
		
		return counterM;
		}
	public static int countF(Mitarbeiter[] mArr){
		int counterF = 0;
		for (int j = 0; j <mArr.length; j++){
			
			if((mArr[j]!=null) && (mArr[j].getSex()=='w')){
				counterF ++;
			}
				
		}
		
		return counterF;
		}
	
	}

 

[A_du]

Habe den letzten Satz erst jetzt gesehen, das war das Problem
Vielen Dank für deine Hilfe

 

[njans]

 public static boolean hasDupl(Mitarbeiter[] mArr){
       
        for (int j=0; j < mArr.length; j++){
            for(int a = 1; a < mArr.length; a++){
                if ((mArr[j]!=null) && (mArr[j].getPnr() == mArr[a].getPnr() && mArr[j] != mArr[a] ))
                  return true;
            }
        }
        return false;  
    }

du prüfst ob (mArr[j]!=null) aber ob (mArr[a]!=null) ist, prüfst du nicht.
Generell wäre es besser eine Liste zu verwenden damit du nicht ständig auf null prüfen müsstest.