PHP

Passer en poo ?

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Passer en poo ?

philipMo

Bonjour,

Je travaille sur un petit jeu de foot en php/mysql. Je me débrouille en PHP mais je ne suis jamais passé à la programmation orientée objet, et je me dis que c’est peut-être l’occasion.

Imaginons donc une rencontre entre 2 équipes. Je pensais faire :

- Une classe « equipe » avec des statistiques propres à l’équipe que je ne détaillerai pas ici pour être plus clair. J’ai donc 2 instances, une pour chaque équipe : $equipe[0] et $equipe[1].

- Une classe « joueur » avec les statistiques personnelles des joueurs : vitesse, fatigue, précision etc… ainsi que son poste sur le terrain (défenseur=0, attaquant=1…), son statut (en jeu=0, expulsé=1) et une référence à son équipe (0 ou 1). Pour les instances, il y en a autant que de joueurs : $joueur[0]… $joueur[21].

Jusque là, ça me paraissait cohérent. Mais à l’utilisation, j’ai un peu de mal sur 2 points :

- J’aimerais avoir un tableau contenant les clés de tous les joueurs (hors expulsés), trié par un attribut personnel, disons ici la fatigue : ($joueur[]->fatigue).

- J’aimerais également connaître le défenseur (hors expulsé) le moins fatigué d’une équipe.

Ce genre de chose est fait vraiment très régulièrement dans le match, et avec différentes variables, j’aimerais donc trouver quelque chose de rapide et réutilisable efficacement.

J’avais réussi à faire tout ça sans programmer en objet, mais je voulais quelque chose de plus clair. Et là, mon code commence à devenir du n’importe quoi, je copie des tableaux dans tous les sens pour les trier, et je me demande si je n’ai pas fait fausse route dès le début avec ces classes. Qu’en pensez-vous ?

Merci.

Publicité

budlite

Fresh

Pour la liste des joueurs expulses, pourquoi ne pas creer une methode
$equipe->getExpelledPlayers(); ( avec pourquoi pas en parametre le champ selon lequel on trie le tableau ).

J'imagine que ta classe Equipe contient un tableau de Joueurs. Dans ce cas la methode est extremement simple, elle itere sur le tableau et si le joueur est expulse, l'ajoute au tableau du resultat.

Bon j'ai fait ca vite fait ya surement des erreurs mais c pour te donner l'idee.

Dans ta classe Equipe :

Code :

public function getExpelledPlayers()
{
$expelledPlayers = array();
foreach( $this->equipe->getPlayers() as $player )
{
if ( $player->isExpelled() )
$expelledPlayers[] = $player;
}
return $expelledPlayers;
}
public function getLessTiredPlayer()
{
$resultPlayer = '';
$minimumFatigue = 100 ; // J'ai mis ca au pif, a toi de mettre la valeur max de fatigue d'un joueur
foreach( $this->equipe->getPlayers() as $player )
{
if ( $player->isDefensor() && $player->getFatigue() < $minimumFatigue )
{
$result = $player;
$minimum = $player->getFatigue();
}
}
return $resultPlayer;
}

Message édité par budlite le 18-08-2008 à 06:50:25

---------------
Fresh

philipMo

Si je comprends bien, tu verrais un truc comme ça ?

Code :

class Equipe{
var $side;
var $joueurs = array();
function Equipe($side){
$this->side = $side;
}
function addPlayer($ligne){ // $ligne est issu d'une requête MySQL
$this->joueurs[] = new Joueur($ligne, $this->side);
}
function getPlayers(){
return $this->joueurs;
}
public function getLessTiredPlayer() {
$resultPlayer = '';
$minimumFatigue = 100 ; // J'ai mis ca au pif, a toi de mettre la valeur max de fatigue d'un joueur
foreach( $this->getPlayers() as $keyPlayer => $objPlayer ) {
if ($objPlayer->isDefensor() && $objPlayer->getFatigue() < $minimumFatigue ) {
$resultPlayer = $keyPlayer;
$minimumFatigue = $objPlayer->getFatigue();
}
}
return $resultPlayer;
}
}
class Joueur{
var $fatigue;
var $poste;
var $status;
var $equipe;
function Joueur($ligne, $equipe){
$this->fatigue = $ligne->fatigue;
$this->poste = $ligne->status;
$this->status = 0;
$this->equipe = $equipe;
}
function addFatigue($howmuch){
$this->fatigue += $howmuch;
}
function getFatigue(){
return $this->fatigue;
}
function isDefensor(){
return ($this->poste == 0) ? true : false;
}
}

Message cité 1 fois

yellu

Moi je te conseil de passer en francais completement ou en anglais completement dans la nomination de tes méthodes / attributs, parce que les getFatigue et isDefensor ça irrite je trouve...

Message cité 1 fois

weed

yellu a écrit :

Moi je te conseil de passer en francais completement ou en anglais completement dans la nomination de tes méthodes / attributs, parce que les getFatigue et isDefensor ça irrite je trouve...

c'est une question d'habitude

sérieusement, le problème c'est que let préfixes set et get sont des normes.

philipMo

Je note, mais j'avoue que pour le moment, cela m'importe peu. Budlite m'a proposé des fonctions (merci d'ailleurs), je cherche seulement à savoir si la structure, dans mon cas de figure, est correcte. Qu'en pensez-vous ? (notamment d'instancier la classe joueur dans la classe equipe, car je ne suis pas sûr que Budlite voulait dire ça).

budlite

Fresh

philipMo a écrit :

Si je comprends bien, tu verrais un truc comme ça ?

Code :

class Equipe{
var $side;
var $joueurs = array();
function Equipe($side){
$this->side = $side;
}
function addPlayer($ligne){ // $ligne est issu d'une requête MySQL
$this->joueurs[] = new Joueur($ligne, $this->side);
}
function getPlayers(){
return $this->joueurs;
}
public function getLessTiredPlayer() {
$resultPlayer = '';
$minimumFatigue = 100 ; // J'ai mis ca au pif, a toi de mettre la valeur max de fatigue d'un joueur
foreach( $this->getPlayers() as $keyPlayer => $objPlayer ) {
if ($objPlayer->isDefensor() && $objPlayer->getFatigue() < $minimumFatigue ) {
$resultPlayer = $keyPlayer;
$minimumFatigue = $objPlayer->getFatigue();
}
}
return $resultPlayer;
}
}
class Joueur{
var $fatigue;
var $poste;
var $status;
var $equipe;
function Joueur($ligne, $equipe){
$this->fatigue = $ligne->fatigue;
$this->poste = $ligne->status;
$this->status = 0;
$this->equipe = $equipe;
}
function addFatigue($howmuch){
$this->fatigue += $howmuch;
}
function getFatigue(){
return $this->fatigue;
}
function isDefensor(){
return ($this->poste == 0) ? true : false;
}
}

Ouais en gros je vois ca comme ca.
Une equipe est composee de plusieurs joueurs, donc un attribut de Equipe doit etre un tableau de Joueurs.

Et sinon toute petite correction

Code :

function isDefensor(){
return ($this->poste == 0) ;
}

est suffisant

Message cité 1 fois

---------------
Fresh

esox_ch

Salut,

Moi en général, dans une fonction comme addFatigue, je colle un return $this->fatigue à la fin.
Même si on peut dire que c'est pas vraiment sémantique et tout et tout, je vois que c'est une pratique courante dans certains langages comme Ruby (et il me semblerait que Python fasse pareil) et je dit que c'est vachement pratique

Message cité 2 fois

---------------
Si la vérité est découverte par quelqu'un d'autre,elle perd toujours un peu d'attrait

budlite

Fresh

esox_ch a écrit :

Ben en ruby c'est pas une "pratique" ( au sens ou c'est toi le developpeur qui le met), c'est juste le language qui le fait automatiquement pour toi.

Message cité 1 fois

---------------
Fresh

masklinn

í dag viðrar vel til loftárása

esox_ch a écrit :

Dans le addFatigue j'aurais plutôt tendance à renvoyer this perso, ça serait largement plus pratique

Message cité 1 fois

---------------
I mean, true, a cancer will probably destroy its host organism. But what about the cells whose mutations allow them to think outside the box by throwing away the limits imposed by overbearing genetic regulations? Isn't that a good thing?

Publicité

budlite

Fresh

masklinn a écrit :

Dans le addFatigue j'aurais plutôt tendance à renvoyer this perso, ça serait largement plus pratique

Pour pouvoir enchainer les commandes ?
genre

Code :

$player->addFatigue(20)->addMachin(30)->addTruc(40) ;

Je ne vois que cet interet, y en a t il d'autre ?

Message cité 1 fois

---------------
Fresh

masklinn

í dag viðrar vel til loftárása

budlite a écrit :

Pour pouvoir enchainer les commandes ?
genre

Code :

$player->addFatigue(20)->addMachin(30)->addTruc(40) ;

Je ne vois que cet interet, y en a t il d'autre ?

Yen a pas d'autre, mais c'est super pratique. Alors que renvoyer la valeur, franchement... [:petrus75]

Message cité 1 fois
Message édité par masklinn le 19-08-2008 à 11:39:45

budlite

Fresh

masklinn a écrit :

Yen a pas d'autre, mais c'est super pratique. Alors que renvoyer la valeur, franchement... [:petrus75]

Je pense pareil :jap:

---------------
Fresh

esox_ch

budlite a écrit :

Ben en ruby c'est pas une "pratique" ( au sens ou c'est toi le developpeur qui le met), c'est juste le language qui le fait automatiquement pour toi.

C'est bien ce que j'entendais.. Si j'avais voulu dire que c'était une pratique que les dev ont, ça pourrait être pareil dans tous les langages..

Concernant le fait de retourner $this ou la valeur ... C'est un choix :

Code :

irb(main):001:0> class Player
irb(main):002:1> def initialize
irb(main):003:2> @fatigue = 0
irb(main):004:2> end
irb(main):005:1> def add_fatigue fatigue
irb(main):006:2> @fatigue += fatigue
irb(main):007:2> end
irb(main):008:1> end
=> nil
irb(main):009:0> player=Player.new
=> #<Player:0xb7d44344 @fatigue=0>
irb(main):010:0> player.add_fatigue 10
=> 10

Message cité 1 fois
Message édité par esox_ch le 19-08-2008 à 20:35:08

---------------
Si la vérité est découverte par quelqu'un d'autre,elle perd toujours un peu d'attrait

masklinn

í dag viðrar vel til loftárása

esox_ch a écrit :

C'est un choix

En l'occurence dans ton exemple c'est pas un choix du tout, juste le comportement par défaut du langage qui est de retourner le résultat de la dernière expression évaluée.

esox_ch

T1 mais je dois mal m'exprimer (ou alors vous comprenez tout de travers).
Je voulais dire que c'est le choix de Ruby de retourner la dernière expression, et non pas $this comme tu l'aurais fait. Il me semble d'ailleurs que Python ait fait le même choix non?

De plus c'est un choix que je trouve assez judicieux, parce que comme ça, si tu veux savoir combien de fatigue a un joueur après lui avoir ajouter X, tu peux le faire en 1 coup..

Message cité 1 fois

---------------
Si la vérité est découverte par quelqu'un d'autre,elle perd toujours un peu d'attrait

masklinn

í dag viðrar vel til loftárása

esox_ch a écrit :

Je voulais dire que c'est le choix de Ruby de retourner la dernière expression, et non pas $this comme tu l'aurais fait.

Renvoyer this/self est un choix qui ne peut être fait qu'au cas par cas, dans la mesure où ça n'a pas de sens en dehors des méthodes.

esox_ch a écrit :

Il me semble d'ailleurs que Python ait fait le même choix non?

Non, Python demande un retour explicite, sinon il renvoie None (/null/nil/whatever).

esox_ch a écrit :

De plus c'est un choix que je trouve assez judicieux, parce que comme ça, si tu veux savoir combien de fatigue a un joueur après lui avoir ajouter X, tu peux le faire en 1 coup..

Franchement, perso j'aurais collé un attr_accessor :fatigue et baste.

Message cité 1 fois
Message édité par masklinn le 19-08-2008 à 21:02:09

CyberDenix

budlite a écrit :

Ouais en gros je vois ca comme ca.
Une equipe est composee de plusieurs joueurs, donc un attribut de Equipe doit etre un tableau de Joueurs.

Et sinon toute petite correction

Code :

function isDefensor(){
return ($this->poste == 0) ;
}

est suffisant

Code :

function isDefensor() {
  return $this->poste;
}

est suffisant

---------------
Directeur Technique (CTO)

esox_ch

masklinn a écrit :

Franchement, perso j'aurais collé un attr_accessor :fatigue et baste.

Certes, mais si tu sais que tu vas souvent devoir ajouter de la fatigue à un joueur (par exemple s'il subit un tacle), et que donc tu vas le changer s'il a plus que X de fatigue, ça peut être pratique de faire un :

Code :

unless( player.add_fatigue(10) < MAX_FATIGUE )
change_player(player, fresh_players.pop)

Alors que sinon ça aurait demandé une ligne de plus (d'abord faire le add_fatigue, puis appeler l'accessor) ... Mais bon au fond les deux ont du sens ...
et pis ça aide pas des masses l'auteur du topic de savoir si Ruby et Python font ça ou pas (à part p-e de se rendre compte qu'au lieu d'écrire son site en PHP (en tentant de l'écrire OO) il aurait pu choisir un langage qui est OO de base)

Message cité 1 fois
Message édité par esox_ch le 19-08-2008 à 23:40:36

---------------
Si la vérité est découverte par quelqu'un d'autre,elle perd toujours un peu d'attrait

masklinn

í dag viðrar vel til loftárása

esox_ch a écrit :

Alors que sinon ça aurait demandé une ligne de plus (d'abord faire le add_fatigue, puis appeler l'accessor) ...

Non mais avec un attr_accessor le add_fatigue dégage hein, il sert plus à rien [:dawa]

esox_ch

Faut voir dans le contexte de l'appli ..
Par exemple si la fatigue ne peut jamais diminuer (à part être remise à 0 entre 2 match), je trouve que ça a pas beaucoup de sens de faire un attr_accessor.
Je ferais un add_fatigue + attr_reader :fatigue

Message cité 1 fois

---------------
Si la vérité est découverte par quelqu'un d'autre,elle perd toujours un peu d'attrait

masklinn

í dag viðrar vel til loftárása

esox_ch a écrit :

Faut voir dans le contexte de l'appli ..
Par exemple si la fatigue ne peut jamais diminuer (à part être remise à 0 entre 2 match)

Dans ce cas tu crées une property avec une vraie méthode en writer [:spamafote]

Non parce que ton add_fatigue, si tu lui passes un nombre négatif sans checker il va diminuer la fatigue hein [:dawa]

esox_ch

Mouais bof je suis pas convaincu .. Mais de toutes façons on s'en fout

Code :

def add_fatigue fatigue
@fatigue += fatigue unless fatigue < 0 # Voilà :)
end

Message cité 1 fois

---------------
Si la vérité est découverte par quelqu'un d'autre,elle perd toujours un peu d'attrait

masklinn

í dag viðrar vel til loftárása

esox_ch a écrit :

Mouais bof je suis pas convaincu ..

Pas convaincu de quoi exactement [:pingouino dei]

Code :

attr_reader :fatigue
def fatigue= fatigue
    @fatigue = fatigue unless fatigue < @fatigue
end

[:spamafote]

esox_ch

Pas convaincu que ce soit beaucoup mieux. Me semble que les 2 approches sont valables

Message édité par esox_ch le 20-08-2008 à 15:09:05

---------------
Si la vérité est découverte par quelqu'un d'autre,elle perd toujours un peu d'attrait

philipMo

Merci pour vos réponses. Mais pour ce qui concerne le ruby, j'avoue que cela ne m'intéresse pas.
Par contre, avec ce modèle, je reviens à la question initiale. J'aimerais obtenir régulièrement un tableau de tous les joueurs (des deux équipes donc), avec en clé l'id du joueur (référence à l'objet joueur en fait) et en valeur un critère défini trié (fatigue par exemple, mais il y en a beaucoup d'autres). Est-ce que cette structure est vraiment adaptée ? Parce que dans le cas présent, si je ne me trompe pas je dois reformer un tableau à partir des deux tableaux ($equipe[0]->joueurs[] et $equipe[1]->joueurs[]). Y a-t-il plus simple ? Merci à vous.

Message édité par philipMo le 20-08-2008 à 18:19:18

esox_ch

Bah tu peux créer un tableau joueurs à partir des 2 autres ayant comme index l'ID du l'utilisateur et comme valeur la valeur en question, après tu fais un sort et c'est bon.. Sinon si tu as déjà à portée de main un array contenant tous les ID des joueurs, tu peux faire une requête SQL sur ces ID avec un ORDER BY fatigue

---------------
Si la vérité est découverte par quelqu'un d'autre,elle perd toujours un peu d'attrait

Publicité

FORUM HardWare.fr

Programmation

PHP

Passer en poo ?

Sujets relatifs
Servlet passer une ligne "\n"	[RESOLU] Pb pour passer des arguments de ligne de commande
Ecrire sur une clé USB sans passer par le FS	[shell] Passer stdin à une commande passé au shell sur stdin
passer de mac à windows	Passer un objet Array à une fonction JS
Passer une valeur dans le liens!? GET ou POST? [RESOLUS]	Comment passer du flash au .html
Passer d'un site en frame en div	Macro excel passer plusieurs variable
Plus de sujets relatifs à : Passer en poo ?

Page générée en 0.116 secondes