[SOLUCIONADO] Llamados a funciones en PHP

[j_silk_h]

Tengo el siguiente codigo:

Código PHP:

  function conectarse(){
    include("config.php");
    $mysqli = new mysqli($host, $user, $password, $db);
    return $mysqli;        
}
 
function funcion1(){
    $mysqli = conectarse();
    
    $query = $mysqli->query("bla bla bla bla bla");
}
 
function funcion2(){
       $query = $mysqli->query("bla bla bla bla bla");
}
 
function funcion3(){
        $query = $mysqli->query("bla bla bla bla bla");
}
 
$mysqli = conectarse(); 
   

Como veran en la funcion1 no hay problema porque vuelvo a llamar a funcion conectarse()

En las demás me da un error de variable no definida:

Undefined variable: mysqli in C:\wamp\www\administrativo\bin\php.php on line

Me pregunto lo siguiente, estoy llamando a mi función conectarse() al final, lo he colocado después de declarar la funcion y en varias partes de mis codigos, solo quiero hacer un solo llamado a la funcion conectarse, para evitar esto:

Código PHP:

  function conectarse(){
    include("config.php");
    $mysqli = new mysqli($host, $user, $password, $db);
    return $mysqli;        
}
 
function funcion1(){
    $mysqli = conectarse();// aqui
    
    $query = $mysqli->query("bla bla bla bla bla");
}
 
function funcion2(){
       $mysqli = conectarse();// aqui
       $query = $mysqli->query("bla bla bla bla bla");
}
 
function funcion3(){
       $mysqli = conectarse();// aqui
        $query = $mysqli->query("bla bla bla bla bla");
}
 
$mysqli = conectarse(); 
   

Algun consejo?

[jonni09lo]

Hola en PHP puedes usar el ámbito de variables este seria un EJEMPLO de como PODRIAS usarlo, ten en cuenta de que le faltan muchos aspectos de tenerse en cuenta pero la idea del funcionamiento es la misma

Código PHP:

<?php
global $mysqli;
 
function conectarse(){ 
    include("config.php");
    $GLOBALS['mysqli'] = new mysqli($host, $user, $password, $db);     
}
 
function funcion1(){ 
    $query = $GLOBALS['mysqli']->query("bla bla bla bla bla"); 
} 
 
function funcion2(){ 
    $query = $GLOBALS['mysqli']->query("bla bla bla bla bla"); 
} 
 
function funcion3(){ 
    $query = $GLOBALS['mysqli']->query("bla bla bla bla bla"); 
}
//llamamos esta funcion ANTES de USAR las funciones que dependan de $mysqli
conectarse();
 

Saludos

[j_silk_h]

Oka, me parece interesante, precisamente estaba bordeando los manuales sobre variables locales, ya lo pruebo!

[ChrisPio]

Me parecería mejor que lo encerraras en una clase, así lo estancias una sola vez y luego en cada página solo será hacer un llamado a la estancia y la función, guardando la conexión en una variable publica de la clase:

Código PHP:

  class db{
  public $con = new mysqli($host, $user, $password, $db);

  //Llamado dentro de la clase
  $this->con;
}
//Llamado fuera
$db=new db();
$db->con; 
   

[j_silk_h]

Perfecto jonni09lo, funciono de maravilla. Me gustaría que me menciones esos aspectos que menciones para evitar problemas a futuro!

[j_silk_h]

A ver ChrisPio, también me gusta tu propuesta, ya la programo

[jonni09lo]

Hola el principal problema es lo que menciona @ChrisPio al ser un código para ingresar a la base de datos es mejor trabajar con clases yo te respondí de la manera como lo hice por dos razones: la primera para que veas el asunto del ámbito y la segunda porque tu código inicial es programación estructurado y es bueno que vayas viendo que existen soluciones variadas a los problemas que se plantean, así NO sea lo mas adecuado ya que como dice un viejo amigo: Si se puede imaginar se puede programar


Has trabajado con clases?
Sabes sobre patrones de diseño?

Saludos

[j_silk_h]

Tengo años trabajando en programación estructurada, desde el bachillerato cuando empece en esto, manejo ajax, JS, jQuery, etc. Pero no se porque no logro entender la POO, algún foro con un tema de conexión a BD usando MySQLi y POO?

[j_silk_h]

@jonni09lo tiene razón tu amigo, "Si se puede imaginar se puede programar" me gusta lo usare porque es verdad, tengo algo de experiencia con clases, y que patrones de diseño me hablas? UML, diagramas, etc?

[j_silk_h]

Listo. Ya le encontré la solución a esto con POO gracias mis amigazos!

[jonni09lo]

Hola perdón la demora me ocupé a último momento... te hablaba de clases porque puedes hacer perfectamente esto:

Código PHP:

<?php
class db{
    private $host = 'localhost';
    private $db = 'prueba';
    private $user = 'root';
    private $pass = '';
    
    private $conex;
    
    public function __construct(){
        try{
            $this->conex = new PDO('mysql:host='.$this->host.';dbname='.$this->db.'', $this->user, $this->pass);
        }catch(Exception $e){
            echo $e->getMessage();
        }
    }
    
    public function query($sql){
        $this->conex->query($sql);
    }
}
 
//llamamos a la clase de esta manera
 
$db = new db();
$sql = 'bloa bla bla';
$db->query($sql);
 

Y con lo anterior tendrás lo mismo que estabas tratando de hacer pero con POO y PDO ten en cuenta lo siguiente:

1. Tus consultas deben de llevar prepare y execute para evitar inyecciones sql
2. Depura tu código y separa las consultas que se ejecutan solamente y devuelven boleanos (update, delete) o el id del insert (usando lastInsertId de PDO) de las que retornan datos (select)
3. Ten en cuenta de que POO se hizo para reusar el código asi que todo lo que haga exactamente lo mismo lo puedes volver función

Esos son algunos consejos... por último te hable de patrones de diseño para decirte que en los casos de conexiones a la bd es muy util usar SINGLETON

Así que con esto tienes buen material para que practiques y mejores

Saludos

[j_silk_h]

Oka, me hice esto. Tengo un archivo

config.php

Código PHP:

  <?php
$host = "localhost";
$user = "root";
$password = "";
$db = "administrativo";
?>   

Tengo mi archivo php.php donde pienso tener tooodas mis funciones y clases en php
PD. Aun estoy terminando este archivo, es un pequeño proyecto personal sobre un sistema que quiero llevar a cabo porque en verdad me gusta esto y desde hace tiempo estoy buscando pasar a mysqli, POO y PDO, hasta los momentos me funciona, aqui les va

Código PHP:

  <?php
session_start();
date_default_timezone_set("America/Caracas");

//conf principalmente me trae los valores del archivo config.php
class Conf{
    private $_user;
    private $_password;
    private $_host;
    private $_database;
    
    static $_instance;
    
    private function __construct(){
        require("config.php");
        $this->_user=$user;
        $this->_password=$password;
        $this->_host=$host;
        $this->_database=$db;
    }
    
    public static function getInstance(){
        if(!(self::$_instance instanceof self)){
            self::$_instance = new self;
        }
        return self::$_instance;
    }
    
    public function getUserDB(){
        $var = $this->_user;
        return $var;
    }
    
    public function getHostDB(){
        $var = $this->_host;
        return $var;
    }
    
    public function getPassDB(){
        $var = $this->_password;
        return $var;
    }
    
    public function getDB(){
        $var = $this->_database;
        return $var;
    }
}

class Db{
    private $servidor;
    private $usuario;
    private $password;
    private $database;
    private $conex;
    
    static $_instance;
    
    private function setConexion(){
        $conf = Conf::getInstance();
        $this->servidor=$conf->getHostDB();
        $this->database=$conf->getDB();
        $this->usuario=$conf->getUserDB();
        $this->password=$conf->getPassDB();
    }
    
    private function __construct(){
        $this->setConexion();

        //gracias a mi amigo @jonni09lo de www.forosdelweb.com
        try{
            $this->conex = new PDO('mysql:host='.$this->servidor.';dbname='.$this->database.'', $this->usuario, $this->password);
        }catch(Exception $e){
            echo $e->getMessage();
        }
    }
    
    public function ejecutar($sql){
        $this->conex->query($sql);//gracias a mi amigo @jonni09lo de www.forosdelweb.com
    }
    
    public static function getInstance(){
        if(!(self::$_instance instanceof self)){
            self::$_instance = new self();
        }
        return self::$_instance;
    }
    
}

function getUbicacionCliente(){
    
    $ip = $_SERVER["REMOTE_ADDR"];
    $bd = Db::getInstance();//instancia de Db (buena idea para que se vea complejo el codigo no? Lo vi por alli)
    $stmt = $bd->ejecutar("select * from maquinas_ubicaciones");//gracias a mi amigo @jonni09lo de www.forosdelweb.com

}
//ejecutandose (funciones que siempre estaran en ejecucion, como un reloj o elaboracion de respaldos automaticos a ciertas cosas o cualquier cosa que se me ocurra a un tiempo)   

Creo que se ve bien y tambien funciona bien, se me escapa algo? Que dicen?