Forum Debian Users Gang

debbie · 2009-09-01 13:32:11

Witam! Mam taki problem - mam skrypt logowania wygląda on tak:

Kod:

<?php
session_start();
mysql_connect("localhost","user","pass");
mysql_select_db("base");

if(isset($_SESSION['zalogowany'])) {
echo "Witam, ".$_SESSION['login'];
}else{

if(isset($_POST['wyslij'])) {


   if(mysql_num_rows(mysql_query("SELECT login, haslo
   FROM konta WHERE login = '".$_POST['login']."'
   && haslo = '".$_POST['haslo']."' ")) > 0) {


       if(mysql_num_rows(mysql_query("SELECT nr FROM konta
       WHERE login = '".$_POST['login']."'
       && haslo = '".$_POST['haslo']."' ")) > 0 ) {


           $_SESSION['zalogowany'] = true;
           $_SESSION['login'] = $_POST['login'];
           $_SESSION['haslo'] = $_POST['haslo'];
           echo "Jesteś zalogowany.";


       } else {

   echo "Złe hasło, proszę spróbować ponownie";
}
} else {
   echo "Nie ma takiego użytkownika";
}
} else {

?>
<form method='POST' action='logowanie.php'>
<b>nazwa uzytkownika:</b> <input type='text' name='login'><br>
<b>haslo:</b> <input type='password' name='haslo'><br>
<input type='submit' value='Wyślij' name='wyslij'>
</form>

<?php
}
}
?>

Jak widać działa na bazie mysql'a
Tylko tak - mam serwer apache na którym działają strony i podstrony. I chciałem zrobić tak aby dopiero po zalogowaniu
użytkownik miał do nich dostęp. Czyli jak go przerobić aby np user po wpisaniu jakiejś podstrony obojętnie której ( oczywiście z tych które
zamieściłem na swoim serwerze apache) mógł z niej korzystać dopiero po zalogowaniu.
Był bym wdzięczny za pomoc.

kamikaze · 2009-09-01 14:01:33

Na każdej podstronie sprawdzaj czy $_SESSION['zalogowany'] == true, jesli nie przekieruj na sronę logowania albo co tam chcesz.

Wieloryb · 2009-09-01 15:07:07

Po co Ci haslo uzytkownika w sesji i 2 zapytania do bazy danych?

jezoo · 2009-09-02 17:34:41

mozesz tez tak to zrobic (wersja do rozbudowy), wez pod uwage tworzenie tpl;ow

Kod:

<?php
session_start();
ob_start();

function checkUser($login){
    $sql = "SELECT COUNT(*) FROM tabela WHERE login='$login'";
    $sql = mysql_query($sql);
    $sql = mysql_fetch_array($sql);
    
    if($sql[0] == 1)
        return true;
    else
        return false;
}

function authenticate($login,$pass){
    $sql = "SELECT COUNT(*) FROM tabela WHERE login='$login' AND passwd='$pass'";
    $sql = mysql_query($sql);
    $sql = mysql_fetch_array($sql);
    if($sql[0] == 1){
        $_SESSION['zalogowany'] = true;
        $_SESSION['login'] = $login;
        header('Location: ./');
    }else
        header('Location: ./');
}


if(checkUser($login)==true && isset($_SESSION['zalogowany'])){
    switch ($_REQUEST['url']){
        case "podstrona1":
            include('podstrona1.phtml');
        break;
        
        default:
            echo "Witaj wodzu!";
        break;
    }
    
} elseif($_REQUEST['akcja']=='logowanie'){
    authenticate($login,$pass);
}
else{
// formularz logowania
}
?>

kamikaze · 2009-09-03 08:43:20

Imho checkUser() jest zbędna. I po co ob_start()?

jezoo · 2009-09-11 22:15:39

kamikaze, nie do konca, jezeli masz do sprawdzenia usera a login lub id jest w sesji zaszyte to mozesz kontrolowac w if;ie, a co do ob_start() bufor i naglowki, przydatna rzecz, jezeli ktos nie chce to nie musi uruchamiac.kiedys mialem problem z sesjami i to mi pomoglo

Time (s)	Query
0.00009	SET CHARSET latin2
0.00004	SET NAMES latin2
0.00117	SELECT u., g., o.logged FROM punbb_users AS u INNER JOIN punbb_groups AS g ON u.group_id=g.g_id LEFT JOIN punbb_online AS o ON o.ident='3.15.203.246' WHERE u.id=1
0.00063	REPLACE INTO punbb_online (user_id, ident, logged) VALUES(1, '3.15.203.246', 1732845747)
0.00035	SELECT * FROM punbb_online WHERE logged<1732845447
0.00053	SELECT t.subject, t.closed, t.num_replies, t.sticky, f.id AS forum_id, f.forum_name, f.moderators, fp.post_replies, 0 FROM punbb_topics AS t INNER JOIN punbb_forums AS f ON f.id=t.forum_id LEFT JOIN punbb_forum_perms AS fp ON (fp.forum_id=f.id AND fp.group_id=3) WHERE (fp.read_forum IS NULL OR fp.read_forum=1) AND t.id=14903 AND t.moved_to IS NULL
0.00023	SELECT search_for, replace_with FROM punbb_censoring
0.00208	SELECT u.email, u.title, u.url, u.location, u.use_avatar, u.signature, u.email_setting, u.num_posts, u.registered, u.admin_note, p.id, p.poster AS username, p.poster_id, p.poster_ip, p.poster_email, p.message, p.hide_smilies, p.posted, p.edited, p.edited_by, g.g_id, g.g_user_title, o.user_id AS is_online FROM punbb_posts AS p INNER JOIN punbb_users AS u ON u.id=p.poster_id INNER JOIN punbb_groups AS g ON g.g_id=u.group_id LEFT JOIN punbb_online AS o ON (o.user_id=u.id AND o.user_id!=1 AND o.idle=0) WHERE p.topic_id=14903 ORDER BY p.id LIMIT 0,25
0.00075	UPDATE punbb_topics SET num_views=num_views+1 WHERE id=14903
Total query time: 0.00587 s

Forum Debian Users Gang

Ogłoszenie

#1 2009-09-01 13:32:11

debbie - Użytkownik

Skrypt logowania + php + dla wszystkich skryptów

Kod:

#2 2009-09-01 14:01:33

kamikaze - Administrator

Re: Skrypt logowania + php + dla wszystkich skryptów

#3 2009-09-01 15:07:07

Wieloryb - Użytkownik

Re: Skrypt logowania + php + dla wszystkich skryptów

#4 2009-09-02 17:34:41

jezoo - Dzięcioł

Re: Skrypt logowania + php + dla wszystkich skryptów

Kod:

#5 2009-09-03 08:43:20

kamikaze - Administrator

Re: Skrypt logowania + php + dla wszystkich skryptów

#6 2009-09-11 22:15:39

jezoo - Dzięcioł

Re: Skrypt logowania + php + dla wszystkich skryptów

Stopka forum

Informacje debugowania