php - Fatal error: Call to a member function - Udvikleren.dk

Tags: php

<< < 1 2 > >>

Bruger #15047 @ 18.05.10 15:17

Hejsa!

Jeg har et problem med noget OOP. Jeg har aldrig haft dette problem før og jeg forstår, samt kan ikke se hvor problemet er.-

Er er min klasse..
class.mysql.php

PHP kode

 class Database{
     var $connection;
     function Database() {
         $this->connection = mysql_connect(localhost, root) or die(mysql_error());
         mysql_select_db(test1, $this->connection) or die(mysql_error());
     }
 
     function Close() {
         mysql_close($this->connection);
     }
     
     function SecureInput($input) {
         if(!get_magic_quotes_gpc()){
             $input = addslashes($input);
         }    
         return htmlentities($input);    
     }
     function get_info($table, $boolean) {
         $table = $this->SecureInput($table);
         $boolean = $this->SecureInput($boolean);
 
         $query = "SELECT * FROM " . TBL_PREFIX . $table . " $boolean";
         $result = mysql_query($query, $this->connection);
 
         if(!$result || (mysql_numrows($result) < 1)){
             return NULL;
         }
 
         $dbarray = mysql_fetch_array($result);
         return $dbarray;
     }
 } 
 $database = new Database;

Her kalder jeg get_list i en anden klasse, som til sidst bliver kaldt.
class.rights.php

PHP kode

 class rights {
         function admin($username) {
             $bob = $database->get_info(not, "WHERE id = '1'");
             if(in_array('Z', $bob)) {
                 return true;
             } else {
                 return false;
             }
         
         }    
     }
     $rights = new rights;

Og her kalder jeg den:
if($rights->admin($_COOKIE['login']) == false) { die(); }

Og fejlen er:
Fatal error: Call to a member function get_info() on a non-object in C:\htdocs\test\system\class\rights.class.php on line 4

Mvh. Magnus.

Indlæg senest redigeret d. 18.05.2010 15:18 af Bruger #15047

Bruger #2695 @ 18.05.10 16:20

1.963

Du har ikke en lokal variabel med navnet $database. Den skal du nok give med som parameter til 'admin' funktionen.

Bruger #15047 @ 18.05.10 17:13

Hvordan får jeg den så lokal? Jeg har den inkluderet

index.php

PHP kode

 <?php
     include("init.php");
     include("system/class/mysql.class.php"); // Her
     include("system/class/rights.class.php"); // Her 
 ?>

Indlæg senest redigeret d. 18.05.2010 17:14 af Bruger #15047

Bruger #2695 @ 18.05.10 17:23

1.963

Hvordan får jeg den så lokal? Jeg har den inkluderet

index.php

PHP kode

 <?php
     include("init.php");
     include("system/class/mysql.class.php"); // Her
     include("system/class/rights.class.php"); // Her 
 ?>

Kode

class rights {
 function admin($username) {
   global $database;
   $bob = $database->get_info(not, "WHERE id = '1'");
   ....

...eller nok lidt kønnere:

Kode

class rights {
 function admin($database, $username) {
   $bob = $database->get_info(not, "WHERE id = '1'");
   ....

Bruger #15047 @ 18.05.10 17:55

Tusinde tak, nu virker det

Bruger #7728 @ 18.05.10 19:19

243

undgå gerne at bruge global...

Lav det hellere i starten af dit objekt eller lav et instance som er static...

Bruger #15047 @ 18.05.10 21:58

undgå gerne at bruge global...

Lav det hellere i starten af dit objekt eller lav et instance som er static...

Hvorfor skal jeg undgå det?
Og hvordan laver jeg det statisk?

Bruger #2695 @ 18.05.10 22:16

1.963

undgå gerne at bruge global...

Lav det hellere i starten af dit objekt eller lav et instance som er static...

Hvorfor skal jeg undgå det?
Og hvordan laver jeg det statisk?

Globale variable er tvivlsomt design som giver en grim binding som kan være svær at slippe af med når man en dag vil ændre sit design eller måske endda implementere unit testing.

Men statiske variable er ikke meget bedre. Dependency injection er en hel del bedre...altså at du leverer din database reference til metoden når den kaldes istedet for at den finder den et sted, som kunne ændre sig en dag.

Bruger #15047 @ 19.05.10 13:55

Hejsa.

Jeg har læst lidt om "Dependency injection".
Og er kommet frem til dette..

PHP kode

 class Factory {
  
     public static $_database;
  
     public static function makeBook() {
  
         $book = new Book();
         $book->setDatabase(self::$_database);
         // more injection...
  
         return $book;
     }
  
 }

Kilde: http://www.potstuck.com/2009/01/08/php-dependency-injection/

Er det sådan at jeg skal bygge min mysql.class.php op?

Bruger #2695 @ 19.05.10 14:38

1.963

Hejsa.

Jeg har læst lidt om "Dependency injection".
Og er kommet frem til dette..
PHP kode
 class Factory {
  
     public static $_database;
  
     public static function makeBook() {
  
         $book = new Book();
         $book->setDatabase(self::$_database);
         // more injection...
  
         return $book;
     }
  
 }
Kilde: http://www.potstuck.com/2009/01/08/php-dependency-injection/

Er det sådan at jeg skal bygge min mysql.class.php op?

Hvis det giver mening for dig. Der er ikke noget korrekt svar på den slags. Public static er ikke meget forskellig fra globale variable og derfor også lidt skidt. Det kunne måske give større fleksibilitet at give en database reference med til makeBook() metoden:

Kode

	public static function makeBook(Database $db) {
 
		$book = new Book();
		$book->setDatabase($db);
		// more injection...
 
		return $book;
	}

Bruger #15047 @ 19.05.10 14:48

Hejsa.

Jeg har læst lidt om "Dependency injection".
Og er kommet frem til dette..
PHP kode
 class Factory {
  
     public static $_database;
  
     public static function makeBook() {
  
         $book = new Book();
         $book->setDatabase(self::$_database);
         // more injection...
  
         return $book;
     }
  
 }
Kilde: http://www.potstuck.com/2009/01/08/php-dependency-injection/

Er det sådan at jeg skal bygge min mysql.class.php op?
Hvis det giver mening for dig. Der er ikke noget korrekt svar på den slags. Public static er ikke meget forskellig fra globale variable og derfor også lidt skidt. Det kunne måske give større fleksibilitet at give en database reference med til makeBook() metoden:
Kode
	public static function makeBook(Database $db) {
 
		$book = new Book();
		$book->setDatabase($db);
		// more injection...
 
		return $book;
	}

Og hvordan kalder jeg så databasen?

Jeg kan ikke få den til at tislutte med kodeord og brugernavn, samt extern db.

Jeg prøver lige denne kilde:
http://refactormycode.com/codes/416-mysql-original-php-5-wrapper-class

Jeg har fået lavet dette og det virker ren faktisk:

PHP kode

 <?php
 class mySQL
 {
     const db_host = 'localhost';        // host
     const db_base = 'spx';        // database  
     const db_user = 'root';        // user
     const db_pass = '';        // password
     private static $db_link = false;    // link
     private static $db_connected = false;    // connection established?
     
     //------------------------------
     // Constructor + Destructor
     
     public function __construct()
     {
         self::connect();
     }
     
     public function __destruct()
     {
         self::disconnect();
     }
     
     //------------------------------
     // Connect + Disconnect
     
     public static function connect()
     {
         if(!self::$db_connected)
         {
             self::$db_link = mysql_connect(self::db_host, self::db_user, self::db_pass);
         }
         return $db_connected = true;
     }
     public static function disconnect() 
     {
         self::$db_link = mysql_close(self::$db_link);
     }
 }
 $db = new mySQL;
 ?>

Indlæg senest redigeret d. 19.05.2010 15:44 af Bruger #15047

<< < 1 2 > >>

Fatal error: Call to a member function

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler