Discussion :

[seb-65]

Bonjour,

Ma variable $this->Client est vide OU pas, suivant la fonction que j'utilise...

Voici ma classe :

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
 
class monTest{
 
var $FichierListe="";
var $Date="";
var $Client="";
 
function setClient($client) {
	$this->Client=$client;
}
 
function setDate($date) {
	$this->Date=$date;
}
 
function setFichier($filename){
	$this->FichierListe=$filename;
}
 
function setItem() {
	if ($this->FichierListe == "File_A") {
		$Separateur = ";";
		return $Separateur;
	} else {
		$Separateur = ":";
		return $Separateur;
	}
}
 
function addItem($categorie,$nombre){
 
$file=fopen($this->FichierListe,"a");
    $MonSeparateur=$this->setItem();
    if ($this->Date == "2012") {
        $newLine="ID = ".$this->Client."|".$categorie.$this->setItem.$nombre;
    } else {
        $newLine="ID Old = ".$this->Client."|".$categorie.$this->setItem.$nombre;
    }
 
	fputs($file,$newLine."\n");
    fclose($file); 
}
 
function copyItem() {
	$file=fopen("copybackup","w+");
	fputs($file,$this->Date."\n");
	fputs($file,$this->FichierListe."\n");
	fputs($file,$this->Client."\n");
	fputs($file,"------------"."\n");
}
 
}

Mon problème est que lorsque je veux utiliser la variable $this->Client de la fonction addItem celle-ci est vide

Le fichier texte contient : "ID = |Legume;4" au lieu de "ID = TEST|Legume;4"

Par contre si j'utilise la même variable $this->Client de la fonction [B]copyItem[B], elle me retourne bien le nom du Client...
Le fichier copybackup contient bien le nom dans $this->Client

Je comprend pas pourquoi

[Benjamin Delespierre]

Comme ça je saurais pas dire, il faudrait que je vois comment tu utilise tes objets.

[seb-65]

Olala j'ai honte

Ma déclaration setClient était après le addItem...

En replacant le setClient avant le addItem cela fonctionne bien.

Je faisait :

$res->setFichier();
$res->setDate();
$res->addItem();
$res->setClient();
$res->copyItem();

au lieu de :

$res->setFichier();
$res->setDate();
$res->setClient();
$res->addItem();
$res->copyItem();

Merci quand même

[Benjamin Delespierre]

Tip of the day: utilise un constructeur

Ah et j'oubliais, toutes tes méthodes setX ne servent de fait à rien vu que les propriétés qu'elles manipulent sont publiques (et qu'elles n'effectuent aucun contrôle).

Sur ce, salut !