PHP OOP, Werte werden nicht in den Eigenschaften gespeichert.

Stef · 6. November 2017

Hey,

ich habe in oop was programmiert, was Gesamtkosten und die Gesamtdauer zurückgeben soll.

Wenn ich diese beiden Punkte ohne function und Eigenschaften mache, dann geht das auch, nur wenn ich nun die value der Gesamtkosten und Gesamtdauer in deren passenden Eigenschaften speichern möchte, wird es nicht gemacht. Verstehe nicht warum.

Mein Code:

PHP

private $bookedDays;
private $totalCost;

public function setBookedDays(){

                $ankunftstag = new Datetime($this->ankunftstag);
                $abfahrtstag = new Datetime($this->abfahrtstag);

                // subtrahieren

                $days = $abfahrtstag->diff($ankunftstag);

                $this->bookedDays = $days->format("%a"); //%a gibt die Anzahl der Tage zurück

        }

        public function getBookedDays(){
            return $this->bookedDays;
        }

        public function setTotalCost(){
            $gesamtKosten = self::PRICE * $this->bookedDays; // -> das ist der Endpreis
            $this->totalCost = $gesamtKosten;
        }

        public function getTotalCost(){
            return $this->totalCost;
        }

Alles anzeigen

Wenn ich getBookedDays() und getTotalCost() ausgeben möchte wird nix ausgegeben.

Hoffe ihr könnt mir helfen.

Stef

JR Cologne · 6. November 2017

Das ist aber nicht der ganze Code...

Bekommst du irgendwelche Fehler?

Was du auf jeden Fall beachten musst, ist, dass alle Attribute/Eigenschaften, die du mit $this->attributname aufrufst, auch oben in der Klasse definiert sein müssen.

Stef · 6. November 2017

Der ganze Code ist 500 Zeilen lang.

Nein bekomme keine Fehler. Habe ja schon gedebugt.

Wenn ich diese Werte ohne function und eigenschaft weiterverarbeite funktioniert es ja. Nur möchte ich aufgrund Sauberkeit des Codes, da ich diese 2 Angaben auch in ner anderen Function brauche, diese in den eigenschaften speichern.

Eigenschaften sind richtig definiert.

JR Cologne · 6. November 2017

Das ist schön und gut, nur leider kann ich damit wenig anfangen, wenn ich nicht genügend Infos habe bzw. dies nicht mit eigenen Augen sehen kann.

Was meinst du genau mit "ohne function und Eigenschaft weiterverarbeiten"?

Wie interagierst du mit der Klasse?

Ich habe dir mal ein paar Kommentare im Code hinterlassen...

PHP

private $bookedDays;
private $totalCost;

public function setBookedDays() {
    $ankunftstag = new Datetime($this->ankunftstag);    // Datetime sollte doch eigentlich DateTime sein
    $abfahrtstag = new Datetime($this->abfahrtstag);    // woher kommt $this->ankunftstag und $this->abfahrtstag?

    // subtrahieren
    $days = $abfahrtstag->diff($ankunftstag);
    $this->bookedDays = $days->format("%a"); //%a gibt die Anzahl der Tage zurück
}

public function getBookedDays() {
    return $this->bookedDays;
}

public function setTotalCost() {
    $gesamtKosten = self::PRICE * $this->bookedDays; // -> das ist der Endpreis    // woher kommt self::PRICE?
    $this->totalCost = $gesamtKosten;
}

public function getTotalCost(){
    return $this->totalCost;
}

Alles anzeigen

Hast du schon die unterschiedlichen Werte der verschiedenen Variablen mittels var_dump() überprüft?

Stef · 6. November 2017

Ja, das ist mir auch bewusst. Nur was bringt dir/euch der ganze Code wenn der Fehler in 2 Funktionen nur auftritt? Da habt ihr viel mehr Arbeit.

$this->ankunftstag & $this->abfahrtstag bekommt die value aus einem Formular. Habe diese beiden Attribute schon per getter-Methode ausgegeben. Die sind vorhanden. Und nach dem erstellen der DateTime sind die auch vorhanden. Nur ab der Subtraktion ist der Wert weg. $days sowie $this->bookedDays gibt "" aus und das selbe ist auch bei $gesamtKosten.

Die Konstante PRICE ist oben bei den attributen definiert.

cottton · 6. November 2017

Kann auch nix sehen. Sehe keine Tippfehler o.ä.

Wenn Du keine Werte bei den get-Funktionen bekommst, dann wurden die set-Funktionen wohl nicht ausgerufen.

Order Du benutzt eine andere Instanz (also zB nicht die var, mit der Du die setter aufgerufen hast).

nur geraten, fehlt ja code

Edit: Posts überschnitten.

Poste doch mal den Code, mit dem Du die Klasse ansprichst.

zB

PHP

$object = new MyClass();
$object->setBookedDays();
var_dump($object->getBookedDays());

Jav · 6. November 2017

Ich habe mal bei "new Datetime" gestoppt zu lesen, denn das müsste ja "new DateTime" sein. Würde raten, daran liegts.

JR Cologne · 6. November 2017

Jep, habe ich ja auch schon drauf hingewiesen.

Habe mich nur dummerweise von abbringen lassen, da Stef ja schrieb, dass die Werte bis zur Subtraktion vorhanden sind. Damit meinte er allerdings die Eigenschaften der Klasse und nicht die lokalen Variablen $ankunftstag bzw. $abfahrtstag. Mein Fehler, nicht gut genug aufgepasst.

Wobei es mich wundert, dass PHP da kein Fehler schmeißt... Sollte doch eigentlich etwas in die Richtung Class not found produzieren.

Stef · 7. November 2017

Hey,

an Datetime liegt es nicht. So funktioniert das auch, egal ob DateTime oder Datetime.

Mein Problem war, dass ich die beiden funktionen nicht aufgerufen habe.

Habe dies ganz vergessen.

Vielen Dank für die Hilfe

JR Cologne · 7. November 2017

Oh, wieder was dazu gelernt. Klassennamen sind case insensitive...

Würde es übrigens trotzdem ändern und die Klasse so schreiben, wie sie definiert wurde. Also DateTime.

Jav · 7. November 2017

Schön

Aber: Waaas? Das funktioniert mit Datetime? Bei PHP ist Gross-/Kleinschreibung relevant, das wär' ja 'n Ding. Muss ich glatt mal selbst testen.

Stef · 7. November 2017

Ja habe ich nochmal überarbeitet.

Jav · 8. November 2017

Mein Welt ist zusammengebrochen.

Das gibt wirklich alles dasselbe aus:

Code

$a=new DateTime('2018-01-20');
$b=new Datetime('2018-01-20');
$c=new dAtEtImE('2018-01-20');
echo $a->format('Y-m-d H:i:s') . '<br>';
echo $b->format('Y-m-d H:i:s') . '<br>';
echo $c->format('Y-m-d H:i:s') . '<br>';

Wer dafür verantwortlich ist gehört auf den Scheiterhaufen.

Zitat von JR Cologne

Oh, wieder was dazu gelernt. Klassennamen sind case insensitive...

Hast Du da vielleicht irgendwas Offizielles? Die Suchmaschine meines Vertrauens gibt mir 0 brauchbare Ergebnisse, und in der Doku konnte ich auch noch nichts finden.

JR Cologne · 8. November 2017

Nope, habe das auf Stackoverflow gefunden:

https://stackoverflow.com/ques…3941/php-case-sensitivity

Steht aber mit Sicherheit irgendwo in der Dokumentation, habe allerdings selber auch nichts gefunden.

PHP OOP, Werte werden nicht in den Eigenschaften gespeichert.

Jetzt mitmachen!

Teilen