[PHP]Nie zakończona pętla w funkcji Opcje

[d0m1n1k_]

Witam,
poszukuję rozwiązania do następującego problemu:

Napisałem prosty zestaw sprawdzania poprawności logowania.
Działać - działa, ale uprzykrza mi życie swoją długością ;-)
Zastanawiałem się jak to "pociąć" aby zamknąć całość do trzech/czterech elementów np. wkładając kolejne części do trzech funkcji.
Niemniej jednak nie znam się na tym na tyle aby to rozwiązać - próbowałem wyszukania takiego problemu ale nic nie znalazłem.

KOD DZIAŁAJĄCY:

[PHP] pobierz, plaintext 
if(isset($_COOKIE['ident']) && isset($_COOKIE['paswrd1']) && isset($_COOKIE['paswrd2']))
	{
		$login_idnrA = $_COOKIE['ident'];
		$login_pas1A = $_COOKIE['paswrd1'];
		$login_pas2A = $_COOKIE['paswrd2'];
		$login_czekA = mysql_query("SELECT id, authorization, password FROM usrs WHERE id = '$login_idnrA' LIMIT 1");
		$login_czekB = mysql_fetch_array($login_czekA);
		$login_idnrB = $login_czekB['id'];
		$login_pas1B = $login_czekB['password'];
		$login_pas2B = $login_czekB['authorization'];
 
		if (($login_idnrA == $login_idnrB) && ($login_pas1A == $login_pas1B) && ($login_pas2A == $login_pas2B))
			{			
				echo "<li><a href=\"panel.php?id=$login_idnrA\">Mój profil</a></li>";
				echo "<li><a href=\"wyloguj.php\">Wyloguj</a></li>";
			}else{
			}
	}else{
		echo "<li><a href=\"logowanie.php\">Zaloguj się</a></li>";
		echo "<li><a href=\"rejestracja.php\">Dołącz do nas!</a></li>";
	}
[PHP] pobierz, plaintext 

MOJA "WIZJA" POCIĘCIA:

Część I

[PHP] pobierz, plaintext 
function czescpierwsza()
{
if(isset($_COOKIE['ident']) && isset($_COOKIE['paswrd1']) && isset($_COOKIE['paswrd2']))
	{
		$login_idnrA = $_COOKIE['ident'];
		$login_pas1A = $_COOKIE['paswrd1'];
		$login_pas2A = $_COOKIE['paswrd2'];
		$login_czekA = mysql_query("SELECT id, authorization, password FROM usrs WHERE id = '$login_idnrA' LIMIT 1");
		$login_czekB = mysql_fetch_array($login_czekA);
		$login_idnrB = $login_czekB['id'];
		$login_pas1B = $login_czekB['password'];
		$login_pas2B = $login_czekB['authorization'];
 
		if (($login_idnrA == $login_idnrB) && ($login_pas1A == $login_pas1B) && ($login_pas2A == $login_pas2B))
			{
}
[PHP] pobierz, plaintext 

Część II

[PHP] pobierz, plaintext 
function czescdruga()
{
      }else{
   }
}else{
}
[PHP] pobierz, plaintext 

Część III

[PHP] pobierz, plaintext 
function czesctrzecia()
{
   }
}
[PHP] pobierz, plaintext 

i takie coś wstawiać w kod w mniej więcej taki sposób:

[PHP] pobierz, plaintext 
czescpierwsza();
echo "tresc dla zalogowanego";
czescdruga();
echo "tresc dla niezalogowanego";
czesctrzecia();
[PHP] pobierz, plaintext 

Czy jest taka możliwość?
Z góry dziękuję za pomoc!

[tzm]

Poczytaj może o sesjach?

[Daimos]

Nie rozbijesz tak instrukcji warunkowych. Możesz wsadzić do funkcji ich zawartość, np:

[PHP] pobierz, plaintext 
if (($login_idnrA == $login_idnrB) && ($login_pas1A == $login_pas1B) && ($login_pas2A == $login_pas2B))
{			
    wyswietlCos($login_idnrA);
}
function wyswietlCos($login_idnrA)
{
	echo "<li><a href=\"panel.php?id=$login_idnrA\">Mój profil</a></li>";
	echo "<li><a href=\"wyloguj.php\">Wyloguj</a></li>";
}
[PHP] pobierz, plaintext 

Jeśli chcesz nauczyć się jakoś logicznie rozbijać aplikację, pomyśl o tym, aby funkcja zwracała Ci dane z bazy, inna funkcja odpowiadała za widoki.
Myślenie o tym, aby funkcja rozbijała kod bo ma za dużo linijek, to zbyt proste rozumowanie (IMG:style_emoticons/default/smile.gif)

[d0m1n1k_]

Nie rozumiem. Mam zastąpić ciasteczka sesją?
Logowanie działa. Wykonuje powierzone jej działania.

Zależy mi na podzielenie kodu w sposób który spowoduje nie zamknięcie pętli (rozdzielenie jej na dwa/trzy "składziki).

Jeśli chcesz nauczyć się jakoś logicznie rozbijać aplikację, pomyśl o tym, aby funkcja zwracała Ci dane z bazy, inna funkcja odpowiadała za widoki.
Myślenie o tym, aby funkcja rozbijała kod bo ma za dużo linijek, to zbyt proste rozumowanie (IMG:style_emoticons/default/smile.gif)

Wiem, że upraszczam, ale na moje potrzeby to uproszczenie jest uzasadnione... no prawie uzasadnione lenistwem ;-)

@Daimos, czyli rozumiem, że raczej się nie da, a na pewno nie powinienem tak tego wykonywać?

Ten post edytował d0m1n1k_ 19.09.2014, 11:12:47

[nospor]

Nie rozumiem. Mam zastąpić ciasteczka sesją?
Logowanie działa. Wykonuje powierzone jej działania.

I co z tego ze dziala jak dziala żle.... No chyba, że Twoim zamierzeniem było, że każdy może zalgoować sie na dowolne konto w Twoim serwisie. Jesli takie bylo zalozenie, to faktycznie, dziala i to wysmienicie (IMG:style_emoticons/default/smile.gif)

[d0m1n1k_]

I co z tego ze dziala jak dziala żle.... No chyba, że Twoim zamierzeniem było, że każdy może zalgoować sie na dowolne konto w Twoim serwisie. Jesli takie bylo zalozenie, to faktycznie, dziala i to wysmienicie (IMG:style_emoticons/default/smile.gif)

Kolejny raz - nie rozumiem, czyli jeśli sesji nie wprowadzę w życie to robię to źle?
Jeśli tak, to zatem jednak jestem w lesie

[KsaR]

Kolejny raz - nie rozumiem, czyli jeśli sesji nie wprowadzę w życie to robię to źle?
Jeśli tak, to zatem jednak jestem w lesie

Ciasteczka da sie edytować,
więc można tak sie zalogować na admina na przykład.

A do sesji użytkownik nie posiada dostępu do edycji, bynajmniej nie wiem jak.

Ten post edytował Ksar 19.09.2014, 11:38:28

[d0m1n1k_]

Ok - przyjęte do informacji.
Spróbuję to wszystko poprawić i dodać sesje.