Меня интересует безопасность и эффективность этого кода.
В начале страницы прописываю:

<? include './units/database.php';
if (isset($_GET['id'])) {$id=intval($_GET['id']);}
else {
	echo '<script type="text/javascript">
	window.location = "news.php"
	</script>';
}

$res = mysql_query("SELECT `title`, `m_description`, `m_keywords`, `text` FROM `ru_pages` WHERE `page`='news'",$db_connect);
if (!$res)
{
	exit (mysql_error());
}
if (mysql_num_rows($res)>0)
{
	$table = mysql_fetch_assoc($res);
}
else 
{
	echo "<p>No rows!</p>";
	exit ();
}


?>

то есть код в начале принимает переменную id новости и дерет информацию о странице, а дальше:

<?
$res2 = mysql_query("SELECT `date`,`text` FROM `news` WHERE `id`='$id'",$db_connect);							
if (!$res2)
{
      exit (mysql_error());
}
if (mysql_num_rows($res2)>0)
{
	while ($table2 = mysql_fetch_assoc($res2))
	{
	      echo "<p>".htmlspecialchars($table2['date'])."</p>".htmlspecialchars($table2['text']);	
	}
}
else
{
	echo "<p>Данной новости не существует. Попробуйте обратиться снова.</p>";
}
?>

Этот код выводит новость на страницу. Я сделал intval вначале, но не уверен что все условия расставлены правильно. Помогите пожалуйста