Génération de classe CSS

Voila, je veux que mon menu soi actif en fonction d'une variable que je récupère dans l'URL :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
<div class="left_box_news">
<ul id="side_menu">
<?php 
$filter = $_GET['filter'];
if ($filter == 'all') {	$menu1='active';	$menu2='';	$menu3='';	}
if ($filter == '1') {	$menu1='';	$menu2='active';	$menu3='';	}
if ($filter == '2') {	$menu1='';	$menu2='';	$menu3='active';	}
?>
<li><a href="news.php?filter=all" class="<?php '.$menu1.' ?>">Toutes mes news</a></li>
<li><a href="news.php?filter=1" class="<?php '.$menu2.' ?>">News Professionnelles</a></li>
<li><a href="news.php?filter=2" class="<?php '.$menu3.' ?>">News Personnelles</a></li>
</ul>	
</div>

Ou est l'erreur ? merci beaucoup