Discussion :

[keokaz]

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 :

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

[k'amm]

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 :

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

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

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 :

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

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/

[rawsrc]

Salut,

ton code se résume à ceci :

Code :

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

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

[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 :

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

1
2
$annee = array("1","2"...)  ?
$promo  = "promo_{$annee}";

si je fait un print_r($promo), j'ai un string ou un tableau ?

[rawsrc]

$annee est un tableau ?
Poste le code entier stp.

[keokaz]

en fait ce que je n'ai pas bien compris c'est l'accolade, voici je test

Code :

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

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

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