[POO] bug? de php

**XtofRoland** · 21/02/2006, 10h56

deux classes, l'une s'appel row l'autre form.
ma classe form contient un array de row
La classe row a ete déclaré a l'exterieur de ma classe form

vous suivez toujours? j'ai parfois tendance a pas etre très clair.

Le fait de racourcir le nom d'une variable de la classe row pour lui donner le meme nom ($errmsg) qu'une variable de form a rendu inaccessible toute les variables (comme si elle n'existait pas ou private) de row.

J'ai utilisé un outil de comparaison de source pour retrouvé mon erreur. et vu que j'ai mis le WE pour la trouver ne me demandez pas si je suis sur qu'il manque pas un ; je risque de trouver ca agaçant

Je lui ai donc rendu sont nom d'origine et le problème est

mais pourquoi ne peut on pas donner un meme nom a des attributs de deux classes distinctes?

qqn a t'il deja rencontré un problème similaire?

PS : PHP 4 (easyphp 1.8)
code a la disposition de celui qui veut tester ;-) (mail par mp)

**Yogui** · 21/02/2006, 11h19

Salut

Est-ce que tu accèdes bien toujours à des propriétés de classe par $this-> ? Si ce n'est pas le cas, peut-être qu'il en fait une variable statique ou globale...
À mon avis, c'est une histoire de scope dans ce style, vu que PHP n'est pas très strict là-dessus.

[Edit] Pour de la POO, préfère PHP5

**polace** · 21/02/2006, 11h52

Slt Kirkis ,
Variable global ou statique, php est strict quand a l'utilisation de plusieurs attributs de meme noms provenant des classes differentes. A mon avis c'est un problème d'aisance dans la manip des objets afin d'éviter des ambiguités ou des conflits d'attributs .c'est aussi ça la POO. Oublier un peu les concepts de procedures et fonctions au profit d' objets .

**XtofRoland** · 21/02/2006, 11h57

Envoyé par Kirkis

Salut

Est-ce que tu accèdes bien toujours à des propriétés de classe par $this-> ? Si ce n'est pas le cas, peut-être qu'il en fait une variable statique ou globale...
À mon avis, c'est une histoire de scope dans ce style, vu que PHP n'est pas très strict là-dessus.

[Edit] Pour de la POO, préfère PHP5

oui j'y accede bien par $this->
la SEULE difféerence entre le script qui fonctionne et celui qui ne va pas étant le nom de la variable.

Kirkis essayes d'etre diplomate avec moi

**Mr N.** · 21/02/2006, 11h59

Envoyé par XtofRoland

mais pourquoi ne peut on pas donner un meme nom a des attributs de deux classes distinctes?

Il n'y a pas de raison.
Sans code on peut pas t'aider...

**Yogui** · 21/02/2006, 12h05

Lol je vais y aller franco : la POO en PHP n'est pas mon domaine donc il me faut du code, à moi aussi, pour tester. Sans code, pas de tests, pas de réponse.

**Mr N.** · 21/02/2006, 12h06

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
<pre><?php
class A {
    var $errormsg;
    var $var_a;
    function A() {
        $this->errormsg = 'Je suis un A !';
        $this->var_a    = 'AA';
    }
}
class B {
    var $errormsg;
    var $var_b;
    var $mes_a;
    function B() {
        $this->errormsg = 'Je suis un B !';
        $this->var_b    = 'BB';
        $this->mes_a    = array();
    }
    function add(&$a) {
        $this->mes_a[] =& $a;
    }
}
 
$a =& new A();
$b =& new B();
$b->add($a);
 
var_dump($b, $b->var_b, $a->var_a);
 
?>

Ce qui donne :

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
 
object(b)(3) {
  ["errormsg"]=>
  string(14) "Je suis un B !"
  ["var_b"]=>
  string(2) "BB"
  ["mes_a"]=>
  array(1) {
    [0]=>
    &object(a)(2) {
      ["errormsg"]=>
      string(14) "Je suis un A !"
      ["var_a"]=>
      string(2) "AA"
    }
  }
}
string(2) "BB"
string(2) "AA"

Je ne vois pas le problème.

**XtofRoland** · 21/02/2006, 14h11

voici le code donc:

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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
<?
//array_keys
define("CONSULT", 0);
define("MODIFY", 1);
define("ADD", 2);
 
class row{
	/*
		
	*/
	var $label      = ""; //
	var $fieldName  = ""; //nom du champs en db
	var $fieldLen   = ""; 
	var $fieldType  = "";
	var $fieldValue ;//sera envoyé a la database
	var $visibleValue="";//sera affiché
	var $fieldErrMsg= "";
	var $visible    = true;/*Un champ peut etre invisible
								- pour permettre des tests
								- car rempli automatiquement et pour le design */
	var $protected  = false;
	var $key        = false;
	var $list       = array();
	var $auto       = "";
	var $emptyisNULL = true;
 
	function row($fieldName, $fieldLen, $fieldType  ){
		$this->fieldName = $fieldName;
		$this->label     = $fieldName;
		$this->fieldLen  = $fieldLen;
		$this->fieldType = $fieldType;	
	}
	function toDB(  ){
		$rtn = $this->fieldValue;
		switch($this->fieldType){
			case  'text' :
			case  'varchar' :
				$rtn = addslashes($rtn);
				$rtn = "'".$rtn."'";
				break; 
			case  'int' :
			case  'datetime' :	
				break;
		}
		return $rtn;
	}
	function show( $mode ){
		$rtn = "";
		 if ($mode && !$this->protected ){
			$class = ($this->fieldErrMsg)? "class=incorrecte":"";
			switch($this->fieldType){
				case  'text':
					$rtn = "<TEXTAREA ".$class." name='".$this->fieldName."' rows=4 cols=40>".nl2br($this->fieldValue)."</TEXTAREA>";
					break;   
				default:
					$rtn = "<input type=text ".$class." name='".$this->fieldName."' value='".$this->fieldValue."'>";
					break;
			}
		}else{//consult
			/*if ($this->fieldName == $this->primaryKey){
				 $form .= "<input name=".$this->rows[$i]->fieldName." type=hidden value=".$this->rows[$i]->fieldValue.">"; 
				if ( ! $this->fieldValue ) $form .= " automatique "; 
			}*/
 
			$rtn = ($this->fieldValue == '' && $this->auto )?'automatique':$this->fieldValue;
		}
		return $rtn;
	}//end show
	function test( ){
		$rtn = true;
		switch($this->fieldType){
			case  'text':
			case  'varchar':
				$rtn = true;
				break;  
			case 'int':
				$rtn = is_numeric($this->fieldValue);
				if ($rtn) $this->fieldErrMsg = "Ceci doit être un nombre";
				break;
			case 'datetime' :
				$this->fieldErrMsg = "ceci est une date";
				$rtn = false;
				break;
		} 
		return $rtn;		
	}
}//end class
 
class formabs{
	var $mode       = CONSULT;
	var $table      = "";
	var $SQL        = "";
	var $rows       = array();
	var $numfields  = 0 ; //nombre de rows
	var $ErrMsg 	= "";	
	var $autocommit     = false;
 
	function formabs( ){
		if (count($_POST)>0){
			$this->autocommit = !$this->mode;
			foreach ($_POST as $key => $val) if(isset($this->rows[$key])) $this->rows[$key]->fieldValue = $val;	
		}  
	}//end _construct
	function nextSequence($col, $tab){}//end abstract fct	
	function mode($mode ){
		$this->mode = $mode;
	}//end mode
	function protect($field ){
		$this->rows[$field]->protected = true;
	}//end protect
	function setAuto($field, $fct ){
		$this->protect($field);
		$this->rows[$field]->auto = $fct;
	}//end setAuto
	function primary($field ){
		$this->rows[$field]->key = true;
		$this->setAuto($field, 'nextSequence');
	}//end primary	
	function traduction($dico ){
		foreach ($dico as $key => $val) if (isset($this->rows[$key]) && $val != "")$this->rows[$key]->label = $val;
	}//end traduction
	function getValue($field){
		return (isset($this->rows[$field]))? $this->rows[$field]->fieldValue: O;
	}//end getValue
	function show(){
		if ($this->autocommit) $this->commit();
		$form = "<form method=POST action=".$_SERVER['PHP_SELF']."?mode=".$this->mode."> ";
		$form .= "<table width=500 cellSpacing=0 cellPadding=0><TBODY>";
		$i = 0;
		foreach ($this->rows as $key => $row) {
   			if ($row->visible){
	   			//libellé
	   			$i++;
	   			$form .= "<tr class=data".(($i % 2) + 1) ."><td>".$row->label."</td>";
	   			//icone
			    $form .= "<td width=16>";
			    if ( $row->fieldErrMsg != "" ) $form .= "<img src='"._HOMEPAGE."/Img/b_drop.png' alt='".$row->fieldErrMsg."'>";
			    $form .= "</td>";
			    //champ
		    	$form .= "<td>";
		    	$form .= $row->show($this->mode) ;
		    	$form .= "</td><td>&nbsp;</td></tr>";
	   		}
		}		
		$form .= "<TBODY></table>";
 
		if ( $this->mode == MODIFY || $this->mode == ADD ){
			 $form .= " <table width=100%><tr><td>";
			 if ( $this->ErrMsg ) $form .= "<marquee>".$this->ErrMsg."</marquee>" ;
			 $form .= "</td><td width=86 height=29><input type=image  src='"._HOMEPAGE."OO/form_bval.gif'></td></tr></table>";
		}
		$form .= "</form>";
		return $form;
	}// end show
	function test( ){
		$test = true;
		foreach ($this->rows as $key => $row) {
			$test = $test & $this->rows[$key]->test();
		}
		if (!$test) $this->ErrMsg = "<font color=red><b>Il  a des champs incorrects</b></font>";
		return $test;
	}//end test
	function fillAutoField(){
		foreach ($this->rows as $key => $row) {
			if ($row->auto){
				switch($row->auto){
					case 'nextSequence':
						$temp = '$temp = $this->nextSequence("'.$key.'");';
						eval($temp);
						$this->rows[$key]->fieldValue = $temp;
						break;
 
				}
			}
		}
	}//end fillAutoField
	function getSQLSyntax(){
		$rtn = "";
		$into = "";
		$values = "";
		$set = "";
		$where  = "";
		foreach ($this->rows  as $key => $row) {
			if ($row->key){
				$where .= ($where == "" )? "" :" and "; 
				$where .= $key."=".$row->toDB()." ";
			}else{
				$set = "";
			}
			$into .= ','.$key;
			$values .= ','.$row->toDB();
		}
		//retirer la premiere ','
		$into = substr($into,1);
		$values = substr($values,1);
 
		switch ($this->mode){
			case MODIFY:
				$rtn = "update ".$this->table." set ".$set." where ".$where.";";
				break;
			case ADD:
				$rtn = "insert into ".$this->table."(".$into.") values(".$values.");";
				break;
		}
		return $rtn;		
	}//end getSQLSyntax
	function sql_exec($SQL ){
		return false;	//abstract
	}
	function commit( ){
		$this->fillAutoField();	
		echo "commit";
		$this->test();
		$SQL = $this->getSQLSyntax();
 
		if ($this->sql_exec($SQL)){
			switch ($this->mode){
				case MODIFY:
					$this->ErrMsg = "<font color=green><b>Enregistrement modifié</b></font>";
					break;
				case ADD:
					$this->ErrMsg = "<font color=green><b>Enregistrement ajouté</b></font>";
					break;
			}
			$this->mode = CONSULT;
		}else{
			$this->ErrMsg = "<font color=red><b>Modification annulée</b></font>";
		}
	}//end commit
}//end class
?>

il plante a la ligne 165 :

Code :

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

if ($row->auto){

si on change le nom de la variable.

**Mr N.** · 21/02/2006, 14h22

Ca veut dire quoi 'il plante' ?

**XtofRoland** · 21/02/2006, 14h33

j'ai retravaillé tte la journée dessus, et j'ai une tonne de nouveau messages d'erreurs ;-)
mais c'etait un message genre.
errmessage not a valid argument ou not exist
ou not a member of class row

**Mr N.** · 21/02/2006, 14h42

Es tu sur que tu travaille sur ce que tu crois ?
var_dump te renseignes les bonnes valeurs ?

Code :

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

1
2
3
 
   var_dump('<pre>', $row, '</pre>');
   if ($row->auto){

[/code]

**XtofRoland** · 21/02/2006, 14h54

on ne peut jamais etre sur ;-)
mais j'y ai travaillé une 20 aine d'hr ce we et malgré mes lacunes en php j'ai 3 ans de dvlp en PB
j'ai du utiliser un utilitaire (compare-it!) pour trouver l'origine de l'erreur.

J'ai d'autres priorités pour le moment, mais je tenterai de remodifier le nom de ma variable par curiosité apres.

Amicalement.

[POO] bug? de php

Langage PHP

Discussions similaires

Partager

Partager