Forum PHP.pl

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 
<?php
session_start ();
include ('header.inc');
 
if (isset ($_POST['submit'])) {
		require_once('./polacz_z_baza.php');
 
 
		$wiadomosc = NULL;
 
		if (empty($_POST['uzytkownik'])) {
				$u=false;
				$wiadomosc.='<p>Wprowadź nazwę użytkownika</p>';
		} else {
 
			   $u=($_POST['uzytkownik']);
		}
 
		if (empty($_POST['haslo'])) {
				$p=false;
				$wiadomosc.='<p>Wprowadź hasło</p>';
		} else {
				$p=($_POST['haslo']);
		}
 
		if ($u && $p) {//Nazwa użytkownika i hasło są podane
 
 
 
 
			$zapytanie = &#092;"SELECT nr, uzytkownik, test  FROM uzytkownicy WHERE uzytkownik = '$u' AND haslo = '$p'\";
			$wynik = mysql_query ($zapytanie);
			$wiersz = mysql_fetch_array ($wynik);
 
			if ($wiersz) {
					//Rejestruję sesję i przekierowuję użytkownika do egzaminu.
 
 
					$_SESSION['uzytkownik'] =$wiersz[1];
					$_SESSION['nr'] =$wiersz[0];
					$test = $wiersz[3];
 
 
 
					 if ($test = tak) {
					 print 'Nie można 2 razy rozwiązywać testu ';	//jesli test juz byl rozwiazywany
					 } else {
							 //jesli nie byl rozwiazywany -  poczatek
/* po pierwszym zalogowaniu ustalam wartość pola logicznego w bazie na tak,
					 aby zapobiec ponownemu rozwiązywaniu testu przez danego użytkownika */
mysql_query(&#092;"UPDATE uzytkownicy SET test = 'tak' WHERE uzytkownik = '$u'\");
					header (&#092;"Location: http://localhost/praca/egzamin.php\");
			} else {
 
					$wiadomosc ='<p>Błedny login lub hasło</p>';
					}
			mysql_close();
			} else {
					$wiadomosc.='<p>Spróbuj ponownie.</p>';
					}
			}
			if (isset($wiadomosc)) {
					echo '<font color =\"red\">', $wiadomosc , '</font>';
					}
 
 
					$_SESSION['data']= date('Y-m-j H:i:s');
					   //jesli nie byl -  koniec
					?>
[PHP] pobierz, plaintext 

Ten post edytował artur81 1.02.2005, 19:48:58

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.

[PHP] pobierz, plaintext 
<?php
 
//...
if ($test = &#092;"tak\") {
//..
 
?>
[PHP] pobierz, plaintext 

?

@ hmmm :A zgadza się, poczytna książeczka (IMG:http://forum.php.pl/style_emoticons/default/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... (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

to policz :P

chodzi mi o ta czesc kodu:

[PHP] pobierz, plaintext 
<?php
 
if ($test = tak) {
					 print 'Nie można 2 razy rozwiązywać testu ';	//jesli test juz byl rozwiazywany
					 } else {
							 //jesli nie byl rozwiazywany -  poczatek
/* po pierwszym zalogowaniu ustalam wartość pola logicznego w bazie na tak,
					 aby zapobiec ponownemu rozwiązywaniu testu przez danego użytkownika */					
mysql_query(&#092;"UPDATE uzytkownicy SET test = 'tak' WHERE uzytkownik = '$u'\");
					header (&#092;"Location: http://localhost/praca/egzamin.php\");
			} else {
 
?>
[PHP] pobierz, plaintext 

po header nie domknales nawiasu ... ;)

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 (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) Proponuję taki układ: ja poczytam manuala, a Wy uznajcie że nie było tematu (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)
Sorki za zajęcie czasu.

Ten post edytował artur81 1.02.2005, 19:52:52

wyrzuca blad, bo wychodzi cos takiego

[PHP] pobierz, plaintext 
<?php
 
if (warunek) {
//wykonaj1
} else {
//wykonaj2
} else {
//wykonaj3
}
 
?>
[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

nie czytasz dokładnie postów,
primo

[PHP] pobierz, plaintext 
<?php
 
if ($test = tak) {
 
?>
[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 
<?php
 
 
            } else {
 
                    $wiadomosc ='<p>Błedny login lub hasło</p>';
                    }
            mysql_close();
            } else { // --------------- jeśli robisz else może być tylko ostatnie, jeśli
						// masz kilka tego typu warunków musi być else if
                    $wiadomosc.='<p>Spróbuj ponownie.</p>';
                    }
            } // --------------- ta klamra jest nie potrzebna
            if (isset($wiadomosc)) {
 
?>
[PHP] pobierz, plaintext 

Ten post edytował ennics 1.02.2005, 19:54:23

@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 ;)

muszę przyznać Ci rację, jakby to powiedzieć... przeliczyłem się :roll2:

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

ennics: jeszcze jedno, niekoniecznie w podwojnym apostrofie, duzo lepiej umiescic w pojedynczym ' '

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 
<?php
session_start ();
include ('header.inc');
 
if (isset ($_POST['submit'])) {
		require_once('./polacz_z_baza.php');
 
 
		$wiadomosc = NULL;
 
		if (empty($_POST['uzytkownik'])) {
				$u=false;
				$wiadomosc.='<p>Wprowadź nazwę użytkownika</p>';
		} else {
 
			   $u=($_POST['uzytkownik']);
		}
 
		if (empty($_POST['haslo'])) {
				$p=false;
				$wiadomosc.='<p>Wprowadź hasło</p>';
		} else {
				$p=($_POST['haslo']);
		}
 
		if ($u && $p) {//Nazwa użytkownika i hasło są podane
 
 
 
 
			$zapytanie = &#092;"SELECT nr, uzytkownik, test  FROM uzytkownicy WHERE uzytkownik = '$u' AND haslo = '$p'\";
			$wynik = mysql_query ($zapytanie);
			$wiersz = mysql_fetch_array ($wynik);
 
			if ($wiersz) {
					//Rejestruję sesję i przekierowuję użytkownika do egzaminu.
 
 
					$_SESSION['uzytkownik'] =$wiersz[1];
					$_SESSION['nr'] =$wiersz[0];
					$test = $wiersz[3];
 
 
 
					 if ($test == 'tak') {
					 print 'Nie można 2 razy rozwiązywać testu ';	//jesli test juz byl rozwiazywany
					 } else {
							 //jesli nie byl rozwiazywany -  poczatek
/* po pierwszym zalogowaniu ustalam wartość pola logicznego w bazie na tak,
					 aby zapobiec ponownemu rozwiązywaniu testu przez danego użytkownika */
mysql_query(&#092;"UPDATE uzytkownicy SET test = 'tak' WHERE uzytkownik = '$u'\");
					header (&#092;"Location: http://localhost/praca/egzamin.php\");
					 }
			} else {
 
					$wiadomosc ='<p>Błedny login lub hasło</p>';
					}
			mysql_close();
			} else {
					$wiadomosc.='<p>Spróbuj ponownie.</p>';
					}
			}
			if (isset($wiadomosc)) {
					echo '<font color =\"red\">', $wiadomosc , '</font>';
					}
 
 
					$_SESSION['data']= date('Y-m-j H:i:s');
					   //jesli nie byl -  koniec
					?>
[PHP] pobierz, plaintext 

A teraz spojrzcie na ten sposob kodowania, widac gdzier sie ktore if i else zaczynaja i widac od razu bledy

[PHP] pobierz, plaintext 
<?php
session_start ();
include ('header.inc');
 
if (isset ($_POST['submit']))
{
    require_once('./polacz_z_baza.php');
    $wiadomosc = NULL;
 
    if (empty($_POST['uzytkownik']))
    {
        $u=false;
        $wiadomosc.='<p>Wprowadź nazwę użytkownika</p>';
    }
    else
    {
       $u=($_POST['uzytkownik']);
    }
 
    if (empty($_POST['haslo']))
    {
        $p=false;
        $wiadomosc.='<p>Wprowadź hasło</p>';
    }
    else
    {
        $p=($_POST['haslo']);
    }
 
    if ($u && $p)
    {
        //Nazwa użytkownika i hasło są podane
 
        $zapytanie = &#092;"SELECT nr, uzytkownik, test  FROM uzytkownicy WHERE uzytkownik = '$u' AND haslo = '$p'\";
        $wynik = mysql_query ($zapytanie);
        $wiersz = mysql_fetch_array ($wynik);
 
        if ($wiersz)
        {
            //Rejestruję sesję i przekierowuję użytkownika do egzaminu.
 
            $_SESSION['uzytkownik'] =$wiersz[1];
            $_SESSION['nr'] =$wiersz[0];
            $test = $wiersz[3];
 
 
 
            if ($test = tak)
            {
                print 'Nie można 2 razy rozwiązywać testu ';    //jesli test juz byl rozwiazywany
            }
            else
            {
                //jesli nie byl rozwiazywany -  poczatek
                /* po pierwszym zalogowaniu ustalam wartość pola logicznego w bazie na tak,
                aby zapobiec ponownemu rozwiązywaniu testu przez danego użytkown
ka */
                mysql_query(&#092;"UPDATE uzytkownicy SET test = 'tak' WHERE uzytkownik = '$u'\");
                // FIXED
                // header (\"Location: http://localhost/praca/egzamin.php\");
                header (&#092;"Location: http://localhost/praca/egzamin.php\");
            }
            else
            {
                // BLAD!!!
                $wiadomosc ='<p>Błedny login lub hasło</p>';
            }
            
            mysql_close();
        } // EDIT: o jeden tab mniej
        else
        {
            $wiadomosc.='<p>Spróbuj ponownie.</p>';
        }
   
    }
    
    if (isset($wiadomosc))
    {
        echo '<font color =\"red\">', $wiadomosc , '</font>';
    }
 
 
    $_SESSION['data']= date('Y-m-j H:i:s');
       //jesli nie byl -  koniec
       
   // BLAD!!! -- brakuje ostatniej zamykacej klamry
?>
[PHP] pobierz, plaintext 

Ten post edytował dr_bonzo 1.02.2005, 20:11:07

piszac w kodzie na samym dole komentarz

[PHP] pobierz, plaintext 
<?php
// BLAD!!! -- brakuje ostatniej zamykacej klamry
 
?>
[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 
<?php
if (warunek)
{
//wykonaj1
}
else
{
//wykonaj2
}
 
?>
[PHP] pobierz, plaintext 

jest malo czytelny ... chyba kwestia przyzwyczajenia.

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


Ja? On?

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