PHP OO Cual seria la mejor forma para este metodo?

[vicluber]

Buenas tardes!

Me enseñaron a crear los métodos de mis objetos de esta forma...

Código:

public function eliminar(){
		
		$base=new BaseDatos();
		if($base->Iniciar()){
			$consultaBorra="DELETE FROM clientes WHERE id_cliente=".$this->getIdCliente();
		
			if($base->Ejecutar($consultaBorra)){
				
			}else{
					echo $base->getError();
					return false;
					
				}			
		}else{
			echo $base->getError();
			return false;
			
		}
	}

Así como esta el método yo debería primero utilizar mi método "buscarCliente" para cargarlo y poder eliminarlo con "$this->getIdCliente()"
Pero esa forma es la mejor? Porque también podría pasarle el id del cliente al método directamente y ahorrarme el cargarlo... de esta forma...

Código:

public function eliminar($id){
		
		$base=new BaseDatos();
		if($base->Iniciar()){
			$consultaBorra="DELETE FROM clientes WHERE id_cliente=".$id;
		
			if($base->Ejecutar($consultaBorra)){
				
			}else{
					echo $base->getError();
					return false;
					
				}			
		}else{
			echo $base->getError();
			return false;
			
		}
	}

Y así evitar el "buscarCliente"... asumo que si hay razones por las que la primera es la correcta... de lo contrario no me lo hubieran enseñado así... (pero me lo enseñaron hace tiempo ya y no tengo a quien preguntar)
Se entiende?

Gracias y saludos!

¿Cual es la mejor forma? y ¿Por que?

[Nemutagk]

La primera no le veo mucho caso, a menos que tu objeto (la clase a la que pertenece el metodo eliminar()) tenga la lógica necesaria para tener un metodo "estatico" como lo tienes tu, se me ocurre por ejemplo un objeto del tipo soldado, en el cual el mismo objeto tiene "definición" propia, en cambio la otra forma da la opción a eliminar el registro que sea, por lo cual es dinamico, todo depende del objeto a la que pertenece dicho metodo, en cualquier forma ambas son validas y correctas usadas en el ambito correcto...

[masterpuppet]

Que tal vicluber,

Antes que nada te sugiero que leas sobre los patrones mas comunes para lo que muestras, ActiveRecord - ActiveTable, Table Data Gateway - Row Data Gateway y algo ya mas lejano(en relación a lo que propones) DataMapper.

Con respecto a cual seria mejor, depende de que representa la clase, si es la unidad, lo lógico seria el primer ejemplo, si es la colección, seria el segundo ejemplo.

Saludos.

[Lokitozzz]

Un poco de seguridad tampoco vendría mal, primero comprobar que el objeto no exista y que el id de usuario sea tratado como entero (int).

Código PHP:

public function eliminar($id)
{
    global $base;
    
    if(!is_object($base) && !($base))
    {
        $base = new BaseDatos();
    }
    
    if($base->Iniciar())
    {
        $consultaBorra = "DELETE FROM clientes WHERE id_cliente=" . (int)$id;
        if($base->Ejecutar($consultaBorra))
        {
            
        }
        else
        {
            echo $base->getError();
            return false;       
        }
    }
    else
    {
        echo $base->getError();
        return false;   
    }
}
 

[vicluber]

Ey gracias gente!

Muy útil!

Saludos!