[php] walidacja formularza + reszta kodu, jak to połączyć?? Opcje

[kononowicz]

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

Ten post edytował kononowicz 14.01.2007, 16:13:23

[Cienki1980]

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]

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

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

Ten post edytował kononowicz 14.01.2007, 14:49:40

[Cienki1980]

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.

Ten post edytował Cienki1980 14.01.2007, 14:52:49

[kononowicz]

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 

Ten post edytował kononowicz 14.01.2007, 16:19:48

[dr_bonzo]

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