Discussion :

[AliciaSH]

Bonjour;

J'ai un champs text dans mon interface qui récupère les données de la base de données selon le choix de l'utilisateur.
Je l'ai changé par un CheckBox à multiple choix. C'est à dire au lieu de taper manuellement un type deLRU maintenant l'utilisateur peut choisir plusieurs types de LRU en même temps.

L'ancienne fonction était manipulé comme ça:

Code :

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

1
2
3
4
public function setLru($lru) {
        $this->_lru = $lru;
        if ($lru == '') $this->_lru ='all';
    }

Comment je peux le changer selon la nouvelle fonctionnalité ? Est-ce-que je dois faire un appel à ma requête ici pour y aller chercher l'article concerné ?

[sabotage]

C'est la logique globale de classe qu'il faudrait connaître, pas cette petite de code.
Est-ce que la classe va gérer la liste de éléments ou est-ce que chaque élément va être une instance de classe.

[AliciaSH]

C'est la logique globale de classe qu'il faudrait connaître, pas cette petite de code.
Est-ce que la classe va gérer la liste de éléments ou est-ce que chaque élément va être une instance de classe.

La classe va gérer la liste des éléments.

En fait, avant l'utilisateur tape un seul choix de LRU et la requête va chercher les bonnes données de la base.
Celle là ma requête:

Code :

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

1
2
$this->_sqlWhere.="`piecearticles`.`ID_Article`=`article`.`ID_Article` 
 AND `article`.`ID_LRU`=`lru`.`ID_LRU` AND `lru`.`LRU`='" . $this->_lru . "'";

Je l'ai changé CheckBox à multiple choix et la liste contient 45 élément, Est-ce-que je dois modifier la requete aussi puisque j'ai changé CheckBox à multiple choix ?
J'ai changé la fonction setter, j'ai mis les 45 élément dans un tableau comme suit:

Code :

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

1
2
3
4
5
6
7
8
public function setLru($lru) {
    $allowed_lru = ['A','B',...........,];
    if(in_array($lru, $allowed_lru)){
        $this->_lru = $lru;
    }else{
        $this->_lru = 'all';
    }
}

[Nazoïde]

1. Tu déclares un tableau pour recevoir les résultats de ta requête;
2. Pour chaque élément du tableau de checkbox posté, tu exécutes ta requête que tu insères dans le tableau précédemment déclaré ($xxx[] = $yyy);
3. Tu Affiche le tableau (print_r(...)) pour voir sa structure (utile pour le parcours lors de l'affichage).

[AliciaSH]

1. Tu déclares un tableau pour recevoir les résultats de ta requête;
2. Pour chaque élément du tableau de checkbox posté, tu exécutes ta requête que tu insères dans le tableau précédemment déclaré ($xxx[] = $yyy);
3. Tu Affiche le tableau (print_r(...)) pour voir sa structure (utile pour le parcours lors de l'affichage).

Bonjour;

Ses étapes je dois les faire dans la fonction setter ou ça concerne m requête ?

Merci

[Nazoïde]

Les deux. Dans ton setter $lru passé en paramètre doit être un tableau.

[Nazoïde]

Tu pourra faire ceci au niveau du setter

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
$lrus[]; // ton tableau de données; à toi de savoir où le placer.
public function setLru($lrus) {
    $allowed_lru = ['A', 'B','C',];
    foreach ($lrus as $lru) {
        if (in_array($lru, $allowed_lru)) {
            $this->_lru[] = $lru;
        } else {
            $this->_lru[] = 'all';
        }
    }
}

[AliciaSH]

Bonjour

Je suis désolée mais je ne comprends pas le tableau des données $lrus[] je dois le placer où exactement, tu veut dire que le tableau des données final où on stocke le résultat final ?
En fait mon constructeur est le suivant, est-ce-que je dois le placer ici mon tableau $lrus ? parce que dans le checkbox l'utilisateur peut choisir un seul choix aussi.

Code :

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

1
2
 public function __construct($lru) {
$this->setLru($lru);

Ma fonction setter est comme tu m'a proposé:

Code :

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

1
2
3
4
5
6
7
8
9
10
public function setLru($lrus) {
    $allowed_lru = ['A', 'B','C',];
    foreach ($lrus as $lru) {
        if (in_array($lru, $allowed_lru)) {
            $this->_lru[] = $lru;
        } else {
            $this->_lru[] = 'all';
        }
    }
}

Concernant la requête, j'ai ajouté une nouvelle et j'ai utilisé implode, et j'ai laissé l'ancienne requête d'une seule recherche parce que dans le checkbox l'utilisateur peut choisir un seul choix, c'est pour ça j'ai ajouté une nouvelle requête est la suivante:

Code :

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

1
2
3
$this->_sqlWhere.="`piecearticles`.`ID_Article`=`article`.`ID_Article`
AND `article`.`ID_LRU`=`lru`.`ID_LRU`
AND `lru`.`LRU` IN (" . implode(",", $this->_lru) . ")";

Qu'est ce que je dois supprimer ou ajouter à ton avis, parce que j'ai essayé par ta solution de la fonction setter il m'a affiché un tableau vide sachant que je n'ai pas ajouté $lrus parce que je ne suis pas certaine où je dois le placer exactement.

Je te remercie beaucoup pour ton aide.

[Nazoïde]

Tu dois passer le tableau à ton constructeur:

Code php :

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

1
2
3
4
 
public function __construct($lrus[]){
   $this->setLru($lrus);
}

Pour le nombre d'élément du tableau pas de souci: si c'est 01, il la variable lru sera un tableau contenant 01 élément, s'il y'a n choix, elle sera un tableau de n éléments.

Au niveau de ton code de requête, implode dans la clause IN affiche les élément du rableau sans les quotes alors qu'il s'agit des chaines de caractère. Je les ai ajoutés:

Code php :

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

1
2
3
4
 
$this->_sqlWhere.="`piecearticles`.`ID_Article`=`article`.`ID_Article`
AND `article`.`ID_LRU`=`lru`.`ID_LRU`
AND `lru`.`LRU` IN ('" . implode("','", $this->_lru) . ")";

Ce code devrait donnez indépendamment du nombre d'éléments sélectionnés.