Discussion :

[greg91]

Bonjour à tous,

J'ai plusieurs méthodes pour afficher une désignation dans différentes langues

Code :

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

1
2
$object->getDesignationFr()
$object->getDesignationEn()

La langue en cours en stockée dans une variable $lang

Lors de l'appel à ces méthodes, voilà ce que je fais pour avoir la bonne langue :

Code :

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

1
2
3
 
$method = "getDesignation".ucfirst($lang);
echo $object->$method();

Ce n'est pas très élégant, n'y a t'il pas un moyen de tout faire tenir en une seule ligne ?
Une sorte de

Code :

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

echo $object->"getDesignation".ucfirst($lang)

Mais qui fonctionne

Merci !

[MaitrePylos]

Ne serait-il pas plus propre et de refactorisé le code et d'ajouter une méthode :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
 
public function getDesignation($lang){
 
    if($lang == 'fr'){
        return $this->getDesignationFr()
    }
    if($lang == 'en'){
        return $this->getDesignationEn()
    }
    return   $this->getDesignationDefault() 
}

[rawsrc]

Salut,

t'es sûr du design de tes classes d'internationalisation ? Parce que créer une méthode par langue pour avoir la traduction c'est sacrément bancal.
Sinon tu peux faire comme ça :

Code :

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

call_user_func([$object, 'getDesignation'.ucfirst($lang)]);

[armel18]

la langue est commune à tous les attributs de la classe. ainsi la langue sera aussi attribut de ladite classe pour pouvoir renseigner la langue une seule fois au moment de l'instanciation de la classe.
A l'instanciation, on invoque n'importe quelle méthode de MyObject sans se préoccuper de la langue ce qui laisse à la vue sa vrai responsabilité celle d'afficher les données sans introduite des conditions.

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
 
class MyObject {
 
    private $designation; // désignation avec la langue par défaut(fr par exemple)
    private $designationEn;
    /**
     * @var null|string
     */
    private $lang;
 
 
    public function __construct(?string $lang ='fr' ) {
        $this->lang = $lang;
    }
 
    /**
     * @return string
     */
    public function getDesignation():?string {
        return
            substr($this->designation, -2) != $this->lang ?
                $this->designation: $this->designation.ucfirst($this->lang) ;
    }
 
 
}

Certainement une classe abstraite définissant la langue comme attribut(au lieu de le faire dans MyObject) laquelle classe serait un modèle pour une classe comme MyObjet serait bien plus intéressante.

[greg91]

Effectivement je vais revoir le code ma classe.
Merci beaucoup !

[Celira]

Et avec des accolades ?

Code :

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

$object->{"getDesignation".ucfirst($lang)}()

[greg91]

Et avec des accolades ?

Code :

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

$object->{"getDesignation".ucfirst($lang)}()

Et bien cela marche !
Merci Celira