Forum PHP.pl

Projekt, który wykonuje razem ze znajomymi na studiach wymaga ode mnie, abym przyswoił w dość szybkim czasie programowanie obiektowe:) jednak jak to zawsze bywa przy nauce, napotkałem pewien problem (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Mam taki skrypt:

[PHP] pobierz, plaintext 
<?php
 
class Druga{
 
public $host = "localhost";
public $user = "root";
public $pass = "";
 
public $zapytanie="SELECT * FROM users";
public $baza="dzienniczek";
 
function polaczenie(){
 
mysql_connect("$this->host", "$this->user", "$this->pass") or die ("Nie mozna sie polaczyc");
			if(mysql_connect){
				echo "Polaczenie do: ".$this->host. " udane";
			}
 
}
 
function wybierz(){
//echo $this->zapytanie;
$query=mysql_query($this->zapytanie);
 
		while($row=mysql_fetch_array($this->query)){
				echo $this->row['login'];
		}
}
 
}
 
 
$polacz=new Druga;
$polacz->polaczenie();
$wyswietl=new Druga;
$wyswietl->wybierz();
 
 
?>
[PHP] pobierz, plaintext 

W linii 25 otrzymuje, że jest błąd: Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in

Widać, że nie otrzymuje polecenia SQL-owego, jednak jak zrobie echo $this->zapytanie, zapytanie wyświetla mi się, więc nie wiem gdzie jest błąd. Proszę o pomoc:)

Przypisujesz w linii 23 wartość do $query a nie do $this->query, innymi słowy ustawioną masz zmienną lokalną, a pobierasz właściwość obiektu.

Zmień $this->query na $query

Niestety to nie rozwiązało problemu (IMG:http://forum.php.pl/style_emoticons/default/sad.gif) A mógłbyś mi wyjaśnić jeśli możesz oczywiście jeszcze jaśniej, różnicę pomiędzy zwykłą zmienną $query a $this->$query. Ja to rozumiem tak, że przez pisanie $this-> wskazuje konkretnie na tą zmienną.

Ten post edytował Danone 25.05.2007, 12:49:33

Danone, nie rozumiem dlaczego tworzysz jeden obiekt klasy w celu połączenia się z bazą, a potem nagle drugi, żeby coś z niej wydobyć. Wydaje mi się, że łatwiej byłoby operować na jednym obiekcie Twojej klasy. Odwołanie do $this->query to odwołanie do zmiennej, będącej właściwością obiektu klasy. Takiej zmiennej nie masz nigdzie zadeklarowanej. Poniżej jest Twój kod po małym tuningu (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) . W każdym razie, inaczej napisałbym tą klasę :roll2: . Staraj się też korzystać ze zmiennych prywatnych i chronionych, gdzie to tylko możliwe. Aha, nigdzie nie wybierałeś tabeli z bazy danych. Poprawione zostało tylko kilka rzeczy. Dalsza koncepcja zależy tylko i wyłącznie od Twojej inwencji.

[PHP] pobierz, plaintext 
<?php
 
class Druga{
 
public $host = "localhost";
public $user = "root";
public $pass = "";
public $query_result;
public $connection;
 
public $zapytanie="SELECT * FROM users";
public $baza="Twoja_baza";
 
public function polacz(){
 
$this->connection = mysql_connect("$this->host", "$this->user", "$this->pass") or die ("Nie mozna sie polaczyc");
			if(is_resource($this->connection)){
				echo "Polaczenie do: ".$this->host. " udane";
			}
mysql_select_db($this->baza);
 
}
 
public function wybierz(){
$this->query_result=mysql_query($this->zapytanie);
 
		while($row=mysql_fetch_array($this->query_result)){
				echo $row['login'];
		}
}
 
}
 
 
$obiektDruga=new Druga();
$obiektDruga->polacz();
$obiektDruga->wybierz();
 
 
?>
[PHP] pobierz, plaintext 

Ten post edytował c0cr0ach 26.05.2007, 00:03:44

Wcześniej udało mi się to poprawić i jakoś działało, ale widzę, że twój przykład jest bardziej obiektowy niż mój (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Dzięki za pomoc (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)