Gdzie ERROR ? :> Opcje

[Balas]

Robie skrypt ktory otwiera sesje jezeli sa odpowiednie dane w cookies. Mam taki kod:

index.php:

[PHP] pobierz, plaintext 
<? 
include "function.php";
 
if (isset($_SESSION["login"]))
{
echo('jestes zalogowany');
echo('<a href="wyloguj.php?wyloguj=1">WYLOGUJ</a>');
}
if (isset($_SESSION["admin"]))
{
echo('jestes zalogowany jako admin');
echo('<a href="wyloguj.php?wyloguj=2">WYLOGUJ</a>');
}
 
 
?>
[PHP] pobierz, plaintext 

Function.php

[PHP] pobierz, plaintext 
<?
$luser = $_COOKIE["user"]; 
$pass = $_COOKIE["pass"]; 
$id = $_COOKIE["id"]; 
 
$strVarHost = 'localhost';
$strVarUser = 'root';
$strVarPasswd = 'Hammett';
$strVarBaza = 'IPB';
 
mysql_connect ("localhost", "root", "Hammett"); // Łączenie się z bazą.
$sqlSelect	= mysql_select_db("IPB");
 
$sqlQuery = mysql_query("SELECT * FROM ipb_members WHERE id = '$id' ");
$sqlArray  = mysql_fetch_array($sqlQuery);
 
if ($sqlArray == TRUE)
{
$sqlPass = $sqlArray['legacy_password'];
$sqlId = $sqlArray['id'];
 
if ($luser == 1 && $id == $sqlId && $pass == $sqlPass)
{
ob_start();
session_start();
session_register("login");
ob_end_flush();
}
if ($luser == 2 && $id == $sqlId && $pass == $sqlPass)
{
ob_start();
session_start();
session_register("admin");
ob_end_flush();
}
 
}
 
?>
[PHP] pobierz, plaintext 

Dane w cookies sa bo sprawadzalem przez echo ($luser); itd.

Gdzie robie błąd

Ten post edytował OnE.Killer 19.09.2005, 17:00:43

[mike]

Poczytaj o priorytetach operatorów a potem użyj nawiasów w warunkach logicznych w if'ach.


A na przyszłość pisząc że masz błąd, że coś nie działa, e.t.c. pisz jakie komunikaty pokazuje php, jakie są konsekwencje błądu. Napisanie że php nic nie wyświetla tez jest wskazówką dla odpowiadających. Jak chcesz dostać odpowiedź, włóż trochę pracy w pytanie.

[Balas]

powiem tak wyswietla zawsze to samo (znaczy jestes zalogowany :] )... nie wazne czy admin czy uzytkwnik (zanim przeszedlem na sesje i tylko na cookies bylo OK)

I nie wiem nadal w czym problem.... jak powinno byc I po co nawiasy w IFach

EDIT: Mam takie pytanie: czy moze lepiej zostac przy ciasteczkach czy lepiej przejsc na sesje Ktos doradzi

Ten post edytował OnE.Killer 19.09.2005, 18:51:47

[Alchemy_A.R.T.]

ja stosuje ciacha - nazwa 'sesja' zle mi sie kojarzy do tej pory walcze hehe

[Balas]

zostalem przy cisateczkach i sie nimi bawie... :]

[mike]

Mam wrażenie że nawet nie zajrzałeś do linka który Ci dałem, a jak zajrzałeś to przeleciałeś wzrokiem żeby wrócić to, nie mieć sobie nic do zarzucenia i dostać gotowca, co :?:

A Tam jest odpowiedź.

Warunki logiczne w if'ach są źle skonstruowane. Żeby były poprawnie musisz zmienić priorytet operatorów logicznych, a to z kolei osiągnisz stosując nawiasy.

Przecież wiadomo że && jest silniejsze od == dlaego Ci źle działa.

[Balas]

to jak powinno byc

[PHP] pobierz, plaintext 
<?
if (($luser == 2) && ($id == $sqlId) && ($pass == $sqlPass))
?>
[PHP] pobierz, plaintext 

Czy ocb A tamto przeczytalem 2 razy i nadal nie wiem ocb

Ten post edytował OnE.Killer 20.09.2005, 15:54:41

[qbuz]

Przecież wiadomo że && jest silniejsze od == dlaego Ci źle działa.

"Raczej" na odwrot.

Poniższa tabela zawiera priorytet operatorów, od najniższego priorytetu na górze.

Powiązanie || Operator
lewe || ,
lewe || or
lewe || xor
lewe || and
prawe || print
lewe || = += -= *= /= .= %= &= |= ^= ~= <<= >>=
lewe || ? :
lewe || ||
lewe || &&
lewe || |
lewe || ^
lewe || &
bez powiązania || == != === !==
bez powiązania || < <= > >=

to jak powinno byc

Dobrze miales


@OnE.Killer: Czy na pewno tamte wszystkie IFy w function.php zwracaja ci TRUE?

Sprawdz to np. w taki sposob:

[PHP] pobierz, plaintext 
<?php
 
if ($sqlArray == TRUE)
{
	print 'dziala 1<br />';
}
 
if ($luser == 1 && $id == $sqlId && $pass == $sqlPass)
{
	print '2 tez dziala<br />';
}
 
// i tak dalej...
 
 
?>
[PHP] pobierz, plaintext 

Gdy bedziesz tak sprawdzal, to wykomentuj najlepiej session_start(); itd., zeby nie wywalalo bledow (nie mozna wyslac header'ow).
Co to sa w ogole te ob_start(); tam?
ob_start(); daj na sam poczatek calego kodu, a na sam koniec ob_end_flush;


PS: Coz to?? ;]

[PHP] pobierz, plaintext 
<?php
 
$strVarHost = 'localhost';
$strVarUser = 'root';
$strVarPasswd = 'Hammett';
$strVarBaza = 'IPB';
 
mysql_connect ("localhost", "root", "Hammett"); // Łączenie się z bazą.
$sqlSelect	= mysql_select_db("IPB");
 
?>
[PHP] pobierz, plaintext 

[mike]

Oj, gafa z mojej strony. Przepraszam ślicznie. <zawstydzony>

[Balas]

qrde... gafe zrobilem sprawdzalem w innym miejscu wartosci z cookies (w function.php i tam wyswietlalo) :] teraz poprawilem i działa (dodalem sprawdzanie cookies tuz przed :])

sry za kłopot