[SOLUCIONADO] amateur POO

[webankenovi]

Hol amigos como el tema indica soy novato en POO , despues de mucho pensarlo voy a pasar a trabajar con objetos me costaba un poco pero ya me he dado cuenta del potencial , ahora ando un poco perdido pero bueno nadie nadie nace sabiendo , he hempezado por hacer mi conexion he entrado en php.net para ver como debea de ser ya que tambien me voy a poner con las consultas preparadas MSQLI asique al lio:

este script lo he bajado del manual de php que indica como debe de ser la conexion

http://www.php.net/manual/es/mysqli....onnections.php

$mysqli = new mysqli("localhost", "usuario", "contraseña", "basedatos");
if ($mysqli->connect_errno) {
echo "Fallo al contenctar a MySQL: (" . $mysqli->connect_errno . ") " . $mysqli->connect_error;
}
echo $mysqli->host_info . "\n";


yo lo he intentado adaptarlo a una clase



<?php

class Conexion{

private $servidor = 'localhost';
private $usuario = 'root';
private $contrasena = '';
private $database = 'test';

public function __constructor($servidor, $usuario, $contrasena, $database)
{
$this->servidor = $servidor;
$this->usuario = $usuario;
$this->contrasena = $contrasena;
$this->database = $database;
}

public function conectar()
{
$mysqli = new mysqli($this->servidor, $this->usuario, $this->contrasena, $this->database);

if ($mysqli->connect_errno)
{
echo "Fallo al conectar a MySQL: (" . $mysqli->connect_errno . ") " . $mysqli->connect_error;
}
else
{
echo $mysqli->host_info . "\n";
}
}
}


$db= new Conexion();
$db->conectar();

?>

y mi pregunta es si esta bien planteado o creen que deberia de cambiar algo ay algo mal no se cualquier cosa ,se que falta cerrar la conexion ojo, vamos que lo que quiero es que me evaluen pero poder ir avanzando , como si fuerais mis profes jajaa

muchas gracias

[GatorV]

Pues lo primero sería guardar la instancia de $mysqli en una propiedad de tu clase, si no solo va a estar disponible en el ámbito de conectar().

En lo personal yo te recomendaría usar PDO directamente por la portabilidad entre bases de datos.

Saludos.

[webankenovi]

Es el primer script que hago en POO , y tienes razon mysqli deberia instanciarlo en el contructor verdad??

Asique me recomiendas mas PDO que mysqli ??? si solo fuera a usar mysql con mysqli valdria , pero si en un futuro puede ser que cambies de bd es mas recomendable pdo a eso te refieres verdad

y otra cosa que hare es pasar las variables desde un archivo de configuracion (server,user,password,bd) es mas recomendable verdad??

[GatorV]

Así es, siempre es mejor usar un archivo de configuración.

[webankenovi]

Oye gatorv una preguntita , para traer esos datos a la clase desde el archivo de configuracion , he estado investigando y incluso he hecho un tema para resolverlo por que no daba con ello http://www.forosdelweb.com/f68/varia...clase-1005959/ despues de un rato he dado con ello y la solucion creo que son las constantes,,¿? eso es asi verdad¿? o me estoy equivocando

[GatorV]

Pues no creo que sea lo más adecuado, ya que una constante es eso, aparte de que queda plasmado en el cídigo, yo usaría mejor un archivo de configuración para que los valores sean más fáciles de modificar.

[webankenovi]

pero no he encontrado otra manera de pasar los valores a las ""propiedades"" desde un archivo externo

y en el archivo de configuracion como guardas los valores en una variable normal

$server = 'locahost';


es que segun he leido en php

// Declaraciones inválida de propiedades:
public $var4 = self::myStaticMethod();
public $var5 = $myVar;

// Declaraciones válida de propiedades:
public $var6 = myConstant; // creia que esta es la unica manera de meter datos a las propiedades externamente , si no como los meto¿?
public $var7 = array(true, false);

como las deberia de psar del config a la clase¿?me estoy volviendo loco con esto la verdad, pero a las propiedades no a un metodo eso si se hacerlo , pero necesito crearlo en las propiedades

[GatorV]

Pues lo que pasa es que no las debes de definir en la clase, la idea es que tu clase sea lo más transparente, recuerda el polimorfismo de la clase.

Si lo que quieres es levantar con propiedades ya definidas tu clase, eso lo debe de hacer un helper, como el patrón factory, donde por medio de una cadena le indiques que DataSource usar, por ejemplo:

Código PHP:

$db = DBFactory::factory('default');
 

La clase DBFactory va a usar el patrón factory para cargar el archivo de configuración, e instanciar tu objeto DB y devolverte un objeto ya listo para que lo consumas.

Saludos.

[webankenovi]

ok creo que me sirvio voy a intentar implementar lo que yo quiero cn vuestras sugerencias y cuando termine lo subo para echarle un vistazo , thanks !!

[webankenovi]

He leido un tema en el que casualmente apareces gatorv jeje y me has dado la solucion a mis problemas

""Porque no haces un "registro" o un "almacén" de variables publicas?""

en esta pagina

http://www.forosdelweb.com/f68/clase...cucion-548021/

podria crear asi el archivo de configuracion y despues extenderlo a las clases donde se necesiten los datos como por ejemplo la clase de conexion y pasarle los datos servidor , user etc.... esto es mucho mejor que pasarlo a traves de constantes......


o la ultima solucion que veo es pasar un include en el constructor

[GatorV]

No era exactamente lo que te decía, pero lo puedes usar, el chiste es que cada clase tenga su diseño, y su alcance bien definido, recuerda que no debes de heredar por heredar, es el concepto de polimorfismo, que cada clase tenga su comportamiento exacto.

[webankenovi]

si si cada clase su archivo su alcance y su diseño bien definido es la idea , por eso no me lo tomo a la ligera y no tomo cualquier medida y busco la accion correcta y gracias a gente como tu que se molestan en ayudar a gente como yo lo conseguire jeje .

mira voy a ser lo mas especfico que pueda ser.

1- Archivo config

iniciamos variables de configuracion

por ejemplo las de conexion mysql

$server ='localhost';
$user ='root';
$pass ='';
$database ='test';

///// seguimos cargando variables etc......


2- Archivo mysqli

contiene la clase de la bd (conexion,selectdb,cambiar usuario,ping() y algun metodo mas)
no las consultas


3 Archivo autoload

carga la clase



ahora bien como introduzco las variables del archivo config , dentro de una clase ( en este caso archivo clase mysqli ) como atributos para poder usarlo en todos los metodos

[GatorV]

Puedes usar un archivo ini, o un array de PHP y pasarlo con un include.

[webankenovi]

gracias creo que esa es la solucion .

[webankenovi]

mira gatorv al final lo he dejado asi y funciona sin problemas aun me falta añadirle cosillas y mas metodos poco mas pero esta es la base , como ves creo un constructor y traigo del config los datos y ya tengo los atributos dentro para todos los metodos que incluya ademas de la conexion , en el metodo connect hago la conexion y ademas creo el atributo mysqli para tenerlo funcional en los demas metodos , dime si esta bien planteado o cometi algun error , por otro lado implemento una interface ya que esto es pensando el el futuro si deseo añadir algun driver mas de base de datos entonces tengo una interface que todos los drivers deben de cumplir , por ejemplo connect() y close()

Código PHP:

<?php
 
class mysqlidriver implements database
{
  public function __construct()
    {
        require_once '/application/core/config.ini';
 
        $this->server   = $server;
        $this->username = $username;
        $this->password = $password;
        $this->database = $database;
    }
 
    public function connect()
    {
        $mysqli = new mysqli($this->server,$this->username,$this->password,$this->database);
 
        $this->mysqli = $mysqli;
    }
 
    public function close()
    {
        $this->mysqli->close();
    }
}
 
 
?>
 

o esta otra forma que acabo de crear no se cual es mas correcta

Código PHP:

<?php
 
require_once '/config.ini';
 
class mysqlidriver{
 
public $server   = 'localhost';
public $username = 'root';
public $password = '';
public $database = 'test';
 
    public function __construct($username=NULL,$password=NULL,$database=NULL,$server=NULL)
    {
        $this->server   = is_null($server)   ? $this->server   : $server ;
        $this->username = is_null($username) ? $this->username : $username ;
        $this->password = is_null($password) ? $this->password : $password ;
        $this->database = is_null($database) ? $this->database : $database ;
    }
     
    public function connect()
    {
        $mysqli = new mysqli($this->server,$this->username,$this->password,$this->database);
     
        $this->mysqli = $mysqli;
    }
     
    public function close()
    {
        $this->mysqli->close();
    }
}
     
     $db = new mysqlidriver($username,$database,$password,$server);
     
?>
 

[GatorV]

¿que pasa en caso de que te quieras conectar a 2 bases de datos?

[webankenovi]

todavia no he planteado ese problema

pero creo que habria un problema de identificador , ademas debo de usar 2 bases de datos jeje

bueno podria crear otro metodo con una conexion que cree otro identificador y pase nuevos parametros ¿?

He pensado esta idea y no se si es valida

Código PHP:

<?php
 
 
require_once '/config.ini';
 
 
     
class mysqlidriver{
 
    // configuracion por defecto si no llegan parametros
 
public $server   = 'localhost';
public $username = 'root';
public $password = '';
public $database = 'test';
 
    public function __construct($username=NULL,$password=NULL,$database=NULL,$server=NULL)
    {
        $this->server   = is_null($server)   ? $this->server   : $server ;
        $this->username = is_null($username) ? $this->username : $username ;
        $this->password = is_null($password) ? $this->password : $password ;
        $this->database = is_null($database) ? $this->database : $database ;
    }
     
    public function connect()
    {
        return  new mysqli($this->server,$this->username,$this->password,$this->database);
    }
     
    public function connecttwo($server,$username,$password,$database)
    {
        return  new mysqli($server,$username,$password,$database);
    }
     
    public function close($val)
    {
        $val->close();
    }
}
     
     $db = new mysqlidriver();
     $db->connect(); // conecto por defecto si no instancio los parametros
     $db->connecttwo('localhost','root','','otra bd');
 
 
?>
 

[GatorV]

Como te digo, cada clase debe de tener su propia responsabilidad, si te fijas son dos cosas diferentes, una es tu clase DB que se encarga de hacer consultas contra una Base de Datos, otra cosa es, el iniciar y levantar tu clase con los datos que necesites, por eso es que te decía que usaras un DB Factory, o algún método similar que sea el encargado de dejar tu objeto DB listo para usarse.

Saludos.

[webankenovi]

vale vale lo acabo de pillar

acabo de leer este articulo

http://www.aramirez.es/blog/patron-f...e-forma-facil/

claro asi es mucho mejor para pasar varias conexiones etc...

[webankenovi]

buenas de nuevo mira he intentao adaptarlo a el patron factory no se si lo hice bien pero bueno , yo le mando a que conexion conectarse , el valor lo extraigo del config.

mas o menos lo que hice lo hice de ejemplo para probar y eso seguro que faltaran cosas , lo que me interesa es si he entendido el concepto

archivo unico factory conexion

Código PHP:

<?php
 
class conexion
{
    public function __construct($tipo)
    {
        $this->db = c_conexion::create_connection($tipo);
    }
        
}
 
?>
 

controlador de la conexion , de aqui iriamos al archivo que controla la aplicacion de la conexion elegida y que esa ya si lleva los metodos

Código PHP:

<?php
 
class c_conexion extends conexion
{
    public static function create_connection($type)
    {
        switch ($type)
        {
            case 'mysql':
            $db = new mysql();
            break;
 
            case 'oracle':
            $db = new oracle();
            break;
 
            default:
            $db = new mysqli;
        }
 
        return $db;
    }
 
 
}
 
?>
 

no se como lo ves , pero asi si se puede inicar mas de una conexion si no me equivoco

[GatorV]

Todavía tienes un poco errado lo que es el patrón Factory, la idea de este es que cree otros objetos y los deje listos para usarse.

Lee este tema es básicamente lo que quieres hacer:
http://www.forosdelweb.com/f18/duda-...-clase-835534/

Saludos.

[webankenovi]

bueno me estoy volviendo un poco loco con esto la verdad pero espero poder llegar a enterderlo, para mi es algo muy nuevo la programacion orientada a objetos.

Creo haber entendido por fin el patron, te pongo este ejemplo que si es correcto doy por finalizado este tema

Código PHP:

<?php
 
class MySQL
{
    public function __construct()
    {
        echo "Constructor MySQL";
    }
 
    public function connect()
    {
        echo "Conectar con MySQL";
    }
}
 
class MySQLi
{
    public function __construct()
    {
        echo "Constructor MySQLi";
    }
 
    public function connect()
    {
        echo "Conectar con MySQLi";
    }
}
 
class database
{
    public static function factory($type)
    {
        if(class_exists($type))
        {
            return new $type();
 
        }else
 
            echo "Error. la clase '".$type."' no existe";
 
        return 0;
    }
}
 
 
$obj1=database::factory("MySQLi");
$obj1->connect();
 
 
?>
 

[GatorV]

Pues sí eso parece más una fábrica de conexiones, yo cambiaría el nombre de las clases para reflejar que es lo que hacen pero básicamente sí es eso.

El patrón factory es muy útil y pues su nombre dice, lo que es, una fabrica de conexiones, debe de crear conexiones, es casi como heredar pero es una relación de uso más que nada, es por eso que no hay herencia.

Muchos tratan de aplicar factory para que sea una fábrica genérica y construir objetos de todo tipo, pero ese también es un horror, ya que cada fábrica debe de tener su alcance.

Te invito a que leas el tema que te puse, ya que habla de todos estos conceptos y tiene muchos códigos de ejemplo.

Saludos.

[webankenovi]

!! ..... muchas gracias por todo ..... !!

[webankenovi]

una ultima pregunta esto que te voy a mostrar queria saber si es equivalente a el ejemplo anterior y si es mejor o peor y los por que si me lo puedes decir claro

Código PHP:

<?php
 
 
function _mysql(){
 
 
echo 'conectado mysql';
 
}
 
function _oracle(){
 
 
echo 'conectado oracle';
 
}
 
function database($type){
 
        if(function_exists($type))
        {
            return $type();
        }
        else
        {
            echo "Error. la funcion '".$type."' no existe";
 
            return 0;
        }
 
 
}
 
database('_mysql');
 
?>
 

Muchas gracias.

[GatorV]

Pues, por lo que veo es programación estructural, la cual es diferente a la POO, en todo.

[webankenovi]

si si , ya se que son diferentes , pero habla sobre tiempo de ejecucion , ventajas desventajas esas cosas , muchas gracias .

[GatorV]

Pues ese código que pones es muy simple como para hablar sobre tiempo de ejecución va a ser el mismo ya que no hacen mucho.

¿Ventajas y desventajas? Pues repito lo mismo es un código muy sencillo como para hablar de ventajas y desventajas ya que es prácticamente lo mismo.

[webankenovi]

a eso me referia , bueno gatorv dejamos ya este tema cerrado , gracias por las aclaraciones las sugerencias y me has ayudado a aprender algo mas , yo nunca estudie nada