Zmiana wartosci rekordu Opcje

[Wolfie]

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.

Ten post edytował Wolfie 12.03.2009, 20:03:47

[Foxx]

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 

[Wolfie]

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

[Foxx]

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.

[Wolfie]



Nazwa artykułu: Korale
Opis artykulu: Korale koloru koralowego
Nowa nazwa artykułu: Trampki
Nowy opis artykulu: Trampki o duzym przebiegu!

Fatal error: Function name must be a string in C:\Program Files\WebServ\httpd\OOP\PHP5\testWidget.php on line 16

Linia 16-sta to ta :

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

Ten post edytował Wolfie 12.03.2009, 20:55:03

[erix]

Funkcji zamykania połączenia/etc nie powinno się umieszczać w destruktorze. (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[Mephistofeles]

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?

[erix]

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...

[Mephistofeles]

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

[Wolfie]

Chyba zaczyna sie robic OT. Ja chce tylko znac poprawne rozwiazanie zeby moc dalej przerabiac material ksiazki (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
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.

Ten post edytował Wolfie 12.03.2009, 21:10:30

[Mephistofeles]

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

[Wolfie]

Wlasnie nie wiedzialem do kogo to pytanie (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
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

Ten post edytował Wolfie 12.03.2009, 21:15:11

[artur_dziocha]

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

[Mephistofeles]

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 (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) .

[Wolfie]

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 

Ten post edytował Wolfie 12.03.2009, 21:36:34

[Foxx]

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 

Ten post edytował Foxx 12.03.2009, 21:38:15

[Mephistofeles]

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 

Ten post edytował Mephistofeles 12.03.2009, 21:41:41

[Wolfie]

Wyswietla sie taki komunikat :

Something is wrong in your syntax obok 'opis = 'Trampki o duzym przebiegu!' WHERE nr_artykulu='1'' w linii 1

Ale wyswietlone zapytanie wydaje sie byc poprawne :

UPDATE artykul SET nazwa = 'Trampki' opis = 'Trampki o duzym przebiegu!' WHERE nr_artykulu='1'

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

Ten post edytował Wolfie 12.03.2009, 21:44:37

[Mephistofeles]

Spójrz wyżej! Brakuje ci przecinka.

[Wolfie]

Tez to zauwazylem po dokladnym przyjrzeniu sie dlatego edytowalem poprzedni post , teraz wszystko dziala , dzieki , moge uczyc sie dalej (IMG:http://forum.php.pl/style_emoticons/default/exclamation.gif) (IMG:http://forum.php.pl/style_emoticons/default/laugh.gif)