Getter & Setter Vergleich funktioniert nicht

[Forumsgast]

Hallo.
Ich versuche mich weiter zu bilden und arbeite mich nun in Getter und Setter ein.
Der Vergleich unten gibt mir andauernd nur "Zugriff verweigert" aus.
Kann mir da jmd helfen, da error_reporting(E_ALL) mir auch nichts anzeigt.

<?php
error_reporting(E_ALL);

class Kunde
{
    private $kontostand;
    private $geheimzahl;
        // Methoden
        
        // Setter: Kontostand
    public function setKontostand($konto)
        {
            $this->kontostand = $konto;
            //echo "Konto = $konto<br><br>";
        }
    // Getter: Kontostand
    public function getKontostand()
        {
            echo "Ihr Kontostand ist ";
            echo $this->kontostand;
            echo " Euro<br>";
        }
        
        // Getter: Geheimzahl
    public function getGeheimzahl()
        {
            echo "Ihre Geheimzahl ist ";
            echo $this->geheimzahl;
            echo "<br>";
        }
        
        public function setGeheimzahl($zahl)
        {
            $this->geheimzahl = $zahl;
            //echo "Zahl = $zahl<br><br>";
        }
}
 
$schmitt = new Kunde();
 
// Kontostand verändern
$schmitt->setKontostand(1000);
 
// Kontostand ausgeben
$schmitt->getKontostand();
 
// Geheimzahl verändern
$schmitt->setGeheimzahl(1321);
 
// Geheimzahl überprüfen
$eingabeausBankingFormular = 1321;  // auf 1321 ändern,
                                    // um funktionalität zu überprüfen

if ($schmitt->getGeheimzahl() == $eingabeausBankingFormular)
{
    echo "Zugang gewährt!";
}
else
{
    echo "Zugang verweigert";
        //echo "<br>Ihre Eingabe war: $eingabeausBankingFormular";
            
}
?>

[Forumsgast]

Hallo
Dein getGeheimzahl() gibt nicht den Wert zurück sondern erzeugt eine Ausgabe.

    public function getGeheimzahl()
        {
            echo "Ihre Geheimzahl ist ";
            echo $this->geheimzahl;
            echo "<br>";
        } 

[Forumsgast]

Ok... Hab mich da zu sehr auf die Ausgabe versteift.

...

public function getGeheimzahl()
        {
            echo "Ihre Geheimzahl ist ";
            echo $this->geheimzahl;
            echo "<br>";
            return $this->geheimzahl;
        } 

...

Jetzt passt es. Danke!

[Forumsgast]

Njo, wozu überhaupt die Ausgabe in dieser Methode?
Lass sie einfach den Wert zurückgeben, ob du die Methode dann aufrufst um etwas auszugeben oder um den Wert irgendwo hin zu speichern ist dann deine Sache. -> Flexibilität weil keine Ausgabe als Nebeneffekt erfolgt.