Discussion :

[Emmanuel Lecoester]

Bonjour à tous,

Je souhaite gérer l'affichage d'évènements par slot d'affichage (ligne) car pour une même journée, je peux avoir plusieurs évènements. J'ai donc pensé définir une matrice avec les slots en axe 1 et les jours en axe 2.

la grande question avant d'aller plus en avant sur le code que j'ai pensé est : peut-on optimiser ces codes php (je suis persuadé que oui mais je ne sais pas trop comment faire) ?

Merci d'avance !

--

N'y connaissant pas grand chose en PHP je suis parti sur ce code pour l'initialisation :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
function InitSlots() {
   global $arrSlots ; 
 
   for ($slot=0; $slot<10; $slot++) 
   {
      for ($jour=0; $jour<7; $jour++) 
      {
         $arrSlots[$slot][$jour] = array('id' => '0', 'contenu' => ' ') ;          
      }	     	
   }
   return true ; 
}

Bien sur, un évènement ne doit être présent qu'une seule fois dans ces slots, donc j'ai fait une fonction pour vérifié qu'il n'est pas déjà présent en faisant une recherche sur son id :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
function EvenementDejaTraite($id) {
   global $arrSlots ; 
 
   for ($slot=0; $slot<10; $slot++) 
   {
      for ($jour=0; $jour<7; $jour++) 
      {
         if ($arrSlots[$slot][$jour]['id'] == $id )
            return true ;
      }	     	
   }
   return false ; 
}

Une fonction pour vérifié que je les cases sont vides pour un évènement donné :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
function Estdisponible ($slot, $jour, $duree)
{
   global $arrSlots ; 
 
   // vérification de la dispo sur la ligne
   for ($delta=0 ; $delta < $duree ; $delta++) 
   {
      if ($arrSlots[$slot][$jour+$delta]['id'] <> '0')
         return false ;
   }
   return true ;     
}

et son pendant pour occuper la place

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
 
function Occuper ($slot, $jour, $duree, $contenu, $id)
{
   global $arrSlots ; 
   $premier = true ; 
 
   // vérification de la dispo sur la ligne
   for ($delta=0 ; $delta < $duree ; $delta++) 
   {
      if ($premier )
      {
         $arrSlots[$slot][$jour+$delta]['contenu']= $contenu ; 
         $arrSlots[$slot][$jour+$delta]['id']= $id ; 
         $premier = false ; 
      }
      else
      {
         $arrSlots[$slot][$jour+$delta]['contenu'] = 'X' ; 
         $arrSlots[$slot][$jour+$delta]['id']= $id ; 
      }
   }
   return $dureeligne ; 
}

[rawsrc]

Salut Emmanuel,

Un gros conseil, tu dois fuir les variables globales comme la peste.
En PHP les variables sont passées par copie intelligente.
Tant que ta fonction ne fait que les lire, elles restent sous forme de référence donc ne consomment quasiment rien.

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
40
41
42
43
44
45
46
47
<?php
 
function InitSlots()
{
    return array_fill(0, 10, array_fill(0, 7, array('id' => '0', 'contenu' => '')));
}
 
function estDisponible(array $data, $slot, $jour, $duree)
{
    for ($i = 0; $i < $duree; ++$i, ++$jour)
    {
        if ($data[$slot][$jour]['id'] <> '0')
        {
            return false;
        }
    }
 
    return true;
}
 
function evenementDejaTraite(array $data, $id)
{
    foreach ($data as $evt)
    {
        foreach ($evt as $details)
        {
            if ($details['id'] == $id)
            {
                return true;
            }
        }
    }
 
    return false;
}
 
// on passe $data par référence car la fonction va modifier son contenu
function Occuper(array &$data, $slot, $jour, $duree, $contenu, $id)
{
    for ($i = 0; $i < $duree; ++$i, ++$jour)
    {
        $data[$slot][$jour]['id']      = $id;
        $data[$slot][$jour]['contenu'] = ($i === 0) ? $contenu : 'X';
    }
}
 
?>

J'ai essayé de faire simple.
Par contre dans ton code, tu ne vérifies rien ; comme par exemple t'assurer que le $slot existe ou que la durée ne dépasse pas le maximum.

[Emmanuel Lecoester]

Un très grand merci à toi pour cette réécriture bien php

Cà fonctionne nickel et je ne constate pas de régression.

Pour ton conseil avisé, j'attendais d'avoir une version stable et bien écrite pour m'assurer que tout est bien vérifié (surtout la limite des slots...)