Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [PHP] wyświetlenie formularza kontaktowego
Forum PHP.pl > Forum > Przedszkole
piotrd
19.08.2011, 20:43:04
poproszę o pomoc kogoś bardziej doświadczonego niż ja gdyż trochę pogubiłem się, mam formularz kontaktowy z walidacją i captcha.
nie mogę dojść do rozwiązania takiego aby po wysłaniu formularza bez błędów pojawił się tylko komentarz
[PHP] pobierz, plaintext 
  1. echo '<p>&nbsp;</p><p class="err">Wiadomość wysłana, dziękujemy.</p>';
[PHP] pobierz, plaintext
kod:
[PHP] pobierz, plaintext 
  1. <?php
  2. session_start();
  3.  
  4. $your_email ="kontakt@xxx.pl";
  5.  
  6. $errors = "";
  7. $name = "";
  8. $visitor_email = "";
  9. $user_message = "";
  10.  
  11. if(isset($_POST["submit"])) {
  12. 	$name = $_POST["name"];
  13. 	$visitor_email = $_POST["email"];
  14. 	$user_message = $_POST["message"];
  15.  
  16. 	if(empty($name)||empty($visitor_email)) {
  17. 		$errors .= "Imię i email są polami wymaganymi!";	
  18. 	}
  19.  
  20. 	if(IsInjected($visitor_email)) {
  21. 		$errors .= "Zły email!";
  22. 	}
  23.  
  24. 	if(empty($_SESSION["6_letters_code"] ) || strcasecmp($_SESSION["6_letters_code"], $_POST["6_letters_code"]) != 0) {
  25. 		$errors .= "Przepisz kod poprawnie!";
  26. 	}
  27.  
  28. if(empty($errors)) {
  29. 	$to = $your_email;
  30. 	$subject = "=?UTF-8?B?" . base64_encode("Poland - kontakt") . "?=";
  31. 	$ip = isset($_SERVER["REMOTE_ADDR"]) ? $_SERVER["REMOTE_ADDR"] : "";
  32.  
  33. 	$body = "<html>";	
  34. 	$body .= "<head></head>";	
  35. 	$body .= "<body>";	
  36. 	$body .= "$name ($visitor_email) napisał/ła:" .
  37. 			 "<p>Treść: " . "$user_message</p>" .
  38. 			 "<p>IP: $ip</p>";	
  39. 	$body .= "</body>";	
  40. 	$body .= "</html>";	
  41.  
  42. 	$headers  = "MIME-Version: 1.0\r\n";
  43. 	$headers .= "Content-type: text/html; charset=UTF-8\r\n";
  44. 	$headers .= "From: Poland <kontakt@XXX.pl>\r\n"; 
  45. 	$headers .= "Reply-To: $visitor_email\r\n";
  46.  
  47. 	mail($to, $subject, $body, $headers);
  48.  
  49. 		echo '<p>&nbsp;</p><p class="err">Wiadomość wysłana, dziękujemy.</p>';
  50. 	} else {
  51. 		echo '<p>&nbsp;</p><p class="err">' . nl2br($errors) . '</p>';
  52. 	}
  53. }
  54.  
  55. function IsInjected($str) {
  56. 	$injections = array("(\n+)", "(\r+)", "(\t+)", "(%0A+)", "(%0D+)",  "(%08+)", "(%09+)");
  57. 	$inject = join("|", $injections);
  58. 	$inject = "/$inject/i";
  59.  
  60. 	if(preg_match($inject, $str)) {
  61. 			return true;
  62. 		} else {
  63. 			return false;
  64. 		}
  65. }
  66. ?>
  67. <p><div id="contact_form_errorloc" class="err"></div></p>
  68. <p>&nbsp;</p>
  69. <form method="POST" name="contact_form" action="<?php echo htmlentities($_SERVER["REQUEST_URI"]); ?>"> 
  70. 	<p>
  71. 		<label for="name">Imię i nazwisko:</label><br>
  72. 		<input type="text" name="name" value="" size="50">
  73. 	</p>
  74. 	<p>
  75. 		<label for="email">Twój email:</label><br>
  76. 		<input type="text" name="email" value="" size="50">
  77. 	</p>
  78. 	<p>
  79. 		<label for="message">Treść wiadomości:</label><br>
  80. 		<textarea name="message" rows="10" cols="50"></textarea>
  81. 	</p>
  82. 	<p>
  83. 		<img src="form/captcha_code_file.php?rand=<?php echo rand(10,100); ?>" id="captchaimg"><br>
  84. 		<small>Nie możesz odczytać kodu? <a href="java script: refreshCaptcha();">Wygeneruj nowy.</a></small><br>
  85. 		<label for="message">Wpisz powyższy kod tutaj:</label>
  86. 		<input id="6_letters_code" name="6_letters_code" type="text"><br>
  87. 		<br>
  88. 		<input type="submit" value="wyślij" name="submit">
  89. 	</p>
  90. </form>
[PHP] pobierz, plaintext
nospor
19.08.2011, 20:55:32
W tej chwili wyświetlasz formularz bezwarunkowo.
Ja nie chcesz go wyświetlać, gdy wszystko było ok, to daj warunek
I wszystko ok to nie wyświetlaj się.

No przecież to jest IF, którego już używasz w swoim kodzie
piotrd
19.08.2011, 20:58:04
właśnie przerzuciłem <form> poza if bo gdy dodam go jako warunek else dla if(isset($_POST["submit"])) wtedy nie wykona mi się if(empty($errors))
ot cały paradoks ;(
mortus
19.08.2011, 21:06:16
Masz taki miszmasz w kodzie. Dodaj sobie dodatkową zmienną np. $nadano = false (linia 10). Po wysłaniu maila (linia 48) zmień jej wartość na true, a przed wyświetlaniem formularza (linia 66) użyj warunku if(!$nadano). Nie zapomnij zamknąć tego ifa po wstawieniu formularza:
[PHP] pobierz, plaintext 
  1. if(!$nadano) : ?>
  2. <!-- tutaj formularz i wszystko inne -->
  3. <?php
  4. endif;
  5. ?>
[PHP] pobierz, plaintext
piotrd
19.08.2011, 23:33:12
mortus, posłuchałem Twojej rady, bo jakże nie można się z nią nie zgodzić i napisałem wszystko od nowa, oto gotowy formularz
[PHP] pobierz, plaintext 
  1. <?
  2. session_start();
  3.  
  4. $imie = htmlspecialchars(strip_tags(trim($_POST['imie'])));
  5. $emajl = htmlspecialchars(strip_tags(trim($_POST['emajl'])));
  6. $tresc = htmlspecialchars(strip_tags(trim($_POST['tresc'])));
  7. $ip = isset($_SERVER["REMOTE_ADDR"]) ? $_SERVER["REMOTE_ADDR"] : "brak IP";
  8.  
  9. $temat = "=?UTF-8?B?" . base64_encode("Poland - kontakt") . "?=";
  10.  
  11. $headers  = "MIME-Version: 1.0\r\n";
  12. $headers .= "Content-type: text/html; charset=UTF-8\r\n";
  13. $headers .= "From: Poland <kontakt@xxx.pl>\r\n"; 
  14. $headers .= "Reply-To: $emajl\r\n";
  15.  
  16. $body = "<html>";	
  17. $body .= "<head></head>";	
  18. $body .= "<body>";	
  19. $body .= "$imie ($emajl) napisał/ła:" . "<p>Treść: " . "$tresc</p>" . "<p>IP: $ip</p>";	
  20. $body .= "</body>";	
  21. $body .= "</html>";	
  22.  
  23. if($_POST["send"] == '1') {
  24. 	if (!empty($_POST['imie']) 
  25. 			&& !empty($_POST['emajl']) 
  26. 				&& !empty($_POST['tresc']) 
  27. 					&& !empty($_POST['kod'])
  28. 						&& strcasecmp($_SESSION["kod"], $_POST["kod"]) == 0) {
  29. 		if (mail('kontakt@xxx.pl', $temat, $body, $headers)) {
  30. 			echo '<p>&nbsp;</p>';
  31. 			echo '<p class="err">Mail został pomyślnie wysłany!</p>';
  32. 		} else {
  33. 			echo '<p>&nbsp;</p>';
  34. 			echo '<p class="err">Error #' . rand() . ' :)</p>';
  35. 		}
  36. 	} else {
  37. 		echo '<p>&nbsp;</p>';
  38. 		echo '<p class="err">Wypełnij wszystkie pola! <a href="java script: history.go(-1)">Powrót</a></p>';
  39. 	}
  40. } else {
  41. 	echo '<form action="' . htmlentities($_SERVER["REQUEST_URI"]) . '" method="post">';
  42. 	echo '<input type="hidden" value="1" name="send">';
  43. 	echo '<br>Twoje imię:<br><input type="text" size="50" id="imie" name="imie" value="' . $_POST["imie"] . '">';
  44. 	echo '<br>Twój mail:<br><input type="text" size="50" id="emajl" name="emajl" value="' . $_POST['emajl'] . '">';
  45. 	echo '<br>Treść:<br><textarea id="tresc" name="tresc" rows="10" cols="50"></textarea>';
  46. 	echo '<br><img id="captchaimg" src="./form/captcha.php?rand=' . rand() . '" />';
  47. 	echo '<br><small>Nie możesz odczytać kodu? <a href="java script: refreshCaptcha();">Wygeneruj nowy kod</a>.</small>';
  48. 	echo '<br><input type="text" id="kod" name="kod" value="">&nbsp;<input type="submit" name="submit" value="wyślij">';
  49. 	echo '</form>';
  50. }
  51. ?>
  52. <script language="JavaScript" type="text/javascript">
  53. 	function refreshCaptcha() {
  54. 		var img = document.images["captchaimg"];
  55. 		img.src = img.src.substring(0,img.src.lastIndexOf("?"))+"?rand="+Math.random()*1000;
  56. 	}
  57. </script>
[PHP] pobierz, plaintext
działa pięknie tylko poprawię zapamiętywanie wartości inputów przy history.go(-1) oraz jakoś dodam walidację po sprawdzeniu $_POST["send"] == '1' a przed mail()

pewnie da się to jeszcze jakoś zoptymalizować ale to będzie na przyszłość.
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.