Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [PHP]Walidacja formularza
rossco
post
Post #1





Grupa: Zarejestrowani
Postów: 72
Pomógł: 0
Dołączył: 17.02.2007

Ostrzeżenie: (0%)
-----

Hejka,

jak zrobić walidacje formularza tak żeby po za komunikatem typu 'nie wypełniony np nick' zaznaczał pola nie wypełnione zmieniając czcionkę danego pola na czerwono.
Wtedy wystarczył by jeden komunikat 'proszę uzupełnić pola zaznaczone na czerwono'

W tej chwili mój formularz wygląda następująco:

[PHP] pobierz, plaintext 
  1. <HTML>
  2.  <HEAD>
  3.  <META HTTP-EQUIV="Content-Type" CONTENT="text/html; charset=utf-8">
  4.  <TITLE>System Ogłoszeń</TITLE>
  5.  </HEAD>
  6.  <BODY>
  7.  <H2>Ogłoszenia</H2>
  8.  <?php
  9.  mysql_connect ("*****", "osobliwi5", "*****") or //staramy połączyć się z bazą danych
  10.  die ("Nie można połączyć się z bazą MySQL"); //jeżeli to się nie uda wyświetlamy
  11.  mysql_select_db ("*****") or //wybieramy bazę
  12.  die ("Nie można połączyć się z bazą ogloszenia");
  13. mysql_query("SET NAMES utf-8");
  14. mysql_query("SET NAMES utf8");
  15. mysql_query("SET CHARACTER SET utf8");
  16. mysql_query("SET collation_connection = utf8_polish_ci");
  17. $id=$_GET["id"];
  18. $tresc = $_POST["tresc"];
  19. $stanowisko = $_POST["stanowisko"];
  20. $nazwa_firmy = $_POST["nazwa_firmy"];
  21. $miejscowosc_zatrudnienia = $_POST["miejscowosc_zatrudnienia"];
  22.  
  23.  if(!isset($_GET["id"])) { //odbieramy wartość zmiennej id
  24.  print"<A HREF=\"index.php?id=nowy\">Dodaj nowe Ogłoszenie</A><HR>"; //jeżeli zmienna id nie ma przypisanej wartości
  25.  $wynik = mysql_query ("SELECT * FROM ogloszenia WHERE zatwierdz=1 order by nr desc;"); //wybieramy zatwierdzone ogłoszenia i segregujemy według pola nr od wartości największej do najmniejszej
  26.  print"<UL>";
  27.  while ($rekord = mysql_fetch_array ($wynik, MYSQL_NUM)) { //mysql_fetch_array() zwraca nam tablicę indeksowaną numerycznie, gdzie indeks zaczyna się od 0 (MYSQL_NUM)
  28.  print "<LI>".$rekord[2] = stripslashes($rekord[2]).", <B> ".$rekord[1] = stripslashes($rekord[1])."</B>, ".$rekord[3].", ".$rekord[5] = stripslashes($rekord[5]).", ".$rekord[6] = stripslashes($rekord[6])."</LI>"; //w pętli while wyświetlamy po kolei wszytskie ogłoszenia w kolejności treść, stanowisko (stanowisko), data
  29.  }
  30.  print"</UL>";
  31.  }elseif ($_GET["id"]=='nowy') { //jeżeli zmienna id zawiera wyraz nowy
  32.  print "<HR>"; //wyświetlamy formularz do wpisania ogłoszenia
  33.  print "<H3>Nowe ogłoszenie:</H3>";
  34.  print "<FORM METHOD=POST action=\"index.php?id=dodaj\"><INPUT TYPE=\"hidden\" NAME=\"id\" VALUE=\"dodaj\">";
  35.  print "<B>Nazwa firmy:</B><BR><INPUT TYPE=\"text\" NAME=\"nazwa_firmy\" VALUE=\"$nazwa_firmy\" SIZE=60><BR>";
  36.  print "<B>Miejscowość zatrudnienia:</B><BR><INPUT TYPE=\"text\" NAME=\"miejscowosc_zatrudnienia\" VALUE=\"$miejscowosc_zatrudnienia\" SIZE=60><BR>";
  37.  print "<B>Stanowisko:</B><BR><INPUT TYPE=\"text\" NAME=\"stanowisko\" VALUE=\"$stanowisko\" SIZE=60><BR>";
  38.  print "<B>Treść ogłoszenia:</B><BR><TEXTAREA NAME=\"tresc\" ";
  39.  print "ROWS=10 COLS=60>$tresc</TEXTAREA><BR>";
  40.  print "<INPUT TYPE=\"submit\" VALUE=\"Wyslij\">";
  41.  print "</FORM>";
  42.  print "<a href=\"index.php\">Powrót</a>";
  43.  }elseif ($_GET["id"]=='dodaj') { // jeżeli zmienna id zawiera wartość dodaj
  44.  //dodajemy ogłoszenie do bazy
  45.  $tresc = addslashes(nl2br(htmlentities ($tresc))); //ponieważ nie można ufać danym wprowadzanym przez użytkowników funkcją addslashes dodajemy ukośniki () w miejscach gdzie napotkanym na apostrof (‘), usuwamy je potem funkcją stripslahes przy wypisywaniu komentarzy, dodatkowo funkcja nl2br zamienia znaki końca linii na tag br
  46.  $stanowisko = addslashes(htmlentities ($stanowisko));
  47.  
  48.  if ($tresc = $_POST["tresc"] && $stanowisko = $_POST["stanowisko"]) { //jeżeli pola treść i stanowisko są uzupełnione
  49.  $zapytanie = "INSERT INTO ogloszenia (tresc, stanowisko, nazwa_firmy, miejscowosc_zatrudnienia, "." data) VALUES ('$tresc', '$stanowisko', '$nazwa_firmy', '$miejscowosc_zatrudnienia', "." now());";
  50.  $wynik = mysql_query ($zapytanie); //dodajemy wtedy dane do bazy wraz z aktualną datą i godziną – funkcja now()
  51.  print "<HR>";
  52.  print"<H4>Twoje ogłoszenie zostało dodane do bazy. Ukaże się, gdy administrator je zatwierdzi</h4><br>"; //jeżeli wszystko przebiegnie bez problemów wyświetlamy komunikat
  53.  print"<a href=\"index.php\">Powrót</a>";
  54.  }else{ //jeżeli jednak nie wyświetlamy stosowną informację
  55.  print "<HR>";
  56.  print"<h4>Pola stanowisko i treść muszą być wypełnione ! Kliknij wstecz w swojej przeglądarce i popraw te dane !</h4>";
  57.  }
  58.  }else { //jeżeli zmienna id zawiera inną wartość niż dodaj lub nowy wyświetlamy wszystkie zatwierdzone ogłoszenia
  59.  print"<A HREF=\"index.php?id=nowy\">Dodaj nowe ogłoszenia</A><HR>";
  60.  $wynik = mysql_query ("SELECT * FROM ogloszenia WHERE zatwierdz=1 order by nr desc;"); //wybieramy zatwierdzone ogłoszenia i segregujemy według pola nr od wartości największej do najmniejszej
  61.  print"<UL>";
  62.  while ($rekord = mysql_fetch_array ($wynik, MYSQL_NUM)) { //mysql_fetch_array() zwraca nam tablicę indeksowaną numerycznie, gdzie indeks zaczyna się od 0 (MYSQL_NUM)
  63.  print "<LI>".$rekord[2] = stripslashes($rekord[2]).", <B> ".$rekord[1] = stripslashes($rekord[1])."</B>, ".$rekord[3].", ".$rekord[5] = stripslashes($rekord[5])."</LI>"; //za pomocą pętli while wyświetlamy wszystkie rekordy, które zostały zatwierdzone (wartość pola zatwierdz wynosi 1)
  64.  }
  65.  print"</UL>";
  66.  }
  67.  ?>
  68.  </BODY>
  69.  </HTML>
[PHP] pobierz, plaintext
Go to the top of the page
+Quote Post
2 Stron V   1 2 >  
 Start new topic
Odpowiedzi (1 - 19)
MateuszS
post
Post #2





Grupa: Zarejestrowani
Postów: 1 429
Pomógł: 195
Dołączył: 6.10.2008
Skąd: Kraków/Tomaszów Lubelski

Ostrzeżenie: (0%)
-----

Odpowiedni warunek if z empty + 

[JAVASCRIPT] pobierz, plaintext 
  1. this.style.color = "red";
[JAVASCRIPT] pobierz, plaintext




--------------------
O! Zimniok :P
Go to the top of the page
+Quote Post
Pawel_W
post
Post #3





Grupa: Zarejestrowani
Postów: 1 675
Pomógł: 286
Dołączył: 15.06.2009
Skąd: Wieliczka

Ostrzeżenie: (0%)
-----

Cytat(MateuszScirka @ 21.03.2010, 16:09:32 ) *
Odpowiedni warunek if z empty + 

[JAVASCRIPT] pobierz, plaintext 
  1. this.style.color = "red";
[JAVASCRIPT] pobierz, plaintext

to jest javascript smile.gif a formularz waliduje się po stronie serwera

najprościej będzie nadać mu jakąś klasę np. zle i potem w css przypisać jej czerwony kolor
Go to the top of the page
+Quote Post
rossco
post
Post #4





Grupa: Zarejestrowani
Postów: 72
Pomógł: 0
Dołączył: 17.02.2007

Ostrzeżenie: (0%)
-----

a możecie mi to wytłumaczyć na przykładzie jeśli będziecie na tylu uprzejmi smile.gif
Go to the top of the page
+Quote Post
MateuszS
post
Post #5





Grupa: Zarejestrowani
Postów: 1 429
Pomógł: 195
Dołączył: 6.10.2008
Skąd: Kraków/Tomaszów Lubelski

Ostrzeżenie: (0%)
-----

Nom o to mi chodziło, można zrobić to na początku w JS lub potem w PHP, no jak kto woli, albo najlepiej tak i tak.

Rossco: choćby i tak:

[PHP] pobierz, plaintext 
  1. <?
  2. if(isset($_POST["nick"])) {
  3. 	if(empty($_POST["nick"]))
  4. 	{
  5. 		$border="border: 1px solid red";
  6. 	}
  7. }
  8. ?>
  9. <form action="" method="POST">
  10. <input type="text" id="nick" name="nick" style="<? echo $border; ?>" />
  11. <br /><input type="submit" value="wyslij" />
  12. </form>
  13.  
  14.  
  15.  
[PHP] pobierz, plaintext


Ten post edytował MateuszScirka 21.03.2010, 16:29:01


--------------------
O! Zimniok :P
Go to the top of the page
+Quote Post
rossco
post
Post #6





Grupa: Zarejestrowani
Postów: 72
Pomógł: 0
Dołączył: 17.02.2007

Ostrzeżenie: (0%)
-----

jak uważacie czy ja nie powinnam wywalić formularz z kodu php? jeśli tak to na co uważać żeby nie przestał działać?
Go to the top of the page
+Quote Post
MateuszS
post
Post #7





Grupa: Zarejestrowani
Postów: 1 429
Pomógł: 195
Dołączył: 6.10.2008
Skąd: Kraków/Tomaszów Lubelski

Ostrzeżenie: (0%)
-----

Możesz zostawić, możesz wywalić, osobiście skłaniam się ku temu drugiemu wariantowi, tzn. ja bym wywalił, trochę to nieestetycznie wygląda, taka masa printów, nie widać kolorowania składni, można gdzieś się pomylić no i pewnie są jeszcze jakieś inne ważniejsze argumenty, uważaj przy tym żeby zamknąć tag php przed kodem HTML i otworzyć po tongue.gif.


--------------------
O! Zimniok :P
Go to the top of the page
+Quote Post
rossco
post
Post #8





Grupa: Zarejestrowani
Postów: 72
Pomógł: 0
Dołączył: 17.02.2007

Ostrzeżenie: (0%)
-----

jestem początkująca ale też mi się wydaje że powinnam wywalić formularz z php. Tylko on jest wrzucony w IF-a ($_GET["id"]=='nowy') i nie wiem za bardzo jak do tego się zabrać.
Go to the top of the page
+Quote Post
MateuszS
post
Post #9





Grupa: Zarejestrowani
Postów: 1 429
Pomógł: 195
Dołączył: 6.10.2008
Skąd: Kraków/Tomaszów Lubelski

Ostrzeżenie: (0%)
-----

To nic że jest w ifie, będzie działało. Warunek zostanie zachowany


--------------------
O! Zimniok :P
Go to the top of the page
+Quote Post
mortus
post
Post #10





Grupa: Zarejestrowani
Postów: 2 178
Pomógł: 596
Dołączył: 25.09.2009
Skąd: Piwniczna-Zdrój

Ostrzeżenie: (0%)
-----

Niestety u Ciebie nie da się zrobić takiej walidacji, jaką chcesz, bo warunki w blokach if...elseif...else Cię nie puszczą. Dokładniej chodzi o to, że nie są w odpowiedniej kolejności. Myślę, że warto przemyśleć jeszcze raz kilka kwestii, np.:
1. kiedy wyświetlany jest formularz,
2. czy potrzebujemy tylu $_GET-ów,
3. co sprawdzamy najpierw (chodzi o to, czy formularz został wysłany, czy może pojawił się pierwszy raz),
4. itp...
Odnośnie wyświetlania formularza, to podpowiem, że powinien on być wyświetlany w którymś z końcowych warunków, albo nawet w zupełnie odrębnym bloku if.
Cytat
jak uważacie czy ja nie powinnam wywalić formularz z kodu php? jeśli tak to na co uważać żeby nie przestał działać?
Kod HTML można przeplatać z kodem php np.
[PHP] pobierz, plaintext 
  1. <?php
  2. if($_GET['id'] == 'nowy') {
  3. ?>
  4.     <form action="..." method="post">
  5.     ...
  6.     </form>
  7. <?php
  8. }
  9. ?>
[PHP] pobierz, plaintext
Możesz też formularz zapisać w osobnym pliku html i użyć funkcji include. Możesz napisać funkcję wstawiającą formularz. Każdy sposób jest dobry.

Ten post edytował mortus 21.03.2010, 17:08:42
Go to the top of the page
+Quote Post
rossco
post
Post #11





Grupa: Zarejestrowani
Postów: 72
Pomógł: 0
Dołączył: 17.02.2007

Ostrzeżenie: (0%)
-----

Zrobiłam tak:

formularz przeniosłam do pliku form.php
[PHP] pobierz, plaintext 
  1. //wyświetlamy formularz do wpisania ogłoszenia
  2. <H3>Nowe ogłoszenie:</H3>
  3. <FORM METHOD=POST action="index.php?id=dodaj"><INPUT TYPE="hidden" NAME="id" VALUE="dodaj">
  4. </B><BR><INPUT TYPE="text" NAME="nazwa_firmy" VALUE="<?$nazwa_firmy?>" SIZE=60><BR>
  5. <B>Miejscowość zatrudnienia:</B><BR><INPUT TYPE="text" NAME="miejscowosc_zatrudnienia" VALUE="<?$miejscowosc_zatrudnienia?>" SIZE=60><BR>
  6. <B>Stanowisko:</B><BR><INPUT TYPE="text" NAME="stanowisko" VALUE="<?$stanowisko?>" SIZE=60><BR>
  7. <B>Treść ogłoszenia:</B><BR><TEXTAREA NAME="tresc" "ROWS=10 COLS=60><?$tresc?></TEXTAREA><BR>
  8. <INPUT TYPE="submit" VALUE="Wyslij">
  9. </FORM>
  10. <a href="index.php">Powrót</a>
[PHP] pobierz, plaintext


Dodałam include

[PHP] pobierz, plaintext 
  1. <HTML>
  2.  <HEAD>
  3.  <META HTTP-EQUIV="Content-Type" CONTENT="text/html; charset=utf-8">
  4.  <TITLE>System Ogłoszeń</TITLE>
  5.  </HEAD>
  6.  <BODY>
  7.  <H2>Ogłoszenia</H2>
  8.  <?php
  9.  mysql_connect ("*****", "osobliwi5", "*****") or //staramy połączyć się z bazą danych
  10.  die ("Nie można połączyć się z bazą MySQL"); //jeżeli to się nie uda wyświetlamy
  11.  mysql_select_db ("*****") or //wybieramy bazę
  12.  die ("Nie można połączyć się z bazą ogloszenia");
  13. mysql_query("SET NAMES utf-8");
  14. mysql_query("SET NAMES utf8");
  15. mysql_query("SET CHARACTER SET utf8");
  16. mysql_query("SET collation_connection = utf8_polish_ci");
  17. $id=$_GET["id"];
  18. $tresc = $_POST["tresc"];
  19. $stanowisko = $_POST["stanowisko"];
  20. $nazwa_firmy = $_POST["nazwa_firmy"];
  21. $miejscowosc_zatrudnienia = $_POST["miejscowosc_zatrudnienia"];
  22.  
  23.  if(!isset($_GET["id"])) { //odbieramy wartość zmiennej id
  24.  print"<A HREF=\"index.php?id=nowy\">Dodaj nowe Ogłoszenie</A><HR>"; //jeżeli zmienna id nie ma przypisanej wartości
  25.  $wynik = mysql_query ("SELECT * FROM ogloszenia WHERE zatwierdz=1 order by nr desc;"); //wybieramy zatwierdzone ogłoszenia i segregujemy według pola nr od wartości największej do najmniejszej
  26.  print"<UL>";
  27.  while ($rekord = mysql_fetch_array ($wynik, MYSQL_NUM)) { //mysql_fetch_array() zwraca nam tablicę indeksowaną numerycznie, gdzie indeks zaczyna się od 0 (MYSQL_NUM)
  28.  print "<LI>".$rekord[2] = stripslashes($rekord[2]).", <B> ".$rekord[1] = stripslashes($rekord[1])."</B>, ".$rekord[3].", ".$rekord[5] = stripslashes($rekord[5]).", ".$rekord[6] = stripslashes($rekord[6])."</LI>"; //w pętli while wyświetlamy po kolei wszytskie ogłoszenia w kolejności treść, stanowisko (stanowisko), data
  29.  }
  30.  
  31. if(isset($_POST["stanowisko"])) {
  32.  
  33. if(empty($_POST["stanowisko"]))
  34.  
  35. {
  36.  
  37. $border="border: 1px solid red";
  38.  
  39. }
  40.  
  41. }
  42. if(isset($_POST["stanowisko"])) {
  43.  
  44. if(empty($_POST["stanowisko"]))
  45.  
  46. {
  47.  
  48. $border="border-top: 1px solid red";
  49.  
  50. }
  51.  
  52. }
  53.  
  54.  print"</UL>";
  55.  }elseif ($_GET["id"]=='nowy') { //jeżeli zmienna id zawiera wyraz nowy
  56.  
  57.  include("form.php");
  58.  
  59.  }elseif ($_GET["id"]=='dodaj') { // jeżeli zmienna id zawiera wartość dodaj
  60.  //dodajemy ogłoszenie do bazy
  61.  $tresc = addslashes(nl2br(htmlentities ($tresc))); //ponieważ nie można ufać danym wprowadzanym przez użytkowników funkcją addslashes dodajemy ukośniki () w miejscach gdzie napotkanym na apostrof (‘), usuwamy je potem funkcją stripslahes przy wypisywaniu komentarzy, dodatkowo funkcja nl2br zamienia znaki końca linii na tag br
  62.  $stanowisko = addslashes(htmlentities ($stanowisko));
  63.  
  64.  if ($tresc = $_POST["tresc"] && $stanowisko = $_POST["stanowisko"]) { //jeżeli pola treść i stanowisko są uzupełnione
  65.  $zapytanie = "INSERT INTO ogloszenia (tresc, stanowisko, nazwa_firmy, miejscowosc_zatrudnienia, "." data) VALUES ('$tresc', '$stanowisko', '$nazwa_firmy', '$miejscowosc_zatrudnienia', "." now());";
  66.  $wynik = mysql_query ($zapytanie); //dodajemy wtedy dane do bazy wraz z aktualną datą i godziną – funkcja now()
  67.  print "<HR>";
  68.  print"<H4>Twoje ogłoszenie zostało dodane do bazy. Ukaże się, gdy administrator je zatwierdzi</h4><br>"; //jeżeli wszystko przebiegnie bez problemów wyświetlamy komunikat
  69.  print"<a href=\"index.php\">Powrót</a>";
  70.  }else{ //jeżeli jednak nie wyświetlamy stosowną informację
  71.  print "<HR>";
  72.  print"<h4>Pola stanowisko i treść muszą być wypełnione ! Kliknij wstecz w swojej przeglądarce i popraw te dane !</h4>";
  73.  }
  74.  }else { //jeżeli zmienna id zawiera inną wartość niż dodaj lub nowy wyświetlamy wszystkie zatwierdzone ogłoszenia
  75.  print"<A HREF=\"index.php?id=nowy\">Dodaj nowe ogłoszenia</A><HR>";
  76.  $wynik = mysql_query ("SELECT * FROM ogloszenia WHERE zatwierdz=1 order by nr desc;"); //wybieramy zatwierdzone ogłoszenia i segregujemy według pola nr od wartości największej do najmniejszej
  77.  print"<UL>";
  78.  while ($rekord = mysql_fetch_array ($wynik, MYSQL_NUM)) { //mysql_fetch_array() zwraca nam tablicę indeksowaną numerycznie, gdzie indeks zaczyna się od 0 (MYSQL_NUM)
  79.  print "<LI>".$rekord[2] = stripslashes($rekord[2]).", <B> ".$rekord[1] = stripslashes($rekord[1])."</B>, ".$rekord[3].", ".$rekord[5] = stripslashes($rekord[5])."</LI>"; //za pomocą pętli while wyświetlamy wszystkie rekordy, które zostały zatwierdzone (wartość pola zatwierdz wynosi 1)
  80.  }
  81.  print"</UL>";
  82.  }
  83.  ?>
  84.  </BODY>
  85.  </HTML>
[PHP] pobierz, plaintext


O dziwo działa smile.gif oceńcie i powiedzcie co powinnam jeszcze zmienić.

I najważniejsze - czy przybliżyłam się chociaż o krok od możliwości zrobienia walidacji jaką opisałam?
Go to the top of the page
+Quote Post
MateuszS
post
Post #12





Grupa: Zarejestrowani
Postów: 1 429
Pomógł: 195
Dołączył: 6.10.2008
Skąd: Kraków/Tomaszów Lubelski

Ostrzeżenie: (0%)
-----

Nie trzeba wszedzie sprawdzac tym issetem ;] Chodzi o to, że sprawdzamy czy formularz został wysłany, żeby od razu nie wyświetlał się ten czerwony border ;] Wystarczy jeden if z issetem który obejmuje cały kod który ma się wykonać po naciśnięciu submita


--------------------
O! Zimniok :P
Go to the top of the page
+Quote Post
rossco
post
Post #13





Grupa: Zarejestrowani
Postów: 72
Pomógł: 0
Dołączył: 17.02.2007

Ostrzeżenie: (0%)
-----


if(isset($_POST["stanowisko"])) {

Tu coś wpisuję?

if(empty($_POST["stanowisko"]))

{

$border="border-top: 1px solid red";

}

}

i co muszę wywalić w obecnym kodzie żeby walidację sobie nie przeszkadzały?
Go to the top of the page
+Quote Post
mortus
post
Post #14





Grupa: Zarejestrowani
Postów: 2 178
Pomógł: 596
Dołączył: 25.09.2009
Skąd: Piwniczna-Zdrój

Ostrzeżenie: (0%)
-----

Cytat(MateuszScirka @ 21.03.2010, 17:42:13 ) *
Nie trzeba wszedzie sprawdzac tym issetem ;] Chodzi o to, że sprawdzamy czy formularz został wysłany, żeby od razu nie wyświetlał się ten czerwony border ;] Wystarczy jeden if z issetem który obejmuje cały kod który ma się wykonać po naciśnięciu submita
Chyba się mylisz, bo skąd będziemy wiedzieć, które pola oznaczyć, jeżeli będzie tylko jeden isset? Przecież nie każde pole trzeba wypełnić i nie wszystkie pola, które musimy wypełnić, muszą być puste po naciśnięciu submita.
@rossco Po pierwsze zwróć uwagę na te $_GET-y. Zauważ, że jeśli wybierzesz opcję dodania nowego ogłoszenia, to w $_GET['id'] masz wartość 'nowy' i wtedy wyświetlasz formularz. Ale gdy przyciśniesz przycisk 'Wyślij', to w $_GET['id'] masz wartość 'dodaj', formularz przechodzi walidację i jeśli nie wszystkie wymagane pola zostały wypełnione to znowu wyświetlasz formularz. Chyba trochę bez sensu, ale przyjmijmy, że tak ma być. A jeśli tak, to skrypt mniej więcej powinien wyglądać tak
[PHP] pobierz, plaintext 
  1. // sprawdzamy, czy użytkownik wcisnął przycisk wyślij
  2. if(!empty($_GET['id']) && $_GET['id'] == 'dodaj' && !empty($_POST)) {
  3.     // formularz został wysłany - walidacja (tutaj na przykładzie pól tresc i stanowisko)
  4.     if(!isset($_POST['tresc']) || $_POST['tresc'] == '') {
  5.         $validation_error = true;
  6.         $pusta_tresc = true;
  7.     }
  8.     if(!isset($_POST['stanowisko']) || $_POST['stanowisko'] == '') {
  9.         $validation_error = true;
  10.         $puste_stanowisko = true;
  11.     }
  12.     // jeżeli zatem wszystkie pola zostały wypełnione to
  13.     if(!validation_error) {
  14.         // pobieramy wszystkie pola z tablicy $_POST i wrzucamy ogłoszenie do bazy danych
  15.     } else {
  16.         // wrzucamy formularz
  17.         include 'form.php';
  18.     }
  19. } elseif(!empty($_GET['id'] && $_GET['id'] == 'nowy')) {
  20.     // wrzucamy formularz
  21.     include 'form.php';
  22. } else {
  23.     // wyświetlamy listę ogłoszeń i udostępniamy możliwość dodania nowego ogłoszenia
  24. }
[PHP] pobierz, plaintext
No i pola tresc i stanowisko w pliku form.php
[PHP] pobierz, plaintext 
  1. <input type="text" name="tresc" value="<?php if(!$pusta_tresc) echo $_POST['tresc']; ?>" <?php if($pusta_tresc) echo 'style="border: 1px solid red;"'; ?> /><br />
  2. <input type="text" name="stanowisko" value="<?php if(!$puste_stanowisko) echo $_POST['tresc']; ?>" <?php if($puste_stanowisko) echo 'style="border: 1px solid red;"'; ?> />
[PHP] pobierz, plaintext
Go to the top of the page
+Quote Post
MateuszS
post
Post #15





Grupa: Zarejestrowani
Postów: 1 429
Pomógł: 195
Dołączył: 6.10.2008
Skąd: Kraków/Tomaszów Lubelski

Ostrzeżenie: (0%)
-----

Mortus, nie mylę się. Isset ma sprawdzać tylko czy formularz został przesłany, równie dobrze może być (powinien) jeden isset na cały kod, który ma się wyk. po przesłaniu formularza. A za to czy pole się zaczerwieni, odpowiada warunek z empty.


--------------------
O! Zimniok :P
Go to the top of the page
+Quote Post
rossco
post
Post #16





Grupa: Zarejestrowani
Postów: 72
Pomógł: 0
Dołączył: 17.02.2007

Ostrzeżenie: (0%)
-----

Mortus ja się dopiero uczę i pewnie dlatego ten kod jest "bez sensu". Skoro tak to nie chce nabrać złych nawyków. Mi się wydaje że ten kod nie wykonuję się w dobrej kolejności. Podpowiedz mi proszę jak najlepiej Ty byś go napisał.
Go to the top of the page
+Quote Post
mortus
post
Post #17





Grupa: Zarejestrowani
Postów: 2 178
Pomógł: 596
Dołączył: 25.09.2009
Skąd: Piwniczna-Zdrój

Ostrzeżenie: (0%)
-----

Cytat(MateuszScirka @ 21.03.2010, 18:38:34 ) *
Mortus, nie mylę się. Isset ma sprawdzać tylko czy formularz został przesłany, równie dobrze może być (powinien) jeden isset na cały kod, który ma się wyk. po przesłaniu formularza. A za to czy pole się zaczerwieni, odpowiada warunek z empty.
Właściwie to nie ma większego znaczenia to w jaki sposób będziemy sprawdzać, czy formularz został wysłany. Zarówno isset, jak i empty doskonale się do tego nadają. Tylko podaj mi źródło, w którym przeczytałeś, że powinien być jeden isset na cały kod. A jak sprawdzisz czy pole, w którym możemy wpisać 0, i które w ostateczności nie może być puste, rzeczywiście nie jest puste? Gwarantuję, że empty nie zadziała tak, jakbyśmy chcieli i musimy użyć isset (nawet dwukrotnie w jednym warunku). Przykładowy kod:
[PHP] pobierz, plaintext 
  1. <?php 
  2. // zakładamy, że pole zmienna w formularzu ma być wypełnione i może być wypełnione 0 (zerem)
  3. if(!empty($_POST['zmienna'])) {
  4. 	echo '$_POST[\'zmienna=\'] is not empty<br/>';
  5. 	echo '<pre>';
  6. 	print_r($_POST);
  7. 	echo '</pre>';
  8. }
  9. ?>
  10. <html>
  11. 	<head>
  12. 		<meta name="content-type" content="text/html; charset=UTF-8" />
  13. 	</head>
  14. 	<body>
  15. 		<form action="<?php $_SERVER['PHP_SELF']; ?>" method="post">
  16. 			<input type="text" name="zmienna" value="Spróbuj wpisać 0" /><br />
  17. 			<input type="submit" value="Zatwierdź" /><br />
  18. 		</form>
  19. 	</body>
  20. </html>
[PHP] pobierz, plaintext
W moim odczuciu empty lepiej się nadaje do sprawdzenia, czy formularz został przesłany. Dlaczego? Dlatego, że tablica $_POST istnieje zawsze i dopóki nie prześlemy czegoś za pomocą metody post, tablica $_POST będzie pusta. Jeżeli natomiast prześlemy puste pole z ustawionym atrybutem name, to tablica $_POST już nie jest pusta, ale $_POST['nazwa_pola'] nadal jest puste i możemy użyć empty, żeby to sprawdzić. Tyle, że w wyżej opisanym przypadku empty nas "oszuka".

@rossco Powyżej Ci podałem jak, to mniej więcej powinno wyglądać. Z tym, że do pierwszego warunku można mieć uwagi. Skoro w formularzu przesyłasz ukryte pole o nazwie id i wartości dodaj, to zamiast
[PHP] pobierz, plaintext 
  1. if(!empty($_GET['id']) && $_GET['id'] == 'dodaj' && !empty($_POST))
[PHP] pobierz, plaintext
możemy użyć tylko
[PHP] pobierz, plaintext 
  1. if(!empty($_POST))
[PHP] pobierz, plaintext
A w akcji formularza nie potrzebujemy index.php?id=dodaj.
Oczywiście to jest tylko jeden ze sposobów rozwiązania kwestii walidacji (takiej, o jaką Tobie chodzi).
Go to the top of the page
+Quote Post
thek
post
Post #18





Grupa: Moderatorzy
Postów: 4 362
Pomógł: 714
Dołączył: 12.02.2009
Skąd: Jak się położę tak leżę :D
Mortus jak widzę czuwa. I ma rację. $_POST to tablica superglobalna i wisi, więc jej wypełnienie musi być sprawdzane występowaniem któregoś z elementów zawartego w niej. Większość osób robi to sprawdzając istnienie submita o określonym name w nim. Ale count jest podstawowym dla mnie gdy mam formularz z kilkoma submitami. Bo przecież nie będę zgadywał który submit wcisnął user lub sprawdzał istnienie wszystkich. Za to zliczę ile mam wpisów w niej winksmiley.jpg Potem switchem mogę modyfikować zachowanie zależnie od submit już, ale sprawdzenie czy właściwie $_POST ma choć jeden element w zupełności wystarczy. Sugestia o isset jest dobra, ale można także używać array_key_exists('name_pola', $_POST) i to rozwiązanie ja stosuję najczęściej, bo nie wywali nam się, podobnie jak isset, na wartości 0, co już mortus zasugerował. Swoja drogą wariacje na temat Dodaj/Edytuj to tylko pierdółka tak naprawdę. Dlaczego? Bo to się realizuje jednym polem hidden. Ustawienie tam 0 = nowy, jakakolwiek liczba = id do edycji. Oczywiście id sprawdzamy po stronie serwera. A musimy sprawdzić czy aby nie brak uprawnień do działań. Wystarczająco się naoglądałem skryptów, gdzie user ma prawo edycji tekstów kogoś innego, bo sobie można spreparować POST lub GET i system nie sprawdza uprawnień winksmiley.jpg Ja takie "akcje" loguję by wiedzieć, który user cwaniakuje.

I ma rację mortus... Sposobów rozwiązania problemu jest więcej niż jedna. Ja osobiście skłaniam się do takiej:
- tworzymy strukturę przechowującą domyślne wartości pól.
- jeśli przechwyciliśmy GET sprawdzamy uprawnienia i zamiast domyślnych wrzucamy pobrane.
- jeśli wyłapujemy POST to walidacja idzie, zapis itp
- dopiero teraz wyświetlamy formularz z polami uzupełnionymi wartościami ze struktury wartości pól.

Czemu tak? Bo to nam kapitalnie upraszcza wszystko, włącznie z komunikatami błędów o formie jaką chcesz zastosować. Wystarczy, że stworzymy drugą strukturę, która przechowuje błędy. Jeśli w czasie walidacji napotkamy błąd, to wrzucamy do niej ów błąd z nazwą pola jako kluczem. A teraz cudujemy... Jeśli owa struktura jest pusta to robi się insert lub update. Jeśli zawiera choć jeden wpis ( count ) to operacje na bazie nie następują, a pod danym polem następuje sprawdzenie czy tablica błędów zawiera wpis o nazwie pola. Tak? No to wyświetlamy ów komunikat smile.gif


--------------------
Najpierw był manual... Jeśli tam nie zawarto słów mądrości to zapytaj wszechwiedzącego Google zadając mu własciwe pytania. A jeśli i on milczy to Twój problem nie istnieje :D
Go to the top of the page
+Quote Post
mortus
post
Post #19





Grupa: Zarejestrowani
Postów: 2 178
Pomógł: 596
Dołączył: 25.09.2009
Skąd: Piwniczna-Zdrój

Ostrzeżenie: (0%)
-----

Cytat(thek @ 22.03.2010, 12:47:08 ) *
Sugestia o isset jest dobra, ale można także używać array_key_exists('name_pola', $_POST) i to rozwiązanie ja stosuję najczęściej, bo nie wywali nam się, podobnie jak isset, na wartości 0, co już mortus zasugerował.
isset na wartości zero się nie wywali, wywali się natomiast, jeżeli nie wpiszemy niczego, dlatego samo sprawdzenie
[PHP] pobierz, plaintext 
  1. if(isset($_POST['jakas_zmienna']))
[PHP] pobierz, plaintext
nie wystarcza i musimy sprawdzać, czy ta zmienna nie jest pustym łańcuchem znaków
[PHP] pobierz, plaintext 
  1. if(isset($_POST['jakas_zmienna']) && $_POST['jakas_zmienna'] != '')
[PHP] pobierz, plaintext
Oczywiście mówimy cały czas o danych przesyłanych za pomocą formularza (choć w przypadku zmiennych inicjowanych w samym skrypcie sprawa wygląda podobnie). Na wartości 0 na pewno wywali się empty, bo $zmienna = '0'; to nadal pusta (empty) zmienna.
Ogólnie to w rzeczywistości mamy tyle rozwiązań ilu programistów (choć pewnie większość rozwiązań będzie podobna, albo identyczna), jedyną kwestią jaką trzeba rozważyć jest natomiast wydajność, która i tak w przypadku samej walidacji formularza może się tylko nieznacznie różnić w różnych rozwiązaniach, no chyba, że ktoś to strasznie zamota.

Ten post edytował mortus 22.03.2010, 13:35:56
Go to the top of the page
+Quote Post
rossco
post
Post #20





Grupa: Zarejestrowani
Postów: 72
Pomógł: 0
Dołączył: 17.02.2007

Ostrzeżenie: (0%)
-----

Chyba na początek nauki w php (podejście trzecie:)) wybrałam za trudne zadanie. Przerobiłam skrypt, ale pokazuję się błąd w linijce 56 "
Parse error: syntax error, unexpected T_BOOLEAN_AND, expecting ')' in /ogloszenia/index.php on line 56" i podejrzewam że źle zastosowałam się to twoich wskazówek Mortus

[PHP] pobierz, plaintext 
  1. <?
  2.  
  3. mysql_connect ("*****", "*****", "*****") or //staramy połączyć się z bazą danych
  4.  die ("Nie można połączyć się z bazą MySQL"); //jeżeli to się nie uda wyświetlamy
  5.  mysql_select_db ("*****") or //wybieramy bazę
  6.  die ("Nie można połączyć się z bazą ogloszenia");
  7. mysql_query("SET NAMES utf-8");
  8. mysql_query("SET NAMES utf8");
  9. mysql_query("SET CHARACTER SET utf8");
  10. mysql_query("SET collation_connection = utf8_polish_ci");
  11. $id=$_GET["id"];
  12. $tresc = $_POST["tresc"];
  13. $stanowisko = $_POST["stanowisko"];
  14. $nazwa_firmy = $_POST["nazwa_firmy"];
  15. $miejscowosc_zatrudnienia = $_POST["miejscowosc_zatrudnienia"];
  16.  
  17.       // sprawdzamy, czy użytkownik wcisnął przycisk wyślij
  18.  
  19. 	if(!empty($_GET['id']) && $_GET['id'] == 'dodaj' && !empty($_POST)) {
  20.  
  21.       // formularz został wysłany - walidacja (tutaj na przykładzie pól tresc i stanowisko)
  22.  
  23. 	if(!isset($_POST['tresc']) || $_POST['tresc'] == '') {
  24.  
  25.       $validation_error = true;
  26.  
  27.       $pusta_tresc = true;
  28.  
  29.       }
  30.  
  31.       if(!isset($_POST['stanowisko']) || $_POST['stanowisko'] == '') {
  32.  
  33.       $validation_error = true;
  34.  
  35.       $puste_stanowisko = true;
  36.  
  37.       }
  38.  
  39.       // jeżeli zatem wszystkie pola zostały wypełnione to
  40.  
  41.       if(!validation_error) {
  42.  
  43.       // pobieramy wszystkie pola z tablicy $_POST i wrzucamy ogłoszenie do bazy danych
  44. 			$zapytanie = "INSERT INTO ogloszenia (tresc, stanowisko, nazwa_firmy, miejscowosc_zatrudnienia, "." data) VALUES ('$tresc', '$stanowisko', '$nazwa_firmy', '$miejscowosc_zatrudnienia', "." now());";
  45.  $wynik = mysql_query ($zapytanie); //dodajemy wtedy dane do bazy wraz z aktualną datą i godziną – funkcja now()
  46.  print "<HR>";
  47.  
  48.       } else {
  49.  
  50.       // wrzucamy formularz
  51.  
  52.       include ('form.php');
  53.  
  54.       }
  55.  
  56.       } elseif(!empty($_GET['id'] && $_GET['id'] == 'nowy')) {
  57.  
  58.       // wrzucamy formularz
  59.       include ('form.php');
  60.  
  61.       } else {
  62.  
  63.       // wyświetlamy listę ogłoszeń i udostępniamy możliwość dodania nowego ogłoszenia
  64.  
  65. 			print"<A HREF=\"index.php?id=nowy\">Dodaj nowe ogłoszenia</A><HR>";
  66.  $wynik = mysql_query ("SELECT * FROM ogloszenia WHERE zatwierdz=1 order by nr desc;"); //wybieramy zatwierdzone ogłoszenia i segregujemy według pola nr od wartości największej do najmniejszej
  67.  print"<UL>";
  68.  while ($rekord = mysql_fetch_array ($wynik, MYSQL_NUM)) { //mysql_fetch_array() zwraca nam tablicę indeksowaną numerycznie, gdzie indeks zaczyna się od 0 (MYSQL_NUM)
  69.  print "<LI>".$rekord[2] = stripslashes($rekord[2]).", <B> ".$rekord[1] = stripslashes($rekord[1])."</B>, ".$rekord[3].", ".$rekord[5] = stripslashes($rekord[5])."</LI>"; //za pomocą pętli while wyświetlamy wszystkie rekordy, które zostały zatwierdzone (wartość pola zatwierdz wynosi 1)
  70.  }
  71.  print"</UL>";
  72.  
  73.  
  74. 	}
  75. ?>
[PHP] pobierz, plaintext



Jeśli można to proszę o podpowiedzi co źle zrobiłam
Go to the top of the page
+Quote Post
« Następny starszy · Przedszkole · Następny nowszy »
 

2 Stron V   1 2 >
Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Przełącz na: Standardowy · Linearny+ · Przełącz na: Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Aktualny czas: 20.08.2025 - 09:02
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn