Forum PHP.pl > [PHP OOP]Wywoływanie bezpośrednie

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [PHP OOP]Wywoływanie bezpośrednie

Levabul

14.07.2005, 16:34:29

Tak wiec mam problem. Mam dwie klasy : klasę mysql i klasę session nie są one w żaden sposób dziedziczne ani podpięte do innych klas. kod wygląda mniej więcej tak:

[PHP] pobierz, plaintext 
<?php
 
class mysql {
#...
function query ($sql) {
		 $this -> sql_query = mysql_query ($sql, $this -> handle);
}
#...
 
class session {
#...
function auth_user () {
		 if (!empty($_POST['login']) && !empty($_POST['password'])) {
			 $username = $_POST['login'];
			$password = md5($_POST['password']);
			 mysql::query (&#092;"SELECT user_id FROM lms_users WHERE username = '$username' AND password = '$password'\");;
			if (mysql::num_rows() == 1) {
			   $row = mysql::fetch_array ();	
			   $this -> user_id	= $row['id'];   
			   return true;
			}
			else 
				 return false;
		 }
		 else
			  return false;
}
#...
}
 
?>
[PHP] pobierz, plaintext

Dlaczego gdy wywołuje mysql::query wyskakuje mi błąd:

Kod

Warning: mysql_query(): supplied argument is not a valid MySQL-Link resource in c:\usr\krasnal\www\php-lms\databases\mysql.php on line 14

Gdy pomienię mysql::query na mysql_query wszystko działa!

Z góry thx za pomoc.

Nievinny

14.07.2005, 16:48:49

Bo $this->handle nie jest obiektem połączenia MySQL, naucz się czytać komunikaty.

Levabul

14.07.2005, 17:01:53

Sry za moją głupotę ale jestem świeżo po nauce OOP (wcześniej pracowałem w strukturalnym. Co na to poradzić (na ten błąd oczywiście)

dasko

14.07.2005, 18:17:32

Eh, ten komunikat jest w tym wypadku nieco mylący. A błąd jest wywalany dlatego, że wywołując metodę poprzez czterokropek(::, T_PAAMAYIM_NEKUDOTAYIM, T_DOUBLE_COLON, czy jak mu tam

), nie można użyć w jej wnętrzu $this - to tak jakby metoda klasy, a nie metoda obiektu. Napisz po prostu tak:

[PHP] pobierz, plaintext 
<?php
 
$db = new mysql;
// ...
// i tutaj wykonujesz zapytanie
$db->query(&#092;"SELECT user_id FROM lms_users WHERE username = '$username' AND password = '$password'\");
// ...
 
?>
[PHP] pobierz, plaintext

Powinno działać

Levabul

14.07.2005, 18:32:15

niestety na powyższe rozwiązanie nie moge sobie pozwolić :/

a jeżeli mysql::query przerobiłbym na :

[PHP] pobierz, plaintext 
<?php
 
function query ($sql, $handle = $this -> handle) {
		 $this -> sql_query = mysql_query ($sql, $handle);
}
#i dodal dodatkowa funkcje
function get_handle () {
		 return $this -> handle;
}
 
#a zapytanie napisal tak
			 mysql::query (&#092;"SELECT user_id FROM lms_users WHERE username = '$username' AND password = '$password'\", mysql::get_handle());
 
 
?>
[PHP] pobierz, plaintext

To zadziała

Ozzy

14.07.2005, 18:54:00

W gre wchodzi tylko self::$handle lub mysql::$handle, o ile pole jest statyczne

Levabul

14.07.2005, 19:00:31

ale jeżeli zamiast mysql::get_handle () wpisze mysql::$handle wyświtla mi brak '('

Kod

Parse error: parse error, expecting `'('' in c:\usr\krasnal\www\php-lms\common\session.php on line 22

dasko

14.07.2005, 19:05:13

Przeciez napisalem ze jak wywolujesz przez :: to nie możesz użyć $this. Nie rozumiem dlaczego nie możesz sobie pozwolić na ten sposób - jedynym problemem moze byc tworzenie dodatkowych obiektów, które łatwo można rozwiązać singletonem.

Odsyłam do manuala -> http://www.php.net/manual/en/language.oop.php - bo z tego co widze, korzystasz z php4.

Levabul

14.07.2005, 19:09:13

tak niestety korzystam z php 4 :/

a co do twojego sposobu - ja chcę urzywać mysql wewnątrz innych klas a przy takim rozwiązaniu wyskakuje mi błąd o używaniu non-objekt czy coś takiego ...

dasko

14.07.2005, 19:12:33

Pokaż kod - i dokładnie pokaż błąd.

Levabul

14.07.2005, 19:19:50

Oto kod:
session.php

[PHP] pobierz, plaintext 
<?php
define ('SESSION_TIME', 24*3600);
class session{
 
function session () {
		 ob_start();
}
function is_session () {
		 if(strlen($_COOKIE['session_id']) == 32) {
										  $this -> session_id = $_COOKIE['session_id'];
										 setcookie ('session_id', $this ->session_id, time()+ SESSION_TIME);
										 $this -> sql_session_id = mysql::real_escape_string ($this -> session_id);
										 return true;
		}
		else
										 return false;													
}
function auth_user () {
		 if (!empty($_POST['login']) && !empty($_POST['password'])) {
			 $username = $_POST['login'];
			$password = md5($_POST['password']);
			 mysql::query (&#092;"SELECT user_id FROM lms_users WHERE username = '$username' AND password = '$password'\");
			if (mysql::num_rows() == 1) {
			   $row = mysql::fetch_array ();	
			   $this -> user_id	= $row['id'];   
			   return true;
			}
			else 
				 return false;
		 }
		 else
			  return false;
}
function new_session () {
		 if(getenv('HTTP_X_FORWARDED_FOR') != '')
						   $this -> ip = (!empty($_SERVER['HTTP_X_FORWARDED_FOR'])) ? $_SERVER['HTTP_X_FORWARDED_FOR'] : (!empty($_ENV['HTTP_X_FORWARDED_FOR'])) ? $_ENV['HTTP_X_FORWARDED_FOR'] : $HTTP_X_FORWARDED_FOR;
		 else
							$this -> ip = (!empty($_SERVER['REMOTE_ADDR'])) ? $_SERVER['REMOTE_ADDR'] : (!empty($_ENV['REMOTE_ADDR'])) ? $_ENV['REMOTE_ADDR'] : $REMOTE_ADDR; 
		 $this -> session_id = md5 (uniqid(str_replace (&#092;".\", \"\", $this -> ip).time ()));
				if ($this -> auth_user ()) {
					 $this -> type = &#092;"human\";
					  setcookie (&#092;"session_id\", $this -> session_id, time () + SESSION_TIME);
					  mysql::query (&#092;"INSERT INTO \".$db_prefix.\"sessions (session_id, session_user, session_ip) VALUES ('$this -> session_id', '$this -> user_id', 'this ->ip')\");
				}
				 else {
						$this -> browser = (!empty($_SERVER['HTTP_USER_AGENT'])) ? $_SERVER['HTTP_USER_AGENT'] : $_ENV['HTTP_USER_AGENT'];
					   if(preg_match('/(lycos|googlebot|slurp@inktomi|ask jeeves|whatuseek|ia_archiver)/i', $this -> browser))
							   $this -> type = &#092;"bot\";
					else {
							$this -> type = &#092;"guest\";
							setcookie (&#092;"session_id\", $this -> session_id, time () + SESSION_TIME);
							mysql::query (&#092;"INSERT INTO \".$db_prefix.\"sessions (session_id, session_user, session_ip) VALUES ('$this -> session_id', '0', 'this ->ip')\");
						}
			   }
}
function load_session () {
		 if ($this -> type == &#092;"human\") {
			 mysql::query(&#092;"SELECT * FROM \".$db_prefix.\"sessions WHERE session_id = '$_COOKIE[session_id]'\");
			$sessiondata = mysql::fetch_array();
			$this -> session_id = $sessiondata['session_id'];
			$this -> session_user = $sessiondata['session_user'];
			$this -> session_ip = $sessiondata['session_ip'];
			if ($this -> session_ip !== $this -> ip)
			   $this -> type = &#092;"guest\";
			else
			   $this -> type = &#092;"human\";		
		 }
}
 
}
?>
[PHP] pobierz, plaintext

mysql.php

[PHP] pobierz, plaintext 
<?php
class mysql {
var $handle;
var $sql_query;
function mysql ($host, $login, $password) {
		 $this -> handle = mysql_connect ($host, $login, $password);
}
function select_db ($name) {
		 mysql_select_db ($name, $this -> handle);
}
function query ($sql, $handle = $this -> handle) {
		 $this -> sql_query = mysql_query ($sql, $handle);
}
function num_rows () {
		 return  mysql_num_rows ($this -> sql_query, $this -> handle);
}
function real_escape_string ($str) {
		 mysql_real_escape_string ($str);
}
function fetch_array () {
		 return mysql_fetch_array ($this -> sql_query, $this -> handle);
}
function free_result () {
		 mysql_free_result ($this -> sql_query, $this -> handle);
}
function close () {
		 mysql_close ($this -> handle);
}
}
?>
[PHP] pobierz, plaintext

przyklad.php

[PHP] pobierz, plaintext 
<?php
include ('./config.cfg');
include ('./databases/mysql.php');
include ('./common/session.php');
$db = new mysql (&#092;"localhost\", \"root\", \"krasnal\");
$db -> select_db (&#092;"lms\");
$session = new Session;
if (!empty($_GET['login']) && !empty($_GET['password']))
   $session -> new_session ();
else {
echo 'brak danych';
	if ($session -> is_session())
	   $session -> load_session();
	else
		$session -> new_session ();   
}
?>
[PHP] pobierz, plaintext

Oto i cały kod. A poniżej błędy.

Błędy

Kod

Warning: mysql_query(): supplied argument is not a valid MySQL-Link resource in c:\usr\krasnal\www\php-lms\databases\mysql.php on line 14

Warning: Wrong parameter count for mysql_num_rows() in c:\usr\krasnal\www\php-lms\databases\mysql.php on line 17

Warning: mysql_query(): supplied argument is not a valid MySQL-Link resource in c:\usr\krasnal\www\php-lms\databases\mysql.php on line 14

Ociu

15.07.2005, 00:19:48

Singleton ?
Poza tym coś mi się wydaje, że masz błąd w zapytaniu.

Nievinny

15.07.2005, 07:51:32

[PHP] pobierz, plaintext 
<?php
 
//zamiast
mysql::query
//używaj 
$this->link = new mysql(); 
$this->link->query( '...' );
 
?>
[PHP] pobierz, plaintext

dasko

15.07.2005, 11:01:27

@Nievinny: patrz kilka postów wyżej - to samo pisałem.

@Levabul: no i co z tego, że w przykładzie zmieniłeś tak jak mówiłem, skoro w klasie session dalej z premedytacją używasz mysql::query, zamiast kulturalnie najpierw stworzyć instancje(tudzież zainicjować połączenie z bazą), a potem wywoływac zapytania. Po drugie mysql_num_rows" title="Zobacz w manualu PHP" target="_manual, przyjmuje jeden argument, a ty na siłe wciskasz mu dwa. Czytaj manuala.

W ogóle sterownik do bazy źle zaprojektowany - cały czas musisz tworzyć nowy obiekt. Przypominam, że w PHP4 też da się zaimplementowac singletony.

Myślę, że najpierw warto troche lepiej poznać podstawy, nauczyć się czytac komunikaty o błędach(podstawowy angielski), a potem zabierać się za CMSa.

Levabul

15.07.2005, 13:39:56

Spokojnie ten sterownik do bazy jest TYMCZASOWY. We właściwym będzie jeszcze cashowanie, obsługa innych baz itd. Ten został stworzony na szybkiego abym mógł stworzyć inne części jądra (od tak mi się zachciało

)

Dzięki za dokładną odpowiedź, poprostu wcześniej nie zrozumiałem twoich rad Dasko

.

A i tak wszystko rozwiązałem singletonem

(thx Ociu)

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.