User Löschen?

[vodka]

Hallo also ich hab ein Login Script gebastelt mit Session etc..!

Geht alles aber jetzt wen ich Users löschen will die Adminrechte haben geht das ned! So wird immer nur der Member gelöschtt der als letztes eingetragen wurde!

<?
	if (!$user_id == "")
    {
	?>
	<?
	switch($_GET[delete]) {
	default:
	?>

            <?php
$db = mysql_connect($db_host, $db_user, $db_pass);
mysql_select_db ($db_name) or die ("Cannot connect to database");

$query = "SELECT * FROM $tabellelogin";
$result = mysql_query($query);
while($r=mysql_fetch_array($result))
{
$username=$r["username"];
$user_id=$r["user_id"];

echo "<table width=450 border=0 cellpadding=0 cellspacing=0>
  <tr>
    <td width=207>$username</td>
    <td width=184>&nbsp;
</td>
    <td width=59><div align=center><a href=?action=userdel&delete=del&user_id=$user_id>L&ouml;schen</a></div></td>
  </tr>
</table>";
  }


?>
            <?
break;
case "del";
$db = mysql_connect($db_host, $db_user, $db_pass);
mysql_select_db ($db_name) or die ("Cannot connect to database");

$loeschen = "DELETE FROM $tabellelogin WHERE user_id = '$user_id'";
$loesch = mysql_query($loeschen);

    echo "<b>User erfolgreich gel&ouml;scht";

?>
            <?
}
?>
            <?
 } 
 else 
{
       echo "Du bist nicht eingeloggt!";
}
?>

 

[worki2k1]

Ich sehe bei deinem Code irgendwie nicht ganz durch. Ich vermute mal du wirst die Variable $user_id irgendwo doppelt benutzern oder so. Mir ist nicht so ganz klar wo einige Variablen herkommen.
Letzendlich brauchst du doch nur die ID des Benutzers an ein Script übergeben, was die Datenbankabfrage dann ausführt. Das wäre die einfachste Variante. Ich lade meist noch die Seite danach neu damit der Code bei einem Reload der Seite nicht nochmal ausgeführt wird.

 

[vodka]

Ich denke es liegt an dem weil man ja mit einer User ID eingeloggt ist und dan einen Löschen will mit einer User ID!

Weis aber ned wie ich das machen soll das man ein Benutzer löschen kann!

 

[worki2k1]

Der Ansatz ist doch schon richtig. Du listest alle User auf mit einem Link zum löschen. An diesen Link hängst du jeweils die user_id des Benutzer an, damit das Script weiß um welchen Benutzer es geht.
Im Löschen-Script liest du die überlieferte ID aus und machst eine SQL-Anweisung, die du auch schon richtig hattest.
Du musst jetzt halt nur mit der Benennung deiner Variablen aufpassen, dass Sie nicht mit anderen in die Quere kommen. Hast du vielleicht auch in der Session eine Variable gespeichert, die "user_id" heißt? Das kann unter umständen auch zu Problemen führen.

 

[Nuka]

Man kann das doch ganz einfach mit $_POST[""] bzw. $_GET[""] lösen

 

[vodka]

Beispiel?

 

[ronin]

mysql_query("DELETE FROM user WHERE id='".$_POST["user_id"]."'");

Du mußt vorher halt ein Formular haben mit dem Input-Feld "user_id" welches per POST übergeben wird. ^^