Bonjour,
depuis quelque temps je me suis mis à la programmation avec les classes en PHP.
 
Cependant, ayant encore trop de reflexes de programmation procédurale, j'aimerais quelques conseils.
 
Alors, déjà voici un exemple de classes d'un projet sur lequel je travaille :
 
Classe affichage :
 

 
 
classe AffichageUser
 

 
 
 
Bien-entendu ce n'est qu'un exemple. Je pense que vous vous en doutez où je veux venir :
 
Est-ce que ce codage là est correct ? (ce dont je doute).
Est-ce que l'heritage est superflu dans ce genre de cas ?
 
Est-ce qu'il vaut mieux que je passe tous les paramètres (connexion, login_user, etc...) au moment de l'instanciation et me servir d'attributs pour appeler les méthodes ?
 
Enfin, est-ce qu'il serait préferable que je fasse une classe "connexion" que j'instancie dans une variable, et que je passe cette variable en parametre dans le constructeur de la classe qu'il me faudra par la suite ?
 
Pouvez-vous me donner un exemple, comment auriez-vous fait ?
 
 
Merci d'avance à ceux qui répondront, ça m'aiderait à comprendre certaines choses  

Je ferais comme toi mais je donnerais des autres noms à mes classes (class DB, class DB_User)
 
Sinon je pense que le constructeur de la deuxième classe hérite automatiquement de l'autre constructeur.
 
 
ps: mais je suis aussi nioub en POO (mais bon, pour le super avis que je donnes )

Je ne suis pas très fan du découpage, mais bon ça peut être bien comme tu as fait...perso j'aurais plutôt un truc qui ressemblerait à ça (fait à l'arrache hein)

class Db
{
    private $server;
    private $pass;
    private $user;
    private $name;
    private $connexion;
    
    
    function __construct($bdserveur, $bduser, $bdpass, $bdnom)
    {
        $this->server = $bdserveur;
        $this->user = $bduser;
        //...
        $this->connexion = false;
    }
    
    function connect(){
        if(!$this->connexion){
            $this->connexion = mysql_connect($this->server, $this->user, $this->pass);
            if(!$this->connexion){
                throw new Exception('connexion ratée, au-secours.');
            }
            mysql_select_db($name);
        }
    }
    
    function execSql($sql){
        //on exécute la requête $sql on on retourne les rows sous foprme de tableau...
    }
    
}
 
class User
{
    private $db;
    
    function __construct(Db $dbInstance)
    {
        $this->db = $dbInstance;
    }
 
    public function getNomUser($login_user)
    {
        try{
            $this->db->connect();
            $requete  = "SELECT nom FROM user WHERE login = '$login_user'";
            $resultats = $this->db->execSql($requete);
        }
        catch(Exception $e){
            die($e->getMessage());
        }
        
        return $resultats[0]['nom'];
    }
 
}

Merci pour vos réponses Comme vous m'avez donné pratiquement la même réponse, je pense que je vais adopter cette manière-là.
 
Si je comprends bien, l'heritage dans ce genre de cas vaut mieux éviter ?
 
Cependant, je me pose encore la question : pour la fonction "getNomUser" est-ce que c'est "propre" de laisser un tel paramètre ?  
Ou faudrait-il plutôt le mettre dans l'appel du constructeur ?
 

Ne pas avoir d'héritage là c'est plutôt une question de logique : a priori ce ne sera pas la seule classe à utiliser la base de données, si tu dois passer tous ces paramètres strictement identiques à tous tes objets t'es pas sorti...

Pour la question suivante ça dépend de ce que tu veux faire - du sens que tu veux donner à ta classe.
Si une instance de cette classe doit représenter un utilisateur de ton appli, alors ce que j'ai fait n'est pas la "bonne" solution. La classe User que j'ai faite plus haut serait faite uniquement pour interagir avec la représentation d'un utilisateur en base de données. On pourrait la renommer en "UserDbFactory" par exemple.

'fin c'est une question bien plus complexe, si on veut rentrer dans le détail et une implémentation complète...

Merci pour ton attention skeye
 
Alors en fait la classes User c'est uniquement pour intéragir avec la base de données, donc ce que tu as fait est bien à priori.
 
Sinon je n'ai pas très bien compris ta réponse concernant l'héritage. Si je fais instancier à chaque fois la classe "Db" y a t-il vraiment un problème ? Ou faut-il que je la fasse hériter ?

est-ce mal de faire quelque chose comme ça ?
 

 
Le but étant d'avoir la classe User la plus lisible possible et aussi d'avoir toutes les classes DB_* contenant les requêtes rassemblée en un endroit.

non, ce n'est pas mal. Enfin pas trop.
Mais personnellement je mettrais plutôt une méthode getUserFromLoginAndPasswd() - bon ok p-e un nom plus simple - qui retourne un User dans DB_user...ça permet de ne pas avoir du tout de référence à la source de données dans User.

oui c'est ce que je fais

j'ai juste donné le schémas en gros

merci

OK merci pour vos indications je verrai ça de plus près alors