Witam

Mam pewien problem z klasami i funkcjami, otóż mam klasę do wykonywania zapytań itp:

[PHP] pobierz, plaintext 
class db
   {
(...)
	public function query($query){
			$dbquery = $this ->dbquery = mysql_query($query);
				if($this ->dbquery ==false) {echo "Nie można wykonać zapytania,<br />";
		echo mysql_errno() . ": " . mysql_error(). "\n";
			}
		return $this->dbquery;
	}
 
(...)
}
[PHP] pobierz, plaintext 

i plik z różnymi funkcjami wywoływanymi ze skryptu np:

[PHP] pobierz, plaintext 
function login() {
(...)
	$db->query("INSERT INTO(...)");	
}
[PHP] pobierz, plaintext 

Moje pytanie brzmi, dlaczego wywala mi błąd gdy w funkcji "login" próbuję użyć "$db->query", zamiast tego muszę używać normalnie mysql_query i czy jest jakis sposób aby to obejśc lub wykonać?

a może tak najpierw?

[PHP] pobierz, plaintext 
$db = new db();
[PHP] pobierz, plaintext 

To akurat raczej logiczne.


Nie dodałem, że w skrypcie wszystko jest ok, tylko gdy użyję $db->query w jakiejś funkcji wtedy nie działa.

Oraz nie dodałeś co to za błąd Ci wywala.

Parafrazując autora tematu:
To raczej logiczne

@indexx nie pisz, że coś jest logiczne a coś innego nie, skoro dla Ciebie logicznie nie jest podanie dokładnie komunikatu błędu

Już daję, miał być wklejony pod kodem, przepraszam za błąd:



A kod odpowiedzialny:

Zasięg zmiennych

No i okazuje się, że wystarczyło pomyśleć.

Dzięki wielkie za pomoc.

Mam jeszcze pytanie czy lepiej teraz w każdej funkcji deklarować $db jako global i używać "$db->query", czy lepiej pozostać przy "mysql_query"

Żadne z powyższych.

Google: wzorce projektowe.

Ok, skleciłem coś takiego:

[PHP] pobierz, plaintext 
 class db
   {
(...)
 
	public function query($query){
			$dbquery = $this ->dbquery = mysql_query($query);
				if($this ->dbquery ==false) {echo "Nie można wykonać zapytania,<br />";
		echo mysql_errno() . ": " . mysql_error(). "\n";
			}
		return $this->dbquery;
	}
 
(...)
}
[PHP] pobierz, plaintext 

rozszerzenie klasy:

[PHP] pobierz, plaintext 
class display extends db {
 
	private $_all = array();
 
public function zlicz($tbl, $where, $paggin){
 
	$sql = "SELECT count(*) AS licz FROM ".$tbl." ".(isset($where) ? 'WHERE '.$where : '')."";
	$result = $this->policz($sql);
	(isset($paggin) ? $res = ceil($result[0]/10) : $res = $result[0]);
	return $res;	
}
public function select($dbs, $where, $order, $limit){
 
  if($where != "brak"){$warunki = "WHERE $where";}else{$warunki = "";}
  $sql = "SELECT * FROM $dbs $warunki ".(isset($order) ? 'ORDER BY '.$order.' DESC' : '')." LIMIT $limit;";
    if ($results = $this->query($sql)){
      $rs = array();
 
 
 
      while ($row = mysql_fetch_array($results)){
        $mx = array();
         foreach ($row as $key => $val):
           $mx[$key] = $val;
         endforeach; 
         $this->_all[] = $mx;
 
         };  
 
     // return $rs;
 
    }else{
      echo 'Błąd pobrania danych mysql: ' . $this->error;
      return false;
   }
}
public function wyswietl_comments(){ 
    $comments =  $this->_all;
  foreach($comments as $r)
        { 
echo '<div class="comment"> 	
				<span class="autor">
					<img src="'.avatar($r["author"]).'">
					<a href="/profil/'.$r["author"].'">'.$r["author"].'</a>
				</span>
				<span class="com_p">'.$r["date"].'</span>
			<span class="com_glosowanie">
					Ocena: <span class="com_punkty">'.$r["ocena"].'</span> 
					(głosów: <span class="com_count">'.$r["glosow"].'</span>)
		 | <a href="#" onclick="msg_raport('.$r["id"].', \'komentarz\'); return false;" id="msg_komentarz_'.$r["id"].'">Raportuj</a>
			</span>
					<div class="com_content">'.$r["content"].'</div>
					<a class="com_odpowiedz" href="/'.$r["id_demot"].'/#komentarze_wybor">[Zobacz demotywator]</a>			
	</div>';
        }
}
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
a w skrypcie wywołuję tak:
$display = new display();
$display->select("tabela", "author='$login' and activ='0'", "date", "$pages[1],$pages[2]");
echo $display->wyswietl_comments();
[PHP] pobierz, plaintext 

Moje pytanie, czy tak na początek może to wyglądać?
Czy za każdym razem muszę wywoływać

[PHP] pobierz, plaintext 
$display = new display();
[PHP] pobierz, plaintext 

czy mogę równie dobrze na początku każdego pliku(raczej chyba nie ale wolę się spytać)

Co mogę poprawić i jakie usprawnienia wprowadzić? Tylko proszę o konkretne uwagi bo kolega "erix" mnie wręcz załamał

kolega erix podał ci słowo kluczowe: wzorce projektowe. Jakbyś poczytał to byś nie tworzył takich potworków jak teraz

Po db nie powinna dziedziczyć żadna klasa do wyświetlania. To są zupełnie różne twory nie powiązane ze sobą przez dziedziczenie.

Poczytaj o wzorcu Rejestr, Singleton, poczytaj o MVC.

Owszem czytałem ale jak na mój poziom to jest rzekł bym "zbyt zawiłe", a najlepiej uczę się na przykładach, które rozumiem.

Może ktoś mógł by mniej więcej pokazać mi jak przekształcić to co mam w coś funkcjonalnego i poprawnego żebym mógł zrozumieć co robię źle?

Wzorce projektowe, konkretnie: MVC.

Jak tego nie zrozumiesz (głównego zamysłu), to zostaw programowanie.

Dobrze omówione wzorce są w książce "PHP. Zaawansowane programowanie."

Dzięki za porady, książkę(inną), pożyczyłem już wczoraj za namową znajomego gdy opisałem mu sprawę.

A ogólne zarysy już łapię, teraz trochę po kombinować i nie będzie problemu.



Chyba raczej nie miało by to sensu, to tak jakby murarz nie umiał kłaść tynków więc pracuje w MAcu.
Jak czegoś nie rozumiem to staram się dwa razy bardziej więc nie będzie problemu

Temat do zamknięcia.


Pozdrawiam