PDO, OOP, Composer

Odpowiedz Nowy wątek
2019-05-14 19:34
0

Witam,

Zacząłem ambitniej uczyć się programowania obiektowego w PHP i natrafiłem na pewien problem z zapytaniami. Z dodawaniem rekordów do bazy sobie poradziłem, mam problem z ich pobieraniem. Tak oto prezentuje się mój kod:

class DB.php


public function select($table,$conditions = array()){
        $sql = 'SELECT ';
        $sql .= array_key_exists("select",$conditions)?$conditions['select']:'*';
        $sql .= ' FROM '.$table;
        if(array_key_exists("where",$conditions)){
            $sql .= ' WHERE ';
            $i = 0;
            foreach($conditions['where'] as $key => $value){
                $pre = ($i > 0)?' AND ':'';
                $sql .= $pre.$key." = '".$value."'";
                $i++;
            }
        }

        if(array_key_exists("order_by",$conditions)){
            $sql .= ' ORDER BY '.$conditions['order_by'];
        }

        if(array_key_exists("start",$conditions) && array_key_exists("limit",$conditions)){
            $sql .= ' LIMIT '.$conditions['start'].','.$conditions['limit'];
        }elseif(!array_key_exists("start",$conditions) && array_key_exists("limit",$conditions)){
            $sql .= ' LIMIT '.$conditions['limit'];
        }

        $query = $this->db->prepare($sql);
        $query->execute();

        return !empty($data)?$data:false;
    }

class User.php


public function loginUser($credentials){
        $db = new DB();
        $email = $credentials['email'];
        $password = $credentials['password'];
        $result = $db->select('users', array('where'=>array('email'=>$email),'password'=>$password));
        if ($result->rowCount() > 0){
            //
        }else{
            //
        }
    }

login.php


<?php
require __DIR__ . '/../../../vendor/autoload.php';

use App\User;

$credentials = [
    'email' => $_POST['email'],
    'password' => $_POST['password']
];
$user = new User();
$user->loginUser($credentials);

błąd który otrzymuję

Fatal error: Uncaught Error: Call to a member function rowCount() on bool

Pozostało 580 znaków

2019-05-14 19:47
0

Może spróbuj zaimplementować MVC, bo te zapytanie do bazy trochę "brudne" z jakimś if, foreach, jak dla mnie to najpierw powinieneś przygotować dane, a dopiero potem na nich pracować, czy foreach we widokach dopiero. Może zobacz jak to we frameworkach wygląda, lub wygooglaj mvc.


Pozostało 580 znaków

2019-05-14 20:05
1

To co robisz pierwszą funkcją, czyli:

public function select

to zwracanie zawsze false, bo tak jak jest to skonstruowane to nigdy nie zwrócisz $data, bo nie przyporządkowujesz tej zmiennej wyniku działania funkcji query dlatego dostajesz ten komunikat:

Fatal error: Uncaught Error: Call to a member function rowCount() on bool

To co próbujesz robić tą funkcją realizują tak zwane ORMy, dzięki którym możesz budować zapytania SQL dynamicznie. Jednym z popularniejszych jest ORM Doctrine, ja używam fremeworka Phalcon i ORMa z tego frameworka.

W niektórych sytuacjach używanie ORMów bardzo ułatwia życie, ale trzeba pamiętać, że zapytania z ORM zawsze będą mniej wydajne chociażby przez narzut wielu funkcji w porównaniu do napisania prostego zapytania SQL tekstowo, po drugie relizacja pewnych natywnych spraw albo związanych z JSON w bazach danych jest utrudnione w większości ORMów. Ja stosuje ORMy głównie w panelach administracyjnych, natomiast do generowania danych na frontend który jest poza panelem administracyjnym dla adminów/właścicela systemu, ORMów zazwyczaj nie potrzeba.


edytowany 4x, ostatnio: TomRZ, 2019-05-14 20:08
Pokaż pozostałe 44 komentarze
To są rozwiązania związane z Big Data, do pewnych rzeczy lepsze niż cache nawet synchronizowany, ale znowu - to trochę nisza, tak samo jak np,. Apache Hadoop. - TomRZ 2019-05-14 22:52
@TomRZ: nie nałozą się procesy bo jest kolejka, jeśli odpalasz równolegle joby które mają obsługiwać tę samą kolejkę to wtedy robi się trick, żeby joby obsługiwały zmiany niezależnych wewnętrznych kolejek czyli zmiany stanu rożnych przesyłek w firmie kurierskiej. - mr_jaro 2019-05-14 22:53
Dobrze, kolejka gwarantuje że nie będzie konfliktów, ale z kolei nie gwarantuje, że określona komenda czy kwerenda odpali się w okreslonym momencie - znowu to jest narzędzie które do czegoś trzeba dopasować, w jednym projekcie/zagadnieniu to będzie ok w innym nie. - TomRZ 2019-05-14 22:54
Dobranoc i dzięki za dyskusję - TomRZ 2019-05-14 22:56
@TomRZ: oczywiście, że masz racje, nie wszystko pasuje wszędzie. - mr_jaro 2019-05-14 23:03

Pozostało 580 znaków

Odpowiedz
Liczba odpowiedzi na stronę

1 użytkowników online, w tym zalogowanych: 0, gości: 1, botów: 0