Discussion :

[Squiz]

Bonjour,

Je m'occupe d'un site web développé intégralement en fonctionnel et je voudrais passer quelques fonctionnalités en objet.
Je débute en POO et je me débrouille avec ce que je trouve sur internet. Je voudrait avoir votre avis sur une de mes premières classes. Elle permet de récupérer la langue d'un client (le client est stocké dans ma base de données) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
class myLangue {
    private $id_Compte;
    private $id_Client;
 
	// fonctions d'initialisation
    public function __construct($ets,$nui) { 
        $this->id_Compte = $ets;
        $this->id_Client = $nui;
    }
 
	// fonction d'enregistrement de la langue
    public function saveLangue($langue) {
        $query = "update clients";
        $query.= " set langue = '$langue'";
		$query.= " where id_compte = '".$this->id_Compte."'";
		$query.= " and id_client = ".$this->id_Client;
		execute_query($query); //Fonction permettant d'exécuter une requete
    }
 
	// fonction donne la langue
    public function getLangue() {
		$tabLng = array();
        $query = "select ifNull(langue,'FR') as langue";
        $query.= " from clients";
		$query.= " where id_compte = '".$this->id_Compte."'";
		$query.= " and id_client = ".$this->id_Client;
		$query.= " limit 1";
		$tabLng= select_query($query); // fonction permettant d'executer la requete et de récupérer une tableau associatif comme résultat
		if (count($tabLng)==0) { 
			return 'FR';
		} else {
			return $tabLng[0]['langue'];
		}
	}
}

D'avance merci pour vos réponses

[rawsrc]

Salut,

Je ne vois du tout l'intérêt de faire une classe myLangue. A la vue de ton code, je dirais que la langue devrait faire normalement partie d'une classe Client.

Ensuite vu que tu es débutant autant apprendre directement les bonnes choses en matière de standard de codage : un peu de lecture ici. Il est toujours bon de suivre à au moins 90% les préconisations PSR-*.
Elle ne sont pas dénuées de bon sens et à l'usage c'est très bénéfique.

Enfin, vu que tu souhaites basculer vers la POO et afin de rester cohérent dans la nouvelle voie empruntée, je t'invite à faire de même pour la couche d'accès aux données en utilisant PDO.

Bon bouquinage et n'oublie pas qu'au début le meilleur outil que tu puisses avoir est la feuille et le crayon. Schématise pour fixer au mieux la théorie, le découpage et les responsabilités de tous les objets que tu vas être amené à manipuler.
N'oublie pas que si tu veux durer : tu dois surtout éviter l'erreur du débutant en t'attaquant trop vite à l'Everest.

Un outil gratuit que j'utilise avant tout codage : yEd

Bon courage.

[ascito]

En ajout a rawsrc,

moi je vois ca un peu comme ca ( on ne dénigre rien, hein),

1er )

quand tu crée une Class myLangue, avec un autoConstructeur, pour moi, cela veux veux dire, que tu aura besoin d'avoir plusieurs langues à gérer sur la même page, ou au moins que tu aura besoin de ces infos plus d'une fois pour mettre en page le site.

2 em)

le $this , pourquoi pas, mais avec le 1 er ) ( ou avec la remarque rawsrc ) , il doit alimenter une class plus standart, en fait celle qui fera ton "Patron" ( ou un de tes patrons ) tout au long de ta découverte des OP.
Pour ca n'ésite pas à faire une class qui ne sert à rien, tu lui donne un nom ( de préférence un qui te tiens à coeur ) , et tu rapatrie, dans la mesure du possible tous les $this dedans.
Amuse toi un peu, regarde le résultat, et déjà de mon point de vue, tu verra certaines diférences

3 em )

le site de php

http://fr2.php.net/manual/fr/funcref.php

http://fr2.php.net/manual/fr/ref.spl.php