Deleting a Row...

Daggeth

The browser sends my script a variable, ID.. my script searches the First column of the MySQL database (Which is ID) till both IDs match.. heres how it goes!

<?php

$db = mysql_connect("$host", "$username", "$password");
mysql_select_db("test",$db);

$sql = "SELECT * FROM ITEMS ";
$result = mysql_query($sql,$db);

// $ID = Get from frickin browser

$loop = mysql_num_rows($result);

$i = 0;
    while ($i < $loop)
	  {		
			$i++;
			$myrow = mysql_fetch_row($result);
			$RowID=$myrow[0];
			echo "$RowID";
			if ($ID===$RowID) 
				{
 				   //BAM! It found the row (Each ID is Unique!)
                                                                   //Now how do I delete this row?
                                                                   break;
				}
	  }
?>

DoD

$sql = "DELETE FROM $table
	WHERE ID=$id";
	if (@mysql_query($sql)) {
	echo("<p>The record has been deleted.</p>");
	} else {
	echo("<p>Error deleting Record: " .
	mysql_error() . "</p>");
	}

rheroux

Try this:

<?php

$db = mysql_connect("$host", "$username", "$password");
mysql_select_db("test",$db);

$sql = "SELECT * FROM ITEMS ";
$result = mysql_query($sql,$db);

// $ID = Get from frickin browser

$loop = mysql_num_rows($result);

$i = 0;
    while ($i < $loop)
      {        

            $i++;
            $myrow = mysql_fetch_row($result);
            $RowID=$myrow[0];
            echo "$RowID";
            if ($ID===$RowID) 
                {
                    $sql1 = "DELETE FROM ITEMS WHERE id = " .$ID;
                    $result1 = mysql_query($sql1,$db);

                                                               break;
            }
  }
mysql_close($db);
?>

philipolson

Why do you select them all? Just run the delete query. The logic here is a little questionable but I'll assume you want to know the information so try something like:

<?php
// Variables are case-sensitive, most prefer lowercase
// but maybe you really want ID and not id.
if (@is_numeric($_REQUEST['id'])) {
    $id = $_REQUEST['id'];
} else {
    print "Please go back and choose an appropriate user id";
    exit;
}

if (!$conn = @mysql_connect($host, $username, $password)) {
    print "Could not connect to mysql : " . mysql_error();
    exit;
}

if (!@mysql_select_db($dbname, $conn)) {
    print "Could not select the db ($dbname) " . mysql_error();
    exit;
}

$sql = "SELECT id,name FROM yourtable WHERE id = $id";
if (!$result = @mysql_query($sql)) {
    print "Could not run query ($sql) : " . mysql_error();
    exit;
}

// If there is a row of data with that id
if (mysql_num_rows($result) > 0) {

$row = mysql_fetch_assoc($result);
extract($row);

print "Hello, we are about to delete the information from id $id, so say goodbye to $name, s/he will be gone forever! RIP.";

$sql = "DELETE FROM yourtable WHERE id = $id";
if (!$result = mysql_query($sql)) {
    print "Could not run query ($sql) : " . mysql_error();
    exit;
}

// This should always return true but hey let's go with it
if (mysql_affected_rows($result) > 0) {
    print "Okay, $name is officially gone now! :)";
}

} else {
    print "There is nobody with that id here";
}

That of course is just an example on what you can do, it needs some work (like the error handling). But the point is I strongly suggest not looping through almost every record like that, what's the point?