Discussion :

[gege2061]

Bonjour,

Je souhaite simplement écrire :

Code :

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

new Bar ()->foo ();

Mais j'obtiens une erreur d'analyse :

Code x :

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

Parse error: parse error

Est-ce normal ?

[Celira]

Hum... franchement, je me rappelle pas avoir déjà essayé... Ce genre de syntaxe implique que tu crées une instance de ton objet mais que tu ne la gardes nulle part. Donc ça me parait logique que ton script râle : à quoi ça sert de faire des instances si on n'en fait rien ?

Si l'instance ne t'intéresse pas, ta méthode devrait sans doute être statique et appelée par ::

[gege2061]

Hum... franchement, je me rappelle pas avoir déjà essayé... Ce genre de syntaxe implique que tu crées une instance de ton objet mais que tu ne la gardes nulle part. Donc ça me parait logique que ton script râle : à quoi ça sert de faire des instances si on n'en fait rien ?

Si l'instance ne t'intéresse pas, ta méthode devrait sans doute être statique et appelée par ::

Si l'instance m'intéresse mais juste pour appeler une fonction. Actuellement je fais :

Code :

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

1
2
$bar = new Bar ();
$bar->foo ();

C'est un peu ce que l'on retrouve dans zend framework :

Code :

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

1
2
$application->bootstrap()
            ->run();

Sauf que je veux le faire en partant du constructeur, ce qui devrait être possible puisqu'il retourne bien un objet, mais je pense que ça n'a pas été prévu par l'analyseur donc peut être un bug à reporter ?

[Celira]

Si l'instance m'intéresse mais juste pour appeler une fonction. Actuellement je fais :

Code :

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

1
2
$bar = new Bar ();
$bar->foo ();

Si tu ne crées $bar QUE pour appeler la fonction et que tu ne t'en ressers pas après, c'est que l'instance ne t'intéresse pas... Auquel cas il faut utiliser une fonction statique.
Par contre, si tu t'en ressers après (pour autre chose qu'appeler cette même fonction) faut ben que tu la stocke quelque part et donc que tu fasses la manœuvre en deux étapes.

Enfin je suppose que ça dépend de ta fonction...

[gege2061]

Si tu ne crées $bar QUE pour appeler la fonction et que tu ne t'en ressers pas après, c'est que l'instance ne t'intéresse pas... Auquel cas il faut utiliser une fonction statique.
Par contre, si tu t'en ressers après (pour autre chose qu'appeler cette même fonction) faut ben que tu la stocke quelque part et donc que tu fasses la manœuvre en deux étapes.

Mais si je veux mon instance

Concrètement c'est une classe qui génère un PDF, j'ai juste besoin de l'afficher à l'écran, il semble logique de n'avoir qu'à appeler la fonction d'affichage, ensuite c'est l'objet qui sait comment il doit s'afficher :

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
class MyPDF extends PDF
{
  public function __construct ()
  {
    parent::__construct ('P', 'mm', 'A4');
    [...]
  }
 
  public function Content ()
  {
  }
 
  public function Output ()
  {
    $this->addPage ();
    $this->Content ();
    parent::Output ();
  }
}
 
new MyPDF ()->Output ();

Après si je veux faire d'autres choses rien ne m'empêche de chainer le tous :

Code :

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

1
2
3
new MyPDF ()->setTitle ('Mon PDF')
            ->setData ($data)
            ->Output ();

Bref, ma question ne porte pas sur la conception de mon code mais sur l'erreur d'analyse qui ne me semble pas justifiée. Est ce voulu ? Est ce normal ? Ou est ce un bug ?

[Petibidon]

Perso ça me parait logique.

Tu crée ton instance puis tu l'utilises. Tu ne peux pas faire les deux en une opération.
Y'a des langages qui permettent ce genre de choses ?

Le chaînage des méthodes après la création de l'instance est envisageable si les méthodes sont définies pour être chaînées.