php oop et mysql

Question

Je dois obtenir des données, les vérifier et les envoyer à db. Programmation avec PHP OOP.

Pourriez-vous me dire si ma structure de classe est bonne et comment supprimer toutes les données? Merci

<?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

C'est un morceau de code horrible.

1. Pour la communication de base de données, utilisez PDO. Ce n'est pas parfait, mais ça va.
2. Chaque fois que vous aurez besoin d'une base de données, vous vous connecterez avec cette base de données?
3. Database::insert() etc. devrait être un vrai magicien de deviner où son paramètre doit être inséré

Answer 2

Vous avez besoin d'un meilleur IDE pour montrer vos variables non définies. J'utilise PHPStorm.

Vous avez besoin de dependency injection. Utilisez encapsulation. Il suffit de suivre SOLID.

Avant-dernière chose, ne faites pas d'écho depuis l'intérieur des objets. Vous ne pouvez pas unit test efficacement votre code de cette façon.

Vous pouvez également essayer TDD.

Answer 3

Vous allez à tort, je pense. Ce que vous construisez est une enveloppe autour des fonctions de base de données déjà existantes. La chose est, ces fonctions déjà définies pour faire exactement ce que vous voulez - l'approche orientée objet ici ne sert à rien parce qu'elle ne peut se connecter qu'à une base de données et les méthodes ne sont pas vraiment bien définies et semblent agir comme des couche sous-jacente.

Comme d'autres l'ont mentionné, le code que vous semblez suivre ne vous convient pas. Quelque chose comme PDO vous servira bien et c'est déjà intégré à PHP. Vous n'avez pas besoin de réinventer la roue pour un code commun comme l'accès à une base de données. L'apprentissage de quelques principes fondamentaux de l'orientation des objets vous fera aussi du bien - regardez d'abord l'encapsulation et la réutilisation du code. Ils vous aideront dans le futur quand vous devrez maintenir ce code que vous écrivez!