[php] 2 klasy, błąd przy obiekcie Opcje

[Lejto]

index.php

[PHP] pobierz, plaintext 
<?php
include_once("db.class.php");
$db = new mysql();
global $db;
 
include("login.class.php");
$log = new Auth();
 
include("upload.class.php");
$up = new upload();
global $up;
?>
[PHP] pobierz, plaintext 

db.class.php

[PHP] pobierz, plaintext 
<?php
class mysql
{
   public $_lacz;
   
   public function __construct()
   {
       $this->_lacz = new mysqli(DB_HOST, DB_USER, DB_PASS, DB_NAME);
   }
 
}
?>
[PHP] pobierz, plaintext 

wywołuje funkcje dodaj z klasy upload
klasa

[PHP] pobierz, plaintext 
<?php
class upload extends mysql {
..[...]
    public function dodaj($n)
    {
        
         
        $result = $this->_lacz->query('zapytanie');    
       
         
        }
        fclose($uchwyt); 
    }
}
?>
[PHP] pobierz, plaintext 

i otrzymuje że funkcja query nie ma obiektu ;/ co nie tak?

Pozdrawiam

[WebCM]

Pokaż wszystkie linijki kodu. Źle to robisz. Jeżeli klasa mysql jest nadrzędna dla upload:

[PHP] pobierz, plaintext 
<?php
$result = parent :: $_lacz -> query('zapytanie')
?>
[PHP] pobierz, plaintext 

Nie widzę potrzeby istnienia klasy mysql, jeżeli ma oferować te same funkcje, co MySQLi. Jeśli już chcesz ja deklarować, niech będzie podrzędną dla MySQLi. Wtedy w konstruktorze tej klasy napisz:

[PHP] pobierz, plaintext 
<?php
parent :: __construct(parametry)
?>
[PHP] pobierz, plaintext 

[Lejto]

nic nie dało ;/
klasa upload

[PHP] pobierz, plaintext 
<?
class upload extends mysql {
    
    public function upload()
    {
        if($_SESSION['login'])
        {
        $nazwa = $_FILES['plik']['name'];
 
        if(move_uploaded_file($_FILES['plik']['tmp_name'],"./csv/".$nazwa))
            {
                ?>
                <div id=good>
                        <p>Plik wysłany pomyślnie</p>
                        <a href=java script:history.back(-1);>Wstecz</a>
                </div>
                
                
                <?
                $this->dodaj($nazwa);
            }
            else
                {
                    switch($_FILES['plik']['error'])
                    {
                        case 1: echo ('<span id=bad>Przesłany plik jest większy niż zezwala konfiguracja serwera.</span>');
                            break;
                        case 2: echo ('<span id=bad>Przesłany plik jest wiekszy niż zezwala formularz.</span>');
                            break;
                        case 3: echo ('<span id=bad>Przesłano tylko część pliku.</span>');
                            break;
                        case 4: echo ('<span id=bad>Plik nie został przesłany.</span>');
                            break;
                    }
                }
        }
    }
    public function formularz()
    {
        ?>
        <div id=good>Wyślij plik</div>
            <form action=?akcja=upload enctype=multipart/form-data method=post>
            <table>
            <tr><td>Ścieżka pliku: </td><td>
            <input type=file name=plik />
            </td></tr>
            <input type=submit value=Wyślij zdjęcie/>
            </td></tr></table>
            </div>
            <?
    }
    public function dodaj($n)
    {
        
        $row = 1;
        $uchwyt = fopen("csv/".$n,r);
 
 
        while (($data = fgetcsv($uchwyt, 1000, ";")) !== FALSE)
        {
        $num = count($data);
         $row;
         
        $result = $this->_lacz->query('INSERT INTO `probid_auctions` (
 
        `name`, `description`, `quantity`, `auction_type`, `start_price`,
        `buyout_price`,`duration`, `shipping_method`, `shipping_int`,
        `payment_methods`, `category_id`,  `hpfeat`, `catfeat`, `bold`,
        `hl`, `postage_amount`, `insurance_amount`, `type_service`,
        `list_in`, `approved`,
        `auto_relist_nb`,`is_relisted_item`,`start_time`,`start_time_type`,
        `is_draft`,`start_on_bid`,`additional_rows_titles`,`additional_rows_values`
    
        )
        VALUES (
        '.$data[3].', '.$data[12].','.$data[13].','.$data[2].','.$data[14].',                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                 '.$data[17].','.$data[23].','.$data[30].','.$data[31].','.$data[37].',
        '.$data[1].','.$data[19].','.$data[20].','.$data[17].','.$data[18].',
        '.$data[32].','.$data[33].','.$data[35].', '.$data[0].','.$data[38].',
        '.$data[29].','.$data[28].','.$data[21].','.$data[21].','.$data[40].',
        '.$data[22].','.$data[4].','.$data[5].')');    
       
         
        }
        fclose($uchwyt); 
    }
}
[PHP] pobierz, plaintext 

[Crozin]

Przepraszam za mały offtop: "class upload extends mysql" co ma piernik do wiatraka? W jaki sposób klasa upload rozszerza klasę mysql?

[Lejto]

ok troche zamotanie, a jak to poprawić?

Ten post edytował Lejto 30.04.2009, 19:56:08

[-=Peter=-]

Przekazując obiekt klasy mysql do obiektu klasy upload...

przykładowo:

[PHP] pobierz, plaintext 
<?php
class mysql implements Database{}
 
 
class upload{
  private $db;
 
  //ciach  
 
  public function setDb(Database $mysql){
    $this->db = $mysql;
  }
 
  public function getDb(){
    return $this->db;
  }
  
  public function add($v){
    $result = $this->getDb()->query('...');
    //ciach
  }
 
}
 
$upload->setDb($mysql);
$upload->add('blebleble');
?>
[PHP] pobierz, plaintext 

A jeśli chodzi o Twoje konkretne pytanie, to w jakiś cudowny sposób metoda upload (nazwa taka sama jak klasa, a w php4 tak się zwie konstruktor) przysłania konstruktor nadklasy mysql. Zmień nazwę tej metody. Sprawdzałem to teraz i to chyba jest bug, że najpierw jest sprawdzane czy w klasie jest konstruktor o nazwie __construct, jak nie ma to o takiej samej nazwie co klasa, a jak wtedy dopiero nie ma to czy w klasie nadrzędnej jest konstruktor __construct... Wg mnie nieporządane i dziwne działanie, więc kwalifikuję to jako bug (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Ten post edytował -=Peter=- 30.04.2009, 22:01:28

[okitoki]

poco dziedziczysz po klasie mysql, nie łatwiej stworzyć obiekt mysql i w konstruktorze go przesłać?, w twoim sposobie będziesz generował zbędne połączenia z bazą danych, bo każdy obiekt będzie miał swoje.


nie działa ci to bo zasłoniłeś konstruktora klasy mysql, wszędzie gdzie dziedziczysz po tej klasie w konstruktorze dodaj wpis parent :: __construct()

[Lejto]

to jak mam sobie klase która ma za zadanie łączenia się z bazą dodaje ją w pliku index.php potem mam 2 klasę która dodaje coś do bazy to żeby nie robić
extends mysql { -,-
to muszę za każdym razem implements Database{}? nie da się inaczej?
@okitoki nie za bardzo cię rozumiem ;/

[marcio]

Da sie inaczej po co tu w ogole sa potrzebne interfejsy tu masz przyklad jak takie cos zrobic: Temat: user_class__dziedziczenie ostatni post.

A jak nie robisz klase i metody statyczne wtedy nie musisz robic instancji danej klasy jednak jest to zbedne przy klasie dla Mysql bo za kazdym razem bedzie musial tworzyc i zamykac polaczenie.

A jak juz tak bardzo chcesz dziedziczyc to tak jak ci WebCm napisal uzywaj metod z klasy bazowej za pomoca parent::

[Lejto]

ok dzięki, jakoś sobie poradzę

[okitoki]

to jak mam sobie klase która ma za zadanie łączenia się z bazą dodaje ją w pliku index.php potem mam 2 klasę która dodaje coś do bazy to żeby nie robić
extends mysql { -,-
to muszę za każdym razem implements Database{}? nie da się inaczej?
@okitoki nie za bardzo cię rozumiem ;/

nie musisz za każdym razem implementować database
tworzysz obiekt

$sql = new mysql(); // on już się łączy z bazą ma tam jakąś metode query

jak chcesz to przekazać do innego obiektu to robisz

$inny_obiekt = new inna_klasa($sql)

a klasa wygląda następująco

Kod

class inna_klasa
{
private $sql;
public function __construct($sql)
  {
    $this->sql = $sql;
  }
}

i w ten oto sposób w 2giej klasie masz zaimplementowanego sql-a.
jak zrobisz poprzez dziedziczenie tak jak na twoim przykładzie to klasa gdzie dziedziczysz musi wyglądać następująco

Kod

class upload extends mysql {
   public function __construct()
   {
     parent::__construct()
   }

    public function dodaj($n)
    {
        
        
        $result = $this->_lacz->query('zapytanie');    
      
        
        }
        fclose($uchwyt);
    }
}

[marcio]

Kod

public function __construct($sql)
  {
    $this->sql = $sql;
  }

Chyba jednak nie tak jak cos to:

Kod

public function __construct(KlasaSql $sql)
  {
    $this -> sql = $sql;
  }

[osl]

marcio, a czym różnią się te dwa konstruktory, poza tym że jeden jest bardziej 'strict'?
Działają identycznie przecież...

[marcio]

Jak moze dzialac skoro nie podajemy klasy z ktorej ma zostac przekazany obiekt?

Jesli dziala to good nie wiedzialem o tym (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) i zwracam honor lecz dziwnie mi to wygladalo.

[okitoki]

działają good (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) , jak określisz klase, to tylko obiekt tej klasy będziesz mógł tam przesłać, jeżeli nie określisz klasy, to tworzysz klasy baza_danych1, baza_danych2, ważne by miały metody i pola te same, w ten sposób możesz działać na różnych silnikach bazodanowych i nie tylko

[Lejto]

ok dzięki trochę mi się rozjaśniło (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[ndx]

marcio, a czym różnią się te dwa konstruktory, poza tym że jeden jest bardziej 'strict'?
Działają identycznie przecież...

Zobacz co sie stanie jak $sql nie bedzie instancja KlasaSql tylko np. stringiem

[osl]

@ndx - zgoda, ale to nie jest aż taki znowu błąd - generalnie w przypadku implementacji takiej klasy wszystko zależy od programisty, więc jeśli tam się pojawi cokolwiek innego niż instancja klasy to znaczy tyle, że gdzieś wcześniej jest błąd.

W żadnym przypadku nie twierdze kod KlasaSql $sql jest błędne, ale po prostu w tym przypadku wszystko powinno być na tyle hermetyczne żeby nic innego się nie pojawiło poza instancją odpowiedniej klasy.

[okitoki]

Zobacz co sie stanie jak $sql nie bedzie instancja KlasaSql tylko np. stringiem

bez znaczenia, czy będzie stricte czy nie będzie błąd jak podepniesz stringa, tyle że na innych liniach i inny błąd (IMG:http://forum.php.pl/style_emoticons/default/exclamation.gif)

[marcio]

Ale wtedy wiadomo ze trzeba podac instancje klasy i nie trzeba analizowac kodu ani rozmyslac.