Forum PHP.pl > [php] walidacja formularza + reszta kodu

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [php] walidacja formularza + reszta kodu

kononowicz

14.01.2007, 04:51:19

Mam problem z połaczeniem kodu z walidacją formularza z kodem zapisujacym dane do pliku txt. Nie moge zrobic aby jezeli walidacja przeszła dobrze to ma zapisac dane do pliku. Walidację nie napisałem sam dlatego mam problem... ale nawet gdybym jakos napisał sam to też pewnie coś byloby nie tak

walidacja:

[PHP] pobierz, plaintext 
<?php
if (isset($_POST['submit']))
		{
	if ($_POST['persona']!='')
				 {
		if (!preg_match('/^[A-ZŁ]([a-ząćęłńóśĽż])+ ([A-ZŁ]([a-ząćęłńóśĽż])+)$$/', $_POST['persona']))
						$personaWalidacja = '<div id=walidacja>Podano nieprawodłowe dane. Format: "Imię Nazwisko"</div>';
			 }
				else
				$personaWalidacja = '<div id=walidacja>Nie podano Imienia i Nazwiska</div>';
 
	if (trim($_POST['temat'])=='')
				$tematWalidacja = '<div id=walidacja>Nie wpisano tematu</div>';
 
	if (trim($_POST['tresc'])=='')
				$trescWalidacja = '<div id=walidacja>Nie wpisano treści wiadomości</div>';
 
		if ($_POST['email']!='')
				 {
		 if (!preg_match('/^(([a-z0-9_]+)-*.?)+@[a-z0-9](([a-z0-9]+)-?_?.?)+([a-z]{2,})$/', $_POST['email']))
						 $emailWalidacja  = '<div id=walidacja>Podano nieprawodłowy adres e-mail</div>';
					 }
						 else
						 $emailWalidacja = '<div id=walidacja>Nie podano adresu e-mail</div>';
						 
						 
	   }
?>
[PHP] pobierz, plaintext

zapisanie wiadomosci z formularza do pliku

[PHP] pobierz, plaintext 
<?php
function kontakt($ip, $email, $temat, $tresc, $persona)
			{
												$ip = $REMOTE_ADDR;
												$data = date('d-m-Y');
								$godz = date('H:i:s');
								$dane = "n<b>data:</b> $data <br>
												<b>godzina:</b> $godz<br>n
												<b>e-mail:</b> $email<br>n
												<b>Imie i Nazwisko:</b> $persona<br>n
												<b>IP:</b> $ip<br>n
												<b>temat:</b> $temat<br>n
												<b>treść:</b> $tresc<br><br>n
												n<img src=images/podzialka.jpg><br><br>n";
								$plikAdr = "wiadomosci/wiadomosci.txt";
								$plik = @fopen($plikAdr, "a");
								flock($plik, 2);
								fwrite($plik, $dane);
								flock($plik, 3);
								fclose($plik);
								print "<br><div id=uwaga>Pytanie zostało wysłane,<br>dziękuję</div>";
			}
 
						kontakt($ip, $email, $temat, $tresc, $persona);
?>
[PHP] pobierz, plaintext

sam formularz

[PHP] pobierz, plaintext 
<?php
print'   <br>
	<font size=4><img src="images/strzalka.jpg"> formularz </font><br>
	<br>
	<center>
 
	<div id="uwaga">  Aby zadać pytanie wyślij formularz<br>
	  odpowiedĽ prześlemy mailem <BR></div>
	  <BR><img src="images/form.jpg">
	  <form action="formularz.php" method=post  id="formform">
			<p>
			<label for="temat">temat:</label>
			<input type=text name=temat class="tekst">
			'; if ($tematWalidacja) echo ($tematWalidacja);
		   print' </p>
			<p>
			<label for="persona">Imie i Nazwisko</label>
			<input type=text name=persona class="tekst">
			'; if ($personaWalidacja) echo ($personaWalidacja);
		   print' </p>
			<p>
			<label for="email">e-mail:</label>
			<input type=text name=email class="tekst">
			'; if ($emailWalidacja) echo ($emailWalidacja);
		   print' </p>
			<p><br>
			<label for="tresc">treść:</label><br>
			<textarea name="tresc" onKeyUp="limit(this,255);" onKeyDown="limit(this,200);"></textarea>
			'; if ($trescWalidacja) echo ($trescWalidacja);
			print'</p>
			 <p>
			 <input type=reset value="wyczyść" name="reset" class="przycisk">
			 <input type=submit value="wyślij" name="submit" class="przycisk">
			  </p>
	  </form>
	</center>';
?>
[PHP] pobierz, plaintext

jak podczepic wszystko do siebie ,żeby było jedną całościa?

? ps. jest godzina 4:50 w nocy wiec troche juz nad tym mysle

Cienki1980

14.01.2007, 10:06:13

Po pierwsze primo: po co przekazujesz do funkcji kontakt zmienną $ip, skoro i tak pobierasz ją w funkcji

Po drugie primo: w wywołaniu funkcji kontakt zmień $tresc na $_POST['tresc'] i analogicznie tak wszystkie zmienne przekazywane do funkcji.

Po trzecie primo: w walidacji brakuje czegoś co będzie pokazywało, czy możesz dodać wpis, bo jak narazie po prostu sprawdzasz czy poszczególne części są OK.... ale brakuje jakiegoś if'a który ostatecznie sprawdzi czy całość jest OK i doda wpis.

Po czwarte primo: jak poprawisz powyższe punkty, sklej te kawałki kodu razem w jednym pliku w kolejności:

- kod funkcji ( chociaż on może być w dowolnym miejscu )
- kod walidacji
- kod forumlarza.

Ja tak zrobiłem i u mnie zadziałało.

kononowicz

14.01.2007, 14:44:38

Cytat(Cienki1980 @ 14.01.2007, 10:06:13 )

Po trzecie primo: w walidacji brakuje czegoś co będzie pokazywało, czy możesz dodać wpis, bo jak narazie po prostu sprawdzasz czy poszczególne części są OK.... ale brakuje jakiegoś if'a który ostatecznie sprawdzi czy całość jest OK i doda wpis.

no własnie jak powinna wyglądać ta pętla

Cytat(Cienki1980 @ 14.01.2007, 10:06:13 )

Po drugie primo: w wywołaniu funkcji kontakt zmień $tresc na $_POST['tresc'] i analogicznie tak wszystkie zmienne przekazywane do funkcji.

[PHP] pobierz, plaintext 
<?php
kontakt($_POST['email'], $_POST['temat'], $_POST['tresc'], $_POST['persona']);
?>
[PHP] pobierz, plaintext

o to chodzi

Cienki1980

14.01.2007, 14:51:57

Tam nie ma żadnej pętli ... tylko same if'y.

Możesz dodać sobie na początku jakąś zmienną ... np $check=0;

W przypadku gdy coś jest nie tak, wypisujesz komunikat ... a jeżeli jest dobrze to zwiększ $check o 1.

Na końcu sprawdź czy $check ma odpowiednią wartość ... ( podpowiem że powinna być równa 4 - zwiększenie o jeden na każde pole formularza ) ... i albo wykonujesz funkcję kontakt() albo nie.

EDIT: wywołanie funkcji poprawiłeś dobrze.

kononowicz

14.01.2007, 15:25:40

dzieki... oto cały kod do walidacji danych z formularza... nie działa mi jedynie wyswietlanie ip :/ (ps. musiałem zrobic jak byłoczyli wstawic w argument funkcji $ip)

[PHP] pobierz, plaintext 
<?php
$personaWalidacja = null;
$adresWalidacja = null;
$tematWalidacja = null;
$trescWalidacja = null;
$emailWalidacja = null;
 
function kontakt($email, $temat, $tresc, $persona)
			{
												$ip = $REMOTE_ADDR;
												$data = date('d-m-Y');
								$godz = date('H:i:s');
								$dane = "n<b>data:</b> $data <br>
												<b>godzina:</b> $godz<br>n
												<b>e-mail:</b> $email<br>n
												<b>Imie i Nazwisko:</b> $persona<br>n
												<b>IP:</b> $ip<br>n
												<b>temat:</b> $temat<br>n
												<b>treść:</b> $tresc<br><br>n
												n<img src=images/podzialka.jpg><br><br>n";
								$plikAdr = "wiadomosci/wiadomosci.txt";
								$plik = @fopen($plikAdr, "a");
								flock($plik, 2);
								fwrite($plik, $dane);
								flock($plik, 3);
								fclose($plik);
								print "<br><div id=uwaga>Pytanie zostało wysłane,<br>dziękuję</div>";
			}
 
 
 
if (isset($_POST['submit']))
		{
		$check=0;
		if ($_POST['persona']!='')
				 {
		if (!preg_match('/^[A-ZŁ]([a-ząćęłńóśĽż])+ ([A-ZŁ]([a-ząćęłńóśĽż])+)$$/', $_POST['persona']))
						$personaWalidacja = '<div id=walidacja>Podano nieprawodłowe dane. Format: "Imię Nazwisko"</div>';
				else
				$check++;
			 }
		else
		$personaWalidacja = '<div id=walidacja>Nie podano Imienia i Nazwiska</div>';
 
 
 
 
	if (trim($_POST['temat'])=='')
				$tematWalidacja = '<div id=walidacja>Nie wpisano tematu</div>';
		else
				$check++;
 
	if (trim($_POST['tresc'])=='')
				$trescWalidacja = '<div id=walidacja>Nie wpisano treści wiadomości</div>';
		else
				$check++;
 
		if ($_POST['email']!='')
				 {
		 if (!preg_match('/^(([a-z0-9_]+)-*.?)+@[a-z0-9](([a-z0-9]+)-?_?.?)+([a-z]{2,})$/', $_POST['email']))
						 $emailWalidacja  = '<div id=walidacja>Podano nieprawodłowy adres e-mail</div>';
				else
						 $check++;
			}
		else
		$emailWalidacja = '<div id=walidacja>Nie podano adresu e-mail</div>';
 
						 
	   }
 
if($check==4)
		kontakt($_POST['email'], $_POST['temat'], $_POST['tresc'], $_POST['persona']);
 
 
 
  print'   <br>
	<font size=4><img src="images/strzalka.jpg"> formularz </font><br>
	<br>
	<center>
 
	<div id="uwaga">  Aby zadać pytanie wyślij formularz<br>
	  odpowiedĽ prześlemy mailem <BR></div>
	  <BR><img src="images/form.jpg">
	  <form action="formularz.php" method=post  id="formform">
			<p>
			<label for="temat">temat:</label>
			<input type=text name=temat class="tekst">
			'; if ($tematWalidacja) echo ($tematWalidacja);
		   print' </p>
			<p>
			<label for="persona">Imie i Nazwisko</label>
			<input type=text name=persona class="tekst">
			'; if ($personaWalidacja) echo ($personaWalidacja);
		   print' </p>
			<p>
			<label for="email">e-mail:</label>
			<input type=text name=email class="tekst">
			'; if ($emailWalidacja) echo ($emailWalidacja);
		   print' </p>
			<p><br>
			<label for="tresc">treść:</label><br>
			<textarea name="tresc" onKeyUp="limit(this,255);" onKeyDown="limit(this,200);"></textarea>
			'; if ($trescWalidacja) echo ($trescWalidacja);
			print'</p>
			 <p>
			 <input type=reset value="wyczyść" name="reset" class="przycisk">
			 <input type=submit value="wyślij" name="submit" class="przycisk">
			  </p>
	  </form>
	</center>';
?>
[PHP] pobierz, plaintext

dr_bonzo

14.01.2007, 15:51:16

Prosze dodac odpowiedni tag to tytulu topika, tak jak stoi w reg. Przedszkola.

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.