réduire mon bout de script ?

Version imprimable

bonjour, j'ai une méthode qui sert à inscrire chaque étudiant dans une groupe, on peut
imaginer que ces groupe s'appelle des cohortes.
j'aimerais savoir si je peux réduire facilement ma méthode la voici

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
39
while ($donnee = $query->fetch()) {
    //echo $donnee['lastname']."<br/>"; 
    //$this->promo[$increment][] = $donnee['id'];
    /*
     * ON ENREGISTRE CHAQUE ETUDIANT DANS LEURS COHORTES
    */
    switch ($annee) {
        case 1:
            $enrol_cohort = "INSERT INTO cohort_members (cohortid,userid,timeadded) 
            VALUES(" . $this->cohorte_1a . "," . $donnee['id'] . "," . $this->timestamp_now . ")";
            $this->promo_1[] = $donnee['id'];
            $this->bdd->exec($enrol_cohort);
            break;
        case 2:
            $enrol_cohort = "INSERT INTO cohort_members (cohortid,userid,timeadded) 
            VALUES(" . $this->cohorte_2a . "," . $donnee['id'] . "," . $this->timestamp_now . ")";
            $this->promo_2[] = $donnee['id'];
            $this->bdd->exec($enrol_cohort);
            break;
        case 3:
            $enrol_cohort = "INSERT INTO cohort_members (cohortid,userid,timeadded) 
            VALUES(" . $this->cohorte_3a . "," . $donnee['id'] . "," . $this->timestamp_now . ")";
            $this->promo_3[] = $donnee['id'];
            $this->bdd->exec($enrol_cohort);
            break;
        case 4:
            $enrol_cohort = "INSERT INTO cohort_members (cohortid,userid,timeadded) 
            VALUES(" . $this->cohorte_4a . "," . $donnee['id'] . "," . $this->timestamp_now . ")";
            $this->promo_4[] = $donnee['id'];
            $this->bdd->exec($enrol_cohort);
            break;
        case 5:
            $enrol_cohort = "INSERT INTO cohort_members (cohortid,userid,timeadded) 
            VALUES(" . $this->cohorte_5a . "," . $donnee['id'] . "," . $this->timestamp_now . ")";
            $this->promo_5[] = $donnee['id'];
            $this->bdd->exec($enrol_cohort);
            break;
    }
}

l'attribut cohorte_1a est un nombre, entre cohorte_1a et cohorte_2a, il n'y a pas de
suite logique quelconque.

je vous remercie de votre réponse si vous avez une idée.

Plusieurs solutions, selon ton besoin : la plus longue et la plus proche de ton code actuel consiste à simplement centraliser ta requête à la fin de ta boucle.

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
while ($donnee = $query->fetch()) {
    /*
     * ON ENREGISTRE CHAQUE ETUDIANT DANS LEURS COHORTES
     */
    switch ($annee) {
        case 1:
            $cohorte = $this->cohorte_1a;
            $this->promo_1[] = $donnee['id'];
            break;
        case 2:
            $cohorte = $this->cohorte_2a;
            $this->promo_2[] = $donnee['id'];
            break;
        case 3:
            $cohorte = $this->cohorte_3a;
            $this->promo_3[] = $donnee['id'];
            break;
        case 4:
            $cohorte = $this->cohorte_4a;
            $this->promo_4[] = $donnee['id'];
            break;
        case 5:
            $cohorte = $this->cohorte_5a;
            $this->promo_5[] = $donnee['id'];
            break;
    }
    $enrol_cohort = "INSERT INTO cohort_members (cohortid,userid,timeadded) 
    VALUES(" . $cohorte . "," . $donnee['id'] . "," . $this->timestamp_now . ")";
    $this->bdd->exec($enrol_cohort);
}

une autre solution : supprimer ton switch en utilisant des variables dynamiques (ce qui est loin d'être ma solution préférée).

Code:

1
2
3
4
5
6
7
8
9
while ($donnee = $query->fetch()) {
    /*
     * ON ENREGISTRE CHAQUE ETUDIANT DANS LEURS COHORTES
     */
    $enrol_cohort = "INSERT INTO cohort_members (cohortid,userid,timeadded) 
    VALUES(" . $this->{'cohorte_'.$annee.'a'} . "," . $donnee['id'] . "," . $this->timestamp_now . ")";
    $this->{'promo_'.$annee}[] = $donnee['id'];
    $this->bdd->exec($enrol_cohort);
}

Une solution plus propre est d'utiliser un tableau : ça nécessite un peu plus de travail de ton côté puisqu'il te faudra adapter le code autour, mais ce sera bien plus propre, lisible et maintenable à terme !

Code:

1
2
3
4
5
6
7
8
9
while ($donnee = $query->fetch()) {
    /*
     * ON ENREGISTRE CHAQUE ETUDIANT DANS LEURS COHORTES
     */
    $enrol_cohort = "INSERT INTO cohort_members (cohortid,userid,timeadded) 
    VALUES(" . $this->cohorte[$annee] . "," . $donnee['id'] . "," . $this->timestamp_now . ")";
    $this->promo[$annee][] = $donnee['id'];
    $this->bdd->exec($enrol_cohort);
}

Dernier point : tu effectues une requête à chaque tour de boucle...tu gagnerais beaucoup à utiliser pour ça une requête préparée, où à grouper tes valeurs à insérer et effectuer ton insertion en dehors de ta boucle !
Tu as quelques exemples et pistes ici : http://www.developpez.net/forums/d13...ertion-double/

Salut,

ton code se résume à ceci :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
$func   = "cohorte_{$annee}a";
$promo  = "promo_{$annee}";
$values = array();
 
while ($v = $query->fetch())
{
    $values[]         = "({$this->$func}, {$v['id']}, {$this->timestamp_now})";
    $this->{$promo}[] = $v['id'];
}
 
$sql = 'INSERT INTO cohort_members (cohortid, userid, timeadded) VALUES '.implode(',', $values);
$this->bdd->exec($sql);

25/02/2013, 15h37
keokaz
merci de ces réponse :)

je pense qu'il faut que j'adopte la 1er solution qui ressemble à la mienne, car il faut
faire un script compréhensible par l'admin réseaux lorsque je ne serais pas là.

Par contre j'ai du mal à comprendre sur la soluce ci-dessous.
Code:

1 2 $annee = array("1","2"...) ? $promo = "promo_{$annee}";
si je fait un print_r($promo), j'ai un string ou un tableau ?
25/02/2013, 15h43
rawsrc

$annee est un tableau ?
Poste le code entier stp.
25/02/2013, 16h12
keokaz
en fait ce que je n'ai pas bien compris c'est l'accolade, voici je test
Code:

1 2 3 4 5 6 7 $tb=array(1,50,5); $promo="promo_{$tb}"; foreach($promo as $P) { echo $P; }
Citation:

Warning: Invalid argument supplied for foreach() in /var/www/test.php on line 6 Call Stack: 0.0218 326956 1. {main}() /var/www/test.php:0

Voilà un petit code d'exemple pour t'aider à mieux comprendre :
Code:

1 2 3 4 5 6 $annee = '2013'; $test = "promo_{$annee}"; echo $test.'<br />'; // affiche "promo_2013" ${'la_'.$test} = 'Lorem ipsum'; // on concatène la chaîne de caractère "la_" et la valeur de $test, donc "promo_2013" pour nommer notre variable. echo $la_promo_2013; // affiche "Lorem ipsum" !
L'acolade permet d'indiquer à php quelle chaîne prendre pour sa variable.
Dans le premier test : $test = "promo_{$annee}";, on pourrait s'en passer, ainsi on aura exactement le même rendu avec ce test : $test = "promo_$annee";.

Dans le deuxième cas par contre, ${'la_'.$test} = 'Lorem ipsum';, on ne peut pas écrire $'la_'.$test, il nous faut donc utiliser les accolades pour bien délimiter nos variables :)

merci :)
je n'avais jamais utiliser cette syntaxe et pourtant j'avais tenter cela sans les accolade.
Code:

1 2 3 4 5 6 7 $promo = array('1_a','2_a','3_a','4_a'); foreach($promo as $P) { $nom_promo= "promo_{$P}"; echo $nom_promo."<br>"; }