[Résolu] Création de classes

[zneman]

Bien le bonjour
Afin de me mettre pleinement dans l'utilisation des fonctions (que je n'utilisais jamais avant), je me suis lancé dans la POO.
Après avoir passé ma matinée devant des tutos et des articles sur la POO, j'ai enfin compris son utilité et pour être franc... Ca me donne très envie (de coder ^^).

Bref... Je suis en ce moment en train de revoir complètement le plan de mon script d'attaque principal.
Je me suis dis "pourquoi faire les choses à moitié, commençons à y mettre de la POO tout plein !" xD

Voila donc le début du début mais je voudrais savoir si je pars dans la bonne direction avant de me lancer dans les milliers de lignes

Code PHP :

<?php

class AttaqueArmesSiege {

    public function __construct($beliers, $trepans, $balistes, $espringales, $trebuchets, 

$forceBeliers, $forceTrepans, $forceBalistes, $forceEspringales, $forceTrebuchets) {

      $this->beliers = $beliers;

      $this->trepans = $trepans;

      $this->balistes = $balistes;

      $this->espringales = $espringales;

      $this->trebuchets = $trebuchets;

      $this->forceBeliers = $forceBeliers;

      $this->forceTrepans = $forceTrepans;

      $this->forceBalistes = $forceBalistes;

      $this->forceEspringales = $forceEspringales;

      $this->forceTrebuchets = $forceTrebuchets;

    }

    

    public function PourcentagesArmesSiege() {

      $this-> Pourcentages = mt_rand(80,100) / 100;

      return $this->Pourcentages;

    }

    

    public function AttaqueBruteArmesSiege() {

      $this-> AttaqueBrute = ($this->beliers * $this->forceBeliers) + ($this->trepans * $this->forceTrepans)

 + ($this->balistes * $this->forceBalistes) + ($this->espringales * $this->forceEspringales)

 + ($this->trebuchets * $this->forceTrebuchets);

      return $this->AttaqueBrute;

    }

    

    public function AttaqueNetArmesSiege() {

      $this-> AttaqueNet = $this->AttaqueBrute * $this->Pourcentages;

      return $this->AttaqueNet;

    }

    

    public function AfficherAttaqueNetArme() {

      echo $this->AttaqueNetArmesSiege();

    }

    

    public function AfficherPourcentagesAttaqueArmesSiege() {

      echo $this->PourcentagesArmesSiege();

    }

}

?>

[Ren Nelos]

Bonjour zneman.

Première remarque qui s'apparente plus à un conseil, tu devrais faire des lignes de code moins longue. C'est très peu confortable de devoir déplacer la vue pour prendre connaissance du code. J'ajouterais également que tu devrais fournir des commentaires dans ta source surtout quand on commence quelque chose de nouveau pour soi. Cela dit, cela vaut pour n'importe quelle situation : commente !
Dans la même catégorie, je te conseillerais d'utiliser des noms de méthodes plus explicites. Par exemple, setMaVariable() plutôt que MaVariable().
Généralement, les noms de méthodes & de variables ne commencent pas par une majuscule ^_^

Ensuite, tu fais des $this->variableName. Seulement, tu as omis de déclarer les variables dans ta classe, qu'on appelle variables de classe.

Ce qui suit est également à proscrire :

Code PHP :

<?php 

public function AfficherAttaqueNetArme() {

      echo $this->AttaqueNetArmesSiege();

    } 

[Cartman34]

Quand je vois ca:

Code PHP :

<?php 

public function AttaqueBruteArmesSiege() {

      $this-> AttaqueBrute = ($this->beliers * $this->forceBeliers) + ($this->trepans * $this->forceTrepans)

 + ($this->balistes * $this->forceBalistes) + ($this->espringales * $this->forceEspringales)

 + ($this->trebuchets * $this->forceTrebuchets);

      return $this->AttaqueBrute;

    } 

[naholyr]

Plusieurs commentaires :
- Tu utilises des attributs d'objet sans les déclarer, ces $this->Variable n'ont pas de sens.
- IGStaff++ : Tu définis des attributs parfois dans une méthode, sans jamais les utiliser ailleurs. Pourquoi alors passer par un attribut ?
- Lignes trop longues => factorisation ! Dans ce cas, la factorisation t'aurait mené peut-être à la solution de Ren, mais qui n'apporte rien d'autre que de réduire la ligne. Je te propose de rendre ton code évolutif, de manière à ce que tu puisses ajouter extrèmement simplement de nouvelles armes.
- Les méthodes "Afficher*" ne devraient pas exister, pour plusieurs raisons :
-- Une méthode d'une classe "utilitaire" ne devrait jamais rien afficher directement
-- Le seul intéret de ces méthodes semblent d'économiser des octets, sinon quel intérêt ? Or "Afficher" (8) prend plus de place que "echo " (5), donc vraiment aucun intérêt

Tu ne t'en es pas rendu compte car tu n'as probablement pas assez réfléchi à ton code, mais tu manipules des objets dans cette classe, des objets de type "arme de siège", définis par :
- un nom
- une force
- une quantité

Code PHP :

<?php 

class ArmeSiege {

    

    public $nom;

    public $quantite;

    public $force;

    

    public function __construct($nom, $quantite, $force) {

        $this->nom = $nom;

        $this->quantite = $quantite;

        $this->force = $force;

    }

    

} 

[Sephi-Chan]

Merci Naholyr, je formulais le même conseil avant de voir ton message en faisant une prévisualisation.


Sephi-Chan

[Cartman34]

Tu poussess la POO extremement loin naholyr.
Même si j'en ai compris le but, je n'ai trop compris comment tu veux utiliser ce code:

Et au lieu d'appeler :

Code PHP :

<?php 

new AttaqueArmesSiege($beliers, $trepans, $balistes, $espringales, $trebuchets, 

$forceBeliers, $forceTrepans, $forceBalistes, $forceEspringales, $forceTrebuchets) 

[naholyr]

Ce n'est pas poussé extrèmement loin, si on fait de l'objet on pense objet. Si c'est juste faire des classes pour faire des classes (histoire de grouper les fonctions) ça n'a pas plus d'intérêt que de faire du procédural et de bien organiser ses includes (ce qui n'est pas foncièrement "mal" soit dit en passant).

La programmation orientée objet, c'est détecter les objets qui sont utilisés dans le processus, ici l'objet de base c'est l'arme de siège. L'objet que tu manipules regroupe une liste d'armes de sièges, donc il est normal de créer une classe "arme de siège"
Cela apparait très nettement quand on voit la liste initiale des paramètres.

[zneman]

Aie, j'en avais oublié ce message :$
Je suis pas mal occupé en ce moment IRL
Honte à moi de n'avoir répondu

Un grand merci à Ren Nelos pour sa réponse
Merci aux autres aussi, je commence à comprendre
Je posterais ce que j'ai fais plus tard dans la semaine je pense