Discussion :

[ozzmax]

Bonjours
je me suis créer une classe pour subvenir a mes besoins dans celle ci j'ai un array contenant environ une 20taine de champs
ex:

arrayDossier['nom'] = "joe";
arrayDossier['age'] = 22;
...

dans une autre page je veux faire afficher dans des texte box les valeurs de chaque champs
ainsi est ce que je suis mieux de me faire une méthode qui me retourne l'array au complet et l'utiliser pour l'affichage

ou

faire afficher une valeur à la fois en appelant toujours une méthode GET avec le nom du champs que je souhaite avoir??

le résultat est le meme mais point de vue ressource et/ou esthétique...lequel est le mieux?

merci beaucoup
ps voici ma méthode GET jusqu'a présent elle est adapter pour les 2 type de solutions et fonctionnel alors c'est pas un soucis, je vous demande une suggestion

Code :

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

1
2
3
4
5
6
7
8
9
 
function getValeur($attribut){
	if($attribut == "empty"){
		return $this->arrayDossier;
	}
	else{
		return $this->arrayDossier[$attribut];
	}
}

[ozzmax]

j'ai p-e pas été clair après relecture
voilà ce que je demande:
est ce mieux de faire:

Code :

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

1
2
3
4
5
6
7
8
9
 
$_SESSION['newDossier'] =& new cModifDossier();
 
//Récupération de l'array des attributs de la classe cNewDossier en passant la valeur "empty"
$dossierCourant = $_SESSION['newDossier']->getValeur("empty");
 
....
 
<input type="text" name="txtNom" value="<? echo $dossierCourant['clientAssureNom']; ?>" >

ou encore:

Code :

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

1
2
3
4
 
$_SESSION['newDossier'] =& new cModifDossier();
 
<input type="text" name="txtNom" value="<? echo $_SESSION['newDossier']->getValeur("clientAssureNom"); ?>" >

j'espere avoir été plus clair

[berceker united]

j'ai p-e pas été clair après relecture
voilà ce que je demande:
est ce mieux de faire:

Code :

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

1
2
3
4
5
6
7
8
9
 
$_SESSION['newDossier'] =& new cModifDossier();
 
//Récupération de l'array des attributs de la classe cNewDossier en passant la valeur "empty"
$dossierCourant = $_SESSION['newDossier']->getValeur("empty");
 
....
 
<input type="text" name="txtNom" value="<? echo $dossierCourant['clientAssureNom']; ?>" >

ou encore:

Code :

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

1
2
3
4
 
$_SESSION['newDossier'] =& new cModifDossier();
 
<input type="text" name="txtNom" value="<? echo $_SESSION['newDossier']->getValeur("clientAssureNom"); ?>" >

j'espere avoir été plus clair

Bof je dirais qu'il est préférable de passer par un autre entité plutot que de passer directement par la session. Ainsi s'il y a un problème dans la session tu peux le gérer avant.

[ozzmax]

hmm mais c'était pas ca que je voulais
oki merci pour la question de session
mais c'est pour garder ma class en session
moi c'était l'affichage que je voulais...
je retourne l'array complet ou une valeur à la fois donc un multiple appel de méthod getValeur?

[berceker united]

hmm mais c'était pas ca que je voulais
oki merci pour la question de session
mais c'est pour garder ma class en session
moi c'était l'affichage que je voulais...
je retourne l'array complet ou une valeur à la fois donc un multiple appel de méthod getValeur?

ha ok , je crois que ça ne change rien fait dans le plus propre, plus logique, le plus compréhensible.

[guitou12]

Sinon pour ta fonction je la ferai un poil plus simple

Code :

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

1
2
3
4
5
6
7
8
function getValeur($attribut=null){
	if(empty($attribut)){
		return $this->arrayDossier;
	}
	else{
		return $this->arrayDossier[$attribut];
	}
}

Ca t'évite d'appeler ta fonction avec 'empty' si tu veux récupérer tout le tableau un simple getValeur(); suffira

[ozzmax]

donc j'imagine qu'a ce moment je suis mieux de retourner toute les valeurs puisque je les fait toutes afficher
alors un seul appel retournant l'array complet
c'est plus propre et plus logic? disons que tu retomberais dans ce code ultérieurement?

[ozzmax]

Sinon pour ta fonction je la ferai un poil plus simple

Code :

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

1
2
3
4
5
6
7
8
function getValeur($attribut=null){
	if(empty($attribut)){
		return $this->arrayDossier;
	}
	else{
		return $this->arrayDossier[$attribut];
	}
}

Ca t'évite d'appeler ta fonction avec 'empty' si tu veux récupérer tout le tableau un simple getValeur(); suffira

Hmm merci
en effet ca fais clean

alors bonne journée a vous 2