Objektówka, niby brak błędów a nie działa Opcje

[primo]

Witam,

próbuję zrozumieć działanie prostego skryptu pobierającego dane z bazy, niby błędy nie wyskakują (nawet jak coś celowo usunę z kodu i powinno je pokazać)

[PHP] pobierz, plaintext 
<?php
require_once(&#092;"BD.class.php\");
 
	class Widget {
		  private $id;
		private $name;
		private $description; 
		private $bd;
		private $needsUpdating = false;
 
 
		public function _construct($widgetID){
 
		$bd = new BD;
 
		if (!$bd->otworz()) {
		die ($bd->blad());
		}
 
		if (!$bd->zapytaj(&#092;"SELECT \"nazwa\", \"opis\" FROM artykul WHERE nr_artykulu = $widgetID\")) {
		die ($db->error());
		}
 
		if (!pobierzTablice) {
			throw new Exception ('Szukanego artykułu nie ma w bazie !');
		}
 
		$bd->pobierzTablice();
 
		$data->pobierzTablice();
		$this->id = $widgetID;
		$this->name = $data['nazwa'];
		$this->description = $data['opis'];		
	}
 
	public function getName() {
		return $this->name;
	}
 
	public function getDescription() {
		return $this->description;
	}
 
	public function setName($name) {
		$this->name = $name;
		$this->needsUpdating = true;
	}
 
	public function setDescription($description) {
		$this->description = $description;
		$this->needUpdating = true;
	}
 
	public function _destruct () {
		if (! $this->needsUpdating){
		return;
	}
 
	$bd = 'UPDATE \"artykul\" SET';
	$bd.= &#092;"\"nazwa\" = '\" .msql_escape_string($this->name) . \"', \";
	$bd.= &#092;"\"opis\" = '\" .msql_escape_string($this->description) . \"' \";
	$bd.= &#092;"WHERE nr_artykulu = \" .$this->id;
 
	}
 
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
require_once('class.Widget.php');
 
		try {
			$objWidget = new Widget (1);
 
			print &#092;"Nazwa artykułu: \" .$objWidget->getName() .\"<br>n\";
			print &#092;"Opis artykułu: \" .$objWidget->getDescription() . \"<br>n\";
 
			$objWidget->setName ('Trampki');
			$objWidget->SetDescription ('Trampki o dużym przebiegu !');
		} catch (Exception $e) {
			die (&#092;"Wystąpił problem: \" .$e->getMessage());
		}
?>
[PHP] pobierz, plaintext 

jak odpalę ten drugi skrypt to pokazuje mi tylko w przeglądarce:
Nazwa artykułu:
Opis artykułu:

nic poza tym.

macie jakieś pomysły ?

[czachor]

[PHP] pobierz, plaintext 
<?php
$bd = 'UPDATE \"artykul\" SET';
	$bd.= &#092;"\"nazwa\" = '\" .msql_escape_string($this->name) . \"', \";
	$bd.= &#092;"\"opis\" = '\" .msql_escape_string($this->description) . \"' \";
	$bd.= &#092;"WHERE nr_artykulu = \" .$this->id;
?>
[PHP] pobierz, plaintext 

to Ci się luzem wala...

[Imperior]

Widzę tutaj dwa zasadnicze błędy:
1. metody specjalne zaczynają się od __ zatem powinno być:

[PHP] pobierz, plaintext 
<?php
public function __construct(...) {}
public function __destruct(...) {}
?>
[PHP] pobierz, plaintext 

2. Dwie rzeczy są ni z gruszki ni z pietruszki:

[PHP] pobierz, plaintext 
<?php
if (!pobierzTablice) {...}
?>
[PHP] pobierz, plaintext 

Oraz:

[PHP] pobierz, plaintext 
<?php
$data->pobierzTablice();
?>
[PHP] pobierz, plaintext 

Skąd to się wzieło (w drugim chodzi mi o $data)?

--EDIT--

No i własnie jeszcze SQL... nazwy pól masz zapisane jako stringi...
Używaj `pole`.

--EDIT2--

Dopiero teraz zrozumiałem post poprzednika
Tworzysz zapytanie, ale go nie wykonujesz.
No i jeszcze podstawianie zapytania do zmiennej o takiej samej nazwie, jak wcześniej była baza danych, daje nieczytelny kod.

Ten post edytował Imperior 26.05.2005, 13:08:25

[primo]

Ad. 1 -- poprawione
Ad. 2

poprawiłem na:

[PHP] pobierz, plaintext 
<?php
$data = $bd->pobierzTablice();
?>
[PHP] pobierz, plaintext 

funkcja ta pochodzi z includowanej clasy Baza - msql_fetch_array

Ad. 3

[SQL] pobierz, plaintext 
$sql = 'UPDATE "artykul" SET';
	$sql.= "\"nazwa\" = $this->name ,";
	$sql.= "\"opis\" = $this->description";
	$sql.= "WHERE nr_artykulu = " .$this->id;
[SQL] pobierz, plaintext 

a co dalej to nie mam zielonego pojęcia ;(

[Imperior]

No jeszcze pozostało Ci wykonać to zapytanie (czyli tak jak u góry z tym bd->zapytaj() )

[primo]

[PHP] pobierz, plaintext 
<?php
if (!$bd->zapytaj(&#092;"UPDATE artykul SET nazwa = $this->name, opis = $this->description WHERE nr_artykulu = $this->id\")) {
		die ($bd->blad());
	}
?>
[PHP] pobierz, plaintext 

Wyskakuje komunikat o błędzie:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'o dużym przebiegu ! WHERE nr_artykulu = 1' at line 1

[nospor]

ale napisy w mysql ttrzeba w ciapkach dawac:

[PHP] pobierz, plaintext 
<?php
 
if (!$bd->zapytaj(&#092;"UPDATE artykul SET nazwa = '$this->name', opis = '$this->description' WHERE nr_artykulu = $this->id\")) {
        die ($bd->blad());
    }
 
?>
[PHP] pobierz, plaintext 

że już nie wspomnę o ich ewentualnym escapowaniu

Ten post edytował nospor 27.05.2005, 08:35:12

[primo]

że już nie wspomnę o ich ewentualnym escapowaniu

a co to oznacza

Wszystkie błędy usunięte, a jak nie było pobrania z bazy tak i dalej nie ma. A cholewcia obiektówka miała być o wiele łatwiejsza a nie mogę nawet rekordu z bazy wyciągnąć, co dopiero przerobić kod strony (za 5 lat się odezwę)

[nospor]

escapowanie: mysql_escape_string

a nie mogę nawet rekordu z bazy wyciągnąć

Nie żebym się czepial, ale update nie slyzu do wyciągania danych

edit: aha, nie doczytalem calego topicu
może pokaż jeszcze raz caly oprawiony kod

Ten post edytował nospor 27.05.2005, 08:58:52

[primo]

no bez przesady, przecież jest powyżej kod z zapytaniem select.

dopisałem jeszcze:

[PHP] pobierz, plaintext 
<?php
if (!$sql = $bd->zapytaj(&#092;"UPDATE artykul SET nazwa = '$this->name', opis = '$this->description' WHERE nr_artykulu = $this->id\")) {
		die ($bd->blad());
	}
 
	$rs = $bd->zapytaj($this->bd, $sql);
		if(! is_resource($rs)) {
			throw new Exception('Wystąpił błąd podczas aktualizacji bazy danych');
	}
 
	$bd->zamknij($this->bd);
?>
[PHP] pobierz, plaintext 

escapowanie: mysql_escape_string

jak teraz zescapuje to zapytanie to czy przypadkiem nie wyskoczy błąd że mój obiekt BD tego nie obsługuje

[nospor]

jak teraz zescapuje to zapytanie

Ale ty nie masz apytania escxapować, a wartości tekstowe, czyli np mysql_escape_string($this->name)
Miales to robione wyżej w postach

[primo]

[SQL] pobierz, plaintext 
IF (!$sql = $bd->zapytaj("UPDATE artykul SET nazwa = '". mysql_escape_string ($this->name)."', opis = '". mysql_escape_string ($this->description)."' WHERE nr_artykulu = $this->id")) {
		die ($bd->blad());
	}
[SQL] pobierz, plaintext 

oki poszło.

jak dodałem ten kod poniżej zapytania to mam błąd: Fatal error: Exception thrown without a stack frame in Unknown on line 0

cały poprawiony kod:

[PHP] pobierz, plaintext 
<?php
require_once(&#092;"BD.class.php\");
 
	class Widget {
		  private $id;
		private $name;
		private $description; 
		private $bd;
		private $needsUpdating = false;
 
 
		public function __construct($widgetID){
 
		$bd = new BD;
 
		if (!$bd->otworz()) {
		die ($bd->blad());
		}
 
		if (!$bd->zapytaj(&#092;"SELECT nazwa, opis FROM artykul WHERE nr_artykulu = $widgetID\")) {
		die ($bd->blad());
		}
 
		$data = $bd->pobierzTablice();
		$this->id = $widgetID;
		$this->name = $data['nazwa'];
		$this->description = $data['opis'];		
	}
 
	public function getName() {
		return $this->name;
	}
 
	public function getDescription() {
		return $this->description;
	}
 
	public function setName($name) {
		$this->name = $name;
		$this->needsUpdating = true;
	}
 
	public function setDescription($description) {
		$this->description = $description;
		$this->needUpdating = true;
	}
 
	public function __destruct () {
		if (! $this->needsUpdating){
		return;
	}
 
	$bd = new BD;
 
	if (!$bd->otworz()) {
		die ($bd->blad());
	}
 
	if (!$sql = $bd->zapytaj(&#092;"UPDATE artykul SET nazwa = '\". mysql_escape_string ($this->name).\"', opis = '\". mysql_escape_string ($this->description).\"' WHERE nr_artykulu =\" .$this->id)) {
		die ($bd->blad());
	}	
 
	$bd->zamknij($this->bd);
 
	}
 
}
?>
[PHP] pobierz, plaintext 

Ten post edytował primo 27.05.2005, 09:07:21

[nospor]

no ale sluchaj. Robisz zabytanie update, a potem sprawdzzasz czy wynik tego zapytania jest resource. Jelsi nie to wyrzucasz wyjątek. Jaki sens i logika? przecież zapytania typu update zwracają true lub false i nigdy nie będą resorcem, więc zawsze po takim zapytaniu wywali ci wyjątek

[primo]

właśnie to dostrzegłem i wywaliłem ale dzięki