[Tableaux] Call to a member function set_var() on a non-object

Version imprimable

Bonjour,

J'utilise le moteur de templates de PHPlib, et je n'utilise pas la POO (comme ça c'est clair :P)

Une de mes pages contient une erreur que je n'arrive pas à trouver :

Citation:

Fatal error: Call to a member function set_var() on a non-object in E:\Programmation\Site\VERSION II\jouez\sante.php on line 33

J'ai été voir sur google ce que veut dire cette erreur, ce sera soit-disant du au fait que je ne déclare pas mes classes, ou un truc du genre (enfin, c'est pas ça parce qu'il y a bien

Code:

include ('includes/template.inc');

au début de mon code

Le bout qui foire c'est ça :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 function traitement($var,$name) { if($var < 0) {$var_img = 0;} elseif($var > 100) {$var_img = 100;} else {$var_img = $var;} $moteur -> set_var($name.'_img',$var_img); $moteur -> set_var($name,$var); }
cette fonction sert à générer deux variables :
$name.'_img' => longueur d'une image
$name => valeur réelle de la variable

l'image en question est d'autant plus grande que $var est grande, mais elle ne peut dépasser 100 pixels, c'est pour ça que j'ai créé cette petite fonction

et donc bin ça bugge, et je sais pas pourquoi :(

voilà, si quelqu'un sait m'aider, c'est bien gentil :king:

merci d'avance

18/12/2007, 16h39
Vell

Bonjour,

La question est de savoir à quoi correspond $moteur ...

C'est de là que vient ton erreur, car tu utilises $moteur->truc(); mais il faut peut-être faire les choses dans l'ordre avec un $moteur = MaClasse(); $moteur->truc();

Enfin bon, comme tu l'as déja lu, il faut instancier ta classe pour pouvoir utiliser ses fonctions.

wé wé ...bin je l'ai fait :s mais ça intervient plus haut dans la page :

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
 
include ('includes/connect_db.inc.php');
include ('includes/template.inc');
$moteur = new Template('templates/');
$moteur -> set_file('donnees_tpl','donnees.html');
 
[...]
 
function traitement($var,$name)
		{
		if($var < 0) 
			{$var_img = 0;} 
		elseif($var > 100)  
			{$var_img = 100;}
		else
			{$var_img = $var;}
 
		$moteur -> set_var($name.'_img',$var_img);
		$moteur -> set_var($name,$var);	
		}
 
	$moteur -> parse('results','donnees_tpl',true);
	$moteur -> p('results');	
?>

18/12/2007, 16h57
Vell

Honnêtement, y'a rien qui te choque dans ton code ?

C'est tout bête, mais je suis sûr qu'en te laissant mariner 5 minutes tu vas trouver tout seul :mouarf:
18/12/2007, 18h50
Hayabusa

euh ...

bin ...

oui il manque les appels à la fonction, mais bon j'ai pas mis tout le code parce que voilà....

sinon je vois pas :oops:
18/12/2007, 22h21
webmatrix

euh!

Code:

function traitement($var,$name,$var_img )

:lol: c'est çà? :aie:
18/12/2007, 23h14
Vell

Non ... la portée de la variable $moteur dans ta fonction ...

Mais le plus propre serait d'utiliser "return" dans ta fonction, car là ta fonction n'est pas réutilisable, puis c'est pas très recommandé de faire comme ça ...
18/12/2007, 23h21
webmatrix

ah oui c'est vrai!! mais de 1 c'est pas ma fonction, c'est pas mon code :evilred:
18/12/2007, 23h25
Vell

Et bien apprends donc à optimiser le code que tu récupères, ça t'évitera de nombreuses heures de prise de tête pour rien ;)
18/12/2007, 23h54
Hayabusa

:king:

merci bien, ça fait plaisir

mais je vois pas pourquoi ça foire sans return ...
19/12/2007, 01h32
Vell

Pour que ça ne foire plus il te suffit de mettre dans ta fonction : global $moteur;

Mais c'est vraiment crade ! Une fonction doit idéalement te retourner un résultat ou alors false.

Si tu utilises return, tu auras une fonction, qui sera propre : elle fera une seule chose, mais elle le fera bien. Ce n'est pas son rôle d'avoir set_var() ! Tu l'ajouteras simplement plus loin dans le code.
19/12/2007, 09h37
Hayabusa

ok mais j'ai 8 variables qui doivent être "traitées" par cette fonction

donc ça fait que je dois mettre 16 fois $moteur -> set_var :?

enfin c'est fait maintenant, mais ça fait pas très propre
19/12/2007, 17h21
Vell

Mets la totalité de ton code ici histoire de voir ce qu'il est possible de faire pour éclaircir tout ça :)

ok ok

en gros ça fait exactement la meme chose pour 8 variables, c'est à dire envoyer la variable au template de trois "façons" différents
- telle quelle
- après avoir "traitée" par la fonction traitement()
- en l'ayant soustrait à 100

tout ça sert à faire une petite barre pour chaque option, de 100 pixel de long. Un partie verte de X pixel de long, et une partie rouge, de 100-x pixels de long, où X est la valeur de l'option...

Comme je fais 8 fois la meme chose je pensais que ce serait plus facile avec une fonction :s

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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
$sql1 = 'SELECT faim,soif,moral,sante,hygiene,confort,condition_physique,forcee,habilite,points,argent FROM joueurs WHERE id='.$_SESSION['id'];
	$res1 = mysql_query($sql1) or die (mysql_error());
 
	while($row1 = mysql_fetch_array($res1))
		{
		$moteur -> set_var('faim_img',traitement($row1['faim']));
		$moteur -> set_Var('rouge_faim',bcsub('100',$row1['faim']));
		$moteur -> set_var('faim',round($row1['faim'],0));
 
		$moteur -> set_var('soif_img',traitement($row1['soif']));
		$moteur -> set_Var('rouge_soif',bcsub('100',$row1['soif']));
		$moteur -> set_var('soif',round($row1['soif'],0));
 
		$moteur -> set_var('moral_img',traitement($row1['moral']));
		$moteur -> set_Var('rouge_moral',bcsub('100',$row1['moral']));
		$moteur -> set_var('moral',round($row1['moral'],0));
 
		$moteur -> set_var('sante_img',traitement($row1['sante']));
		$moteur -> set_Var('rouge_sante',bcsub('100',$row1['sante']));
		$moteur -> set_var('sante',round($row1['sante'],0));
 
		$moteur -> set_var('hygiene_img',traitement($row1['hygiene']));
		$moteur -> set_Var('rouge_hygiene',bcsub('100',$row1['hygiene']));
		$moteur -> set_var('hygiene',round($row1['hygiene'],0));
 
		$moteur -> set_var('confort_img',traitement($row1['confort']));
		$moteur -> set_Var('rouge_confort',bcsub('100',$row1['confort']));
		$moteur -> set_var('confort',round($row1['confort'],0));
 
		$moteur -> set_var('condition_img',traitement($row1['condition_physique']));
		$moteur -> set_Var('rouge_condition',bcsub('100',$row1['condition_physique']));
		$moteur -> set_var('condition',round($row1['condition_physique'],0));
 
		$moteur -> set_var('force_img',traitement($row1['forcee']));
		$moteur -> set_Var('rouge_force',bcsub('100',$row1['forcee']));
		$moteur -> set_var('force',round($row1['forcee'],0));
 
		$moteur -> set_var('habilite_img',traitement($row1['habilite']));
		$moteur -> set_Var('rouge_habilite',bcsub('100',$row1['habilite']));
		$moteur -> set_var('habilite',round($row1['habilite'],0));
 
		if($row1['moral'] < 50)
			{$moteur -> set_Var('imagemoral','triste.jpg');}
		else
			{$moteur -> set_Var('imagemoral','sourire.jpg');}	
		}
 
	function traitement($taille)
		{
		if($taille < 0) 
			{$taille_img = 0;} 
		elseif($taille > 100)  
			{$taille_img = 100;}
		else
			{$taille_img = $taille;}
 
		return $taille_img;
		}
 
	$moteur -> parse('results','sante_tpl',true);
	$moteur -> p('results');

19/12/2007, 23h10
Hayabusa

c'est pas trop beau comme ça, c'est super, mais ça marche ... :yaisse2:

donc tu tues pas à trouver une solution de taré ;)
20/12/2007, 00h30
Vell

Hello,

Si ton moteur de template accepte les tableaux, tu peux tout passer dans un tableau ça simplifiera déja beaucoup le code :)

Sinon, il y'a un problème dans ton code : tu redéfinis toutes les variables à chaque boucle while. Bon, ok à priori il n'y a qu'une seule ligne de retournée ... mais à ce moment là il faut enlever le while ;)