Simplification de fonction avec variable dynamique

Version imprimable

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:

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:

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.

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

Code:

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());

Bonsoir,

Code:

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:

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

25/05/2018, 21h56
moimp
@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')

Ah cest deja plus clair

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
 
$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

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

À tout hasard :

Code:

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:

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

26/05/2018, 09h02
moimp

@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.

On peut rendre les identifiants paramétrables :

Code:

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:

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

26/05/2018, 19h39
moimp
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:

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