(Consulta) Archivo de conexión en una función

[reborn]

Buenas...

Estoy haciendo una funcion para insertar registros en la bd pero no me toma el archivo de conexion, solo me lo toma cuando lo pongo dentro de la funcion.

Tengo el archivo conexion y el archivo con la funcion por separados y habia hecho lo siguiente:

Código PHP:

require_once('conexion.php');
require_once('funciones.php');
 
//****aca demas codigo y prueba de la funcion
 
$probar = funcion('1','2'...);
 

Despues probe poniendo el archivo de conexion dentro del archivo funciones.php:

Código PHP:

require_once('conexion.php');
 
function ($1,$2..) {
 
   //intrucciones y todo eso...
 
}
 

...y tampoco anduvo, pero si le pongo el archivo de conexion dentro de la funcion si funciona, pero no se si es correcto eso, no me convence:

Código PHP:

function ($1,$2..) {
    require_once('conexion.php');
   //intrucciones y todo eso...
 
}
 

Aclaro q la funcion funciona bien, el tema es el archivo de conexion.
Me parece raro porq he visto ejemplos y ninguno incluye la conexion dentro de la funcion.

Gracias.
Saludos.

[IEKK]

La conexión se puede separar de la función, y de hecho tenerla dentro es un poco un sin sentido.

¿Qué ocurre cuando lo dejas fuera? ¿Te da algún error? (En ese caso dinos cual)
¿No coge la conexión? ¿No entra en la función?

¿Como tienes hecha la conexión?
Sin más datos es difícil saber que puede estar pasando.

Saludos

[reborn]

Claro, no toma la conexion, no hace el insert.
Cuando coloco la conexion dentro de la funcion si hace el insert lo mas bien.
No tira ningun error de nada, simplemente no toma la conexion.
La funcion es comun, hice una rapida para probar:

De esta forma NO funciona

Código PHP:

require_once('conexion.php');
 
function insertDatos($dato1,$dato2,$dato3){
 
$insert = mysql_query("INSERT INTO tabla VALUES ($dato1,$dato2,$dato3)", $link)
             or die(mysql_error());
 
return $insert;
 
}
 
 
$insertar = insertDatos('dato1', 'dato2', 'dato3');
 

De esta forma SI funciona

Código PHP:

function insertDatos($dato1,$dato2,$dato3){
 
require_once('conexion.php');
$insert = mysql_query("INSERT INTO tabla VALUES ($dato1,$dato2,$dato3)", $link)
             or die(mysql_error());
 
return $insert;
 
}
 
 
$insertar = insertDatos('dato1', 'dato2', 'dato3');
 

[reborn]

Este es el archivo de conexion.php:

Código PHP:

$link = mysql_connect('localhost', 'user', 'pass');
if(!$link){
die('No se pudo conectar: ' . mysql_error());
}
 
$db_selected = mysql_select_db('database', $link);
if(!$db_selected){
die('No se selecciono la base de datos: ' . mysql_error());
 
}
 

[IEKK]

Claro es porque no te está cogiendo la variable $link.
Supongo que la gente mete en las consultas la conexión porque habá algún maldito tutorial que lo pone así xD (Se suele ver mucho)
Pero es totalmente innecesario.

Edito y te pongo un ejemplo.


conexion.php

Código PHP:

  $conexion = mysql_connect("***","***","***") or die ( "No se ha podido establecer la conexión con el servidor"); 
mysql_select_db("nombreBD",$conexion);  
   

Tú pagina:

Código PHP:

  require_once('conexion.php');

function insertDatos($dato1,$dato2,$dato3){

$insert = mysql_query("INSERT INTO tabla VALUES ($dato1,$dato2,$dato3)")
             or die(mysql_error());
 
return $insert;
 
}

$insertar = insertDatos('dato1', 'dato2', 'dato3');  
   

Fíjate que no se le añade un segundo parámentro al mysql_query, sólo la consulta.
De todas formas yo prefiero hacerlo con una clase y con php en modo objeto y limpiando memoria y cerrando la conexión, pero eso debería servirte.

[reborn]

No lo sabia.
Al consultar si habia hecho bien ese archivo de conexion me dijeron q si.
Por q no es necesario incluir de esa manera la conexion en las consultas?, porq la verdad q me saca de esquema.

[IEKK]

Iniciado por reborn

No lo sabia.
Al consultar si habia hecho bien ese archivo de conexion me dijeron q si.
Por q no es necesario incluir de esa manera la conexion en las consultas?, porq la verdad q me saca de esquema.

No es que esté mal hecha tu conexión. El problema venía por $link.
Ponías el include de la conexión fuera de la función y cuando entraba en la función no te cogía el valor de esa variable.
Por eso cuando el include estaba dentro si cogía el valor.

De la otra forma que te puse ahí tienes la conexión abierta y no tienes necesidad de ponerlo como segundo parámentro en el mysql_query. De hecho no le veo el sentido de ponerlo dentro del query si total están dejando la conexión abierta y sin limpiar memoria... Para eso es más práctico abrir la conexión como te puse y listo xD.

Siendo un maníatico lo más correcto sería tener una función que estableciera una conexión y otra para las consultas la cual limpiara la memoria de las consultas y cerrara la conexión.