Do i need to select before delete in mysql?:

Question

In my websites admin page, i can delete products from the webshop, with this php code below. (Ajax calls this php file).

My question is, that do i need to select and check the num rows, or drop that, and just keep and run the delete queries?

    <?php
include_once("../../files/connect.php");
if($_POST)
{
    $id = mysqli_real_escape_string($kapcs, $_POST['id']);
    $check = mysqli_query($kapcs, "SELECT termek_id, termek_thumb, termek_big FROM termek WHERE termek_id='$id' LIMIT 1");
    if(mysqli_num_rows($check) > 0 )
    {


        /*Galéria törlése, ha van*/
        $check_gallery = mysqli_query($kapcs, "SELECT * FROM gallery WHERE gallery_termek_id = '$id'");
        if(mysqli_num_rows($check_gallery) > 0 )
        {
            while($gall = mysqli_fetch_assoc($check_gallery))
            {
                $DestinationDirectory = "../../images/gallery/";
                if(file_exists($DestinationDirectory.$gall['gallery_thumb']))
                {
                    unlink($DestinationDirectory . $gall['gallery_thumb']); 
                }
                if(file_exists($DestinationDirectory.$gall['gallery_big']))
                {
                    unlink($DestinationDirectory . $gall['gallery_big']);   
                }
                mysqli_query($kapcs, "DELETE FROM gallery WHERE gallery_termek_id = '$id'") or die(mysqli_error($kapcs));
            }
        }

        /* Címkék törlése, ha van */
        $check_cimke = mysqli_query($kapcs, "SELECT termek_cimke_termek_id FROM termek_cimke WHERE termek_cimke_termek_id = '$id'");
        if(mysqli_num_rows($check_cimke) > 0 )
        {   
            mysqli_query($kapcs, "DELETE FROM termek_cimke WHERE termek_cimke_termek_id = '$id'");
        }



        mysqli_query($kapcs, "DELETE FROM hir_termek_kapcsolo WHERE hir_termek_kapcs_termek = '$id' ") or die("Delete error 2 - " . mysqli_error($kapcs));


        $del = mysqli_query($kapcs, "DELETE FROM termek WHERE termek_id = '$id'");
        if($del)
        {
            echo 'Delete ok';
        }
        else
        {
            echo 'Delete error;
        }
    }
    else
    {
        echo 'Nincs ilyen azonosítójú termék.';
    }
}
?>

you need to be specific which sql ? – Ravi Jan 13 '18 at 16:57 — Ravi, Jan 13 '18 at 16:57

Qirel · Answer 1 · 2018-01-13T17:12:39.980

You do not need to select anything before deleting any rows. Just use the same condition as you would selecting the rows.

Run the delete query, and you can check the number of affected rows to see if anything was deleted or not, and if so, how many rows. This function returns and integer with the number of deleted rows for a DELETE query.

mysqli_query($kapcs, "DELETE FROM hir_termek_kapcsolo WHERE hir_termek_kapcs_termek = '$id' ") or die("Delete error 2 - " . mysqli_error($kapcs));
echo mysqli_affected_rows($kapcs)." rows deleted";

http://php.net/manual/en/mysqli.affected-rows.php

WARNING
You're already using an API that supports prepared statements with bounded variable input, you should utilize parameterized queries with placeholders (prepared statements) to protect your database against SQL-injection! Get started with mysqli::prepare() and mysqli_stmt::bind_param().

score 1 · Answer 2 · answered Jan 13 '18 at 16:58

1

You do not need to select before you delete.

If no rows match the conditions, then the delete will not delete any rows.

answered Jan 13 '18 at 16:58

Gordon Linoff

1,242,037
58
646
786

So why to select? If there arent any rows that match, it wouldnt delete anything. – KissTom87 Jan 13 '18 at 17:00
It is your code. I don't know why there is a `select` in it. – Gordon Linoff Jan 14 '18 at 23:07

Do i need to select before delete in mysql?:

2 Answers2