ich habe der Code geschrieben wie du sagst , aber bekomme nichts...

[Rolf B]

Hallo liebewinter,

die Funktion GetPostData, die ich Dir vorgeschlagen hatte, soll die Werte aus den Eingabefeldern einlesen und in Eigenschaften des ClassProveContakt3-Objekts speichern. Wenn Du auch noch Filter-Funktionen verwenden willst, dann gehört das nach GetPostData.

function GetPostData()
{
   if (isset($_POST["Name"]) && isset($_POST["Email"]) && isset($_POST["Message"]))
   {
      $this->Name = trim(filter_input( INPUT_POST, 'Name', FILTER_SANITIZE_STRING));
      $this->Email = trim(filter_input( INPUT_POST, 'Email', FILTER_SANITIZE_STRING));
      $this->Message = trim(filter_input( INPUT_POST, 'Message', FILTER_SANITIZE_STRING));
   }  
} 

In mella() verwendest Du dann nur noch $this->Name, $this->Email und $this->Message, und du prüfst mit empty(), ob etwas drin steht. So wie in PostOk.

Die mella-Funktion sieht ganz gut aus, rufst Du sie denn auch auf? Irgendwo muss $this->mella() stehen.

Ich möchte noch etwas dazu sagen, WIE man Programme aufschreibt. Es muss einheitlich sein. Dann kannst Du sie besser lesen. Es gibt zum Beispiel unterschiedliche Möglichkeiten, wo man ein { hinschreibt.

So:

function GetPostData() {
   if (isset($_POST["Name"])) {
      // ...
   }
}

Oder so:

function GetPostData()
{
   if (isset($_POST["Name"]))
   {
      // ...
   }
}

Es gibt da kein falsch. Du musst wissen was dir besser gefällt. Aber es muss überall gleich sein. Sonst ist es schwer zu lesen.

Einrückung ist auch wichtig. Programmzeilen werden eingerückt wenn mit {...} ein neuer Block gebildet wird, oder auch bei einem if, wenn zum if nur ein Befehl gehört. Anfang und Ende vom Block müssen genau untereinander stehen.

function hallo($welt) {
    echo "Das ist " . $welt;
       for ($a=0; $a<100; $a++)
   { echo "Nummer ".$a;
}
  }

Das versteht keiner. So ist es besser:

function hallo($welt) {
   echo "Das ist " . $welt;
   for ($a=0; $a<100; $a++)
   {
      echo "Nummer ".$a;
   }
}

Rolf