Simplification de fonction avec variable dynamique

**moimp** · 25/05/2018, 20h00

Bonjour,

Dans le code qui suit:

La classe optListSelect crée une liste d'options, le paramètre est le nom et l'identifiant de la balise.
La classe optListOption ajoute une option à la liste avec comme premier paramètre le texte à afficher et comme deuxième paramètre la valeur de l'option.

Ici, les deux personnes sont identifiées et chaque liste ne contient qu'une option. Dans la suite du développement, il est prévu de fixer une personne connue (option unique) et d'ajouter d'autres personnes dans la deuxième liste.
Code original

Code php :

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
$lstMen = new optListSelect('id_m');
$lstWomen = new optListSelect('id_f');
 
if (!empty($mainPerson) and $mainPerson->gender === 'M')
{
	$lstMen->addOption(new optListOption("$mainPerson->id, $mainPerson->full_name ($mainPerson->live_dates)", $mainPerson->id));
}
if (!empty($mainPerson) and $mainPerson->gender === 'F')
{
	$lstWomen->addOption(new optListOption("$mainPerson->id, $mainPerson->full_name ($mainPerson->live_dates)", $mainPerson->id));
}
 
if (!empty($mainPerson) and $mainPerson->gender === 'F')
{
	$lstWomen->addOption(new optListOption("$mainPerson->id, $mainPerson->full_name ($mainPerson->live_dates)", $mainPerson->id));
}
if (!empty($mainPerson) and $mainPerson->gender === 'M')
{
	$lstMen->addOption(new optListOption("$mainPerson->id, $mainPerson->full_name ($mainPerson->live_dates)", $mainPerson->id));
}
 
if (!empty($joinedPerson) and $joinedPerson->gender === 'M')
{
	$lstMen->addOption(new optListOption("$joinedPerson->id, $joinedPerson->full_name ($joinedPerson->live_dates)", $joinedPerson->id));
}
if (!empty($joinedPerson) and $joinedPerson->gender === 'F')
{
	$lstWomen->addOption(new optListOption("$joinedPerson->id, $joinedPerson->full_name ($joinedPerson->live_dates)", $joinedPerson->id));
}
 
if (!empty($joinedPerson) and $joinedPerson->gender === 'F')
{
	$lstWomen->addOption(new optListOption("$joinedPerson->id, $joinedPerson->full_name ($joinedPerson->live_dates)", $joinedPerson->id));
}
if (!empty($joinedPerson) and $joinedPerson->gender === 'M')
{
	$lstMen->addOption(new optListOption("$joinedPerson->id, $joinedPerson->full_name ($joinedPerson->live_dates)", $joinedPerson->id));
}

Code simplifié

Code php :

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
function makeList($person)
{
	if (empty($person)) return false;
 
	$lstMen		= new optListSelect('id_m');
	$lstWomen	= new optListSelect('id_f');
	$arrLists	= ['lstMen','lstWomen'];
	$arrGenders	= ['M', 'F'];
	$gender		= $person->gender;
 
	foreach ($arrLists as $key=>$list)
	{
		${$list}->addOption(new optListOption("$person->id, $person->full_name ($person->live_dates)", $person->id));
	}
	return ${$list};
}
 
var_dump(makeList($mainPerson));
var_dump(makeList($joinedPerson));

Dans mon code simplifié, le nom de la liste est toujours 'id_f' et je n'arrive pas à tenir compte du genre de la personne, c'est à dire à prendre en compte les variables $arrGender et $gender. Je suppose que ce n'est pas la seule erreur.

**Dendrite** · 25/05/2018, 20h25

Comprends rien.
Tu peux nous montrer ta classe optListOption ? et ta méthode addOption de la classe Person ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
function makeList(){
	$lstMen		= new optListSelect('id_m');
	$lstWomen	= new optListSelect('id_f');
	$arrLists	= array_merge($lstMen,$lstWomen);
        $lists=array();
	foreach ($arrLists as $person){
		$lists[]=$person->addOption(new optListOption($person->id, $person->full_name,$person->live_dates, $person->id));
	}
	return $list;
}
var_dump(makeList());

**bonjourajax** · 25/05/2018, 21h15

Bonsoir,

Code :

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

1
2
3
4
5
 
foreach ($arrLists as $key=>$list)
	{
		${$list}->addOption(new optListOption("$person->id, $person->full_name ($person->live_dates)", $person->id));
	}

Jpense pas que ca va faire quelque chose

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
 
function makeList($person) { 
	$lstMen = new optListSelect('id_m');
	$lstWomen	= new optListSelect('id_f');
	$arrLists = [$lstMen,$lstWomen];
	$arrGenders = ['M', 'F'];
	$gender = $person->gender;
 
        $person->addOption(new optListOption("$person->id, $person->full_name ($person->live_dates)", $person->id, $arrLists, $arrGenders, $gender));
 
}
 
foreach($person as $personne) {
makeList($personne);
}

j'ai pas tout compris mais en tout cas tu peux faire quelque chose comme ca

**moimp** · 25/05/2018, 21h56

@Dendrite: Je ne suis pas jaloux de mon code de classe mais je pense que ça ne servira qu'à alourdir la discussion:

La classe optListSelect crée une balise <select id='nom' name='nom'></select>
La classe optListOption crée une balise <option value='value'>Texte</option>
La méthode addOption ajoute une option à la liste.
La variable personne n'est pas une classe mais un objet issu d'une base de données.

Mon sujet est de simplifier le code original en remplaçant toutes les conditions sur les cas particuliers grâce à l'utilisation des variables dynamiques pour $mainPerson->gender et $mainPerson/$joinedPerson.
Je pense que ma fonction est foireuse et qu'il vaut mieux se concentrer su l'idée de simplifier le code original.
Pour plus d'infos, le but de ce code lorsqu'il sera complet sera de créer une liste de conjoints et une liste de conjointes possibles pour une application de généalogie. Au départ, je connais l'une des personnes et je fixe sa valeur et je veux sélectionner l'autre personne parmi les conjoints possibles en les regroupant en plusieurs groupes (optgroup) ('Enfants en commun', 'Faible écart d'âge', 'Gros écart d'âge', date de naissance inconnue')

**bonjourajax** · 25/05/2018, 22h23

Ah cest deja plus clair

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
 
$lstMen = new optListSelect('id_m');
$lstWomen = new optListSelect('id_f');
 
if (!empty($mainPerson) && $mainPerson->gender === 'M')
{
	$lstMen->addOption(new optListOption("$mainPerson->id, $mainPerson->full_name ($mainPerson->live_dates)", $mainPerson->id));
	$lstMen->addOption(new optListOption("$mainPerson->id, $mainPerson->full_name ($mainPerson->live_dates)", $mainPerson->id));
}
else if (!empty($mainPerson) && $mainPerson->gender === 'F')
{
	$lstWomen->addOption(new optListOption("$mainPerson->id, $mainPerson->full_name ($mainPerson->live_dates)", $mainPerson->id));
	$lstWomen->addOption(new optListOption("$mainPerson->id, $mainPerson->full_name ($mainPerson->live_dates)", $mainPerson->id));
} 
 
if (!empty($joinedPerson) && $joinedPerson->gender === 'M')
{
	$lstMen->addOption(new optListOption("$joinedPerson->id, $joinedPerson->full_name ($joinedPerson->live_dates)", $joinedPerson->id));
	$lstMen->addOption(new optListOption("$joinedPerson->id, $joinedPerson->full_name ($joinedPerson->live_dates)", $joinedPerson->id));
}
else if (!empty($joinedPerson) && $joinedPerson->gender === 'F')
{
	$lstWomen->addOption(new optListOption("$joinedPerson->id, $joinedPerson->full_name ($joinedPerson->live_dates)", $joinedPerson->id));
	$lstWomen->addOption(new optListOption("$joinedPerson->id, $joinedPerson->full_name ($joinedPerson->live_dates)", $joinedPerson->id));
}

Mais j'ai pas compris pourquoi il faut ajouter 2 fois addoption

et y a combien d'objet person?
et comment tu le sors de la base de données un peu plus de code et on fait la function

Invité · 25/05/2018, 23h23

C'est clair comme du jus de pieds...
Et dans ton "code original", toutes les conditions sont en double...

À tout hasard :

Code php :

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
function makeList( $persons )
{
   $lst = [
           'M' => 'lstMen',
           'F' => 'lstWomen'
   ];
 
   $lstMen = new optListSelect('id_m');
   $lstWomen = new optListSelect('id_f');
 
   foreach( $persons as $person )
   {
      if ( !empty($person) ) 
      {
	${$lst[$person->gender]}->addOption(new optListOption("$person->id, $person->full_name ($person->live_dates)", $person->id));
      }
   }
	return [$lstMen, $lstWomen];
}

Code php :

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

1
2
$persons = [$mainPerson, $joinedPerson];
list($lstMen, $lstWomen) = makeList( $persons );

**moimp** · 26/05/2018, 09h02

@bonjourajax: C'est une erreur: une fois suffit.
@jreaux62: Il semble que ce soit la bonne solution, mais je n'ai pas le temps de d'approfondir ce matin.

Invité · 26/05/2018, 13h51

On peut rendre les identifiants paramétrables :

Code php :

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
function makeList( $persons, $id_M, $id_F )
{
   $genders = ['M','F']; // au cas où tu voudrais ajouter un "3ème sexe";)
 
   foreach( $genders as $gender )
   {
      $lst[$gender] = new optListSelect( ${'id_'.$gender} );
   }
   foreach( $persons as $person )
   {
      if ( !empty($person) && in_array($person->gender, $genders) ) 
      {
	$lst[$person->gender]->addOption(new optListOption("$person->id, $person->full_name ($person->live_dates)", $person->id));
      }
   }
	return $lst; // array
}

Code php :

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

1
2
$persons = [$mainPerson, $joinedPerson]; // array : on peut en ajouter autant qu'on veut
list($lstMen, $lstWomen) = makeList( $persons, 'id_m', 'id_f' );

**moimp** · 26/05/2018, 19h39

Merci pour ce super boulot. Ton code est bon à un détail près: list n'accepte pas les tableaux associatifs. J'ai donc du traiter le résultat de la fonction comme ceci:

Code php :

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

1
2
3
$lst = makeList( $persons, 'id_m', 'id_f' );
$lstMen		= $lst['M'];
$lstWomen	= $lst['F'];

Simplification de fonction avec variable dynamique

Langage PHP

Discussions similaires

Partager

Partager