[SQL][PHP]Przekazanie obiektu z metody do metody Opcje

[smiesznylolek]

mam taki mały problem, tworzę sobie klasę do obsługi bazy danych. I dla tesów zrobiłem taką prosty skrypt do sprawdzenia czy działa:

[PHP] pobierz, plaintext 
<?php
include_once 'dbmenager.php';
$foo = new dbmenager();
 
$foo -> sqlQuery("SELECT * FROM user");
 
while ($row = $foo -> sqlResult())
{
     echo $data->imie;
 
};
[PHP] pobierz, plaintext 

Problem się pojawia w metodzie sqlResult(). W przeglądarce wyskauje mi bład Call to a member function fetch_object() on a non-object in D:\xampp\htdocs\baza\dbmenager.php on line 36($record = $this->result->fetch_object()(IMG:style_emoticons/default/wink.gif) . Wydaje mi się żę w jakiś zły sposób próbuję przesłać wynik zapytania do tej metody przez co pojawia się błąd. Ale już się trochę pogubiłem i nie wiem co robie nie tak. W każdym razie sprawdziłem że "$result" jest obiektem. Tylko coś źle chyba robię żeby to $result było użyte w metodzie sqlResult(). Czy mógłbym prosić o jakąś radę?

[PHP] pobierz, plaintext 
class dbmenager {
     private $conn;
     private $query;
     private $result;
     private $record;
 
             function __construct() {
                  $this->conn = mysqli_connect('localhost', 'root', 'root', 'bazaobiekt');
     if(mysqli_connect_errno()){
          echo 'Błąd połączenia z bazą danych :'.mysqli_connect_errno(); exit();
     }
 
}
          function sqlQuery($query){
               $this->query = $query;
               $result = mysqli_query($this->conn, $query);
               //print_r($result);
 
               return $this->result;
          }
 
          function sqlResult(){
 
               [b]$record = $this->result->fetch_object();[/b]
               return $this->record;
          }
}
[PHP] pobierz, plaintext 

[Turson]

Nie prościej wykonać zapytanie i pobrać wyniki w jednej metodzie?

Ten post edytował Turson 28.12.2013, 21:00:52

[com]

http://pl.wikipedia.org/wiki/CRUD

[smiesznylolek]

Nie prościej wykonać zapytanie i pobrać wyniki w jednej metodzie?

ale czy takie rozwiązanie jest poprawne? Bo chciałem zrobić tak żeby klasa dbmenager była odpowiedzialna właśnie za operacje na bazie, a takim wypadku będę musiał powielać kod...
Chyba że chodzi Ci o to żeby w metodzie sqlQuery umieścić kod z sqlResult?

Ten post edytował smiesznylolek 30.12.2013, 13:09:27

[Turson]

Chyba że chodzi Ci o to żeby w metodzie sqlQuery umieścić kod z sqlResult?

Dokładnie o to. Powiedzmy, że pobierasz dane użytkownika

[PHP] pobierz, plaintext 
function getUserData($id){
	$stmt=$this->db->prepare("SELECT * FROM users WHERE id=:id");
	$stmt->bindValue(':id',$id,PDO::PARAM_INT);
	$stmt->execute();
	return $stmt;
}
[PHP] pobierz, plaintext 

W metodzie getUserData otrzymujesz tablicę z danymi.

Zamiast w każdej klasie pisać konstruktor do połączenia z bazą, polecam zrobić osobną klasę z połączeniem, a potem w kolejnych klasach tylko ją połączać przez extends

[com]

jakbyś poczytał na temat tego co dałem Ci definicje to byś nie zadał takich pytań, ludzie trochę myślcie (IMG:style_emoticons/default/smile.gif)

[smiesznylolek]

jakbyś poczytał na temat tego co dałem Ci definicje to byś nie zadał takich pytań, ludzie trochę myślcie (IMG:style_emoticons/default/smile.gif)

nie do końca rozumiem, o co Ci chodzi, i co objaśnienie CRUD ma do mojego problemu?

[bostaf]

Call to a member function fetch_object() on a non-object in D:\xampp\htdocs\baza\dbmenager.php on line 36($record = $this->result->fetch_object()(IMG:style_emoticons/default/wink.gif) .

Błąd "Call to a member function on a non-object" oznacza, że próbowano wywołać metodę klasy na zmiennej nie będącej instancją żadnej klasy. Zwróć uwagę na przedimek nieokreślony "a" w komunikacie błędu: "a ... function" - "jakaś funkcja" - parser nawet nie wie, o jaką funkcję chodzi. I nic dziwnego - nie jest taki bystry jak my tutaj (IMG:style_emoticons/default/smile.gif) W tym nieszczęsnym wierszu 36 wywołujesz metodę fetch_object() na polu result, a wcześniej nigdzie nie ustawiłeś wartości tego pola - ma ono wartość NULL (bystry parser by chociaż taką podpowiedź dał). Mógłbyś w metodzie sqlQuery() zamiast zwracać pustą wartość pola (return $this->result;), ustawić ją: $this->result = $result;, wtedy może coś zacznie hulać (IMG:style_emoticons/default/wink.gif)

PS. CRUDem się nie przejmuj - to faktycznie nie ma tutaj nic do rzeczy.

[smiesznylolek]

Dzięki wielkie Stary:) jak można było się spodziewać, to pierdółka była, ale dzięki za konkretną odp:)