Discussion :

[sff]

Bonjours, j'ai fais le code suivant, et j'aimerai savoir si il y avait possiblité de le simplifier, et de le rendre plus propre, si cela est possible :

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
	$clan_elementsquery = doquery("SELECT id, type, clan FROM {{table}} WHERE latitude='$latitude' AND longitude='$longitude' LIMIT 1", "clans_elements");
	if (mysql_num_rows($clan_elementsquery) > 0) {
		$clan_elementsrow = mysql_fetch_array($clan_elementsquery);
 
		if($clan_elementsrow['type'] != 3 AND $userrow['team'] == $clan_elementsrow['clan'] AND $userrow['team_ok'] == 1){
 
			$clanquery = doquery("SELECT admin FROM {{table}} WHERE id='$userrow[team]' LIMIT 1", "clans");
			$clanrow = mysql_fetch_array($clanquery);
 
			switch ($clan_elementsrow['type']) {
			case 1: $text_type = 'ce rempart'; break;
			case 2: $text_type = 'cette tour'; break;
			}
 
			$text_delete = ($clanrow['admin'] == $userrow['id']) ? '- Effacer '.$text_type : null;
 
			$_SESSION['infos_map'] = 'Vous êtes sur un élément de la cité de votre clan. Ici vous pouvez :<br><br>- <a href="#">Réparer '.$text_type.'</a><br>'.$text_delete;
 
		}else{
 
			include('clan.php');
			travel_castle($clan_elementsrow['id']);
			die();
		}
 
	}else{
 
		if($userrow['team'] != 0 AND $userrow['team_ok'] == 1){
 
			$clanquery = doquery("SELECT admin, nb_walls_wait, nb_towers_wait, longitude, latitude FROM {{table}} WHERE id='$userrow[team]' LIMIT 1", "clans");
			$clanrow = mysql_fetch_array($clanquery);
 
			if(($clanrow['nb_walls_wait'] > 0 OR $clanrow['nb_towers_wait'] > 0) AND ($userrow['id'] == $clanrow['admin'] AND abs($longitude - $teamrow['longitude']) <= 6 AND abs($latitude - $teamrow['latitude']) <= 6)){				
				$text_wall = ($clanrow['nb_walls_wait'] > 0)? '- <a href="#">Ajouter un rempart</a>' : null;
				$text_tower = ($clanrow['nb_towers_wait'] > 0)? '<br>- <a href="#">Ajouter une tour</a>' : null;
 
				$_SESSION['infos_map'] = 'Ici vous pouvez ajouter un élement à votre cité :<br><br>'.$text_wall.$text_tower;
 
			}else{
 
				$_SESSION['infos_map'] = 'Aucune infos';
			}
		}
	}

Merci à ceux qui m'aideront.

[Le_Che]

Bonjour,

Pour ma part, je ne vois pas comment tu pourrait optimiser mieux ton code qu'il ne l'est à l'heure actuelle. Tu as déjà simplifié certain if, après vu ce que tu fait dans ton code, si tu le fait plus propre (si c'est possible) il sera plus compliqué à lire...

Et puis c'est un petit bout de code qui est très bien comme il est, non? Pourquoi vouloir le simplifier plus?