Forum PHP.pl

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 (IMG:style_emoticons/default/wink.gif)

@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ł (IMG:style_emoticons/default/smile.gif)

kolega erix podał ci słowo kluczowe: wzorce projektowe. Jakbyś poczytał to byś nie tworzył takich potworków jak teraz (IMG:style_emoticons/default/tongue.gif)

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 (IMG:style_emoticons/default/smile.gif)

Temat do zamknięcia.


Pozdrawiam