[POO] Using $this when not in object context

Version imprimable

Bonjour j'ai une erreur "Using $this when not in object context" dans ce bout de code et je ne comprend pas (la ligne concernée est en gras) :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 <?php class TemplateEngine { private $template; private $variables; public function __construct($template) { $this->template = addslashes($template); $this->variables = array(); } public function addVariable($variable, $content) { $this->variables[$variable] = $content; } ... } ?>
Je suis bien dans un "contexte objet", j'ai bien instancié mon tableau dans le constructeur, qu'est-ce qui cloche ?

05/03/2010, 22h05
hornetbzz

je crois que private $template; est en contradiction avec ton constructeur, il y a double emploi.
05/03/2010, 22h12
Gaetch

Même en enlevant la ligne "// $this->variables = array();" j'obtiens toujours la même erreur. Et ça veut dire quoi double emploi ? On peut pas instancier un membre de la classe dans son constructeur ?
05/03/2010, 22h38
nosferapti

montre nous comment tu utilises cette classe pour avoir cette erreur

Voici la classe en entier :

Code:

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
27
28
29
30
31
32
33
34
35
36
37
 
<?php
	class TemplateEngine
	{
		private $template; // Fichier template à utiliser, contenant les variables à substituer.
		private $variables; // Tebleau associatif qui contiendra les variables et les valeurs à leur substituer.
 
		public function __construct($template)
		{
			$this->template = addslashes($template);
			// $this->variables = array();
		}
 
		// Ajouter une nouvelle variable à substituer.
		public function addVariable($variable, $content)
		{
			$this->variables[$variable] = $content;
		}
 
		// Affiche la page 
		public function outputModifiedPage()
		{
			if (file_exists($this->template))
			{
				// Récupération du contenu du template.
				$modifiedPage = file_get_contents($this->template);
				// Parcours du template et substitution des variables.
				foreach ($this->variables as $variable)
				{
					$modifiedPage = preg_repalce('#${'.$variable.'}#', $this->variables[$variable], $modifiedPage);
				}
				echo $modifiedPage;
			}
			else throw new Exception('Le fichier '.$template.' n\'existe pas');
		}
	}
?>

Voici le fichier php qui l'utilise :

Code:

1
2
3
4
5
6
7
8
9
10
 
<?php 
	require('templateEngine.php');
	$page = new TemplateEngine('default.tpl');
	$page::addVariable('description', 'Page de test');
	$page::addVariable('keywords', 'php, test, template');
	$page::addVariable('charset','utf-8');
	$page::addVariable('language','fr');
	$page::outputModifiedPage();
?>

Et juste au cas où voilà le fichier default.tpl :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" lang="en" xml:lang="en">
	<head>
		<meta name="Author" content="{$author}"/>
		<meta name="Description" content="{$description}"/>
		<meta name="Keywords" content="{$keywords}"/>
		<meta http-equiv="Content-Type" content="text/html;charset={$charset}"/>
		<meta http-equiv="Content-Language" content="{$language}"/>
	</head>
	<body>
	</body>
</html>

En gros ce que c'est sensé faire vous l'aurez compris c'est remplacer les variables entre {$...} par les valeurs dans le fichier php.

05/03/2010, 23h16
Gaetch

En remplaçant les :: par des -> dans le fichier php ça fonctionne. Quelqu'un pourrait m'éclairer sur quand utiliser l'un ou l'autre ?
06/03/2010, 00h29
Invité
Salut

Il ne faut pas confondre méthode membre et méthode statique :
- Une méthode membre (public/protected/private function toto()) s'utilise pour effectuer des traitements sur un objet instancié.
  Code:
  
  1 2 $audiA3 = new Voiture(); $audiA3->demarrer();
- Une méthode statique (public/protected/private static function titi()) s'utilise pour effectuer des opérations sur la classe d'objets et non pas sur un objet en particulier.
  
  Code:
  
  Voiture::getModelesDisponibles(); // renvoie une liste d'objets Voiture
Donc dans ton cas tu as bien besoin d'appeler une méthode membre, vu que tu as instancié un objet de ta classe TemplateEngine. C'est donc la flèche qui est utilisée.

Voilou

ps : contrairement à ce que dit hornetbzz, ton $templates est parfaitement déclaré dans ta classe et initialisé dans ton constructeur.
06/03/2010, 01h32
Gaetch

Merci beaucoup, ça fonctionne maintenant après quelques modifications.

Par contre je ne suis pas sûr d'avoir compris pour l'opérateur ::

Il s'utilise pour les membres "static", c'est à dire ceux qui n'existent qu'en un seul exemplaire, quel que soit le nombres d'instances de la classe ? Par exemple un compteur d'instances justement ?
06/03/2010, 05h22
hornetbzz

:oops: désolé
06/03/2010, 07h15
RunCodePhp
Citation:

Envoyé par Gaetch

Par contre je ne suis pas sûr d'avoir compris pour l'opérateur ::

Il s'utilise pour les membres "static", c'est à dire ceux qui n'existent qu'en un seul exemplaire, quel que soit le nombres d'instances de la classe ? Par exemple un compteur d'instances justement ?

Je ne pense pas que ce soit une question d'exemplaire.

La déclaration "static" est faite pour définir une portée (particulière), une visibilité, on dit aussi accessibilité d'une propriété, ou d'une méthode (pour une classe).

La définition du manuel me semble parfaite :
-> Php5 : static
-> L'opérateur de résolution de portée ::

Citation:

Statique
Le fait de déclarer des propriétés ou des méthodes comme statiques vous permet d'y accéder sans avoir besoin d'instancier la classe. On ne peut accéder à une propriété déclarée comme statique avec l'objet instancié d'une classe (bien qu'une méthode statique le puisse).

Comme les méthodes statiques peuvent être appelées sans objet, la pseudo-variable $this n'est pas disponible dans la méthode déclarée en tant que statique.

On ne peut pas accéder à des propriétés statiques à travers l'objet en utilisant l'opérateur ->.

A partir du moment où tu parviens à bien saisir la nuance de cette déclaration statique, de ces avantages et inconvénients du faite de cette visibilité particulière, tu ne devrais rencontrer moins de problème sur sa syntaxe.

Attention, Php a évolué sur cette aspect à partir de Php5.3.0, ce qui peut être déroutant, et même provoquer des bugs sur un projet développé en local sur Php5.3 et un site en production sur une version antérieure, genre Php5.2.* si on y fait pas gaffe.

Code:

print $classname::$my_static; // Depuis PHP 5.3.0

Aussi, la syntaxe d'une constante de classe est assez proche de celle d'une propriété statique :
Code:

1 2 3 echo UneClasse::constante; // Une constante de classe echo AutreClasse::$propriete; // Une propriété statique d'une classe
06/03/2010, 11h35
Invité

Citation:

Envoyé par Gaetch

Il s'utilise pour les membres "static", c'est à dire ceux qui n'existent qu'en un seul exemplaire, quel que soit le nombres d'instances de la classe ? Par exemple un compteur d'instances justement ?

C'est exactement ça. Ce que dit RunCodePhp est très juste techniquement. Les membres static peuvent être accédés (et je dirais même plus doivent être accédés) sans instancier d'objet (c'est pour ça que $this n'est pas autorisé dans ces méthodes). La question est donc à quoi ça sert, et quand l'utiliser ? Et bien comme tu dis, typiquement un compteur d'instances, ou une méthode qui va chercher des données dans une base et qui construit des instances (mon exemple d'hier Voiture::getModelesDisponibles()), ou toute méthode qui ne s'applique pas à un objet en particulier mais bien à la classe regroupant tous ces objets.

Citation:

Envoyé par hornetbzz

:oops: désolé

T'inquiète cacahuète, on est tous là pour apprendre ;)