Valueobject

[srm]

Et en Scala, comme d'habitude c'est super simple :

case class Numero(value: Int) {

    require(value > 0, "Le numéro doit être positif")

}



case class Rue(value: String) {

    require(value.length > 0, "La rue ne doit pas être une chaîne vide")

}



case class Ville(value: String) {

    require(value.length > 0, "La ville ne doit pas être une chaîne vide")

}



case class Adresse(numero: Numero, rue: Rue, ville: Ville) {

    override def toString = numero.value + " " + rue.value + " " + ville.value

}



val adresse_de_jean = Adresse(Numero(44), Rue("de la mairie"), Ville("Paris"))

println("Jean habite au " + adresse_de_jean);



val adresse_de_bob = Adresse(Numero(44), Rue("de la mairie"), Ville("Paris"))



if (adresse_de_bob == adresse_de_jean) {

    println("Adresses identiques");

} else {

    println("Adresses différentes");

}

[Max72]

La vache, vous êtes vraiment des 'Posting Freak' ! On s'en va 3H et y'a une page en plus ^^
C'est parti :

Effectivement, plutôt qu'une interface, je ferais plutot une abstract class avec une exception dans la méthode __set.

Les classes abstraites je les réserves à autre chose : le numéro de rue pourrait très bien étendre une classe abstraites Integer, avec les fonctions communes déjà implémentées, comme :

Code PHP :

<?php 

abstract class ValueObjectInteger

{

  private $number;



  public function __construct($number)

  {

    if (!is_int($number)) 

    {

      throw new InvalidArgumentException('le nombre doit être un entier');

    } 

    $this->number = $number;

  }



  public function __set($nom, $valeur)

  {

    throw new Exception('Tentative de modification du ValueObject Integer');

  }



  public function isEqual(ValueObjectInterface $number)

  {

    if (!$number instanceof $this)

    return false;



    return ($this->number === $number->number);

  }



  public function __toString()

  {

    return strval($this->number);

  }



} 

[niahoo]

@Max72 ta classe value integer peut elle même étendre une classe abstraite. L'idée est de bloquer les setters et cela devrait se faire à un seul endroit.

@srm Tu as oublié de comparer les noms de ville en lowercase

Enfin, les numéros de rue sont des String.

edit: les case class qui n'ont qu'un seul membre ont une fonction toString définie automatiquement qui output ce membre ?

[Xenos]

Qui va comparer deux adresses juste sur le nom de la rue ? // Elle porte le même nom, alors elle est égale. // Xenos, tu vas toujours trop loin

Je vous parie que sur le long terme, dans quelques années, vous allez avoir besoin de comparer deux rues de deux villes, vous allez être content de voir qu'il existe une classe "Rue" avec une méthode "isEquals()", et vous allez vous arracher les cheveux quand il faudra faire en sorte que deux rues de même nom de deux villes ne soient pas considérées comme identiques. A moins qu'on ne puisse me proposer une méthode non-destructive des API existantes et qui ferait cette comparaison.

Pour que ce soit plus propre, il faudrait que la méthode isEqual() de Rue ne puisse être appelée que par un objet Adresse, ça je suis d'accord.

Nope, c'est le principe de l'OO: n'importe qui/quoi peut demander un truc à une classe (aka appeler la méthode de l'objet). Si tu commences à vouloir restreindre "qui appelle la méthode", tu vas dans le mur (en tous cas, en PHP). Accessoirement, tu t'assoies sur la combinatoire (le truc génial là, qui fait qu'avec 10 classes t'as 100 assemblages possibles).

Si une méthode de classe A n'a vocation à être appelée que par une seule classe B, alors A est trop fortement couplée à B, et les deux classes ne devraient pas être séparées.

---

@srm Ca compare quoi, le "==" en fin de ton Scala? Les valeurs des membres? Ou le résultat du toString?

Parce que je me demande ce que ceci donnerait:

val adresse_de_jean = Adresse(Numero(2), Rue("Jean André"), Ville("Bouzy Foissons"))

val adresse_de_bob = Adresse(Numero(2), Rue("Jean Adré Bouzy"), Ville("Foissons"))

 

if (adresse_de_bob == adresse_de_jean) {

    println("Adresses identiques ?!");

Ah, l'injection... ^^

[niahoo]

srm ça compile pas ton truc

Si une méthode de classe A n'a vocation à être appelée que par une seule classe B, alors A est trop fortement couplée à B, et les deux classes ne devraient pas être séparées.

Ou bien A devrait être définie dans B :p

vous allez vous arracher les cheveux quand il faudra faire en sorte que deux rues de même nom de deux villes ne soient pas considérées comme identiques.

J'ai vraiment du mal à imaginer ce cas ! Là on bosse sur des value objects qui transportent des données. pas sur des modèles ou autre. Mais bon oui dans l'absolu si le pape change de sexe et que des alien agoraphobes débarquent, ça pourra toujours un jour poser problème. ça sera pas dur à mettre à jour

[Xenos]

Ouep, comme cela est autorisé en Java. Mais pas en PHP.

[niahoo]

Ah, l'injection...

Il a mis des espaces hein !

[Max72]

Enfin, les numéros de rue sont des String.

edit: les case class qui n'ont qu'un seul membre ont une fonction toString définie automatiquement qui output ce membre ?

Pas compris

Nope, c'est le principe de l'OO: n'importe qui/quoi peut demander un truc à une classe (aka appeler la méthode de l'objet). Si tu commences à vouloir restreindre "qui appelle la méthode", tu vas dans le mur (en tous cas, en PHP). Accessoirement, tu t'assoies sur la combinatoire (le truc génial là, qui fait qu'avec 10 classes t'as 100 assemblages possibles).

Si une méthode de classe A n'a vocation à être appelée que par une seule classe B, alors A est trop fortement couplée à B, et les deux classes ne devraient pas être séparées.

Je crois que tu n'as pas compris le sens : Rue pourrait être une classe enfant d'Adresse, et sa méthode isEqual() passerait en protected. Ainsi, seule Adresse peut appeler cette méthode. Ca créé quand même du couplage mais nécessaire (comme dans l'exemple actuel).

[Xenos]

J'ai édité:

val adresse_de_jean = Adresse(Numero(2), Rue("Jean André"), Ville("Bouzy Foissons"))

val adresse_de_bob = Adresse(Numero(2), Rue("Jean Adré Bouzy"), Ville("Foissons"))

Rue pourrait être une classe enfant d'Adresse, et sa méthode isEqual() passerait en protected. Ainsi, seule Adresse peut appeler cette méthode. Ca créé quand même du couplage mais nécessaire (comme dans l'exemple actuel).

J'ai un doute... Si tu as un typehinting quelque part genre setAdresseJoueur(Adresse $adresse), alors tu peux lui passer une Rue...

[niahoo]

Enfin, les numéros de rue sont des String.

edit: les case class qui n'ont qu'un seul membre ont une fonction toString définie automatiquement qui output ce membre ?

Pas compris

Et ben pour la première phrase : « 1 bis rue des machin truc »

Pour la seconde, c'est pour srm. soit il a eu la flemme d'implémenter toString pour ses 3 sous-classes, soit c'est automatique. C'est ce que je voudrais savoir. Mais ça compile pas et j'ai pas de REPL scala pour tester un script.