Parametro en blanco? vacio? mm problema de filtrado.

[Hidek1]

Después de mucho meditarlo no pude encontrar solución :P les dejo un problemita q tengo para ver si alguno se le ocurre algo

tengo una classe para administrar mi database SQL
con un método constructor para tomar los datos y hacer la conexión, hasta aquí todo bien, todo normal, el tema es que tengo un archivo de configuración. en donde defino unas constantes que serian por decirlo así la conexión por defecto a SQL

planteo aquí el ejemplo

Código PHP:

define('DBHOST','localhost');
define('DBUSER','user');
define('DBPASS','pass');
define('DBDDBB','nombre_db');
 

entonces el metodo constructor quedaria asi

Código PHP:

public function __construct($db_host = "", $db_user = "", $db_pass = "", $db_ddbb = ""){
        $this->host = (!empty($db_host)) ? $db_host : DBHOST;
        $this->user = (!empty($db_user)) ? $db_user : DBUSER;
        $this->pass = (!empty($db_pass)) ? $db_pass : DBPASS;
        $this->ddbb = (!empty($db_ddbb)) ? $db_ddbb : DBDDBB;
        $this->conectar();
    }
 

hasta aquí todo bien también

pruebas de ejemplo

Código PHP:

$db = new Database(); // conexión sin problemas
$db = new Database("","","","OtraBaseDeDatos");  // conexión sin problemas
$db = new Database("","OtroUser","OtraPass"); // conexión sin problemas
/* pero aqui o.O? 
    quiero cambiar el user a root, y sin password.
*/
$db = new Database("","root","");
 

Obviamente estoy enviando la variable vacía, así que el empty del método constructor lo filtra y me deja como contraseña de la db la asignada por la constante de mi archivo de configuración.

alguna idea de como mantener este algoritmo y dejarlo 100% funcional?

PD: soy pésimo explicando problemas, intente dejarlo lo mas claro posible.. cualquier duda comentan...

saludos

[pateketrueke]

usa is_null(), pues si usas comillas vacías no son iguales a NULL

y en caso de querer usar un valor por defecto usas las comillas vacías... (:


adicionalmente dejo este enlace, es una tabla comparativa de tipos:
http://www.blueshoes.org/en/developer/php_cheat_sheet/

[Hidek1]

es mas o menos lo que necesito pero mi idea era evitar tener que entregar una instrucción al que use la classe... por que no muchos la leerían y se darían cuenta de eso

y para que lo que tu dices funcione
tendria q hacerlo asi

Código PHP:

$db = new Database(NULL,NULL,NULL,"otraDatabase");
 

lo cual rara ves e visto q alguien lo haga :P..

hay alguna otra forma?

[pateketrueke]

entonces el problema radica en el diseño de tu clase... osea, si no quieres que se haga eso tan simple como crear métodos setters para la configuración...

Código PHP:

  function setHost($host) 
{ 
  // ... 
} 
 
// .. 
 
$foo = new Bar; 
$foo->setHost('localhost');  
   

por cierto... esto debería ir en el foro de OOP

[Hidek1]

ajaja si al parecer no me queda otra que usar setters :P
eso me pasa por cerrarme tanto en intentar dejarlo todo desde el __construct :3

gracias por la ayuda @pateketrueke
te dejo karma, ya que aquí tienen la mala costumbre de ni agradecer la ayuda que uno brinda !!

saludos :P!

[HackmanC]

Hola,

Posiblemente ...

Para mantener las características básicas de la programación orientada a objetos, la clase 'Conexión' no debería inicializar por defecto ningún valor. Por abstracción, la clase no debería conocer los detalles externos de la implementación.

De otra forma se vuelve confuso para los programadores que usan o extienden dichas clases, y resultas usando 'siempre' la versión 'con parámetros'. Si no se le pasan los parámetros necesarios debe tirar una excepción.

Así tenemos como ejemplo los frameworks, creo que ningún framework va a inicializar por defecto ningún valor de la conexión (sino posiblemente resultaría conectandome a Zend sin saberlo ). Eso permite que la clase sea lo suficientemente abstracta para que cada uno la inicializemos a nuestro gusto.

Ahora bien ... si quieres crear esa clase para ahorrarte el trabajo de estar escribiendo el nombre de usuario, la contraseña, entonces lo mas aconsejable sería usar una clase nueva que extienda la anterior, que implemente el patrón singleton y que tenga un método Factory para obtener una conexión específica de tu base de datos obtenerConexionLocal(), y en ese caso no acepta parámetros.

Por ejemplo, la programación a objetos debe simular el comportamiento normal de las cosas, si yo digo Casa() me estoy refiriendo a un prototipo de casa pero ninguna en especial. Si digo new Casa("25"), estoy refiriendome a la casa 25.

Si digo Conexion no me estoy refiriendo a ningúna en específico. new Conexion(25) entonces ya sé que es la conexión número 25. Pero no espero que new Conexion() me devuelva la conexión 8 ... ¿Cómo lo voy a saber exactamente, si eso lo oculta la implementación de Conexion?.

Pero esa es solamente mi opinión ,
Saludos,

[Hidek1]

@HackmanC gracias por tu comentario.. en realidad no llevo mucho con esto de POO y todo lo que se de php es por manuales y ese tipo de cosas..
me a costado mucho salir de la programación estructurada por un tema de costumbre y no tengo aun las bases que enseñan bien el tema este :P

como lo había planteado esto va con un archivo de configuración por eso había pensado en esa opción en caso de que en 1 u otra pagina se necesitara cambiar solo de base de datos u otro parecido :P.. pero ya que lo comentas creo que revisare para ver si mejoro un poco la vision que tengo sobre la classe..

saludos.

[GatorV]

Tema movido desde PHP a PHP orientado a objetos