simplificar consulta

[WinderJerter]

Hola que tal, lo que pasa es que tengo un código que sirve para activar, borrar imágenes y activar usuarios y funciona correctamente, aquí el código:

Código PHP:

<?php
 
include_once("connectBD.php");
 
if($_GET['a'] == "activate" && $_GET['type'] == "img") {
    $query = "SELECT * FROM IMAGES WHERE ID = '".$_GET['id']."' AND PIN = '".$_GET['pin']."'";
    $result = mysql_query($query);
    
    $row = mysql_fetch_assoc($result);
    
    $query2 = "SELECT * FROM TMP_IMAGES WHERE ID = '".$row['ID']."'";
    $result2 = mysql_query($query2);
    
    if(mysql_num_rows($result) == 1 && mysql_num_rows($result2) == 0) {
        header("refresh: 3; url= http://web.com/preview.php?id=".$row['ID']);
        echo "<div class=\"error\">Su imagen ya a sido activada anteriormente.</div>";
    } elseif(mysql_num_rows($result) == 1) {
        $sql_del = "DELETE FROM TMP_IMAGES WHERE ID = '".$row['ID']."'";
        $result_del = mysql_query($sql_del);
        header("refresh: 3; url= http://web.com/preview.php?id=".$row['ID']);
        echo "<div class=\"success\">La imagen ha sido activada correctamente.</div>";
    } else {
        echo "<div class=\"error\">ERROR</div>";
    }
}
 
elseif($_GET['a'] == "delete"  && $_GET['type'] == "img") {
    $query = "SELECT * FROM IMAGES I INNER JOIN THUMBNAILS T ON I.ID = T.ID WHERE I.ID = '".$_GET['id']."' AND I.PIN = '".mysql_real_escape_string($_GET['pin'])."'";
    $result = mysql_query($query);
    
    if(mysql_num_rows($result) == 1) {
        $row = mysql_fetch_assoc($result);
        $delete_link = array("http://", ".web.com");
        $str_link = str_replace($delete_link, '', $row['LOCATION']);
        $str_link2 = str_replace($delete_link, '', $row['SQUARE_75']);
        $str_link3 = str_replace($delete_link, '', $row['MEDIUM_640']);
        unlink('servers/' . $str_link);
        unlink($str_link2);
        unlink($str_link3);
        
        $sql_del_img = "DELETE FROM IMAGES WHERE ID = '".$row['ID']."'";
        $result_del_img = mysql_query($sql_del_img);
        
        $sql_del_img_tmp = "DELETE FROM TMP_IMAGES WHERE ID = '".$row['ID']."'";
        $result_del_img_tmp = mysql_query($sql_del_img_tmp);
        
        $sql_del_thumb = "DELETE FROM THUMBNAILS WHERE ID = '".$row['ID']."'";
        $result_del_thumb = mysql_query($sql_del_thumb);
        
        $sql_del_user = "UPDATE USERS SET UPLOADS = UPLOADS - 1 WHERE IUID = '".$row['IUID']."'";
        $result_del_user = mysql_query($sql_del_user);
        
        echo "<div class=\"success\">La imagen ha sido borrada correctamente.</div>";
    } else {
        echo "<div class=\"error\">ERROR</div>";
    }
}
 
elseif($_GET['a'] == "activate"  && $_GET['type'] == "user") {
    $query = "SELECT * FROM USERS WHERE IUID = '".$_GET['iuid']."' AND EMAIL = '".$_GET['email']."'";
    $result = mysql_query($query);
    
    $row = mysql_fetch_assoc($result);
    
    $query2 = "SELECT * FROM TMP_USERS WHERE IUID = '".$row['IUID']."'";
    $result2 = mysql_query($query2);
    
    if(mysql_num_rows($result) == 1 && mysql_num_rows($result2) == 0) {
        header("refresh: 3; url=http://web.com/user.php?iuid=".$row['IUID']);
        echo "<div class=\"error\">Su usuario ya a sido activado.</div>";
    } elseif(mysql_num_rows($result) == 1) {
        $sql_del = "DELETE FROM TMP_USERS WHERE IUID = '".$row['IUID']."'";
        $result_del = mysql_query($sql_del);
        header("refresh: 3; url=/");
        echo "<div class=\"success\">Thank you for registering in Imagupload.</div>";
    } else {
        echo "<div class=\"error\">ERROR</div>";
    }
}
 
else {
    echo "<div class=\"error\">La pagina no existe</div>";
}
 
?>
 

Pero lo que quiero hacer es simplificar el código para que no se vea y se me haga engorroso para cuando lo quiera modificar.

Alguien que me pueda ayudar???

Saludos y gracias de antemano

[efedefernan]

Una pequeña recomendación es que uses programacion orientada a objetos.
Create una clase y creas unos metodos en funcion de lo que necesites, el codigo lo tendras en la clase y cada vez que necesites actualizar, solamente tendrás que modificar el codigo dentro de tu clase, porque la llamada a los metodos desde tu pagina no cambiará.

Un saludo.

[WinderJerter]

Hola que tal, muchas gracias por responder, la verdad no se nada de PHP OO y ayer leyendo la documentación de PHP OO en php.net y con un ejemplo en foro logre hacer este código pero la verdad no se si este bien y no he podido probarlo porque mi cuanta en el hosting esta suspendida. Aquí el código:

Código PHP:

<?php
 
class activarUsuario {
    
    public function consultarBD($query) {
        $this->consulta=mysql_query($query, $this->enlace);
    }
    
    public function obtenerdatosBD() {
        $this->resultado=mysql_fetch_assoc($this->consulta);
        return $this->resultado;
    }
    
    public function comprobar() {
        if($_GET['a'] == "activate" && $_GET['type'] == "img") {
            $this->consultarBD("SELECT * FROM USERS WHERE IUID = '".$_GET['iuid']."' AND EMAIL = '".$_GET['email']."'");
            $row=$obj->obtenerdatosBD();
        }
    }
    
    public function activar() {
        if(mysql_num_rows($row) == 1) {
            $this->consultarBD("DELETE FROM TMP_USERS WHERE IUID = '".$row['IUID']."'");
        }
    }
    
}
 
?>
 

La verdad se que tiene muchos errores y si me pudieran ayudar a corregirlos se los agradecería mucho.

Saludos y gracias de antemano

[efedefernan]

Antes de que te den el código hecho, te recomiendo te leas un pequeño manual sobre la POO en php.

Una vez entiendas la dinamica de la programación POO te resultará muy facil crear tus propias clases.

La esencia de la clase que has creado es buena, pero sería recomendable crear unas propiedades, un constructor (si lo ves necesario), y unos metodos que en ocasiones querrás que sean privados y otras que sean públicos o estaticos , etc. etc. etc.

Luego te tienes que fijar que los metodos son funciones al uso, por lo que si vas a utilizar variables como en tu caso $_GET, una de dos, o las declaras como variables globales, o las tendras que pasar como parametros.

Mi pequeño consejo es que leas un pequeño manual acerca de la POO.

Un saludo.