Discussion :

[Emmanuel Lecoester]

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 :

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

[MaitrePylos]

Salut,

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

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

[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
Tu peux arriver au même résultat avec ceci :

Code :

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

1
2
3
4
5
$tab = [
    ['`#__agenda`', 'agenda_id'],
    ['`#__activities`', 'activity_id'],
    ['`#__categories`', 'category_id']
];

[MaitrePylos]

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

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

[Emmanuel Lecoester]

lol les mecs j'ai pas dit noob non plus

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