Kleine Frage zu OOP

[mihe7]

Das siehst Du, wenn Du die Fragen beantwortest.

 

[jono]

Das bedeutet dass b durch a bestimmt wird

 

[mihe7]

Jetzt beantwortet Ihr noch die Fragen aus #142 (oder wenn Euch Fragen in Kommentaren überfordern #141).

 

[temi]

Es ist doch wirklich ganz einfach:

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

Welcher Variablen wird hier etwas zugewiesen?

Und behaupte bloß nicht wieder dass alles richtig ist!

 

[1337Avdu]

Also laut #141 bewirkt der erste code das x 1 gesetzt wird wenn x kleiner als 1 ist,
Und der zweite code deklariert dann x = bewohner.lenght bzw 1 = bewohner.lenght

 

[Xknight]

in der methode doSomething wird überprüft ob x kleiner eins ist und im Konstruktor haus ist x die Länge des arrays d.h. die Länge des Arrays ist eins

 

[mihe7]

in der methode doSomething wird überprüft ob x kleiner eins ist

Eine Überprüfung bewirkt aber noch nichts.

im Konstruktor haus ist x die Länge des arrays d.h. die Länge des Arrays ist eins

Woher nimmst Du die Information, dass das Array eine Länge von 1 hätte?

 

[temi]

Oh Gott @mihe7, was hast du getan!

 

[jono]

@mihe

1.
void doSomething(int x) {  // hier wird die Bedingung festgelegt, falls x kleiner eins sein sollte, soll es der 1 gleichgesetzt werden. 
   if (x < 1) {
       x = 1;
   }
}
2.
Haus(int x) {
    x = bewohner.length;
}
Hier wird dem x die Länge des Bewohnerarrays zugewiesen.

 

[jono]

Aufrunden in jedem Fall für kleiner 1 auf 1 wäre dann doch korrekt?

 

[temi]

Hier wird dem x die Länge des Bewohnerarrays zugewiesen.

Und ist das sinnvoll? Was ist denn "x"?

 

[jono]

mit Math.ceil(anzahlWohneinheiten)

 

[jono]

x soll die Anzahl der Wohneinheiten darstellen

 

[temi]

Leute ihr schweift ab. Kommt zurück!

 

[Xknight]

Nein @jono man soll das Array im Konstruktor definieren.

 

[temi]

x soll die Anzahl der Wohneinheiten darstellen

Könnte x der Parameter sein? Vielleicht?

 

[jono]

Bewohner.length =anzahlWohneinheiten

 

[temi]

Bewohner.length =anzahlWohneinheiten

Du kannst bewohner.length nichts zuweisen.

 

[kneitzel]

Bewohner.length =anzahlWohneinheiten

Wir kommen in die richtige Richtung!

Was ist bewohner für ein Typ? Wie kann man den initialisieren, so dass bewohner.length = anzahlWohneinheiten ist?

 

[jono]

Stimmt sorry

 

[Xknight]

Wir sollen das Array im Konstruktor übernehmen.

 

[mihe7]

@jono In #159 hast Du die Fragen schon mal nicht falsch beantwortet. Ein wesentlicher Punkt:

Nein @jono man soll das Array im Konstruktor definieren.

Ihr greift auf ein Array zu, das noch gar nicht(*) initialisiert ist. Welchen Wert hat bewohner also zu diesem Zeitpunkt? Richtig, null. Folglich wird beim Versuch, auf die Längen-Eigenschaft des bewohner-Arrays (bewohner.length) eine NullPointerException auftreten. Das war die Frage zu 2.

Bei der Frage zu 1.: x ist ein Parameter. Dem wird nun ein Wert zugewiesen. Was soll das bringen?

EDIT: (*) das war von mir schlampig formuliert. Es wurde noch kein Array erzeugt. Die Instanzvariable bewohner ist selbstverständlich mit null initialisiert.

 

[Xknight]

Wir haben den Array schon oberhalb des Konstruktors definiert.

 

[temi]

Wir haben den Array schon oberhalb des Konstruktors definiert.

Ihr habt eine Feldvariable oder Instanzvariable für ein Array deklariert, aber diese Variable ist null solange sie nicht initialisiert wird.

 

[kneitzel]

Wir haben den Array schon oberhalb des Konstruktors definiert.

Dann bitte noch einmal anschauen, wie man ein Array deklariert und wie man es initialisiert. Was ihr habt ist nur eine Deklaration. Daher kommt auch die NullPointerException, wenn ihr darauf zugreift.
Wie initialisiert man ein Array mit einer bestimmten Länge?

 

[jono]

Familie[] Bewohner = new int [10]

 

[Xknight]

this.bewohner = new Familie[anzahlWohneinheiten];

Ich habe es doch hier den Array initialisiert.

 

[mihe7]

this.bewohner = new Familie[anzahlWohneinheiten];

Ich habe es doch hier den Array initialisiert.

Ja, aber erst nachdem Du auf bewohner.length zugreifst.

Familie[] Bewohner = new int [10]

Das wäre Deklaration und Initialisierung in einem. Schreib das mal getrennt. EDIT: Und warum int???

 

[temi]

this.bewohner = new Familie[anzahlWohneinheiten];

Ich habe es doch hier den Array initialisiert.

Korrekt, aber eine Zeile oberhalb versuchst du auf das noch nicht initialisierte Array zuzugreifen (mit benutzer.length).

 

[jono]

Die Zeile löschen?

 

[mihe7]

Die Zeile löschen?

Worauf bezieht sich Dein Kommentar?

 

[Xknight]

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

Das hatte ich schon die ganze Zeit

 

[jono]

@mihe int ist ein Fehler, war son Automatismus

 

[temi]

Das hatte ich schon die ganze Zeit

Ein paar Seiten vorher hast du etwas anderes behauptet.

 

[temi]

Ein paar Seiten vorher hast du etwas anderes behauptet.

Sind aber immer noch zwei Fehler drin.

 

[jono]

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

auf die Zeile die vorher überhalb der Array Initialisierung stand

 

[mihe7]

Das hatte ich schon die ganze Zeit

Dann ist Punkt 2 bei Dir auch kein Problem. Bleibt Punkt 1: was bringt es, wenn Du dem Parameter anzahlWohneinheiten die Länge des Arrays zuweist?

auf die Zeile die vorher überhalb der Array Initialisierung stand

Ja. Im Folgenden gehe ich von dem Code aus #182 aus. Sprich: die Frage bleibt, was es bringt, einem Parameter einen Wert zuzuweisen.

 

[jono]

 * Implementieren Sie einen Konstruktor mit adresse, besitzer und anzahlWohneinheit als Parameter.
Haus(String adresse,  Person besitzer, int anzahlWohneinheiten, Familie[] bewohner){ // Familie[]bewohner muss raus

 

[Xknight]

Habe ich schon

 

[jono]

@mihe7
Bringt einem gar nichts.

 

[mihe7]

@mihe7
Bringt einem gar nichts.

Und warum macht Ihr es dann?

 

[1337Avdu]

@mihe7 ich glaube du brauchst kein Bier sondern was hochprozentiges mein Freund!

 

[mihe7]

* Implementieren Sie einen Konstruktor mit adresse, besitzer und anzahlWohneinheit als Parameter.
Haus(String adresse,  Person besitzer, int anzahlWohneinheiten, Familie[] bewohner){ // Familie[]bewohner muss raus

BTW: genau das ist das Prinzip: sich zu 100 % an die Aufgabenstellung halten. Die sind am Anfang so genau geschrieben, dass man sie fast 1:1 in Code übersetzen kann, wie man an diesem Beispiel sieht.

 

[jono]

Gute Frage, da man das Array schon initialisiert hat , ist die Länge des Bewohnerarrays sowieso schon durch die anzahlWohneinheiten definiert

 

[mihe7]

@mihe7 ich glaube du brauchst kein Bier sondern was hochprozentiges mein Freund!

Seit dem Java-Forum bin ich auf intravenös umgestiegen

 

[temi]

Schaut euch mal eure Instanzvariablen an. Gibt es da evtl. eine die auch so heißt, wie der Parameter?

 

[kneitzel]

Seit dem Java-Forum bin ich auf intravenös umgestiegen

Das erklärt Alles! Den Tipp hättest Du mir schon viel früher geben können! Ich hätte hier schon eine eigene Einspritzanlage in Betrieb haben können!

 

[temi]

Und bezogen auf den Code von #182: Ruft mal die Methode mit anzahlWohneinheiten = -10 auf oder spielt diesen Aufruf mal gedanklich durch.

 

[mihe7]

Das erklärt Alles! Den Tipp hättest Du mir schon viel früher geben können! Ich hätte hier schon eine eigene Einspritzanlage in Betrieb haben können!

Alles Junkies hier.

 

[Xknight]

Schaut euch mal eure Instanzvariablen an. Gibt es da evtl. eine die auch so heißt, wie der Parameter?

Unsere Instanzvariablen heißen genau wie unsere Parameter @temi