Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: Problem z if/elseif
Forum PHP.pl > Forum > Przedszkole
artur81
1.02.2005, 18:53:24
Witam, problem jak widać podstawowy ale już na tyle razy zmieniałem kod że zauważyłem iż wstawiam już te ify na "łapu capu", mogłby mi ktoś powiedzieć jak to poprawić, ja już nie mam koncepcji ( wyskakuje mi taki błąd Parse error: parse error, unexpected T_ELSE in c:\program files\easyphp1-7\www\praca\logowanie.php on line 52 ). chodzi o to żeby sprawdzić czy zmienna test jest ustawiona na "tak" czy na "nie", Jesli na tak to wyswietlic komunikat "Nie można 2 razy rozwiązywać testu" i zakonczyc skrypt a jesli na nie to ustwic ją na "tak" i przejsc dalej.
Kod:
[PHP] pobierz, plaintext 
  1. <?php
  2. session_start ();
  3. include ('header.inc');
  4.  
  5. if (isset ($_POST['submit'])) {
  6. 		require_once('./polacz_z_baza.php');
  7.  
  8.  
  9. 		$wiadomosc = NULL;
  10.  
  11. 		if (empty($_POST['uzytkownik'])) {
  12. 				$u=false;
  13. 				$wiadomosc.='<p>Wprowadź nazwę użytkownika</p>';
  14. 		} else {
  15.  
  16. 			   $u=($_POST['uzytkownik']);
  17. 		}
  18.  
  19. 		if (empty($_POST['haslo'])) {
  20. 				$p=false;
  21. 				$wiadomosc.='<p>Wprowadź hasło</p>';
  22. 		} else {
  23. 				$p=($_POST['haslo']);
  24. 		}
  25.  
  26. 		if ($u && $p) {//Nazwa użytkownika i hasło są podane
  27.  
  28.  
  29.  
  30.  
  31. 			$zapytanie = &#092;"SELECT nr, uzytkownik, test  FROM uzytkownicy WHERE uzytkownik = '$u' AND haslo = '$p'\";
  32. 			$wynik = mysql_query ($zapytanie);
  33. 			$wiersz = mysql_fetch_array ($wynik);
  34.  
  35. 			if ($wiersz) {
  36. 					//Rejestruję sesję i przekierowuję użytkownika do egzaminu.
  37.  
  38.  
  39. 					$_SESSION['uzytkownik'] =$wiersz[1];
  40. 					$_SESSION['nr'] =$wiersz[0];
  41. 					$test = $wiersz[3];
  42.  
  43.  
  44.  
  45. 					 if ($test = tak) {
  46. 					 print 'Nie można 2 razy rozwiązywać testu ';	//jesli test juz byl rozwiazywany
  47. 					 } else {
  48. 							 //jesli nie byl rozwiazywany -  poczatek
  49. /* po pierwszym zalogowaniu ustalam wartość pola logicznego w bazie na tak,
  50. 					 aby zapobiec ponownemu rozwiązywaniu testu przez danego użytkownika */
  51. mysql_query(&#092;"UPDATE uzytkownicy SET test = 'tak' WHERE uzytkownik = '$u'\");
  52. 					header (&#092;"Location: http://localhost/praca/egzamin.php\");
  53. 			} else {
  54.  
  55. 					$wiadomosc ='<p>Błedny login lub hasło</p>';
  56. 					}
  57. 			mysql_close();
  58. 			} else {
  59. 					$wiadomosc.='<p>Spróbuj ponownie.</p>';
  60. 					}
  61. 			}
  62. 			if (isset($wiadomosc)) {
  63. 					echo '<font color =\"red\">', $wiadomosc , '</font>';
  64. 					}
  65.  
  66.  
  67. 					$_SESSION['data']= date('Y-m-j H:i:s');
  68. 					   //jesli nie byl -  koniec
  69. 					?>
[PHP] pobierz, plaintext
hmmm
1.02.2005, 19:00:00
ladnie napisany kod - czytelnie :P
korzystasz czasem z jakiejs ksiazki? bo jedna mi nachodzi na mysl ...

na pierwszy rzut oka brakuje mi kilku domkniec nawiasow klamrowych.
nie wczytuje sie w kod, ale albo nie domknales else dla if ($test = tak) albo zamiast drugiego else powinienes wstawic elseif

posprawdzaj domkniecia.
ennics
1.02.2005, 19:00:08 
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. //...
  4. if ($test = &#092;"tak\") {
  5. //..
  6.  
  7. ?>
[PHP] pobierz, plaintext

?
artur81
1.02.2005, 19:37:17
@ hmmm :A zgadza się, poczytna książeczka tongue.gif Swoją drogą podomykane mam wszystkie if'y, sprawdzalem, tak mysle ze jak nie dojde jak to zrobic to bede musial dac przed tą strona jeszcze jedną i tam sprawdzic ten warunek. Pod górę, ale trudno... tongue.gif
hmmm
1.02.2005, 19:42:39
to policz :P

chodzi mi o ta czesc kodu:
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. if ($test = tak) {
  4. 					 print 'Nie można 2 razy rozwiązywać testu ';	//jesli test juz byl rozwiazywany
  5. 					 } else {
  6. 							 //jesli nie byl rozwiazywany -  poczatek
  7. /* po pierwszym zalogowaniu ustalam wartość pola logicznego w bazie na tak,
  8. 					 aby zapobiec ponownemu rozwiązywaniu testu przez danego użytkownika */					
  9. mysql_query(&#092;"UPDATE uzytkownicy SET test = 'tak' WHERE uzytkownik = '$u'\");
  10. 					header (&#092;"Location: http://localhost/praca/egzamin.php\");
  11. 			} else {
  12.  
  13. ?>
[PHP] pobierz, plaintext
po header nie domknales nawiasu ... ;)
artur81
1.02.2005, 19:45:18
Teraz to już przegiąłem na maxa ( zmiany w kodzie- sam się już pogubiłem), aha i tak nawiasem mówiąc mogłbu któryś z moderatorów usunąc tego posta questionmark.gif Proponuję taki układ: ja poczytam manuala, a Wy uznajcie że nie było tematu tongue.gif
Sorki za zajęcie czasu.
hmmm
1.02.2005, 19:50:23
wyrzuca blad, bo wychodzi cos takiego
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. if (warunek) {
  4. //wykonaj1
  5. } else {
  6. //wykonaj2
  7. } else {
  8. //wykonaj3
  9. }
  10.  
  11. ?>
[PHP] pobierz, plaintext
a przeciez else moze byc tylko jedno ...
wstaw domkniecie nawiasu i sprawdz, a jezeli tego domkniecia tam nie powinno byc, bo zamien srodkowe else na elseif
ennics
1.02.2005, 19:53:59
nie czytasz dokładnie postów,
primo
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. if ($test = tak) {
  4.  
  5. ?>
[PHP] pobierz, plaintext

tak nie może być !
'tak' jest ciągiem i musi być zapisany w podwójnym apostrofie ""
po drugie jak już pisał hmm
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3.  
  4.             } else {
  5.  
  6.                     $wiadomosc ='<p>Błedny login lub hasło</p>';
  7.                     }
  8.             mysql_close();
  9.             } else { // --------------- jeśli robisz else może być tylko ostatnie, jeśli
  10. 						// masz kilka tego typu warunków musi być else if
  11.                     $wiadomosc.='<p>Spróbuj ponownie.</p>';
  12.                     }
  13.             } // --------------- ta klamra jest nie potrzebna
  14.             if (isset($wiadomosc)) {
  15.  
  16. ?>
[PHP] pobierz, plaintext
hmmm
1.02.2005, 19:58:12
@ennics co do primo to sie zgodze, a do drugiej uwagi tylko po czesci, bo tamta klamra, przy ktorej napisales komentarz w kodzie, ze jest niepotrzebna, w rzeczywistosci jest potrzebna ;)
ennics
1.02.2005, 19:59:56
muszę przyznać Ci rację, jakby to powiedzieć... przeliczyłem się :roll2:
artur81
1.02.2005, 20:01:17
Dziękuję jeszcze raz za pomoc , ale po moich przeróbkach kod już wcale nie przypomina tego który jest na górze, na dzisiaj to ja już podziękuję, jutro przeanalizuje kod jeszcze raz, uwzględnie Wasze poprawki i ewentualnie się odezwę ... Sorki za zajęcie czasu tongue.gif
Puciek
1.02.2005, 20:02:37
ennics: jeszcze jedno, niekoniecznie w podwojnym apostrofie, duzo lepiej umiescic w pojedynczym ' '
hmmm
1.02.2005, 20:05:03
twoj kod, z trzema poprawkami:
1. domkniecie nawiasu
2. == zamiast =
3. warunek powinien byc w apostrofie: 'tak'
nie wiem, czy gdzies jeszcze sa bledy :P

[PHP] pobierz, plaintext 
  1. <?php
  2. session_start ();
  3. include ('header.inc');
  4.  
  5. if (isset ($_POST['submit'])) {
  6. 		require_once('./polacz_z_baza.php');
  7.  
  8.  
  9. 		$wiadomosc = NULL;
  10.  
  11. 		if (empty($_POST['uzytkownik'])) {
  12. 				$u=false;
  13. 				$wiadomosc.='<p>Wprowadź nazwę użytkownika</p>';
  14. 		} else {
  15.  
  16. 			   $u=($_POST['uzytkownik']);
  17. 		}
  18.  
  19. 		if (empty($_POST['haslo'])) {
  20. 				$p=false;
  21. 				$wiadomosc.='<p>Wprowadź hasło</p>';
  22. 		} else {
  23. 				$p=($_POST['haslo']);
  24. 		}
  25.  
  26. 		if ($u && $p) {//Nazwa użytkownika i hasło są podane
  27.  
  28.  
  29.  
  30.  
  31. 			$zapytanie = &#092;"SELECT nr, uzytkownik, test  FROM uzytkownicy WHERE uzytkownik = '$u' AND haslo = '$p'\";
  32. 			$wynik = mysql_query ($zapytanie);
  33. 			$wiersz = mysql_fetch_array ($wynik);
  34.  
  35. 			if ($wiersz) {
  36. 					//Rejestruję sesję i przekierowuję użytkownika do egzaminu.
  37.  
  38.  
  39. 					$_SESSION['uzytkownik'] =$wiersz[1];
  40. 					$_SESSION['nr'] =$wiersz[0];
  41. 					$test = $wiersz[3];
  42.  
  43.  
  44.  
  45. 					 if ($test == 'tak') {
  46. 					 print 'Nie można 2 razy rozwiązywać testu ';	//jesli test juz byl rozwiazywany
  47. 					 } else {
  48. 							 //jesli nie byl rozwiazywany -  poczatek
  49. /* po pierwszym zalogowaniu ustalam wartość pola logicznego w bazie na tak,
  50. 					 aby zapobiec ponownemu rozwiązywaniu testu przez danego użytkownika */
  51. mysql_query(&#092;"UPDATE uzytkownicy SET test = 'tak' WHERE uzytkownik = '$u'\");
  52. 					header (&#092;"Location: http://localhost/praca/egzamin.php\");
  53. 					 }
  54. 			} else {
  55.  
  56. 					$wiadomosc ='<p>Błedny login lub hasło</p>';
  57. 					}
  58. 			mysql_close();
  59. 			} else {
  60. 					$wiadomosc.='<p>Spróbuj ponownie.</p>';
  61. 					}
  62. 			}
  63. 			if (isset($wiadomosc)) {
  64. 					echo '<font color =\"red\">', $wiadomosc , '</font>';
  65. 					}
  66.  
  67.  
  68. 					$_SESSION['data']= date('Y-m-j H:i:s');
  69. 					   //jesli nie byl -  koniec
  70. 					?>
[PHP] pobierz, plaintext
dr_bonzo
1.02.2005, 20:09:32
A teraz spojrzcie na ten sposob kodowania, widac gdzier sie ktore if i else zaczynaja i widac od razu bledy
[PHP] pobierz, plaintext 
  1. <?php
  2. session_start ();
  3. include ('header.inc');
  4.  
  5. if (isset ($_POST['submit']))
  6. {
  7.     require_once('./polacz_z_baza.php');
  8.     $wiadomosc = NULL;
  9.  
  10.     if (empty($_POST['uzytkownik']))
  11.     {
  12.         $u=false;
  13.         $wiadomosc.='<p>Wprowadź nazwę użytkownika</p>';
  14.     }
  15.     else
  16.     {
  17.        $u=($_POST['uzytkownik']);
  18.     }
  19.  
  20.     if (empty($_POST['haslo']))
  21.     {
  22.         $p=false;
  23.         $wiadomosc.='<p>Wprowadź hasło</p>';
  24.     }
  25.     else
  26.     {
  27.         $p=($_POST['haslo']);
  28.     }
  29.  
  30.     if ($u && $p)
  31.     {
  32.         //Nazwa użytkownika i hasło są podane
  33.  
  34.         $zapytanie = &#092;"SELECT nr, uzytkownik, test  FROM uzytkownicy WHERE uzytkownik = '$u' AND haslo = '$p'\";
  35.         $wynik = mysql_query ($zapytanie);
  36.         $wiersz = mysql_fetch_array ($wynik);
  37.  
  38.         if ($wiersz)
  39.         {
  40.             //Rejestruję sesję i przekierowuję użytkownika do egzaminu.
  41.  
  42.             $_SESSION['uzytkownik'] =$wiersz[1];
  43.             $_SESSION['nr'] =$wiersz[0];
  44.             $test = $wiersz[3];
  45.  
  46.  
  47.  
  48.             if ($test = tak)
  49.             {
  50.                 print 'Nie można 2 razy rozwiązywać testu ';    //jesli test juz byl rozwiazywany
  51.             }
  52.             else
  53.             {
  54.                 //jesli nie byl rozwiazywany -  poczatek
  55.                 /* po pierwszym zalogowaniu ustalam wartość pola logicznego w bazie na tak,
  56.                 aby zapobiec ponownemu rozwiązywaniu testu przez danego użytkown
  57. ka */
  58.                 mysql_query(&#092;"UPDATE uzytkownicy SET test = 'tak' WHERE uzytkownik = '$u'\");
  59.                 // FIXED
  60.                 // header (\"Location: http://localhost/praca/egzamin.php\");
  61.                 header (&#092;"Location: http://localhost/praca/egzamin.php\");
  62.             }
  63.             else
  64.             {
  65.                 // BLAD!!!
  66.                 $wiadomosc ='<p>Błedny login lub hasło</p>';
  67.             }
  68.             
  69.             mysql_close();
  70.         } // EDIT: o jeden tab mniej
  71.         else
  72.         {
  73.             $wiadomosc.='<p>Spróbuj ponownie.</p>';
  74.         }
  75.    
  76.     }
  77.     
  78.     if (isset($wiadomosc))
  79.     {
  80.         echo '<font color =\"red\">', $wiadomosc , '</font>';
  81.     }
  82.  
  83.  
  84.     $_SESSION['data']= date('Y-m-j H:i:s');
  85.        //jesli nie byl -  koniec
  86.        
  87.    // BLAD!!! -- brakuje ostatniej zamykacej klamry
  88. ?>
[PHP] pobierz, plaintext
hmmm
1.02.2005, 20:18:26
piszac w kodzie na samym dole komentarz
[PHP] pobierz, plaintext 
  1. <?php
  2. // BLAD!!! -- brakuje ostatniej zamykacej klamry
  3.  
  4. ?>
[PHP] pobierz, plaintext
ktos moze pomyslec, ze to akurat tam ja trzeba wstawic ...

a co do czytelnosci ... to zaczal dobrze, zle skonczyl :P
jak dla mnie zapis:
[PHP] pobierz, plaintext 
  1. <?php
  2. if (warunek)
  3. {
  4. //wykonaj1
  5. }
  6. else
  7. {
  8. //wykonaj2
  9. }
  10.  
  11. ?>
[PHP] pobierz, plaintext
jest malo czytelny ... chyba kwestia przyzwyczajenia.
dr_bonzo
1.02.2005, 20:25:51
Cytat(hmmm @ 2005-02-01 20:18:26)
piszac w kodzie na samym dole komentarz
[PHP] pobierz, plaintext 
  1. <?php
  2. // BLAD!!! -- brakuje ostatniej zamykacej klamry
  3.  
  4. ?>
[PHP] pobierz, plaintext
ktos moze pomyslec, ze to akurat tam ja trzeba wstawic ...

No bo tam jej brakuje, tylko nie wstawilem jej bo nie sprawdzalem logiki programu, byc moze miala by byc przed $_SESSION...

Cytat
a co do czytelnosci ... to zaczal dobrze, zle skonczyl tongue.gif

Ja? On?
hmmm
1.02.2005, 20:39:05
hehe. co do czytelnosci to on - @artur81 ;)

a na logike, to domkniecia brakuje tam gdzie je dostawilem - na moja logike.
no ale moge sie mylic :P
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.