Skip to main content
Una amplia modificación del código para sugerir formas correctas de usar las funciones mysqli
Origen Enlace
OscarGarcia
  • 28.5k
  • 3
  • 29
  • 64

Prueba con estaEditado para mostrarte una forma de conectarterealizar el trabajo sin hacer uso de variables globales.

Crea un archivo que se llame funciones.bd.php con este contenido:

<?php
function/**
 conectar()* {Establece la conexión con la base de datos y finaliza la ejecución
 * globalen $conexion;caso de error.
 *
 * @return resource Recurso asociado a la conexión MySQLi
 */
function conectar() {
  $conexion = mysqli_connectnew mysqli('localhost', 'root', '', 'vasco');
  /* Presuponemos PHP >= 5.3 si usas fetch_all */
  if ($conexion->connect_error ===!== falsenull) {
    die(mysqli_connect_error()$conexion->connect_error);
  }
  if (mysqli_set_charset($conexion, ->set_charset('utf8') === false) {
    die(mysqli_error($conexion)->error);
  }
  return $conexion;
}

/**
 * Cierre la conexión MySQLi y devuelve el resultado.
 *
 * @param resource $conexion Recurso de conexión MySQLi
 *
 * @return boolean Estado del cierre de la conexión
 */
function desconectar($conexion) {
  /* Forma correcta de cerrar la conexión */
  return $conexion->close();
}

Y esta forma correcta de realizar la consultausar el archivo anterior:

/* Incluimos una única vez la definición de funciones de MySQLi */
require_once 'funciones.bd.php';

/* Forma de realizar el trabajo */
$conexion = conectar();
/* Obtenemos los periodos introduciendo como parámetro la conexión MySQLi */
$periodos = muestraPeriodos($conexion);
/* A partir de aquí podemos trabajar con los datos de periodos */
/* ... */
/* Tras terminar el trabajo con la base de datos cerramos la conexión */
desconectar($conexion);


/**
 * Obtiene todos los registros de periodos.
 *
 * @param resource $conexion Recurso de conexión MySQLi
 *
 * @return array Matriz asociativa con todos los registros
 */
function muestraPeriodos($conexion) {
  global/* $conexion;Realizamos la consulta SQL */
  $respuesta = mysqli_query($conexion, ->query("SELECT * FROM periodos");
  /* En caso de error terminamos (tabla no existente, por ejemplo) */
  if ($respuesta === false) {
    die(mysqli_error($conexion)->error);
  }
  return/* Obtenemos los datos y liberamos la memoria asociada al resultado */
  $resultado = $respuesta->fetch_all(MYSQLI_ASSOC);
  $respuesta->free();
  /* Una vez liberados los recursos podemos devolver el resultado */
  return $resultado;
}

De esta manera sólo se incluirá una única vez la definición de las funciones de conexión y desconexión a tu base de datos. También sabrás qué ha ocurrido con la consulta y la ejecución de tu aplicación tampoco continuará en caso de error.

Por cierto, cuidado con esaHe eliminado la necesidad de mantener una variable global llamada $conexion. Al ser global, cualquier modificación que hagas sobre ellapero como puedes ver en cualquier parte de tu código podría afectar su valor yel ejemplo, tendrás que dejarahacer las llamadas a tus funciones enviando como parámetro el recurso de funcionar tu aplicaciónla conexión.

Espero que esto solucione tus problemas.

Prueba con esta forma de conectarte:

<?php
function conectar() {
  global $conexion; 
  $conexion = mysqli_connect('localhost', 'root', '', 'vasco');
  if ($conexion === false) {
    die(mysqli_connect_error());
  }
  if (mysqli_set_charset($conexion, 'utf8') === false) {
    die(mysqli_error($conexion));
  }
  return $conexion;
}

Y esta forma de realizar la consulta:

function muestraPeriodos() {
  global $conexion;
  $respuesta = mysqli_query($conexion, "SELECT * FROM periodos");
  if ($respuesta === false) {
    die(mysqli_error($conexion));
  }
  return $respuesta->fetch_all();
}

De esta manera sabrás qué ha ocurrido con la consulta y la ejecución de tu aplicación tampoco continuará en caso de error.

Por cierto, cuidado con esa variable global llamada $conexion. Al ser global, cualquier modificación que hagas sobre ella en cualquier parte de tu código podría afectar su valor y que dejara de funcionar tu aplicación.

Editado para mostrarte una forma de realizar el trabajo sin hacer uso de variables globales.

Crea un archivo que se llame funciones.bd.php con este contenido:

<?php
/**
 * Establece la conexión con la base de datos y finaliza la ejecución
 * en caso de error.
 *
 * @return resource Recurso asociado a la conexión MySQLi
 */
function conectar() {
  $conexion = new mysqli('localhost', 'root', '', 'vasco');
  /* Presuponemos PHP >= 5.3 si usas fetch_all */
  if ($conexion->connect_error !== null) {
    die($conexion->connect_error);
  }
  if ($conexion->set_charset('utf8') === false) {
    die($conexion->error);
  }
  return $conexion;
}

/**
 * Cierre la conexión MySQLi y devuelve el resultado.
 *
 * @param resource $conexion Recurso de conexión MySQLi
 *
 * @return boolean Estado del cierre de la conexión
 */
function desconectar($conexion) {
  /* Forma correcta de cerrar la conexión */
  return $conexion->close();
}

Y esta forma correcta de usar el archivo anterior:

/* Incluimos una única vez la definición de funciones de MySQLi */
require_once 'funciones.bd.php';

/* Forma de realizar el trabajo */
$conexion = conectar();
/* Obtenemos los periodos introduciendo como parámetro la conexión MySQLi */
$periodos = muestraPeriodos($conexion);
/* A partir de aquí podemos trabajar con los datos de periodos */
/* ... */
/* Tras terminar el trabajo con la base de datos cerramos la conexión */
desconectar($conexion);


/**
 * Obtiene todos los registros de periodos.
 *
 * @param resource $conexion Recurso de conexión MySQLi
 *
 * @return array Matriz asociativa con todos los registros
 */
function muestraPeriodos($conexion) {
  /* Realizamos la consulta SQL */
  $respuesta = $conexion->query("SELECT * FROM periodos");
  /* En caso de error terminamos (tabla no existente, por ejemplo) */
  if ($respuesta === false) {
    die($conexion->error);
  }
  /* Obtenemos los datos y liberamos la memoria asociada al resultado */
  $resultado = $respuesta->fetch_all(MYSQLI_ASSOC);
  $respuesta->free();
  /* Una vez liberados los recursos podemos devolver el resultado */
  return $resultado;
}

De esta manera sólo se incluirá una única vez la definición de las funciones de conexión y desconexión a tu base de datos. También sabrás qué ha ocurrido con la consulta y la ejecución de tu aplicación tampoco continuará en caso de error.

He eliminado la necesidad de mantener una variable global llamada $conexion, pero como puedes ver en el ejemplo, tendrás que hacer las llamadas a tus funciones enviando como parámetro el recurso de la conexión.

Espero que esto solucione tus problemas.

Ampliada explicación de la comprobación de error en la consulta SQL
Origen Enlace
OscarGarcia
  • 28.5k
  • 3
  • 29
  • 64

Tienes dos errores dependiendo de qué forma de conectarte uses.

El primer error apunta a que no se establece correctamente la conexión a la base de datos por algún motivo y por falta de código de detección de errores no eres consciente de ello, dejando en un valor false la variable $conexion o bien la consulta no se realiza por algún motivo y tampoco eres consciente de ello.

El segundo (Fatal error: Cannot redeclare conectar()) está provocado por definir en dos archivos la función conectar(). Revisa bien tus archivos para no repetir la definición de esa función en varios puntos.

Prueba con esta forma de conectarte:

<?php
function conectar() {
  global $conexion; 
  $conexion = mysqli_connect('localhost', 'root', '', 'vasco');
  if ($conexion === false) {
    die(mysqli_connect_error());
  }
  if (mysqli_set_charset($conexion, 'utf8') === false) {
    die(mysqli_error($conexion));
  }
  return $conexion;
}

De esta manera, si tienes problemas de conexión, te aparecerán por pantalla y detendrán la ejecución del resto de tu aplicación.

Y esta forma de realizar la consulta:

function muestraPeriodos() {
  global $conexion;
  $respuesta = mysqli_query($conexion, "SELECT * FROM periodos");
  if ($respuesta === false) {
    die(mysqli_error($conexion));
  }
  return $respuesta->fetch_all();
}

De esta manera sabrás qué ha ocurrido con la consulta y la ejecución de tu aplicación tampoco continuará en caso de error.

Por cierto, cuidado con esa variable global llamada $conexion. Al ser global, cualquier modificación que hagas sobre ella en cualquier parte de tu código podría afectar su valor y que dejara de funcionar tu aplicación.

Tienes dos errores dependiendo de qué forma de conectarte uses.

El primer error apunta a que no se establece correctamente la conexión a la base de datos por algún motivo y por falta de código de detección de errores no eres consciente de ello, dejando en un valor false la variable $conexion.

El segundo (Fatal error: Cannot redeclare conectar()) está provocado por definir en dos archivos la función conectar(). Revisa bien tus archivos para no repetir la definición de esa función en varios puntos.

Prueba con esta forma de conectarte:

<?php
function conectar() {
  global $conexion; 
  $conexion = mysqli_connect('localhost', 'root', '', 'vasco');
  if ($conexion === false) {
    die(mysqli_connect_error());
  }
  if (mysqli_set_charset($conexion, 'utf8') === false) {
    die(mysqli_error($conexion));
  }
  return $conexion;
}

De esta manera, si tienes problemas de conexión, te aparecerán por pantalla y detendrán la ejecución del resto de tu aplicación.

function muestraPeriodos() {
  global $conexion;
  $respuesta = mysqli_query($conexion, "SELECT * FROM periodos");
  if ($respuesta === false) {
    die(mysqli_error($conexion));
  }
  return $respuesta->fetch_all();
}

Por cierto, cuidado con esa variable global llamada $conexion. Al ser global, cualquier modificación que hagas sobre ella en cualquier parte de tu código podría afectar su valor y que dejara de funcionar tu aplicación.

Tienes dos errores dependiendo de qué forma de conectarte uses.

El primer error apunta a que no se establece correctamente la conexión a la base de datos por algún motivo y por falta de código de detección de errores no eres consciente de ello, dejando en un valor false la variable $conexion o bien la consulta no se realiza por algún motivo y tampoco eres consciente de ello.

El segundo (Fatal error: Cannot redeclare conectar()) está provocado por definir en dos archivos la función conectar(). Revisa bien tus archivos para no repetir la definición de esa función en varios puntos.

Prueba con esta forma de conectarte:

<?php
function conectar() {
  global $conexion; 
  $conexion = mysqli_connect('localhost', 'root', '', 'vasco');
  if ($conexion === false) {
    die(mysqli_connect_error());
  }
  if (mysqli_set_charset($conexion, 'utf8') === false) {
    die(mysqli_error($conexion));
  }
  return $conexion;
}

De esta manera, si tienes problemas de conexión, te aparecerán por pantalla y detendrán la ejecución del resto de tu aplicación.

Y esta forma de realizar la consulta:

function muestraPeriodos() {
  global $conexion;
  $respuesta = mysqli_query($conexion, "SELECT * FROM periodos");
  if ($respuesta === false) {
    die(mysqli_error($conexion));
  }
  return $respuesta->fetch_all();
}

De esta manera sabrás qué ha ocurrido con la consulta y la ejecución de tu aplicación tampoco continuará en caso de error.

Por cierto, cuidado con esa variable global llamada $conexion. Al ser global, cualquier modificación que hagas sobre ella en cualquier parte de tu código podría afectar su valor y que dejara de funcionar tu aplicación.

Origen Enlace
OscarGarcia
  • 28.5k
  • 3
  • 29
  • 64

Tienes dos errores dependiendo de qué forma de conectarte uses.

El primer error apunta a que no se establece correctamente la conexión a la base de datos por algún motivo y por falta de código de detección de errores no eres consciente de ello, dejando en un valor false la variable $conexion.

El segundo (Fatal error: Cannot redeclare conectar()) está provocado por definir en dos archivos la función conectar(). Revisa bien tus archivos para no repetir la definición de esa función en varios puntos.

Prueba con esta forma de conectarte:

<?php
function conectar() {
  global $conexion; 
  $conexion = mysqli_connect('localhost', 'root', '', 'vasco');
  if ($conexion === false) {
    die(mysqli_connect_error());
  }
  if (mysqli_set_charset($conexion, 'utf8') === false) {
    die(mysqli_error($conexion));
  }
  return $conexion;
}

De esta manera, si tienes problemas de conexión, te aparecerán por pantalla y detendrán la ejecución del resto de tu aplicación.

function muestraPeriodos() {
  global $conexion;
  $respuesta = mysqli_query($conexion, "SELECT * FROM periodos");
  if ($respuesta === false) {
    die(mysqli_error($conexion));
  }
  return $respuesta->fetch_all();
}

Por cierto, cuidado con esa variable global llamada $conexion. Al ser global, cualquier modificación que hagas sobre ella en cualquier parte de tu código podría afectar su valor y que dejara de funcionar tu aplicación.