News Problem

[Forumsgast]

Hallo Liebe Leute.

Ich habe folgendes Problem, ich gibe mit einer While Schleife ein Informationen aus der Datenbank aus, es wird ein Newsystem. Das Ausgeben klappt alles, nur wenn ich einen Newsartikels anklicke, gibt er alles miteinander aus, obwohl die ID gesetzt ist, und eine Abfrage gemacht ist?
Ich hoffe ihr könnt mir helfen,

Liebe Grüsse Stefan.

<?php 
if(isset($_GET["id"]))
$rowquery = mysql_query("SELECT * FROM news");
$news_id  = mysql_fetch_assoc($rowquery);
$query = mysql_query("SELECT * FROM news WHERE id = '".$news_id["id"]."");
while($row = mysql_fetch_assoc($query))
{
   echo"<a class=\"sendeplan\" href=\"news.php?id=".$row["id"]."\">".$row["title"]."</a>";
}
?>

[Forumsgast]

"SELECT * FROM `news` WHERE `id` = '".$_GET['id']."' LIMIT 1" 

musst du eigentlich schon am anfang machen.
Du solltest allerdings $_GET['id'] noch escapen.

[Forumsgast]

http://de.wikipedia.org/wiki/SQL-Injection

[Forumsgast]

JA ich weiss ich werde es noch escapen, habs noch vergessen.

[Forumsgast]

du kannst eigentlich den oberen Teil deines Codes weglassen:

$query = mysql_query("SELECT * FROM news WHERE id = '".$_GET["id"]."");
while($row = mysql_fetch_assoc($query))
{
   echo"<a class=\"sendeplan\" href=\"news.php?id=".$row["id"]."\">".$row["title"]."</a>";
} 

durch deinen oberen Codeteil entsteht auch dein Problem, dass zu viel ausgegeben wird.

[Forumsgast]

*seuftz*

$id = isset($_GET['id']) 
   ? is_numeric($_GET['id'])
      ? (int)$_GET['id'] 
      : false
   : false;

$linkPattern = '<a class="sendeplan" href="news.php?id={id}">{title}</a>';

if ( $id ) {

   $stmt = "SELECT id, title FROM news WHERE id = ".$id;
   $result = mysql_query($stmt);

   while( $rowdata = mysql_rech_assoc($result) )
   {
      echo str_replace(
         array('{id}', '{title}'),
         array($rowData['id'], $rowData['title']),
         $linkPattern
      );
   }

} 

Für den Fall dass das Datenbankfeld "id" wirklich eine Id im Sinne von auto-inkrementeller numerischer Id ist.