Réécriture de code ?

Version imprimable

Bonjour à tous,

Vous trouverez ci dessous un bout de code qui me semble optimisable d'une point de vue ligne de code. En effet, on teste un paramètre qui ne fait "que" change la table et la colonne à utiliser pour la sélection... Sauf que mon niveau actuel en php ne me permet pas de voir comment commencer... vous auriez une piste ? Merci.
Manu

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
40
41
42
43
44
45
46
47
		if (empty($data['vignette']))
		{
			$lvignette = "" ;
			switch ($params['gdisplay_colors'])
			{
				case 0: // AGENDA
					if (isset($data['agenda_id'])) 
					{
						$query = $db->getQuery(true);
						$query->select('vignette')->from('`#__agenda`')->where('id = ' . (int)$data['agenda_id']);
						$db->setQuery($query);
						$results = $db->loadObject();
						if ($results)
						{
							$lvignette = $results->vignette;
						}
					}
					break;
				case 1: // ACTIVITY
					if (isset($data['activity_id'])) 
					{
						$query = $db->getQuery(true);
						$query->select('vignette')->from('`#__activities`')->where('id = ' . (int)$data['activity_id']);
						$db->setQuery($query);
						$results = $db->loadObject();
						if ($results)
						{
							$lvignette = $results->vignette;
						}
					}
					break;
				case 2: // CATEGORY
					if (isset($data['category_id'])) 
					{
						$query = $db->getQuery(true);
						$query->select('vignette')->from('`#__categories`')->where('id = ' . (int)$data['category_id']);
						$db->setQuery($query);
						$results = $db->loadObject();
						if ($results)
						{
							$lvignette = $results->vignette;
						}
					}
					break;
			}
			$data['vignette'] = $lvignette ;
		}

Salut,

Teste ceci pour voir si tu as de meilleurs résultats :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
if (empty($data['vignette'])) {
 
    $lvignette = null;
    $tab = new SplFixedArray(3);
    $tab[0] = ['`#__agenda`', 'agenda_id'];
    $tab[1] = ['`#__activities`', 'activity_id'];
    $tab[2] = ['`#__categories`', 'category_id'];
 
    $query = $db->getQuery(true);
    $query->select('vignette')->from($tab[$params['gdisplay_colors']][0])->where('id = ' . (int)$data[$params['gdisplay_colors']][1]);
    $db->setQuery($query);
    $results = $db->loadObject();
    if ($results) {
        $lvignette = $results->vignette;
    }
    $data['vignette'] = $lvignette;
}

02/12/2015, 16h02
rawsrc
Salut,

le code de MaitrePylos est tiptop, le seul point un peu tordu c'est d'utiliser SplFixedArray quand l'apprenant dit clairement qu'il a un niveau faible :lol:
Tu peux arriver au même résultat avec ceci :
Code:

1 2 3 4 5 $tab = [ ['`#__agenda`', 'agenda_id'], ['`#__activities`', 'activity_id'], ['`#__categories`', 'category_id'] ];
02/12/2015, 16h17
MaitrePylos

Citation:

Envoyé par rawsrc

le code de MaitrePylos est tiptop, le seul point un peu tordu c'est d'utiliser SplFixedArray quand l'apprenant dit clairement qu'il a un niveau faible :lol:

Je m'excuse du peu, mais il a demandé 'optimalisable' et la classe SplFixeArray est la plus rapide ;)

Après s'il veut plus de commentaire, je suis son homme ;)
02/12/2015, 19h09
Emmanuel Lecoester

lol les mecs j'ai pas dit noob non plus :aie:

merci beaucoup pour la réponse.

Juste que je n'ai pas les réflexes d'optimisation de bout de code avec des tableaux&cie :)