[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 (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)


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?(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) ? ps. jest godzina 4:50 w nocy wiec troche juz nad tym mysle (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

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 (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

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 (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

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 (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

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