php oop и mysql

Question

Мне нужно получить данные, проверить и отправить в db. Программирование с помощью PHP OOP.

Не могли бы вы рассказать мне, хороша ли моя структура классов и как dislpay все данные ?. Спасибо

<?php 
class Database{ 

    private $DBhost = 'localhost'; 
    private $DBuser = 'root'; 
    private $DBpass = 'root'; 
    private $DBname = 'blog'; 

    public function connect(){ 
     //Connect to mysql db 
    } 

    public function select($rows){ 
     //select data from db 
    } 

    public function insert($rows){ 
     //Insert data to db 
    } 

    public function delete($rows){ 
     //Delete data from db 
    } 
} 

class CheckData{ 

    public $number1; 
    public $number2; 

    public function __construct(){ 
     $this->number1 = $_POST['number1']; 
     $this->number2 = $_POST['number2']; 
    } 

    function ISempty(){ 
     if(!empty($this->$number1)){ 
      echo "Not Empty"; 

      $data = new Database(); 
      $data->insert($this->$number1); 
     } 
     else{ 
      echo "Empty1"; 
     } 

     if(!empty($this->$number2)){ 
      echo "Not Empty"; 

      $data = new Database(); 
      $data->insert($this->$number2); 
     } 
     else{ 
      echo "Empty2"; 
     }  
    } 
} 

class DisplayData{ 

    //How print all data? 
    function DisplayNumber(){ 
     $data = new Database(); 
     $data->select(); 
    } 
} 

$check = new CheckData(); 
$check->ISempty(); 

$display = new DisplayData() 
$display->DisplayNumber(); 
?> 

Answer 1

Это ужасный фрагмент кода.

1. Для связи с базой данных используется PDO. Это не идеально, но все в порядке.
2. Каждый раз, когда вам понадобится база данных, вы будете связываться с этим db?
3. Database::insert() и т.д. должны быть настоящим магом, чтобы догадаться, где его параметр должен быть вставлен

Answer 2

Вам нужно лучше IDE, чтобы показать свои неопределенные переменные. Я использую PHPStorm.

Вам понадобится dependency injection. Используйте encapsulation. Просто следуйте SOLID.

Во-вторых, не эхо изнутри объектов. Вы не можете unit test эффективно использовать свой код таким образом.

Вы также можете попробовать TDD.

Answer 3

Вы думаете, что здесь не так. То, что вы создаете, представляет собой оболочку вокруг уже существующих функций базы данных. Дело в том, что те функции, которые уже определены, чтобы сделать именно то, что вы хотите - объектно-ориентированный подход здесь не имеет реальной цели, поскольку он может подключаться только к одной базе данных, а методы не очень четко определены и, по-видимому, действуют как чистые сквозные нижний слой.

Как уже упоминалось, код, который вы, кажется, собираетесь здесь, не подходит для ваших целей. Что-то вроде PDO послужит вам хорошо и уже встроено в PHP. Вам не нужно повторно изобретать колесо для такого общего кода, как доступ к базе данных.

Изучение нескольких основных принципов ориентации на объекты также принесет вам пользу - сначала просмотрите инкапсуляцию и повторное использование кода. Они помогут в будущем, когда вы будете поддерживать этот код, который вы пишете!