Первый ООП код

[_vitalja__]

Мой первый ООП код. Как вам? Все ли правильно или что изменить лучше?

<?php
define('DBHOST', 'localhost');
define('DBNAME', 'abc');
define('DBUSER', '12');
define('DBPASSWORD', '12');

class DB{
    private $DB;
   
    function Connect(){
        try { 
          $DB = new PDO("mysql:host=".DBHOST."; dbname=".DBNAME, DBUSER, DBPASSWORD);
          $DB->setAttribute( PDO::ATTR_ERRMODE, PDO::ERRMODE_SILENT );
        } 
 
        catch(PDOException $error) { 
        echo $error->getMessage(); 
        }
        return $DB;
    }
   
   
    function Select($rows ,$table, $where = null, $orderby = null){
        if($where != null) $where = 'WHERE '.$where.'';
        if($orderby != null) $orderby = 'ORDER BY '.$order.'';
        $select = $this->Connect()->query('SELECT '.$rows.' from '.$table.' '.$where.' '.$orderby.'');
        return $select;
    }
   
}
?>

 

[Вурдалак]

Как вам?

Полное говно.

 

[_vitalja__]

Полное говно.

Почему

 

[Mad Dogg]

Почему

не обращай внимание на школьников. Кароч тут так. Для начала сходи на php.net в мануал по ООП в документации прочти. Это первое. Во вторых, инициализацию вынеси в конструктор, во вторых, метод Select реально пугает. Лучше уж туда просто строку запроса передавать с массивом аргументов во втором параметре, а не так, как сейчас.

 

[AnrDaemon]

Полное говно.

Вынужден согласиться с предыдущим оратором.
Писать ООП обёртку над ООП - видимо, вам за это хорошо платят?

 

[c0dex]

Плейсхолдеры где?

Да и метод select надо бы как-то представить в более нормальном виде. Он должен выполнять запрос на выборку. Сейчас же у тебя там свой построитель запроса+еще что-то

 

[Вурдалак]

Писать ООП обёртку над ООП - видимо, вам за это хорошо платят?

«Обертку» над ООП? PDO — это конкретная реализация. http://en.wikipedia.org/wiki/Dependency_inversion_principle

 

[Mad Dogg]

Вынужден согласиться с предыдущим оратором.
Писать ООП обёртку над ООП - видимо, вам за это хорошо платят?

вот почему всегда такие неадекватные ответы ? человек начал пробовать ООП и тут понеслось ) обертки какие-то, реализации ) полегче

 

[AmdY]

У тебя Conect каждый раз инициирует подключение, потому что объект не имеет состояния. Ты завёл приватный атрибут DB, надо в него записывать и проверять состояние.

 

[Mad Dogg]

еще вот посмотри http://www.fluffycat.com/PHP-Design-Patterns/Singleton/

 

[Фанат]

1. К ООП данный опус не имеет ни малейшего отношения
2. За такие вот вещи

$DB->setAttribute( PDO::ATTR_ERRMODE, PDO::ERRMODE_SILENT );

echo $error->getMessage();

и

function Select($rows ,$table, $where = null, $orderby = null){

в приличном обществе бьют канделябрами.
А в неприличном - ногами.

Тебе это для использования в работе или потренироваться?
Если в работе, то https://github.com/colshrapnel/thebestpdowrapper
Если потренироваться, то https://toster.ru/answer?answer_id=396395#answers_list_answer

 

[WMix]

Полное говно.
peon и Andkorol нравится это.

Копрофилия

 

[Вурдалак]

еще вот посмотри http://www.fluffycat.com/PHP-Design-Patterns/Singleton/

Зачем смотреть на говно?

 

[_vitalja__]

А что вот здесь на так

$DB->setAttribute( PDO::ATTR_ERRMODE, PDO::ERRMODE_SILENT );

 

[Mad Dogg]

Зачем смотреть на говно?

а можно поподробнее, а то я как то в сортах не шибко разбираюсь...

 

[WMix]

 

[AnrDaemon]

А что вот здесь на так

Всё.

 

[Вурдалак]

Если в работе, то https://github.com/colshrapnel/thebestpdowrapper

Блин, ну ты и тролль. Это просто кусок всего самого плохого, что можно только представить.

 

[Absinthe]

Вынужден согласиться с предыдущим оратором.
Писать ООП обёртку над ООП - видимо, вам за это хорошо платят?

Хочешь сказать, что PDO можно использовать без обертки?

 

[Фанат]

Хочешь сказать, что PDO можно использовать без обертки?

Ну, аффтарами задумывалось имено так. И оно даже работает.
Единственный существенный минус - это идиотизм, с которым execute() возвращает буль, а не себя. Заставляя писать три строчки, когда достаточно одной.

Ну, это если не считать оберткой ваш обожаемый фетиш - DI контейнер, который выдаст инстанс по запросу.