Witam

Napisalem taki skrypcik treningowy, szczerze to praktycznie przepisalem go z ksiazki, niestety w kodzie jak sie okazuje jest sporo bledow a errata online wydaje sie niekompletna. Poproawilem z 3 bledy ale z tym nie potrafie sobie poradzic.
Chodzi o zmiane wartosci w tabeli mysql przez wywolanie odpowiedniej metody w klasie.
Oto kod.

[PHP] pobierz, plaintext 
<?php
//class.Widget.php
 
class Widget {
 
    private $id;
    Private $name;
    private $description; private $hDB;
    private $needsUpdating = false;
    
    public function __construct($widgetID) {
        //parametr widgetID to klucz główny rekordu
        //w bazie danych zawierajacego dane
        //tego obiektu
        
        //tworzy uchwyt połaczenia i zachowuje go w prywatnej zmiennej skladowej
        $this->hDB = mysql_connect('localhost', 'root', "");
        if(!is_resource($this->hDB)) {
            throw new Exception ('Nie mozna polaczyc sie z baza danych.');
            }
            
        $db = mysql_select_db ('webserv',$this->hDB);
        
        $sql = "SELECT nazwa, opis FROM artykul WHERE nr_artykulu = '$widgetID' ";
        $rs = mysql_query($sql, $this->hDB);
        if(!is_resource($rs)) {
            throw new Exception ("Blad przy wykonywaniu instrukcji wyboru.");
            }
            
        if(!mysql_num_rows($rs)) {
            throw new Exception('Szukanego artykulu nie ma w bazie!');
            }
            
        $data = mysql_fetch_array($rs);
        $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->needsUpdating = true;
        }
    
    public function __desctruct() {
        if(!$this->needsUpdating) {
            return;
            }
        
        $sql = 'UPDATE artykul SET';
        $sql .= "nazwa = '" .mysql_escape_string($this->name). "'.";
        $sql .= "opis = '" .mysql_escape_string($this->description). "'.";
        $sql .= "WHERE nr_artykulu = " .$this->id;
        
        $rs = mysql_query($sql, $this->hDB);
        if(!is_resource($rs)) {
            throw new Exception ('Wystapil blad podczas aktualizacji bazy danych');
        }
        
    //koniec operacji na bazie danych. Zamkniecie połączenia.
    mysql_close($this->hDB);
    
    }    
 
}
?>
[PHP] pobierz, plaintext 

A tutaj miejsce inicjalizacji obiketu oraz wywolanie metod :

[PHP] pobierz, plaintext 
<?php
 
require_once('class.Widget.php');
 
try {
    $objWidget = new Widget(1);
    
    print "Nazwa artykułu: " . $objWidget->getName(). "<br>\n";
    print "Opis artykulu: " . $objWidget->getDescription(). "<br>\n";
    
    $objWidget->setName('Trampki');
    $objWidget->setDescription('Trampki o duzym przebiegu!');
} catch (Exception $e) {
    die ("Wystapil problem: " . $e->getMessage());
    }
?>
[PHP] pobierz, plaintext 

Jak zapewnia autor po kolejnym wywolaniu strony automatycznie powinny zmienic sie wartosci w bazie danych poprzez uzycie metod setName oraz setDescription. Jednak kolejne wywolania nic nie daja, dopiero po wyswietleniu wartosci po instrukcji try...catch widac wyswietla sie zmieniona wartosc , lecz w bazie danych pozostaje bez zmian.

Musisz dodać jeszcze jedną linijkę niszczącą obiekt (wywołać destruktor, czyli metodę __destruct()):

[PHP] pobierz, plaintext 
<?php
$objWidget->setDescription('Trampki o duzym przebiegu!');
unset($objWidget); // to wywoła destruktor a on updatuje Twój obiekt
?>
[PHP] pobierz, plaintext 

Niestety nic to nie pomoglo , ciagle to samo.....

Zmień metodę __destruct klasy Widget tak:

[PHP] pobierz, plaintext 
<?php
public function __desctruct() {
       if(!$this->needsUpdating) {
          echo 'needs no updating'; 
          return;
       }
       echo 'needs updating';
       $sql = 'UPDATE artykul SET';
       $sql .= "nazwa = '" .mysql_escape_string($this->name). "'.";
       $sql .= "opis = '" .mysql_escape_string($this->description). "'.";
       $sql .= "WHERE nr_artykulu = " .$this->id;
 
       echo $sql;
 
       $rs = mysql_query($sql, $this->hDB) or die(mysql_error());
       if(!is_resource($rs)) {
           throw new Exception ('Wystapil blad podczas aktualizacji bazy danych');
       }
       
   //koniec operacji na bazie danych. Zamkniecie połączenia.
   mysql_close($this->hDB);
   
   }
?>
[PHP] pobierz, plaintext 

A plik, w którym jej używasz na taki:

[PHP] pobierz, plaintext 
<?php
require_once('class.Widget.php');
 
try {
   $objWidget = new Widget(1);
   
   print "Nazwa artykułu: " . $objWidget->getName(). "<br>\n";
   print "Opis artykulu: " . $objWidget->getDescription(). "<br>\n";
   
   $objWidget->setName('Trampki');
   $objWidget->setDescription('Trampki o duzym przebiegu!');
   
   print "Nowa nazwa artykułu: " . $objWidget->getName(). "<br>\n";
   print "Nowy opis artykulu: " . $objWidget->getDescription(). "<br>\n";
 
   $unset($objWidget);
} catch (Exception $e) {
   die ("Wystapil problem: " . $e->getMessage());
}
?>
[PHP] pobierz, plaintext 

I powiedz co wyświetla.



Linia 16-sta to ta :

[PHP] pobierz, plaintext 
<?php
unset($objWidget);
?>
[PHP] pobierz, plaintext 

Funkcji zamykania połączenia/etc nie powinno się umieszczać w destruktorze.

A zwróciłeś uwagę na " __desctruct()"? Poza tym wcale nie trzeba usuwać obiektu, PHP robi to automatycznie (w normalnych warunkach oczywiście).
Erix, czemu się nie powinno?

Ponieważ prowadzi to nieraz do nieprzewidywalnych sytuacji, typu resource of unknown, czy innych błędów.

Niestety, jeśli chodzi o destruktory, to developerzy PHP się nie popisali...

No OK, ale jak chcę automatycznie zapisać dane do bazy po zniszczeniu obiektu, ale nie zamykam połączenia, to mogę zostawić to w destruktorze?

Chyba zaczyna sie robic OT. Ja chce tylko znac poprawne rozwiazanie zeby moc dalej przerabiac material ksiazki
Wiec chodzi o pojedyncze wywolanie parametrow a po przeladowaniu zmiane wartosci w tabelach.
Niestety zadna z powyzszych metod tego nie zapewnia.

Wartosci w tabeli ciagle pozostaja bez zmian , co prawda nastepuje zmiana nazwy na poziomie klienta ale nie nastepuje zadna zmiana w tabelach.

Zapytałem, czy zwróciłeś uwagę na błąd w nazwie destruktora?

Wlasnie nie wiedzialem do kogo to pytanie
Teraz juz lepiej , za to pojawil sie blad aktualizacji bazy danych.

Czy ta czesc kodu jest poprawna ?
Głównie chodzi o ta dluga zmienna $sql ktora przechowuje zapytanie.

[PHP] pobierz, plaintext 
<?php
public function __destruct() {
        if(!$this->needsUpdating) {
            echo 'needs no updating';
            return;
            }
        
        $sql = 'UPDATE artykul SET';
        $sql .= "nazwa = '" .mysql_escape_string($this->name). "'.";
        $sql .= "opis = '" .mysql_escape_string($this->description). "'.";
        $sql .= "WHERE nr_artykulu = " .$this->id;
        
        $rs = mysql_query($sql, $this->hDB);
        if(!is_resource($rs)) {
            throw new Exception ('Wystapil blad podczas aktualizacji bazy danych');
        }
?>
[PHP] pobierz, plaintext 

Wywala sie ostatni exception z pliku class.Widget.php

no to przejrzyj/wyświetl sobie zapytanie
chyba coś ze spacjami jest nie tete i nie wiem czy przecinki:)

To nie przecinki a kropki. Popraw. No i brakuje spacji.
PS. Jak używasz już apostrofów, to zostaw tak do końca, jest to lepsza technika .

Wprowadzilem nieznaczne zmiany w zapytaniu , teraz wyswietla sie znakomicie , jednak problem ciagle istnieje , wyskakuje exception zwiazany z aktualizacja.

Pokaze jeszcze poprawny kod zapytania $sql:

[PHP] pobierz, plaintext 
<?php
$sql = 'UPDATE artykul SET ';
        $sql .= "nazwa = '" .mysql_escape_string($this->name). "' ";
        $sql .= "opis = '" .mysql_escape_string($this->description). "' ";
        $sql .= "WHERE nr_artykulu = " .$this->id;
?>
[PHP] pobierz, plaintext 

Wprowadź taką zmianę (o której już pisałem raz)

[PHP] pobierz, plaintext 
<?php
$rs = mysql_query($sql, $this->hDB) or die(mysql_error());
?>
[PHP] pobierz, plaintext 

i zobaczysz komunikat.

Edit: popraw też ostatnią część zapytania sql:

[PHP] pobierz, plaintext 
<?php
$sql .= "WHERE nr_artykulu='".$this->id."'"; // tak jest poprawnie
?>
[PHP] pobierz, plaintext 

Skoro trzymasz się wyjątków to lepiej - throw new Exception (mysql_error());

Edit: Wciąż nie jest poprawnie, brakuje ci przecinka tu: $this->name). "' ";, wstaw takie coś:

[PHP] pobierz, plaintext 
<?php
$sql = 'UPDATE artykul SET ';
$sql .= "nazwa = '" .mysql_escape_string($this->name). "', ";
$sql .= "opis = '" .mysql_escape_string($this->description). "' ";
$sql .= "WHERE nr_artykulu = " .$this->id;
?>
[PHP] pobierz, plaintext 

Wyswietla sie taki komunikat :



Ale wyswietlone zapytanie wydaje sie byc poprawne :



OK!
Znalazlem blad , brak przecinka w zapytaniu po pierwszym parametrze komendy SELECT

Spójrz wyżej! Brakuje ci przecinka.

Tez to zauwazylem po dokladnym przyjrzeniu sie dlatego edytowalem poprzedni post , teraz wszystko dziala , dzieki , moge uczyc sie dalej