[PHP]Erstes mini mini Script. Wo ist der Fehler ?

[Apocalyptica]

huhu zusammen
also ich mache grade meine ersten schritte mit php
(versuche es zumindest )

zum anfang wollte ich mal eine kleines mini script machen:
es werden 2 verschiedene namen eingegeben und diese werde dann in einem satz auf der nächsten seite verbunden.

der anfang sieht so aus:

<form action="test.php" method="post">
Erster Name:<br>
<input type="Text" name="erstername">
<br>
Zweiter Name:<br>
<input type="text" name="zweitername"><br>
<input type="submit" value="Senden">
</form>

Die Datei test.php sieht dann so aus:

<?php
$mann = $_post["erstername"];
$frau = $_post["zweitername"];
echo "$mann mag $frau";
?>

leider werden die variablen nicht angezeigt, sondern nur das "mag". wo ist der fehler ?

 

[Darklord]

$_POST

Wird großgeschrieben.

Ansonsten wäre es eine normale Variable. (Die nicht definiert und initialisiert ist.)

 

[Apocalyptica]

$_POST

Wird großgeschrieben.

Ansonsten wäre es eine normale Variable. (Die nicht definiert und initialisiert ist.)

ah danke
aller anfang ist schwer

 

[bastie]

schöner wäre es so:

<?php
$mann = $_POST['erstername'];
$frau = $_POST['zweitername'];
echo $mann .' mag ' . $frau;
?>

 

[Darklord]

schöner wäre es so:

Code hat nichts mit Schönheit zu tun.
Begründe doch einfach dein Vorschlag, und zwar das dein Code etwas schneller ist da PHP den String nicht nach Variablen durchsuchen muss, da Singelqoutes (').
Dann macht dein Vorschlag auch Sinn und Anfänger verstehen ihn.

 

[bastie]

Hi,

sicher auf die minimalen Geschwindigkeitsvorteile könnte man eingehen.

Aber viel wertvoller ist die bessere Überschaubarkeit des Codes, wenn die Variablen getrennt eingefügt werden.

Diesen Vorteil sieht hier wohl jeder (dank des Syntax-Highlightings) - auch ohne Anmerkung

 

[ice-breaker]

Code hat nichts mit Schönheit zu tun.

"Schönheit" hat mit Wartbarkeit zu tun
Umso besser ich den Source lesen und verstehen kann (auf den ersten Blick ) umso wartbarer bleibt der Code über die Jahre.

Begründe doch einfach dein Vorschlag, und zwar das dein Code etwas schneller ist da PHP den String nicht nach Variablen durchsuchen muss, da Singelqoutes (').
Dann macht dein Vorschlag auch Sinn und Anfänger verstehen ihn.

nö, diese Microbenchmarks sind alles andere als ein Argument.

 

[Darklord]

"Schönheit" Kommentare hat mit Wartbarkeit zu tun
Umso besser ich andere den Source lesen und verstehen kann (auf den ersten Blick ) umso wartbarer bleibt der Code über die Jahre.

Mein IDE kann auch " Variablen anständig Highlighten, daher auch kein richtiges Argument.

nö, diese Microbenchmarks sind alles andere als ein Argument.

Gut, von mir aus;PHP ist eh schon so lahm da kommt es auf ein paar Microsekunden auch nicht drauf an.

Aber das ist eh alles OT und darum gings hier eigentlich auch nicht.

 

[theHacker]

Umso besser ich andere den Source lesen und verstehen kann (auf den ersten Blick ) umso wartbarer bleibt der Code über die Jahre.

Richtig. Und je schöner der Code geschrieben is, desto leichter kann man ihn lesen.

Man könnte ja auch so schreiben (und wehe, einer macht das nach ):

<?="{$_POST['erstername']} mag {$_POST['zweitername']}"?>

Selber Code, selbe Ausgabe. Nur ich bezweifle, dass da einer auch nur Lust hat, so einen Kot Code zu warten...

"Schöner Code" ist mehr, als nur ordentlich zu dokumentieren.

 

[B2T]

Wie wäre es mit inkrementeller Schreibweise?

<?php
$mag = $_POST['erstername'];
$mag .= ' mag ';
$mag .= $_POST['zweitername'];
echo $mag;
?>

 

[Snyke]

Niemals Daten die per form uebermittelt wurden direkt wieder ausgeben sag ich nur, das fuehrt zu potentiellen XSS Angriffen (einfach Javascript in eines der Felder einfuegen und due hast ein 1a angriff).
Ich empfehle in erster linie immer ein htmlentities() um die Werte.