[POO] Using $this when not in object context

**Gaetch** · 05/03/2010, 21h57

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 :

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
<?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 ?

**hornetbzz** · 05/03/2010, 22h05

je crois que private $template; est en contradiction avec ton constructeur, il y a double emploi.

**Gaetch** · 05/03/2010, 22h12

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 ?

**nosferapti** · 05/03/2010, 22h38

montre nous comment tu utilises cette classe pour avoir cette erreur

**Gaetch** · 05/03/2010, 22h52

Voici la classe en entier :

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
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 :

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

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 :

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

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.

**Gaetch** · 05/03/2010, 23h16

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 ?

[POO] Using $this when not in object context

Langage PHP

Vue hybride

Discussions similaires

Partager

Partager