Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> ADODB i inna klasa ...
Luke
post
Post #1





Grupa: Przyjaciele php.pl
Postów: 215
Pomógł: 0
Dołączył: 28.10.2003
Skąd: -

Ostrzeżenie: (0%)
-----


[php:1:f3d8be1558]<?php
class usersAuthenticate {

var $login;
var $password;
var $db;

function loginUser() {
if(empty($this->login) || empty($this->password)) {
return(false);
} else {
$row = $this->db->GetRow("SELECT password, date, userName FROM users WHERE login='".$this->login."'");
}

if($row[0] != md5($this->password)) {
return(false);
} else {
$this->db->Execute("UPDATE users SET session='".session_id()."', ip='".$_SERVER['REMOTE_ADDR']."', date=NOW() WHERE login='".$this->login."'");
}

if(!$this->db->Affected_Rows()) {
return(false);
} else {
$_SESSION['date'] = $row[1];
$_SESSION['pelnaNazwa'] = $row[2];
return(true);
}
}

?>[/php:1:f3d8be1558]

Czy odwolanie sie do ADODB (po przez $db) w tym fragmencie klasy to jedyny sposob? Czy jest to optymalne rozwiazanie? Czy mozna zrobic to w inny sposob np. przez "extends"? Jakie są Wasze rozwiązania i doświadczenia?
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi
Nalfein][WR
post
Post #2





Grupa: Zarejestrowani
Postów: 66
Pomógł: 0
Dołączył: 22.04.2003
Skąd: Żory / K-ce

Ostrzeżenie: (0%)
-----


Luca: a o co Ci chodzi? O to, że nie chce Ci się odwoływywać przez właściwość? Jeśli tak, to napisz na początku każdej takiej metody:

Kod
$db = &$this->db;


i używaj $db tak jakbybyś używał go po global. Używanie zmiennych globalnych to nieelegancki nawyk - wytrych, powinno się go unikać jak ognia.
Go to the top of the page
+Quote Post

Posty w temacie


Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Aktualny czas: 5.10.2025 - 20:51