Parse error: (was hab ich falsch gemacht?)

[ultracoder]

mir hat jemand die domain www.modeinbau.de verkauft, da hab ich jetzt ein forumular drauf, das geht auch alles soweit, aber wenn man es absenden will kommt


Parse error: parse error, unexpected $ in /var/www/web12/html/modeinbau/anfrage.php on line 34


der php code von anfrage.php ist folgender:

Code: Alles auswählen

<?
$vorname = $_POST ["vorname"];
$nachname = $_POST ["nachname"];
$email = $_POST ["email"];
$telefon = $_POST ["telefonnummer"];
$url = $_POST ["url"];
$styls = $_POST ["styls"];
$anzahl_eingebaut = $_POST ["dateien"];
$anzahl_einbauen = $_POST ["anzahl"];
$preis = $_POST ["preis"];
$links = $_POST ["links"];
$sonstiges = $_POST ["sonstiges"];
$myemail = "info@modeinbau.de";
$betreff = "modeinbau.de - Anfrage";
$nutzer = "$vorname $nachname";
$zusatz = "---------------------Zusatzangaben--------------------- Telefonnummer ist $telefon und unter sonstiges hat er
folgendes eingetragen: $sonstiges und das war es auch schon";
$text = "Sehr geehrtes Team von modeinbau.de, jemand hat das Formular auf modeinbau.de ausgefüllt, hier sind seine Angaben:
Angefragt hat $vorname $nachname ----------- Die Url zu seinem Forum ist $url ------------- Er hat $styls
eingebaut -------------- er hat schon $anzahl_eingebaut Mods eingebaut und möchte gerne $anzahl_einbauen Mods
eingebaut haben------------- seine preisvorstellung war $preis -------------- hier sind nun die links zu den mods
die er eingebaut haben möchte $links ich hoffe es hat alles geklappt";
if ($vorname or $nachname or $email or $url or $styls or $anzahl_eingebaut or $anzahl_einbauen or $links == "")
{
echo "Du musst alle Felder die mit einem * markiert sind ausfüllen, bitte probiere es noch einmal, indem du auf den
zurückbutton in deinem browser klickst";
}
else
{
mail ($myemail, $betreff, $text,
"From: $nutzer <$email>");
echo "Deine Anfrage wurde abgeschickt, ein Mitarbeiter von modeinbau.de wird sich nun so schnell wie möglich um deine Anfrage kümmern
und dir eine Antwort senden, dies dauert in der Regel nicht mehr als 24 Stunden, aber es ist nicht garantiert, dass modeinbau.de deinen Auftrag übernimmt";
?>

wenn ihr den html code des formulars braucht, geht einfach auf www.modeinbau.de und lasst ihn euch anzeigen, meine frage ist jetzt, wodurch kommt der fehler und wie behebe ich ihn


vielen dank im vorraus

[Olli Oberhausen]

Code: Alles auswählen

mail ($myemail, $betreff, $text,
"From: $nutzer <$email>");

Ein zeilenumbruch ist an dieser Stelle nicht erlaubt.

Code: Alles auswählen

if ($vorname or $nachname or $email or $url or $styls or $anzahl_eingebaut or $anzahl_einbauen or $links == "") 

und das funktioniert vielleicht in perl, aber nicht in php....

Ausserdem:
Bist du sicher, das du diese Spamschleuder online stellen möchtest?
Ich wäre dir sehr dankbar wenn du dieses Script schnellstmöglich löschen würdest. Ich hab nämlich schon genug Viagra und Porno spam.

Gruß, Olli

[S2B]

@Olli: Das ist beides in PHP möglich, wäre auch schlimm, wenn es das nicht wäre...

@ultracoder: Füge mal vor dem ?> in der letzten Zeile das ein:

Code: Alles auswählen

}

Ansonsten kann ich mich nur anschließen: Besorg dir ein gescheites Script.

[ultracoder]

ich will das ja nicht direkt nutzten, ich lerne gerade php und hab damit erst vor 3 tagen angefangen, ich muss das ja mal alles ausprobieren und ich finde, dafür, dass ich vor 3 tagen angefangen habe php zu lernen geht das doch

[Olli Oberhausen]

@Olli: Das ist beides in PHP möglich, wäre auch schlimm, wenn es das nicht wäre...

Ok, bei mail() geb ich dir recht, aber ($1 or $2 or $3 == "") geht nicht, da php das ungefähr so interpretiert:
if( !empty($1) or !empty($2) or $3 == "")

Das meinte ich

Olli

[ultracoder]

was soll den empty bedeuten?

[Jensemann]

was soll den empty bedeuten?

Ich will ja nicht böse sein, aber wenn du vor 3 Tagen angefangen hast PHP zu lernen, solltest du wissen das man Funktionen in der PHP-Dokumentation nachschlagen kann.

Ausserdem: Wenn du es gerade erst lernst, dann stelle deine Ergebnisse nicht gleich Online. Lernen tut man besser auf localhost. Wir lernen Autofahren ja auch nicht gleich in der Innenstadt um 16 Uhr zum Feierabendverkehr in der Weihnachtszeit.

[Dr.Death]

Hier noch ein passendes Keyboard:

[ externes Bild ]

* SCNR *

[ultracoder]

ok, ich habs schon gefunden