dante: parse error.... komm nich drauf was falsch ist

hallo und guten tag

hab als totaler php newb versucht nen php mail script zu schreiben beim testen zeigt er mir aber nen parse error in line 19 an:

<?php
//gueltigkeitspruefung
if ($email=="" || $ansprechpartner=="" || $firma=="")
{
echo"nochmal!";
}
else {
if(!eregi('^([._a-z0-9-]+[._a-z0-9-]*)@(([a-z0-9-]+.)*([a-z0-9-]+)(.[a-z]{2,3})?)$', $email)) {
echo"keine emailadresse!";
exit;
}
// Email wird vorbereitet
$youremail="markus.dauth@volk-marketing.de";
$headers = "Von: "$ansprechpartner" <$email>\n";
$subject="Feedbackformular: Telefrank";
$message = $ansprechpartner.";".$email.";".$firma.";".$strasse.";".$plz.";".$ort.";".$telefon.";".$telefax.";"

//schoener machen
$message=str_replace("\"",""",$message);   ***********line 19 **********
$message=str_replace("'","'",$message);
urlencode ($message);
$message=str_replace("%0D","\n",$message);

//versenden
mail("$youremail", "$subject", $message, $headers);
echo "Danke $name, Ihre Email wurde versendet.";
}
?>

und ich weiss leider überhaupt nicht woran das liegt. es müsste doch eigentlich funktionieren?

lg

dante

  1. hi dante
    schau mal zeile 16?! am ende der zeile fehlt das semikolon vergessen

    gruss armin

  2. hi,

    $message = $ansprechpartner.";".$email.";".$firma.";".$strasse.";".$plz.";".$ort.";".$telefon.";".$telefax.";"

    hinter dieser zeile fehlt ein semikolon.

    gruss,
    wahsaga

    1. oh ja wie peinlich. danke

      *schäm*

      funktioniert jetzt wie es sollte =)

      ich ahbe da aber noch eine frage:

      jetzt kommt der text bei mir als textwüste an:
      name;markus.dauth@volk-marketing.de;firma;leer;leer;01234;statd;leer;

      wie kann ich das hübscher formatieren?
      und wie kann ich den wert einer chechbox übergeben?

      1. hi,

        jetzt kommt der text bei mir als textwüste an:
        name;markus.dauth@volk-marketing.de;firma;leer;leer;01234;statd;leer;

        natürlich, genau so setzt du es ja auch in deinem script zusammen.

        wie kann ich das hübscher formatieren?

        definiere "hübscher"!

        z.b. mit leerzeilen zwischen den einzelnen wert? -> \n

        und wie kann ich den wert einer chechbox übergeben?

        http://dclp-faq.de/q/q-formular-checkbox.html

        gruss,
        wahsaga

        1. natürlich, genau so setzt du es ja auch in deinem script zusammen.

          wie kann ich das hübscher formatieren?

          hab jetzt einfach leerzeichen reingehauen und mehr absätze gemacht ---> passt scho

          mich würde halt nur interesieren wie man dann zum beispiel fett setzen kann oder kursiv oder sowas

          http://dclp-faq.de/q/q-formular-checkbox.html

          super danke für den link!

          schönes wochenende noch
          (nur noch das hier fertig und ich haeb auch WE ^^)

          dante

          1. hi,

            mich würde halt nur interesieren wie man dann zum beispiel fett setzen kann oder kursiv oder sowas

            in einer reinen text-mail - natürlich gar nicht.

            dann müsstest du schon eine HTML-mail verschicken, wenn du das haben willst ...

            gruss,
            wahsaga

          2. hab da jetzt so rumprobiert und noch ne andere seite gefunden, wo es aber auch nciht gerade einfahc erklärt wird

            der name beider chekboxen ist cbutton[]

            und im php script steht:

            // Email wird vorbereitet
            if (isset($_REQUEST['cbutton'])) {
              reset($_REQUEST['cbutton']);
              foreach ($_REQUEST['cbutton'] as $k => $v) {
                $checkbox+=$k.$v."\n";
              }
            }

            $checkbox sollte den wert der chekcboxen enthalten, damit ich die dann an meine nachricht hinendran hängen kann. ich bekomme aber nur die zahl 1 als ergebnis

            ????

            was mache ich denn flasch?

            1. Hallo,

              reset($_REQUEST['cbutton']);

              das reset kannst du dir sparen, das macht foreach selber: http://de2.php.net/foreach

              foreach ($_REQUEST['cbutton'] as $k => $v) {
                  $checkbox+=$k.$v."\n";

              php != javascript
              du möchtest statt dem + einen . verwenden: http://www.php.net/language.operators.string.
              Außerdem ist es besser die Variable vorher zu initialisierten (schreib also vor die Schleife noch ein $checkbox = ''; o.ä. hin (sofern du nicht schon vorher was in die Variable schreibst).

              Grüße aus Nürnberg
              Tobias

              --
              Selfcode: sh:( fo:) ch:? rl:( br:< n4:& ie:% mo:| va:) de:] zu:) fl:( ss:| ls:[ js:|
      2. http://php.net/explode
        http://php.net/if

        rgds, sx