Objekt mit Hashmap vergleichen

[Mert1296]

Hi,
hab da ne kleine Frage.
Undzwar will ich einen Schlüssel von meiner Hashmap mit einem Integer Objekt vegleichen, um das Objekt daraufhin in eine Arraylist zu stecken..
Nur weiß ich leider nicht wie das geht und hab auch überall gesucht und finde nichts..

Map <Integer, Praxis> praxisMap = new HashMap<>();
    
        public void pranlegen() {
            Praxis praxis = new Praxis();

        
            System.out.println("Name der Praxis: ");
            praxis.name = sc.next();
        
            System.out.println("Nummer der Praxis: ");
            praxis.praxisnr = sc.nextInt();
        
            System.out.println("Gründung der Praxis: ");
            praxis.gruendung = sc.next();
        
            praxisMap.put(praxis.praxisnr, praxis);
        
        
        }
    
    
        public void anlegen() {
        
        
            System.out.println("Patientennummer: ");
            patient.pnr = sc.nextInt();
          
            System.out.println("Name des Patienten: ");
            patient.name = sc.next();
        
            System.out.println("Erstbesuch: ");
            patient.erstbesuch = sc.next();
        
            System.out.println("Praxis?");
            patient.praxis.praxisnr = sc.nextInt();   //Hier löst die Exception aus
        
            praxisMap.get(praxis.praxisnr);
            praxis.patient.add(patient);
        
        }

Ich hab in meiner Methode pranlegen() die Praxisnr in meine Hashmap eingefügt und in der anlegen() Methode lege ich einen Patienten an und will ihn einer Praxis zuweisen. Nur wenn ich das Programm laufen lasse, eine Praxis anlege und daraufhin einen Patienten, löst es bei mir eine Exception aus bei der mich die Konsole nach der PraxisNr fragt, aber ich habe keine Ahnung wieso. Ich denke mal, dass ich die patient.praxis.praxisnr erst mit meiner Hashmap vergleichen muss aber hab echt keine Ahnung

Danke schon mal für die Antworten!!

 

[Robat]

Um was für eine Exception handelt es sich denn?

 

[Mert1296]

Um was für eine Exception handelt es sich denn?

NullPointer Exception

 

[Robat]

Wie/Wo wird denn patient bzw dessen praxis initialisiert?

 

[fhoffmann]

Wahrscheinlich hat dein Patient noch keine Praxis.

 

[Mert1296]

Wie/Wo wird denn patient bzw dessen praxis initialisiert?

Also ich habe eine Klasse Praxis und eine Klasse Patient.
In der Klasse Patient habe ich noch zusätzlich die Klasse Praxis mit

Praxis praxis;

aufgerufen.
In der Klasse Praxis hab ich noch eine ArrayList

ArrayList<Patient> patienten = new ArrayList <Patient>();

.
In meiner Main Klasse erstelle ich von Praxis und Patient eine Objekt (Praxis praxis = new Praxis(); ) und zusätzlich eine HashMap praxisMap, und habe dann jeweils eine Methode bei der ich diese anlege.

 

[Robat]

Aber wo weist du dem Patienten die Praxis zu?

 

[Mert1296]

Aber wo weist du dem Patienten die Praxis zu?

Meinst du sowas wie praxis.praxisnr = patient.praxisnr; ?

 

[Robat]

In der Klasse Patient habe ich noch zusätzlich die Klasse Praxis mit

Du sagst selbst, dass jeder Patient ein Praxis-Objekt besitzt - erklärst aber nirgends wo/wie dieses Objekt initialisiert wird.
Irgendwo musst du doch die Praxis zuweisen.

 

[Mert1296]

Du sagst selbst, dass jeder Patient ein Praxis-Objekt besitzt - erklärst aber nirgends wo/wie dieses Objekt initialisiert wird.
Irgendwo musst du doch die Praxis zuweisen.

Ich steh grad voll auf dem Schlauch

 

[Robat]

public class Patient {
     private Praxis praxis; // das ist bei dir momentan immer NULL
     public Patient(Praxis praxis) { // du müsstest schon (bspw) die Praxis im Konstruktor übergeben
          this.praxis = praxis;  // und damit praxis initialisieren, damit es nicht mehr NULL ist
     }
}

// Anwenden kannst du es dann so
public class Test {
    public static void main(String[] args) {
          Praxis p = new Praxis(..);
          Patient patient = new Praxis(p); // ab dem Zeitpunkt ist patient.praxis nicht mehr null!
    }
}

Das ganze ist nur ein Beispiel um dir das Problem zu verdeutlichen. Kann gut sein dass du es für eine Klassenstruktur etwas abändern musst.

 

[Mert1296]

Das hab ich schon so gemacht:

public class Patient {
    int pnr;
   
    String name;
    String erstbesuch;
    Praxis praxis;
   
    public Patient() {
       
    }
   
    public Patient(int pnr, String name, String beginn, Praxis praxis) {
        this.pnr = pnr;
        this.name = name;
        erstbesuch = beginn;
        this.praxis = praxis;
       
    }
   
   
}

 

[stg]

Lies dir doch das hier einmal durch und beachte das bei deinen zukünftigen Postings:
https://stackoverflow.com/help/mcve

löst es bei mir eine Exception aus

und bei Exeptions zusätzlich immer den gesamten StackTrace posten.

 

[Robat]

Und welchen der beiden Konstruktoren rufst du auf?

 

[Mert1296]

Und welchen der beiden Konstruktoren rufst du auf?

Ich weiß es ehrlich gesagt nicht der default-Konstruktor kann es glaube ich nicht sein, da eclipse in meiner Main-Klasse die Objekte erkennt.

 

[Robat]

Am Besten postest du mal deinen kompletten Code sonst raten wir noch eine Weile rum

 

[Mert1296]

Okay, also das ist meine Klasse Patient:

public class Patient {
    int pnr;
    String name;
    String erstbesuch;
    Praxis praxis;
    public Patient() {
  
    }
    public Patient(int pnr, String name, String beginn, Praxis praxis) {
        this.pnr = pnr;
        this.name = name;
        erstbesuch = beginn;
        this.praxis = praxis;
  
    }
}

Hier meine Klasse Praxis:

public class Praxis {


    String name;

    int praxisnr;

    String gruendung;

    ArrayList <Patient> patienten = new ArrayList <Patient>();


    public Praxis() {

  

    }


    public Praxis(String name, int praxisnr, String gruendung, ArrayList<Patient> patient) {

        this.name = name;

        this.praxisnr = praxisnr;

        this.gruendung = gruendung;

        this.patienten = patient;

    }


}

Und meine Klasse Main:

public class Main {

    SimpleDateFormat format = new SimpleDateFormat("dd-MM-yyyy");

    boolean run = true;

    Scanner sc = new Scanner(System.in);

    int zaehler;

    Patient patient = new Patient();

    Praxis praxis = new Praxis();


    public static void main(String [] args) throws Exception{

        Main main = new Main ();

        main.menue();

  

    }


    public void menue() {

  

        while (run) {

  

        System.out.println("Hauptmenü ");

        System.out.println("---------");

        System.out.println("Neue Praxis anlegen         < 1");

        System.out.println("Neuen Patient anlegen          < 2");
        int answer = 0;

        answer = sc.nextInt();

  

        switch(answer) {

        case 1: pranlegen(); break;

        case 2: anlegen(); break;

            run = false;
            break;

      

        }

  

    }

        sc.close();

  

    }

        Map <Integer, Praxis> praxisMap = new HashMap<>();

  

        public void pranlegen() {



            System.out.println("Name der Praxis: ");

            praxis.name = sc.next();

      

            System.out.println("Nummer der Praxis: ");

            praxis.praxisnr = sc.nextInt();

      

            System.out.println("Gründung der Praxis: ");

            praxis.gruendung = sc.next();

      

            praxisMap.put(praxis.praxisnr, praxis);

      

      

        }

  

  

  

        

        public void anlegen() {

      

      

            System.out.println("Patientennummer: ");

            patient.pnr = sc.nextInt();

      

            System.out.println("Praxis?");

            patient.praxis.praxisnr = sc.nextInt();

      

            System.out.println("Name des Patienten: ");

            patient.name = sc.next();

      

            System.out.println("Erstbesuch: ");

            patient.erstbesuch = sc.next();
        }

 

[Robat]

Also in deiner Main-Klasse erstellst du an folgender Stelle einen neuen Patienten mit dem default-Konstruktor. Das bedeutet, dass das Attribut praxis deines Patientens null bleibt.

Patient patient new Patient();

Jetzt wird die Methode anlegen() aufgerufen und du versuchst in der Zeile

patient.praxisnr =  sc.nextInt();

auf die Praxisnummer des Praxisobjektes deines Patienten zuzugreifen. Wie oben aber bereits geschildert, ist praxis null und dadurch fliegt die Exception.

Was du also machen müsstest ist vorher das Praxis-Objekt zu initialisieren. Ob du das schon beim erstellen des Patienten machst (indem du bspw. ein dummy-Praxis-Objekt übergibst dessen Werte du später ändern kannst) oder erst in der Methode anlegen() eine neue Praxis erstellst ist relativ egal. Wichtig ist, dass dies geschieht bevor du über den Patienten auf die Praxis zugreifen willst.

 

[stg]

Du hast echt Glück, das Robat dich nicht einfach genauso ignorierst, wie du es mit dem Inhalt meines Postings #13 getan hast

 

[Mert1296]

Was du also machen müsstest ist vorher das Praxis-Objekt zu initialisieren. Ob du das schon beim erstellen des Patienten machst (indem du bspw. ein dummy-Praxis-Objekt übergibst dessen Werte du später ändern kannst) oder erst in der Methode anlegen() eine neue Praxis erstellst ist relativ egal. Wichtig ist, dass dies geschieht bevor du über den Patienten auf die Praxis zugreifen willst.

könntest du mir da ein kleines Code-Beispiel zeigen? Ich verstehe zwar was du meinst aber weiß nicht was ich machen soll.. wie schon gesagt bin frischer Anfänger.. Wäre echt nett!

 

[Mert1296]

Du hast echt Glück, das Robat dich nicht einfach genauso ignorierst, wie du es mit dem Inhalt meines Postings #13 getan hast

Oh, sry das ist komplett untergegangen hab das nicht gesehen..

 

[Mert1296]

und bei Exeptions zusätzlich immer den gesamten StackTrace posten.

Exception in thread "main" java.lang.NullPointerException

at Main.anlegen(Main.java:83)

at Main.menue(Main.java:39)

at Main.main(Main.java:15)

 

[Robat]

Deine anlegen() Methode sollte von der Logik her in etwa so aufgebaut sein...

public void anlegen() {
      // Patientennummer, Praxisnummer, Patientenname, Erstbesuch abfragen
      int patNr = ....
      int praxNr = ...
      String patName = ....
      String erstbesuch = ...

      // aus deiner Map das Praxis-Objekt mit der korrespondierenden Praxisnummer suchen
      Praxis praxis = ..

      // Patienten erstellen indem du alle Informationen dem Konstruktor mitgibst  Patient(patNr, patName, erstbesuch, praxis)    
      patient = ...
}