Umfrage mit php

flo96 · 19. Dezember 2013

Hey Leute, ich bastle an einer Umfrage mit php "Auswertung", die mit E-Mail dann versandt werden soll. Dazu sollen leere benutzereingaben fehler ausgeben, wenn das Formular abgesendet wurde

hier mal das was ich im Moment habe, komme nich weiter... http://website2.dietotenkoepfe.de/umfr.php

wer am php-code interessiert ist schaut unter: http://pastebin.com/TiEAtQfs

vor allem bringt mir die captcha und des mehrfachnennen probleme

Bitte um Hilfe

Roland · 19. Dezember 2013

du kannst ja um deine ganze eingabenprüfung einen if block machen und diesen nur ausführen wenn das Formular abgesendet wurde
indem du deinen submit button auch noch einen namen gibtst z.b. und diesen dann prüftst
oder einfach fragst ob isset($_POST['geschlecht']) z.b.

cottton · 19. Dezember 2013

Guck mir das grad an und das erste (was eigtl nicht gefragt war :D) was ich sehe ist:
- selected="selected" oder auch checked="checked" kann bei radio buttons / chkboxen weg. Ich halte es für sinnvoller dem Nutzer am Anfang ein komplett neutrales Formular zu übergeben.
- bei "Anregungen/Wünsche/Sonstiges" ist ein td "kaputt" ( td> )
- bei <select name="meinung" multiple="multiple"> bekommst Du nur eine Auswahl zurück.
besser: <select name="meinung[]" multiple="multiple">
durch das [] bekommst Du ein array zurück

edit:
also was da bei der sicherheitsabfrage passiert, kann ich nicht nachvollziehen.
Aber den rest hab ich mal überarbeitet:

PHP

<form action="" method="post">
<table class="mitgliedsantrag" cellspacing="5">
	<tr>
		<th colspan="4">Wir bitten Sie um Ihre Meinung</th>
	</tr>
	<tr>
		<td class="erklaerung" colspan="4">Bitte füllen Sie alle mit <span class="pflichtfeld1">*</span> gekennzeichneten Felder aus (Pflichtfelder)</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Welches Geschlecht haben Sie?</td>
		<td>&nbsp;</td>
		<td><input type="radio" name="geschlecht" value="männlich" />&nbsp;männlich</td>
		<td><input type="radio" name="geschlecht" value="weiblich" />&nbsp;weiblich</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Welcher Altersgruppe gehören Sie an?</td>
		<td>&nbsp;</td>
		<td colspan="2">
			<select name="altersgruppe">
				<option value="Bitte wählen">Bitte wählen</option>
				<option value="12-16">12-16</option>
				<option value="17-21">17-21</option>
				<option value="22-26">22-26</option>
				<option value="27-31">27-31</option>
				<option value="32-45">32-45</option>
				<option value="46-72">46-72</option>
			</select>
			<br>
		<span class="pflichtfeld"></span>
		</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Wie haben Sie uns gefunden?</td>
		<td>&nbsp;</td>
		<td colspan="2">
			<select name="fund">
				<option value="Bitte wählen">Bitte wählen</option>
				<option value="Google">Google</option>
				<option value="Facebook">Facebook</option>
				<option value="Werbung">Werbung</option>
				<option value="Bekannte/Verwandte">Bekannte/Verwandte</option>
				<option value="Mitglieder">Mitglieder</option>
			</select>
			<br>
		<span class="pflichtfeld"></span>
		</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Haben Sie schon einmal gekegelt?</td>
		<td>&nbsp;</td>
		<td><input type="radio" name="gekegelt" value="ja" />&nbsp;ja</td>
		<td><input type="radio" name="gekegelt" value="nein" />&nbsp;nein</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Wie finden Sie unsere Website?
			<br>
			&nbsp;&nbsp;<span class="hilfetext">(Mehrfachnennung mit STRG möglich)</span></td>
		<td>&nbsp;</td>
		<td colspan="2">
			<select name="meinung[]" multiple="multiple">
				<option value="interessant">Interessant</option>
				<option value="informativ">Informativ</option>
				<option value="zu wenig Info's">Zu wenig Informationen</option>
				<option value="mehr Bilder/Impressionen">Mehr Bilder/Impressionen</option>
				<option value="der HAMMER">Der HAMMER</option>
				<option value="verbesserungswürdig">Verbesserungswürdig</option>
			</select>
			<br>
		<span class="pflichtfeld">Bitte sagen Sie uns Ihre Meinung!		</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Waren Sie schon einmal bei uns Gastkegler?</td>
		<td>&nbsp;</td>
		<td><input type="radio" name="gastwar" value="ja" />&nbsp;ja</td>
		<td><input type="radio" name="gastwar" value="nein" />&nbsp;nein</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Könnten Sie sich vorstellen, bei uns Gastkegler zu sein?</td>
		<td>&nbsp;</td>
		<td colspan="2">
			<select name="gastsein">
				<option value="Bitte wählen">Bitte wählen</option>
				<option value="ja">ja</option>
				<option value="nein">nein</option>
				<option value="vielleicht">vielleicht</option>
				<option value="war schon">war schon</option>
			</select>
			<br>
		<span class="pflichtfeld">		</td>
	</tr>
	<tr>
		<td>&nbsp;&nbsp;Anregungen/Wünsche/Sonstiges<br>
			&nbsp;&nbsp;<span class="hilfetext">(Optional)</span>
		</td>
		<td><!--<span class="nachrichtlength">min. 10 Zeichen</span>--></td>
		<td colspan="2"><textarea rows="5" name="nachricht" class="nachricht"></textarea></td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Spam-Schutz</td>
		<td colspan="3">
			9 + 8 =
			<input type='number' name='zip' size='4' autocomplete='off'>
			<input type='hidden' name='zip2' value='70efdf2ec9b086079795c442636b55fb'>			<br>
			<span class="pflichtfeld">Bitte lösen Sie die Rechenaufgabe richtig!</span>




		</td>
	</tr>
	<tr>
		<td colspan="2" class="right"><input type="submit" id="login" value="Senden" /></td>
		<td colspan="2" class="left"><input type="reset" id="zuruecksetzen" value="Zurücksetzen" /></td>
	</tr>
</table>
</form>




<?php
	error_reporting(E_ALL); /*testphase alle fehler/warnings ect ausgeben*/
	if(!$_POST){
		exit(0); /*wenn formular nicht abgeschickt, dann ende*/
	}




	/*
	kurzer erklärung zu ()?:;
	() == if
	? == dann
	: == else
	; == ende
	(if diese bedingung true)? dann das hier : else das hier;
	*/




	function isPost($offset){
		return (isset($_POST[$offset]))?$_POST[$offset]:null;
	}
	function valOrErr($val,$not=array(),$err){
		return (!in_array($val,$not))?$val:$err;
	}







	// Sicherheitsabfrage - Zufallszahlen generieren
	$Z0 = array(mt_rand(1, 9), mt_rand(1, 9));
	$Z1 = max($Z0);
	$Z2 = min($Z0);
	$Spam = $Z1 . " + &#" . (48 + $Z2) . ";";
	$Schutz = md5($Z1 + $Z2);







	//Variablen deklarieren um gesendete Objekte einfacher zu packen
	$geschlecht = isPost('geschlecht');
	$alter = isPost('altersgruppe');
	$fund = isPost('fund');
	$gekegelt = isPost('gekegelt');
	$meinung = isPost('meinung');
	$meinung = (is_array($meinung))?implode(', ',$meinung):$meinung;
	$gastwar = isPost('gastwar');
	$gastsein = isPost('gastsein');
	$nachricht = isPost('nachricht');
	$zip = isPost('zip');
	$zip2 = isPost('zip2');
	$fehler['sicherheit'] = (md5(isPost('zip')) != (isPost('zip2')));
	$fehler = array("sicherheit"=>"",);
	$sicherheit = 'test';







	echo '<hr>';
	echo valOrErr($geschlecht,array(null),'Sie haben kein Geschlecht gewählt!')."<br>";
	echo valOrErr($alter,array(null,'Bitte wählen'),'Bitte geben Sie Ihre Altersgruppe an!')."<br>";
	echo valOrErr($fund,array(null,'Bitte wählen'),'Bitte geben Sie an, wie Sie uns gefunden haben!')."<br>";
	echo valOrErr($gekegelt,array(null),'Sie haben nicht gewählt, ob Sie schon einmal bei uns gekegelt haben!')."<br>";
	echo valOrErr($meinung,array(null),'Bitte sagen Sie uns Ihre Meinung!')."<br>";
	echo valOrErr($gastwar,array(null),'Bitte sagen Sie uns, ob Sie einmal Gastkegler gewesen sind!')."<br>";
	echo valOrErr($gastsein,array(null),'Bitte sagen Sie uns, ob Sie einmal Gastkegler sein wollen!')."<br>";










/*if($zip <= $Schutz or $zip >= $Schutz)
{
	$Fehler["sicherheit"] = (md5($_POST["zip"]) != $_POST["zip2"]) ? "Die Rechenaufgabe ist leider falsch!" : "";
}
else
{
	echo "" . $sicherheit . "<br>";
}




if($Schutz == $zip)
{
	echo 'test';
}*/
?>

Alles anzeigen

flo96 · 20. Dezember 2013

1. Danke für die Hilfe,

2. Bin ich noch nicht sehr geübt in php,

3. Kann man das z.B. echo valOrErr($gekegelt,array(null),'Sie haben nicht gewählt, ob Sie schon einmal bei uns gekegelt haben!')."<br>"; auch direkt in das Feld wo ich <?php if(isset($fehler_gekegelt) [...] ?> reinschreiben?

4. Habe ich auch ein kleines Problem mit der Captcha... könnte mir jemand entweder bei dieser Captcha helfen, oder mir eine funktionierende senden? Bin wie gesagt noch nicht so fix

5. soll der footer auch vor dem absenden des Formulars da sein...

6. Die Weiterleitung funkst auch nicht

cottton · 20. Dezember 2013

1. Gerne. Hab momentan nix zu schreiben, nur am lesen =)

2. Daher lesen lesen lesen =)

3. Die echo sind eigtl erstmal nur zur Testausgabe gedacht. Im eigtl Einsatz wirst Du sicherlich nicht alle Eingaben des Benutzers noch einmal anzeigen lassen.
Die Funktion valOrErr sollte nur zeigen, dass man die Prüfung vieler gleicher Abläufe in einer Funktion auslagern kann.
Ich habs mal umgestaltet. Ist halt wichtig, dass Du verstehst was da passiert.
Ach ja - Dir fehlt ein name="senden" beim Senden Button.

4. hab mir das mal angesehen. Ich denke ich habs verstanden =)

5. durch if gefixt

6. welche Weiterleitung?

hier also mal die geänderte Version:
(Prüfung immer vor der Ausgabe des Formulars)

PHP

<?php
	error_reporting(E_ALL); /*testphase alle fehler/warnings ect ausgeben*/




	function isPost($offset){
		if(isset($_POST[$offset])){ /*wenn $_POST['xyz'] gesetzt ist*/
			return $_POST[$offset]; /*dann gibt $_POST['xyz'] zurück*/
		}
		return ''; /*sonst gibt leeren string zurück*/
	}




	//Variablen deklarieren um gesendete Objekte einfacher zu packen
	$geschlecht = isPost('geschlecht');
	$alter = isPost('altersgruppe');
	$fund = isPost('fund');
	$gekegelt = isPost('gekegelt');
	$meinung = isPost('meinung');
	$meinung = (is_array($meinung))?implode(', ',$meinung):$meinung;
	$gastwar = isPost('gastwar');
	$gastsein = isPost('gastsein');
	$nachricht = isPost('nachricht');







	if(isset($_POST['senden'])){ /*wenn button senden*/




		/*bestimmen was nicht akzeptiert wird*/
		$leeres_dropdown_menu = 'Bitte wählen';
		$leeres_textfeld = '';




		/*die function zum prüfen*/
		function isErr($val,$not=array(),$err){
			if(in_array($val,$not)){ /*wenn im array $not $val enthalten ist (siehe in_array() @php.net)*/
				return $err.'<br>'; /*dann gib den error $err zurück*/
			}
			return ''; /*sonst gibt leeren string zurück*/
		}




		/*verwendung:
		echo isErr(
			$dasZuPrüfende,
			array($dasSollNichtVorkommen, $undDasAuchNich),
			'derAusgegebeneStringFallsEinesDavonVorkommt'
		);




		Bsp:
		echo isErr($alter,array($leeres_textfeld,$leeres_dropdown_menu),'Bitte geben Sie Ihre Altersgruppe an!');
		- es wird entweder die fehlermeldung 'Bitte geben Sie Ihre Altersgruppe an!', oder nichts (also '') ausgegeben
		- $alter wird geprüft
		- array($leeres_textfeld,$leeres_dropdown_menu) -> wir wollen kein leeres textfeld und auch kein 'Bitte wählen'




		natürlich kannst du auch das ergebnis in einer var packen:
		$fehler_soundso = isErr($alter,array($leeres_textfeld,$leeres_dropdown_menu),'Bitte geben Sie Ihre Altersgruppe an!');
		*/
		echo isErr($geschlecht,array($leeres_textfeld),'Sie haben kein Geschlecht gewählt!');
		echo isErr($alter,array($leeres_textfeld,$leeres_dropdown_menu),'Bitte geben Sie Ihre Altersgruppe an!');
		echo isErr($fund,array($leeres_textfeld,$leeres_dropdown_menu),'Bitte geben Sie an, wie Sie uns gefunden haben!');
		echo isErr($gekegelt,array($leeres_textfeld),'Sie haben nicht gewählt, ob Sie schon einmal bei uns gekegelt haben!');
		echo isErr($meinung,array($leeres_textfeld),'Bitte sagen Sie uns Ihre Meinung!');
		echo isErr($gastwar,array($leeres_textfeld),'Bitte sagen Sie uns, ob Sie einmal Gastkegler gewesen sind!');
		echo isErr($gastsein,array($leeres_textfeld),'Bitte sagen Sie uns, ob Sie einmal Gastkegler sein wollen!');




	}







	/*rechenaufgabe checken
	$_POST['zip'] ist die benutzer eingabe -- seine lösung (eingabefeld name="zip")
	$_POST['zip2'] ist der md5 hash der richtigen lösung (hiddefield name="zip2")
		es muss nun also die benutzereingabe (des benutzers lösung)
		in einen md5 hash gewandelt werden um zu prüfen, ob beide hashes gleich sind.
	*/
	if(md5($_POST['zip']) != $_POST['zip2']){
		echo 'Die Rechenaufgabe ist leider falsch!';
	}




	/*für erneute verwendung des rechenaufgabe neue zahlen ect generieren*/
	$Z0 = array(mt_rand(1, 9), mt_rand(1, 9)); /*ein array mit zufälligen zahlen von 1-9*/
	$Z1 = max($Z0); /*aus dem array den max wert holen (größte zahl)*/
	$Z2 = min($Z0); /*aus dem array den min wert holen (kleinstee zahl)*/
	$Spam = $Z1 . " + &#" . (48 + $Z2) . ";"; /*der string (die rechenaufgabe) erstellen*/
	$Schutz = md5($Z1 + $Z2); /*den md5 hash erstellen (hash wert der richtigen lösung)*/
?>










<form action="" method="post">
<table class="mitgliedsantrag" cellspacing="5">
	<tr>
		<th colspan="4">Wir bitten Sie um Ihre Meinung</th>
	</tr>
	<tr>
		<td class="erklaerung" colspan="4">Bitte füllen Sie alle mit <span class="pflichtfeld1">*</span> gekennzeichneten Felder aus (Pflichtfelder)</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Welches Geschlecht haben Sie?</td>
		<td>&nbsp;</td>
		<td><input type="radio" name="geschlecht" value="männlich" />&nbsp;männlich</td>
		<td><input type="radio" name="geschlecht" value="weiblich" />&nbsp;weiblich</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Welcher Altersgruppe gehören Sie an?</td>
		<td>&nbsp;</td>
		<td colspan="2">
			<select name="altersgruppe">
				<option value="Bitte wählen">Bitte wählen</option>
				<option value="12-16">12-16</option>
				<option value="17-21">17-21</option>
				<option value="22-26">22-26</option>
				<option value="27-31">27-31</option>
				<option value="32-45">32-45</option>
				<option value="46-72">46-72</option>
			</select>
			<br>
		<span class="pflichtfeld"></span>
		</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Wie haben Sie uns gefunden?</td>
		<td>&nbsp;</td>
		<td colspan="2">
			<select name="fund">
				<option value="Bitte wählen">Bitte wählen</option>
				<option value="Google">Google</option>
				<option value="Facebook">Facebook</option>
				<option value="Werbung">Werbung</option>
				<option value="Bekannte/Verwandte">Bekannte/Verwandte</option>
				<option value="Mitglieder">Mitglieder</option>
			</select>
			<br>
		<span class="pflichtfeld"></span>
		</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Haben Sie schon einmal gekegelt?</td>
		<td>&nbsp;</td>
		<td><input type="radio" name="gekegelt" value="ja" />&nbsp;ja</td>
		<td><input type="radio" name="gekegelt" value="nein" />&nbsp;nein</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Wie finden Sie unsere Website?
			<br>
			&nbsp;&nbsp;<span class="hilfetext">(Mehrfachnennung mit STRG möglich)</span></td>
		<td>&nbsp;</td>
		<td colspan="2">
			<select name="meinung[]" multiple="multiple">
				<option value="interessant">Interessant</option>
				<option value="informativ">Informativ</option>
				<option value="zu wenig Info's">Zu wenig Informationen</option>
				<option value="mehr Bilder/Impressionen">Mehr Bilder/Impressionen</option>
				<option value="der HAMMER">Der HAMMER</option>
				<option value="verbesserungswürdig">Verbesserungswürdig</option>
			</select>
			<br>
		<span class="pflichtfeld">Bitte sagen Sie uns Ihre Meinung!		</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Waren Sie schon einmal bei uns Gastkegler?</td>
		<td>&nbsp;</td>
		<td><input type="radio" name="gastwar" value="ja" />&nbsp;ja</td>
		<td><input type="radio" name="gastwar" value="nein" />&nbsp;nein</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Könnten Sie sich vorstellen, bei uns Gastkegler zu sein?</td>
		<td>&nbsp;</td>
		<td colspan="2">
			<select name="gastsein">
				<option value="Bitte wählen">Bitte wählen</option>
				<option value="ja">ja</option>
				<option value="nein">nein</option>
				<option value="vielleicht">vielleicht</option>
				<option value="war schon">war schon</option>
			</select>
			<br>
		<span class="pflichtfeld">		</td>
	</tr>
	<tr>
		<td>&nbsp;&nbsp;Anregungen/Wünsche/Sonstiges<br>
			&nbsp;&nbsp;<span class="hilfetext">(Optional)</span>
		</td>
		<td><!--<span class="nachrichtlength">min. 10 Zeichen</span>--></td>
		<td colspan="2"><textarea rows="5" name="nachricht" class="nachricht"></textarea></td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Spam-Schutz</td>
		<td colspan="3">
			<?php echo $Spam;?>
			<input type='number' name='zip' size='4' autocomplete='off'>
			<input type='hidden' name='zip2' value='<?php echo $Schutz;?>'>			<br>
			<span class="pflichtfeld">Bitte lösen Sie die Rechenaufgabe richtig!</span>




		</td>
	</tr>
	<tr>
		<td colspan="2" class="right"><input type="submit" id="login" name="senden" value="Senden" /></td>
		<td colspan="2" class="left"><input type="reset" id="zuruecksetzen" value="Zurücksetzen" /></td>
	</tr>
</table>
</form>

Alles anzeigen

flo96 · 23. Dezember 2013

1. Wiederum Danke für die Hilfe

2. Gibt es noch ein kleines Problemchen mit dem Rechencaptcha, da schon beim öffnen der Umfrage die Fehlermeldung kommt...

3. Sollte dann die Benutzereingabe per Mail übertragen werden, wenn alles richtig ist.

Bitte um kleine Hilfe

cottton · 24. Dezember 2013

2. dann das hier:

PHP

/*rechenaufgabe checken
    $_POST['zip'] ist die benutzer eingabe -- seine lösung (eingabefeld name="zip")
    $_POST['zip2'] ist der md5 hash der richtigen lösung (hiddefield name="zip2")
        es muss nun also die benutzereingabe (des benutzers lösung)
        in einen md5 hash gewandelt werden um zu prüfen, ob beide hashes gleich sind.
    */
    if(md5($_POST['zip']) != $_POST['zip2']){
        echo 'Die Rechenaufgabe ist leider falsch!';
    }

mit in die if Abfrage

PHP

if(isset($_POST['senden']))

damit es nur beim Senden des Formulars geprüft wird.

Alternativ geht auch:

PHP

if(isset($_POST['senden'] AND md5($_POST['zip']) != $_POST['zip2']){
        echo 'Die Rechenaufgabe ist leider falsch!';
    }

3.
Naja, Du musst nun prüfen, ob ein Fehler vorkam und darauf hin handeln.
Also wenn kein Feher vorkam, dann kannst Du per mail() (http://www.php.net/manual/de/function.mail.php) die eMail senden.
Falls doch ein Fehler vorkam gibst Du dem Benutzer den Fehler aus.

Es gibt viele Wege zum Ziel. Du kannst zB:

PHP

$fehler_sammlung = array(); # ist ein leeres array




$x = isErr($geschlecht,array($leeres_textfeld),'Sie haben kein Geschlecht gewählt!');
if($x != ''){
 $fehler_sammlung[] = $x;
}
# ...




# und am ende prüfst du ob in $fehler_sammlung ein fehler ist ...
if(!empty($fehler_sammlung)){
 # es kommen fehler for - also keine eMail
 # und fehler ausgeben natürlich 
 foreach($fehler_sammlung as $fehler){
  echo $fehler.'<br>';
 }
}else{
 # keine fehler - also email schicken
}

Alles anzeigen

flo96 · 24. Dezember 2013

Ich habe es jetzt so gelöst:

PHP

if(isset($_POST['senden']) AND md5($_POST['zip']) != $_POST['zip2']){ 
echo 'Die Rechenaufgabe ist leider falsch!'; 
} 
else 
{ 
$mail = mail($empfaenger, $betreff, $text . "\n" . 'Content-Type:text/plain; charset="UTF-8"'); 
}

Allerdings wird bei jedem Öffnen der Umfrage eine mail versendet....

muss ich dass dann mit

PHP

elseif(isset($_POST['senden'])) 
{ 
mail(....); 
}

lösen?

flo96 · 24. Dezember 2013

ok hat sich erledigt... muss ich mit elseif machen

Jetzt habe ich es so gelöst:

PHP

<?php
    error_reporting(E_ALL); /*testphase alle fehler/warnings ect ausgeben*/
    function isPost($offset){
        if(isset($_POST[$offset])){ /*wenn $_POST['xyz'] gesetzt ist*/
            return $_POST[$offset]; /*dann gibt $_POST['xyz'] zurück*/
        }
        return ''; /*sonst gibt leeren string zurück*/
    }
    //Variablen deklarieren um gesendete Objekte einfacher zu packen
    $geschlecht = isPost('geschlecht');
    $alter = isPost('altersgruppe');
    $fund = isPost('fund');
    $gekegelt = isPost('gekegelt');
    $meinung = isPost('meinung');
    $meinung = (is_array($meinung))?implode(', ',$meinung):$meinung;
    $gastwar = isPost('gastwar');
    $gastsein = isPost('gastsein');
    $nachricht = isPost('nachricht');




    if(isset($_POST['senden'])){ /*wenn button senden*/
        /*bestimmen was nicht akzeptiert wird*/
        $leeres_dropdown_menu = 'Bitte wählen';
        $leeres_textfeld = '';
        /*die function zum prüfen*/
        function isErr($val,$not=array(),$err){
            if(in_array($val,$not)){ /*wenn im array $not $val enthalten ist (siehe in_array() @php.net)*/
                return $err.'<br>'; /*dann gib den error $err zurück*/
            }
            return ''; /*sonst gibt leeren string zurück*/
        }
        /*verwendung:
        echo isErr(
            $dasZuPrüfende,
            array($dasSollNichtVorkommen, $undDasAuchNich),
            'derAusgegebeneStringFallsEinesDavonVorkommt'
        );
        Bsp:
        echo isErr($alter,array($leeres_textfeld,$leeres_dropdown_menu),'Bitte geben Sie Ihre Altersgruppe an!');
        - es wird entweder die fehlermeldung 'Bitte geben Sie Ihre Altersgruppe an!', oder nichts (also '') ausgegeben
        - $alter wird geprüft
        - array($leeres_textfeld,$leeres_dropdown_menu) -> wir wollen kein leeres textfeld und auch kein 'Bitte wählen'
        natürlich kannst du auch das ergebnis in einer var packen:
        $fehler_soundso = isErr($alter,array($leeres_textfeld,$leeres_dropdown_menu),'Bitte geben Sie Ihre Altersgruppe an!');
        */







    }
 $empfaenger = "umfrage@dietotenkoepfe.de";
 $betreff = "Eine neue Teilnahme an der Umfrage";
 $text = "
    Geschlecht: ".$geschlecht."
    Altersgruppe: ".$alter."
    Gefunden über: ".$fund."
    Schon mal gekegelt: ".$gekegelt."
    Website: ".$meinung = (is_array($meinung))?implode(', ',$meinung):$meinung."
    Gastkegler gewesen: ".$gastwar."
    Gast sein: ".$gastsein."
    Anregungen/Wünsche/Sonstiges: ".$nachricht."
   ";

    /*rechenaufgabe checken
    $_POST['zip'] ist die benutzer eingabe -- seine lösung (eingabefeld name="zip")
    $_POST['zip2'] ist der md5 hash der richtigen lösung (hiddefield name="zip2")
        es muss nun also die benutzereingabe (des benutzers lösung)
        in einen md5 hash gewandelt werden um zu prüfen, ob beide hashes gleich sind.
    */

 if(isset($_POST['senden']) AND $geschlecht =="")
 {
  echo isErr($geschlecht,array($leeres_textfeld),'Sie haben kein Geschlecht gewählt!');
 }
 if(isset($_POST['senden']) AND $alter =="Bitte wählen")
 {
  echo isErr($alter,array($leeres_textfeld,$leeres_dropdown_menu),'Bitte geben Sie Ihre Altersgruppe an!');
 }
 if(isset($_POST['senden']) AND $fund =="Bitte wählen")
 {
  echo isErr($fund,array($leeres_textfeld,$leeres_dropdown_menu),'Bitte geben Sie an, wie Sie uns gefunden haben!');
 }
 if(isset($_POST['senden']) AND $gekegelt =="")
 {
  echo isErr($gekegelt,array($leeres_textfeld),'Sie haben nicht gewählt, ob Sie schon einmal bei uns gekegelt haben!');
 }
 if(isset($_POST['senden']) AND $meinung = (is_array($meinung))?implode(', ',$meinung):$meinung)
 {
  echo isErr($meinung,array($leeres_textfeld),'Bitte sagen Sie uns Ihre Meinung!');
 }
 if(isset($_POST['senden']) AND $gastwar =="")
 {
  echo isErr($gastwar,array($leeres_textfeld),'Bitte sagen Sie uns, ob Sie einmal Gastkegler gewesen sind!');
 }
 if(isset($_POST['senden']) AND $gastsein =="Bitte wählen")
 {
  echo isErr($gastsein,array($leeres_textfeld),'Bitte sagen Sie uns, ob Sie einmal Gastkegler sein wollen!');
 }
 if(isset($_POST['senden']) AND md5($_POST['zip']) != $_POST['zip2'])
 {
        echo 'Die Rechenaufgabe ist leider falsch!';
    }
 elseif(isset($_POST['senden']))
 {
  $mail = mail($empfaenger, $betreff, $text . "\n" . 'Content-Type:text/plain; charset="UTF-8"');
 }
 
    /*für erneute verwendung des rechenaufgabe neue zahlen ect generieren*/
    $Z0 = array(mt_rand(1, 9), mt_rand(1, 9)); /*ein array mit zufälligen zahlen von 1-9*/
    $Z1 = max($Z0); /*aus dem array den max wert holen (größte zahl)*/
    $Z2 = min($Z0); /*aus dem array den min wert holen (kleinstee zahl)*/
    $Spam = $Z1 . " + &#" . (48 + $Z2) . ";"; /*der string (die rechenaufgabe) erstellen*/
    $Schutz = md5($Z1 + $Z2); /*den md5 hash erstellen (hash wert der richtigen lösung)*/
?>

Alles anzeigen

Allerdings wird kein Fehler bei der Meinung ausgegeben... Wie muss ich das dann machen?

cottton · 24. Dezember 2013

Die ganzen IFs ab Zeile 73 -> du moppelst doppelt
Über die Funktion isErr() wird geprüft, ob das Feld leer oder "Bitte...." ist. Und Du prüfst extra noch in einer IF davor =)

Und die eMail wird immer gesendet, sobald POST "senden" gesetzt ist.
Also auch wenn Fehler vorkommen.

Du musst die Funktion isErr() auch nicht nutzen. War nur ein Bsp wir man Wiederholungen einsparen kann.
Und wie Du im Moment siehst hast Du nun viele Wiederholungen in den IFs ab Zeile 73.

Ausserdem ist in der ersten IF ab Zeile 20 jetzt nur die Funktion drin Das mach keinen Sinn.
Dort waren doch vorher die Fehlerabfragen drin.

Du musst Dir den Code in ruhe ansehen und verstehen. Sonst können konische Sachen passieren und/oder Du siehst irgentwann garnicht mehr durch.
=)

flo96 · 24. Dezember 2013

wie kann ich dass dann machen ohne die if's?

weil wenn kein fehler ist soll erst die mail versendet werden

könnte ich das machen mit

PHP

if(isset($_POST['senden']) AND isErr($gastsein,array($leeres_textfeld)))
{
 echo 'Bitte sagen Sie uns, ob Sie einmal Gastkegler sein wollen!';
}

???

cottton · 24. Dezember 2013

Nein. Ich denke wir gehen das Ganze mal ohne functions an.
So könnte das Ganze funktionieren:

PHP

<?php
	error_reporting(E_ALL); /*testphase alle fehler/warnings ect ausgeben*/




	if(isset($_POST['senden'])){ /*wenn button "senden" gedrückt*/




		$fehler = array(); /*$fehler ist anfangs ein leeres array*/




		/*eingabefelder ect prüfen*/
		if(isset($_POST['geschlecht'])){
			$geschlecht = $_POST['geschlecht'];
		}else{
			$fehler[] = 'Sie haben kein Geschlecht gewählt!';
		}




		if(isset($_POST['altersgruppe']) and $_POST['altersgruppe'] != 'Bitte wählen'){
			$alter = $_POST['altersgruppe'];
		}else{
			$fehler[] = 'Bitte geben Sie Ihre Altersgruppe an!';
		}




 		if(isset($_POST['fund']) and $_POST['fund'] != 'Bitte wählen'){
			$fund = $_POST['fund'];
		}else{
			$fehler[] = 'Bitte geben Sie an, wie Sie uns gefunden haben!';
		}




		if(isset($_POST['gekegelt'])){
			$gekegelt = $_POST['gekegelt'];
		}else{
			$fehler[] = 'Sie haben nicht gewählt, ob Sie schon einmal bei uns gekegelt haben!';
		}




		if(isset($_POST['meinung'])){
			$meinung = implode(', ',$_POST['meinung']);
		}else{
			$fehler[] = 'Bitte sagen Sie uns Ihre Meinung!';
		}




		if(isset($_POST['gastwar'])){
			$gastwar = $_POST['gastwar'];
		}else{
			$fehler[] = 'Bitte sagen Sie uns, ob Sie einmal Gastkegler gewesen sind!';
		}




		if(isset($_POST['gastsein'])){
			$gastsein = $_POST['gastsein'];
		}else{
			$fehler[] = 'Bitte sagen Sie uns, ob Sie einmal Gastkegler sein wollen!';
		}




		/*rechenaufgabe checken
		$_POST['zip'] ist die benutzer eingabe -- seine lösung (eingabefeld name="zip")
		$_POST['zip2'] ist der md5 hash der richtigen lösung (hiddefield name="zip2")
			es muss nun also die benutzereingabe (des benutzers lösung)
			in einen md5 hash gewandelt werden um zu prüfen, ob beide hashes gleich sind.
		*/
		if(md5($_POST['zip']) != $_POST['zip2']){
			$fehler[] = 'Die Rechenaufgabe ist leider falsch!';
		}







		/*fehler vorhanden?*/
		if(empty($fehler)){
			/*keine fehler aufgetaucht*/
			$mail = mail($empfaenger, $betreff, $text . "\n" . 'Content-Type:text/plain; charset="UTF-8"');
		}else{
			/*es gibt fehler -> fehler ausgeben*/
			foreach($fehler as $err){
				echo $err.'<br>';
			}
		}
	} /*if(isset($_POST['senden'])) -ENDE*/







	/*für erneute verwendung des rechenaufgabe neue zahlen ect generieren*/
	$Z0 = array(mt_rand(1, 9), mt_rand(1, 9)); /*ein array mit zufälligen zahlen von 1-9*/
	$Z1 = max($Z0); /*aus dem array den max wert holen (größte zahl)*/
	$Z2 = min($Z0); /*aus dem array den min wert holen (kleinstee zahl)*/
	$Spam = $Z1 . " + &#" . (48 + $Z2) . ";"; /*der string (die rechenaufgabe) erstellen*/
	$Schutz = md5($Z1 + $Z2); /*den md5 hash erstellen (hash wert der richtigen lösung)*/
?>










<form action="" method="post">
<table class="mitgliedsantrag" cellspacing="5">
	<tr>
		<th colspan="4">Wir bitten Sie um Ihre Meinung</th>
	</tr>
	<tr>
		<td class="erklaerung" colspan="4">Bitte füllen Sie alle mit <span class="pflichtfeld1">*</span> gekennzeichneten Felder aus (Pflichtfelder)</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Welches Geschlecht haben Sie?</td>
		<td>&nbsp;</td>
		<td><input type="radio" name="geschlecht" value="männlich" />&nbsp;männlich</td>
		<td><input type="radio" name="geschlecht" value="weiblich" />&nbsp;weiblich</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Welcher Altersgruppe gehören Sie an?</td>
		<td>&nbsp;</td>
		<td colspan="2">
			<select name="altersgruppe">
				<option value="Bitte wählen">Bitte wählen</option>
				<option value="12-16">12-16</option>
				<option value="17-21">17-21</option>
				<option value="22-26">22-26</option>
				<option value="27-31">27-31</option>
				<option value="32-45">32-45</option>
				<option value="46-72">46-72</option>
			</select>
			<br>
		<span class="pflichtfeld"></span>
		</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Wie haben Sie uns gefunden?</td>
		<td>&nbsp;</td>
		<td colspan="2">
			<select name="fund">
				<option value="Bitte wählen">Bitte wählen</option>
				<option value="Google">Google</option>
				<option value="Facebook">Facebook</option>
				<option value="Werbung">Werbung</option>
				<option value="Bekannte/Verwandte">Bekannte/Verwandte</option>
				<option value="Mitglieder">Mitglieder</option>
			</select>
			<br>
		<span class="pflichtfeld"></span>
		</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Haben Sie schon einmal gekegelt?</td>
		<td>&nbsp;</td>
		<td><input type="radio" name="gekegelt" value="ja" />&nbsp;ja</td>
		<td><input type="radio" name="gekegelt" value="nein" />&nbsp;nein</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Wie finden Sie unsere Website?
			<br>
			&nbsp;&nbsp;<span class="hilfetext">(Mehrfachnennung mit STRG möglich)</span></td>
		<td>&nbsp;</td>
		<td colspan="2">
			<select name="meinung[]" multiple="multiple">
				<option value="interessant">Interessant</option>
				<option value="informativ">Informativ</option>
				<option value="zu wenig Info's">Zu wenig Informationen</option>
				<option value="mehr Bilder/Impressionen">Mehr Bilder/Impressionen</option>
				<option value="der HAMMER">Der HAMMER</option>
				<option value="verbesserungswürdig">Verbesserungswürdig</option>
			</select>
			<br>
		<span class="pflichtfeld">Bitte sagen Sie uns Ihre Meinung!		</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Waren Sie schon einmal bei uns Gastkegler?</td>
		<td>&nbsp;</td>
		<td><input type="radio" name="gastwar" value="ja" />&nbsp;ja</td>
		<td><input type="radio" name="gastwar" value="nein" />&nbsp;nein</td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Könnten Sie sich vorstellen, bei uns Gastkegler zu sein?</td>
		<td>&nbsp;</td>
		<td colspan="2">
			<select name="gastsein">
				<option value="Bitte wählen">Bitte wählen</option>
				<option value="ja">ja</option>
				<option value="nein">nein</option>
				<option value="vielleicht">vielleicht</option>
				<option value="war schon">war schon</option>
			</select>
			<br>
		<span class="pflichtfeld">		</td>
	</tr>
	<tr>
		<td>&nbsp;&nbsp;Anregungen/Wünsche/Sonstiges<br>
			&nbsp;&nbsp;<span class="hilfetext">(Optional)</span>
		</td>
		<td><!--<span class="nachrichtlength">min. 10 Zeichen</span>--></td>
		<td colspan="2"><textarea rows="5" name="nachricht" class="nachricht"></textarea></td>
	</tr>
	<tr>
		<td><span class="pflichtfeld1">*</span> Spam-Schutz</td>
		<td colspan="3">
			<?php echo $Spam;?>
			<input type='number' name='zip' size='4' autocomplete='off'>
			<input type='hidden' name='zip2' value='<?php echo $Schutz;?>'>			<br>
			<span class="pflichtfeld">Bitte lösen Sie die Rechenaufgabe richtig!</span>




		</td>
	</tr>
	<tr>
		<td colspan="2" class="right"><input type="submit" id="login" name="senden" value="Senden" /></td>
		<td colspan="2" class="left"><input type="reset" id="zuruecksetzen" value="Zurücksetzen" /></td>
	</tr>
</table>
</form>

Alles anzeigen

Aber wie gesagt - lesen und verstehen. Sonst bringt Dir das alles nix.
=)

flo96 · 24. Dezember 2013

Hab ich das jetzt richtig verstanden?

Wenn kein fehler im Array ist, soll er die Benutezreingabe in der Variable für das passende speichern?

cottton · 24. Dezember 2013

Ja, also Du musst halt dort (bei /*keine fehler aufgetaucht*/) Deinen Text für die eMail ect zusammen stellen.

flo96 · 24. Dezember 2013

kann ich aber auch vorher schon in Variablen speichern und da die variablen reinsetzen oder?

so hab ichs ja auch schon vorher gemacht

cottton · 24. Dezember 2013

Ist doch schon drin. $alter, $geschlecht ect - ist doch alles da so wie Du sie benannt hattest =)

flo96 · 26. Dezember 2013

merci für die Hilfe es funktioniert jetzt super und ich hab wieder was dazu gelernt

Umfrage mit php

Jetzt mitmachen!

Teilen