[PHP] String aus mehreren Variablen bilden

[Harry2k]

Hallo,

evtl kann mir jemand helfen:

Ich habe 3 Variablen:
Ich würde diese gerne im Betreff einer Mail verketten, leider wird immer der letzte wert genommen
Was mache ich falsch? Es wird leider immer nur der letzte Wert genommen. (Priorität)


$contact->subject = $_POST['name']." ".$_POST['type']." priorität ".$_POST['priority'];

Max Mustermann Service Priorität hoch

soll das Ergebnis im Mailbetreff sein.

Die Variablen bzw Eingaben werden aus einem HTML Formular übergeben -ich denke aber das Problem ist das verketten das ich nicht auf die Kette bekomme :-), wenn ich nur

$contact->subject = $_POST['name']

wähle klappts.

falls jemand einen Lösungsansatz für mich hat...

GRuß

Harry2k

 

[netzgestaltung]

was macht ein

echo $_POST['name'] . " " . $_POST['type'] . " priorität " . $_POST['priority'];

?

 

[Bagbag]

Dann will ich mal noch String-Interpolation in den Raum werfen, was es lesbarer macht: echo "{$a} {$b} priorität {$c}".

 

[Harry2k]

was macht ein

echo $_POST['name'] . " " . $_POST['type'] . " priorität " . $_POST['priority'];

?

das klappt leider nicht :
Error:
Parse error: syntax error, unexpected 'echo' (T_ECHO)

 

[Bagbag]

-

 

[netzgestaltung]

was es lesbarer macht:

also ich finde das fürchterlich unleserlich. aber jeder wie er mag ;-) ich mag nicht in anführungszeichen, die nur strings enthalten sollten noch variablen suchen.

Parse error: syntax error, unexpected 'echo'

da passt was in der Zeile davor nicht, vermutlich fehlt ein semicolon?

 

[leonM]

das klappt leider nicht :
Error:
Parse error: syntax error, unexpected 'echo' (T_ECHO)

Probier es pro Variable. z.B echo $_POST['name'] . " " . $_POST['type'];
und wenns klappt, fügst du die nächste hinzu.

 

[Bagbag]

@netzgestaltung du findest die erste Variante wirklich besser lesbar?

$name = "Max Mustermann"
$age = 35
    
echo "Hallo " . $name . ", du bist " . $age . " Jahre alt."
echo "Hallo {$name}, du bist {$age} Jahre alt."

 

[netzgestaltung]

Ja auf alle Fälle, würde das aber nochmal anders schreiben. Jedenfalls aber mit Semicolons. Ansonsten zu OT ;-)

 

[Harry2k]

ok danke an alle habs dann doch hinbekommen durch rumprobieren - muss ' anstatt " verwenden dann passts

$contact->subject = $_POST['name'].' '.$_POST['type'].' priorität '.$_POST['priority'];

das frisst er :-)

 

[Michael-Menten]

Für sowas geht auch der implode Befehl.

 

[sandreas]

Vielleicht noch ein paar Tipps:

• Du solltest prüfen, ob die POST-Variablen gesetzt und nicht leer sind (Input Validation)
• Du solltest auf keinen Fall POST-Variablen ungefiltert in eine E-Mail reinschreiben... wenn dir da jemand JavaScript reinposted, dann landet das nachher in der Mail (Input Sanitization)

$name = filter_var($_POST['name'] ?? "", FILTER_SANITIZE_STRING);
$type = filter_var($_POST['type'] ?? "", FILTER_SANITIZE_STRING);
$prio = filter_var($_POST['priority'] ?? "", FILTER_SANITIZE_STRING);

$contact->subject = $name.' '.$type.' priorität '.$prio;

Weil du mehrere Datenfelder hast und ohnehin schon OOP verwendest, könntest du auch eine Klasse "MailForm" mit den Eigenschaften "name", "type" und "priority" erstellen und damit die Validierung und Filterung übernehmen.

 

[floq0r]

@netzgestaltung du findest die erste Variante wirklich besser lesbar?

$name = "Max Mustermann"
$age = 35
  
echo "Hallo " . $name . ", du bist " . $age . " Jahre alt."
echo "Hallo {$name}, du bist {$age} Jahre alt."

Das ist Gewohnheitssache, ich sehe das persönlich so wie @netzgestaltung : Zwischen Quotes sollen nur Strings stehen und keine veränderlichen Bestandteile. Ich lasse auch die spaces vor und nach den . weg. Ich finde auch one-lined if-conditions ohne curly brackets furchtbar.

 

[netzgestaltung]

hier hab ich ein paar beispiele in unterschiedlichen situationen: https://stackoverflow.com/questions...hout-plugin-learning-method/60470320#60470320

ich denk da muß eh jeder für sich seinen weg finden...