Wieder mal isset . . . .

Alles, was PHP betrifft, kann hier besprochen werden.

Wieder mal isset . . . .

Postby Gutschy » 08. July 2003 20:31

Bin mal wieder zum lernen von PHP gekommen, und natürlich hackt es gleich wieder *umpf*. Na ja, und da ich gerade sowieso online war . . .*ggg*

Hier mal das Skript:

<html>
<head>
<title>Newsletter</title>
</head>
<body>
<h1>Newsletter</h1>
<h3>Möchten Sie den Newsletter abonieren?</h3>
<form action="<?php $PHP_SELF ?>" method="post">
Anrede. <input type="radio" name="Anrede" value="Frau"> Frau
<input type="radio" name="Anrede" value="Herr"> Herr
Ihr Nachname: <input type="text" name="Name"><br>
Ihre E-Mail-Adresse: <input type="text" name="Mail"><br>
Ja ich möchte den Newsletter. <input type="radio" name="Bestätigung" value="ja"> Ja
<input type="radio" name="Bestätigung" value="nein"> Nein

<input type="submit" name="senden" value="Newsletter bestellen">
</form>
<?php
if (isset(SMail) && $Mail != "" && (isset($Bestätigung) && $Bestätigung == "ja") {
if (mail("MeineAdresse@hhh.de", "Neuer Newsletter", "$Anrede","$Name", "From: $Mail")){
if ($Anrede=="Herr") {
echo "<p>Hallo Herr ";
}
else {
echo "<p>Hallo Frau ";
}
echo "$Name, danke für ihr Interesse </p>";}
elseif (isset(SMail) && $Mail != "" && $Bestätigung=="nein") {
if ($Anrede=="Herr") {
echo "<p>Hallo Herr ";
}
else {
echo "<p>Hallo Frau ";
}
echo "$Name, dann hätten Sie sich das Ausfüllen auch sparen können.</p>";}
else {
echo "<p>Leider gab es einen Sendefehler</p>\n";
}
}
?>
</body>
</html>



Vielen Dank für das anschauen.
Warum Schwerkraft wenn ich sie auch nackt haben kann.
Gutschy
 
Posts: 76
Joined: 28. May 2003 20:12
Operating System: Win7

Postby boppy » 08. July 2003 22:50

*megaHeul* - wenn du gerade anfängst PHP zu lernen kauf dir bitte ein buch oder lerne nah an guten scripts.

Ein paar (nicht-muss-aber-sollte) regeln:

- NIEMALS (doch ein muss *fg*) Sonderzeichen (unter anderem äöüß) einsetzen.
- möglichst alles klein schreiben um probleme zu vermeiden
- du musst nicht doppelt moppeln! es reicht:

if($_POST["mail"] && $_POST["bestätigung"] == "ja") {

Denke unbedingt an $_POST[], $_GET[] o. ä. um fehler zu vermeiden. Die einfache bedingung if($vari) beduetet im übrigen das gleiche wie if(isset($vari)){ bla; }
...in diesem Sinne
yours boppy

Interpunktion und Orthographie dieses Beitrags sind frei erfunden.
Eine Übereinstimmung mit aktuellen oder ehemaligen Regeln wäre rein zufällig und ist nicht beabsichtigt.
User avatar
boppy
AF Moderator
 
Posts: 501
Joined: 27. December 2002 02:15
Location: W-E-City

Postby Gutschy » 09. July 2003 05:43

*boppy Taschentuch reich*

Is ja gut, hast ja recht, aber Skripte abschreiben ist doch so langweilig. *gg* Ok wenn man zwischen den Lernabschnitten eine längere Pause machen muß, sollte man sich nicht gleich übernehmen indem man versucht alles besser zu machen. Was die Umlaute angeht so werden sie ja so von ziemlich allen Browsern korrekt angezeigt, aber natürlich ist es "purer" sie zu maskieren. Außerdem sollte es doch für einen String egal was er für Zeichen übergeben bekommt, oder????? Aber vielleicht kann ja PHP mit Umlauten wirklich nichts anfangen. (Ok, gerade gegoogelt. Im ASCII Zeichensatz gibt es sie nicht *oh wunder*, könnten also eine Fehlerquelle sein.)

Lieber boppy, hiermit gelobe ich Besserung und fange erstmal mit SQL an.*lol*

Ok, und danach mal ein richtiges Buch. *gg*

Hab jetzt auf jeden Fall deinen Code mal Eingefüg und hat geholfen, leider noch natürlich noch nicht so ganz. Aber lass das ganze mal jetzt so stehen, und falls noch jemand lust dazu hat . . . :D
Warum Schwerkraft wenn ich sie auch nackt haben kann.
Gutschy
 
Posts: 76
Joined: 28. May 2003 20:12
Operating System: Win7

Postby ClausVB » 09. July 2003 16:23

Ich wollte ja zuerst ausrasten, weil hier jemand Variablennamen mit Umlauten verwendet ... dann habe ich es getestet und es geht ... upps

Aber der Code ist so grottig formatiert (nicht eingerückt, etc.) da möchte man ja heulen ...

*Claus sich boppy anschließt*

Folgender Seite sei Dir ans Herz gelegt:

PHP Coding Standard

Zwecks Mail Skripten würde ich folgendes vielleicht versuchen:
vlibMimeMail
User avatar
ClausVB
 
Posts: 32
Joined: 12. June 2003 10:46
Location: NRW - Germany

Postby Gutschy » 09. July 2003 19:01

@ClausVB, besten Dank für die Links. Natürlich werd ich mir die mal reinziehen. Und ansonsten nochmal Entschuldigung und als kleine Warnung, *gg* ich bin leider ein Wiederholungstäter, na gut und ein wenig lernfähig auch. ;)
Warum Schwerkraft wenn ich sie auch nackt haben kann.
Gutschy
 
Posts: 76
Joined: 28. May 2003 20:12
Operating System: Win7


Return to PHP

Who is online

Users browsing this forum: No registered users and 8 guests