Kleine Frage zu OOP

[jono]

Danke @temi ,somit weiß ich für das nächste Mal Bescheid wie ich bei einem ähnlichen Problem eventuell vorgehen kann. Ja es ist schwer zu sagen ab wann man lernerfolg sehen sollte abgesehen kann man so sehen dass ich es können müsste nach den Hilfeanweisungen oder nicht, vielleicht wird es besser und da bin ich ziemlich optimistisch weil ich mir wirklich mühe gebe. Will euch ja gar nicht in dem Sinne schlecht beurteilen das hat damit ja gar nichts zu tun, das ist ja denke ich auch klar.
Eine Frage hätte ich da noch zu der vorher beschriebenen Fehlermeldung, warum habe ich da eine nullpointerexception?

 

[temi]

Eine Frage hätte ich da noch zu der vorher beschriebenen Fehlermeldung, warum habe ich da eine nullpointerexception?

Kannst du dir vorstellen, was die Meldung bedeutet?

Schau dir mal diese beiden Zeilen an:

anzahlWohneinheiten = bewohner.length;
this.bewohner = new Familie[anzahlWohneinheiten];

Edit: Außerdem sind noch mehr Fehler in dieser Methode im Konstruktor, sowie in der Methode anmelden().

 

[Xknight]

Was für Fehler denn??? Ich sehe da keine Fehler. Auch in der Methode anmelden sind keine Fehler.

 

[temi]

Was für Fehler denn??? Ich sehe da keine Fehler. Auch in der Methode anmelden sind keine Fehler.

Ja ja

 

[Xknight]

Wo sind denn Fehler und was würdest du besser machen?

 

[1337Avdu]

Ich sehe da auch keine Fehler, bis auf die Nullpointer ^^

 

[temi]

Witzig! Ich nehme mal an es handelt sich um Kommilitonen...

Haus(String adresse,  Person besitzer, int anzahlWohneinheiten){
    this.adresse = adresse;
    this.besitzer = besitzer;
     
    anzahlWohneinheiten = bewohner.length; // Welcher Variablen wird hier etwas zugewiesen?
    this.bewohner = new Familie[anzahlWohneinheiten];
     
    if (anzahlWohneinheiten < 1) { // Was soll das bezwecken?
        anzahlWohneinheiten = 1;
    }
}

 

[Xknight]

Laut der Aufgabe bestimmt die azahlWohneinheiten die Länge des Arrays und wenn die anzahlWohneinheiten kleiner eins ist soll anzahlWohneinheiten eins sein.
Das soll damit gemeint sein. Darum sehe ich im Konstruktor keine Fehler.

 

[1337Avdu]

Wir sind Schulkameraden und am verzweifeln haben Informatik als Grundkurs aber die Aufgabe hats schon in sich ^^

 

[temi]

Laut der Aufgabe bestimmt die azahlWohneinheiten die Länge des Arrays und wenn die anzahlWohneinheiten kleiner eins ist soll anzahlWohneinheiten eins sein.
Das soll damit gemeint sein. Darum sehe ich im Konstruktor keine Fehler.

Das ist mir schon klar, aber was soll die Abfrage nachdem das Array bereits erzeugt wurde?

 

[Xknight]

Welche Abfrage meinst du denn?

 

[temi]

Heiligs Blechle!

if (anzahlWohneinheiten < 1)

 

[Xknight]

In der Methode anmelden() sollen wir in einer freien Stellen einen Bewohner hinzufügen.
So lautet mein Quellcode.
Da sehe ich keinen Fehler.

public boolean anmelden(Familie someFam) {
         for(int i = 0; i < bewohner.length; i++) {
                if(bewohner[i] == null) {
                    bewohner[anzahlWohneinheiten++] = someFam;
                }
         }
      
         for (int i = 0; i < bewohner.length-1; ++i)
              for (int j = i+1; j < bewohner.length; ++j)
                if (bewohner[i] == bewohner[j]) {
                    return false;
                }
        return true;
    }

Mit dieser Abfrage überprüfe ich ob die anzahlWohneinheiten kleiner 1 ist und mit den for-Schleifen überprüfe ich ob Redundanzen im Array vorhanden sind.

 

[temi]

Da sehe ich keinen Fehler.

Doch. Ist fehlerhaft.

Warum will hier jeder das Feld anzahlWohneinheiten hochzählen? Die Anzahl der Wohneinheiten wird im Konstruktor festgelegt und ändert sich nicht mehr. Das hier Werte über Setter-Methoden geändert werden können ist gequirlte Scheiße, aber da könnt ihr nichts dafür, dass will eure Aufgabe so. Aber soweit ich die Aufgabe für anmelden() richtig im Kopf habe, muss hier die Anzahl der Wohneinheiten dieses Hauses nicht geändert werden. Eine freie Wohneinheit soll gesucht und durch eine Familie belegt werden, sofern diese Familie nicht schon im Haus wohnt.

überprüfe ich ob Redundanzen im Array vorhanden sind.

Sollte man das nicht vor dem Einfügen der neuen Familie machen?

 

[Xknight]

Da hast du recht mit den Rudandanzen, aber bei einer frei leer markierten Stelle soll ein Bewohner hinzugefügt werden und dass habe ich mit diesem code bewohner[anzahlWohneinheiten++] gemacht oder wie siehst du es?

 

[Xknight]

So sieht meine neue Methode aus

public boolean anmelden(Familie someFam) {
         for (int i = 0; i < bewohner.length-1; ++i)
              for (int j = i+1; j < bewohner.length; ++j)
                if (bewohner[i] == bewohner[j]) {
                    return false;
                }else  if(bewohner[i] == null) { 
                    bewohner[anzahlWohneinheiten++] = someFam; 
                }
        return true;
    }

 

[temi]

Da hast du recht mit den Rudandanzen, aber bei einer frei leer markierten Stelle soll ein Bewohner hinzugefügt werden und dass habe ich mit diesem code bewohner[anzahlWohneinheiten++] gemacht oder wie siehst du es?

Hast du nicht.

Nehmen wir mal an es gibt 10 Wohneinheiten. Du hast also ein Array "bewohner" mit der Größe 10 erstellt. Mit deinem Code weist du bewohner[10] (weil anzahlWohneinheiten = 10) eine neue Familie zu und erhöhst danach die Anzahl auf 11.

1. bewohner[10] verursacht eine Exception, weil der höchste Index 9 ist
2. Du möchtest an der freien Stelle eine Familie zuweisen. Machst du aber nicht.
3. Die Anzahl der Wohneinheiten soll sich nicht ändern.
4. Du solltest vor der Zuweisung prüfen, ob die Familie schon im Haus ist

 

[temi]

So sieht meine neue Methode aus

Du brauchst KEINE verschachtelte Schleife.

 

[Xknight]

Warum das nicht?

 

[Xknight]

Wie sieht denn dein Lösungsvorschlag denn aus?

 

[lennero]

Warum das nicht?

Was macht die verschachtelte Schleife deiner Meinung nach?

 

[Xknight]

Die überprüft ob ein Wert im Array doppelt vorkommt und wenn ja gibt er es als false zurück und wenn nicht fügt er ein Bewohner hinzu in der frei markierten Stelle

 

[temi]

Wie sieht denn dein Lösungsvorschlag denn aus?

Ich schlage vor, ihr beginnt mal zu denken. Das ist echt sehr einfach (also das Problem, das Denken anscheinend nicht).

Die erste Schleife deiner ursprünglichen Lösung war schon fast richtig. Du musst nur die Familie an die richtige Stelle im Array zuweisen. Und was ist die richtige Stelle? Wo hast du denn den freien Platz gefunden?

Für die Prüfung, ob die Familie bereits existiert, kopiere die Schleife und setze sie vor der Schleife mit der Zuweisung. Dann passt du die Abfrage an und sobald eine identische Familie gefunden ist, kannst du mit return false; die Methode verlassen.

 

[Xknight]

Danke, aber was war noch mal meine ursprüngliche Lösung?

 

[temi]

Warum das nicht?

Weil du eigentlich nicht wissen willst, ob es doppelte Einträge gibt. Du möchtest wissen, ob die Familie, die angemeldet werden soll, bereits vorhanden ist. Dafür musst du nur einmal durch die bewohner schauen. Und das machst du bevor du die Familie einfügst.

 

[temi]

In der Methode anmelden() sollen wir in einer freien Stellen einen Bewohner hinzufügen.
So lautet mein Quellcode.
Da sehe ich keinen Fehler.

public boolean anmelden(Familie someFam) {
         for(int i = 0; i < bewohner.length; i++) {
                if(bewohner[i] == null) {
                    bewohner[anzahlWohneinheiten++] = someFam;
                }
         }
     
         for (int i = 0; i < bewohner.length-1; ++i)
              for (int j = i+1; j < bewohner.length; ++j)
                if (bewohner[i] == bewohner[j]) {
                    return false;
                }
        return true;
    }

Mit dieser Abfrage überprüfe ich ob die anzahlWohneinheiten kleiner 1 ist und mit den for-Schleifen überprüfe ich ob Redundanzen im Array vorhanden sind.

Oh Mann. Ihr seid ja echt drauf!

 

[Xknight]

Soll ich dann schrreiben:

bewohner[i] = somefam

 

[temi]

Muss ich dann bewohner = somefam schreiben. Wäre das korrekt?

Nein. Du musst schon einen Index angeben. Schau dir doch einfach den Vergleich eine Zeile vorher an! Wo hast du die freie Stelle gefunden?

 

[mihe7]

Anstatt weiterzuhelfen einfach hängen lassen oder einfach mal diese Kleinigkeit vorzusagen wäre echt kein Akt, das merke ich mir und lerne auch daraus so ist es nicht aber sich so zu äußern wie ihr bringt mich auch null weiter, außerdem ist das eine kleinigkeit an der ich hänge die ich einfach nicht im stande bin zu beheben, weil ich das noch NIE gemacht habe. Der Rest ist schon alles korrekt außer das mit der Methode.

Jono, das Thema Arrays hatten wir ausführlichst unter https://www.java-forum.org/thema/zw...-und-in-eindimensionales-ueberfuehren.186409/ Dort wurde Dir in über 300(!) Beiträgen der Spaß x-mal erklärt und das ist jetzt 14 Tage her.

Hier geht es um die Umsetzung eines einfachen Algorithmus, den Du erstens sofort hinschreiben können müsstest und Dir @temi in #73 haarklein aufgeschrieben hat.

Noch genauer "helfen" würde bedeuten, Dir den fertigen Code zu geben. Das gibt es hier normalerweise nicht.

 

[kneitzel]

Haus(String adresse,  Person besitzer, int anzahlWohneinheiten){
    ...
    anzahlWohneinheiten = bewohner.length; // Welcher Variablen wird hier etwas zugewiesen?
    ...
}

Also was für einen Sinn macht ein Parameter, wenn ihr den Wert ignoriert und einfach irgendwie überschreibt?

public boolean anmelden(Familie someFam) {
         for(int i = 0; i < bewohner.length; i++) {
                if(bewohner[i] == null) {
                    bewohner[anzahlWohneinheiten++] = someFam;
                }
         }
    ...
    }

Was macht Ihr da? Ihr sucht eine freie Stelle und dann ignoriert ihr die freie Stelle und packt es einfach woanders rein?

 

[Xknight]

Nein. Du musst schon einen Index angeben. Schau dir doch einfach den Vergleich eine Zeile vorher an! Wo hast du die freie Stelle gefunden?

Habe ich doch mit

bewohner[i]=someFam

Ist das endlich korrekt?

 

[temi]

Habe ich doch mit

bewohner[i]=someFam

Ist das endlich korrekt?

Halleluja!
Zeig noch mal den gesamten Code.

 

[Xknight]

Danke, aber was ist noch im Konstruktor falsch, da ich laut der Aufgabe, vermute ich mal, es richtig programmiert habe?

 

[temi]

Danke, aber was ist noch im Konstruktor falsch, da ich laut der Aufgabe, vermute ich mal, es richtig programmiert habe?

#107 und #110 und #130 die obere Hälfte.

 

[Xknight]

Bin neu hier und darum weiss ich nicht was mit diesen # gemeint sind.

 

[mihe7]

Bin neu hier und darum weiss ich nicht was mit diesen # gemeint sind.

Das sind die Kommentarnummern, die Du über jedem Kommentar rechts oben siehst.

 

[kneitzel]

Das sind die Nummern der Beiträge. Findest Du oberhalb auf der Rechten Seite.

Und #130 zeigt es auch noch einmal etwas auf. Wie kann es richtig programmiert sein, wenn Du ein Argument, das vorgegeben ist, nicht verwendest?

Wie kann man fragen, ob etwas richtig ist, wenn ihr eine NullPointerException bekommen habt?

Was macht der Konstruktor? Ein Konstruktor ist doch dazu da, Dinge zu initialisieren. Wie kommt ihr auf die Idee, auf ein Element zugreifen zu können, dass noch nicht initialisiert ist?

@temi: Wie schaffst Du es, das auszuhalten?

 

[temi]

Bin neu hier und darum weiss ich nicht was mit diesen # gemeint sind.

Die Nummer des Beitrags, wird ganz rechts angezeigt.

 

[temi]

@temi: Wie schaffst Du es, das auszuhalten?

Ach, ich hol mir gleich ein Bier.

 

[Xknight]

Das Ding ist wir haben doch beim Konstruktor alles definiert und wir sollten nur die übernehmen.

 

[mihe7]

1. Was bewirkt dieser Code?

void doSomething(int x) {
   if (x < 1) {
       x = 1;
   }
}

2. Was bewirkt dieser Code?

Haus(int x) {
    x = bewohner.length;
}

 

[temi]

Das Ding ist wir haben doch beim Konstruktor alles definiert und wir sollten nur die übernehmen.

Ach Leute. Was ist denn so schwer daran mal die Kommentare durchzulesen und dann darüber nachzudenken.

Haus(String adresse,  Person besitzer, int anzahlWohneinheiten){
    this.adresse = adresse;
    this.besitzer = besitzer;
    
    anzahlWohneinheiten = bewohner.length; // Welcher Variablen wird hier etwas zugewiesen?
    this.bewohner = new Familie[anzahlWohneinheiten];
    
    if (anzahlWohneinheiten < 1) { // Was soll das bezwecken?
        anzahlWohneinheiten = 1;
    }
}

 

[Xknight]

* Implementieren Sie einen Konstruktor mit adresse, besitzer und anzahlWohneinheit als Parameter.
* Die Adresse und der Besitzer sollen übernommen werden.
* Wenn die Anzahl der Wohneinheiten kleiner als 1 sein sollte, dann soll 1 dafür verwendet werden.
* Die Anzahl der Wohneinheiten definiert die Länge des Arrays der Bewohner.
* Zu Beginn wohnen keine Bewohner und die Elemente des Arrays sind alle "null".
Also habe ich genau so programmiert im Konstruktor.

 

[temi]

* Implementieren Sie einen Konstruktor mit adresse, besitzer und anzahlWohneinheit als Parameter.
* Die Adresse und der Besitzer sollen übernommen werden.
* Wenn die Anzahl der Wohneinheiten kleiner als 1 sein sollte, dann soll 1 dafür verwendet werden.
* Die Anzahl der Wohneinheiten definiert die Länge des Arrays der Bewohner.
* Zu Beginn wohnen keine Bewohner und die Elemente des Arrays sind alle "null".
Also habe ich genau so programmiert im Konstruktor.

Ist dein Konstruktor anders, als der von jono aus #142? Der in #142 ist fehlerhaft.

 

[albanhu]

* Implementieren Sie einen Konstruktor mit adresse, besitzer und anzahlWohneinheit als Parameter.
* Die Adresse und der Besitzer sollen übernommen werden.
* Wenn die Anzahl der Wohneinheiten kleiner als 1 sein sollte, dann soll 1 dafür verwendet werden.
* Die Anzahl der Wohneinheiten definiert die Länge des Arrays der Bewohner.
* Zu Beginn wohnen keine Bewohner und die Elemente des Arrays sind alle "null".
Also habe ich genau so programmiert im Konstruktor.

Ich hatte die Aufgabe auch mal und bei mir wurde die ganze Zeit auch die Fehlermeldung angezeigt …
Falls du es gelöst hast könntest mal erklären wie du es geschafft hast ? wäre echt cool

 

[Xknight]

Haus(String adresse,  Person besitzer, int anzahlWohneinheiten){
        this.adresse=adresse;
        this.besitzer=besitzer;
        
        anzahlWohneinheiten = bewohner.length;
        this.bewohner = new Familie[anzahlWohneinheiten];
        
        if(anzahlWohneinheiten<1) {
            anzahlWohneinheiten=1;
        }

Genau so steht es bei mir?

 

[temi]

Genau so steht es bei mir?

Ist das jetzt eine Antwort oder fragst du dich noch, ob es wirklich so bei dir steht?

Ist auf jeden Fall nicht korrekt.

 

[mihe7]

@temi mit nem Bier ist es da nicht getan.

 

[kneitzel]

* Die Anzahl der Wohneinheiten definiert die Länge des Arrays der Bewohner.

Was bedeutet A definiert B?

Bedeutet das, dass A durch B gegeben (definiert) wird oder dass b durch a bestimmt (definiert) wird?

 

[Xknight]

Warum sollte es nicht korrekt sein? Genau das habe ich getan was in der Aufgabe steht.