Salut,

Au boulot je reprends du code PHP écrit par un autre dev que je ne connais pas et je trouve par exemple cette class:

class MaClass
{

public static function build() {
return new MaClass();
}

...
}

Pour obtenir une instance de MaClass, je dois donc écrire $objet = MaClass::build();
Au lieu de simplement faire un $objet = new MaClass()

Est ce que cela à un intérêt et si oui est ce que ça porte un nom?

Merci pour vos avis, je passe peut être à côté d'un truc évident

comme ça ...
aucune
mais ta surement une raison

surtout que niveau nommage ...

Alors oui c'est un pattern qui se nomme : Facade, il me semble.

C'est quelques choses d'assez utile mais dans ce cas la ... Peut-être qu'il s'imaginait pouvoir compléter la fonction plus tard.

D'après une petite recherche, façade c'est le fait d'appelez une classe au travers une autre classe. La il construit un objet à partir de ... lui même ... je suis un peu perplexe...

A mes yeux aucun intérêt ... new MonObject(); c'est plus générique ... et puis si il y a des choses particulière à faire dans la construction, autant utiliser une fonction fait pour : __construct.

Mais bon ... peut-être que c'est une norme de création d'un framework ?

Effectivement c'est peut-être pas forcément une Facade encore mais plus une factory

https://www.grafikart.fr/tutoriels/factory-570 ?

C'est une syntaxe de constructeur alternative, il y a surement une raison derrière ce choix notamment lors de ses tests ou pour une utilisation très spécifique de la classe.

Regarde la classe DateTime en PHP. La méthode createFromFormat répond au même pattern.
Pour ton cas par contre, j'ai du mal à voir l'avantage d'utiliser ça.

http://www.lephpfacile.com/manuel-php/datetime.createfromformat.php

Quelle horreur cette syntaxe..

Je suis quasiment sûr c'est une façade similaire à celles de Laravel : https://laravel.com/docs/5.7/facades

Le principe semble le même, sauf que le système de façade Laravel passe un container : le service container.

Et puisque toutes les classes façades héritent de la classe parente qui contient la méthode magique, __callStatic() , Laravel est capable de te retrouver la méthode que tu cherches à appeler, simplement par appel statique depuis ta classe.

------------------

$user = Cache::get('user:'.$id);

class Cache extends Facade
{
    /**
     * Get the registered name of the component.
     *
     * @return string
     */
    protected static function getFacadeAccessor() { return 'cache'; }
}

    protected static function getFacadeAccessor() 
    { 
        return App\Providers\ReportGeneratorServiceProvider::class;
    }

public function register()
   {
        $this->app->bind('Le\Chemin\Vers\Ta\Classe', function($app){
        return new Classe();
        });
   } 

public static function build() {
return new MaClass();

------------------

C'est plutôt bien pensé, mais perso j'aime pas.

D'ailleurs c'est soit le système de façade, soit l'injection de dépendance. Tu ne peux pas pas faire les deux, ou alors t'es juste un masochiste.

Laravel d'ailleurs te vante les mérites de son système de façade par rapport à l'injection de dépendance, (forcément) :

• https://laravel.com/docs/5.7/facades#facades-vs-dependency-injection

Et puis Laravel il est bien sympa, il te fait tout le taf pour la plupart de tes classes, et l'enregistre dans son propre code-source qu'il te dit à ne surtout pas modifier.

Et toi du coup t'as pas besoin de comprendre le mécanisme derrière, tu fais un simple, Articles::all() et t'es bien content d'avoir la liste de tout tes articles

• Enfin, jusqu'à ce que tu sois forcé à créer une façade dès l'instant où le projet se complexifie

Pauvre op, l'autre dev est probablement un Laravel-addict.

------------------

Le 08 mars 2020 à 20:45:07 Marav a écrit :
Pour ton cas par contre, j'ai du mal à voir l'avantage d'utiliser ça.

Une alternative à l'injection de dépendance.

Moche à souhait.

Complétement d'accord avec mon vdd, le système de Facade pose certaine problématique (bien que comme dit, très bien pensé) et se révèle être une horreur dans l'IDE

Le 09 mars 2020 à 15:48:52 UndeadMarston6 a écrit :
Complétement d'accord avec mon vdd, le système de Facade pose certaine problématique (bien que comme dit, très bien pensé) et se révèle être une horreur dans l'IDE

La même, je trouve ça très moche