Mejorar este código INSERT con PHP

[alyciashape]

Desde que he añadido este fragmento he aumentado el tiempo de ejecución del script en unos 20-30 segundos. Muchísimo. Aunque es un proceso automático que nadie va a estar esperando el resultado.

Código PHP:

  $conexion = odbc_connect ( "origen_de_datos", "", "") or die(exit("Imposible conectar")); 
$sql = "SELECT codigo, exist5, pvp6 FROM articulo WHERE (exist5 IS NOT NULL) AND (exist5 >0)"; 
$resultado=odbc_exec($conexion,$sql)or die(exit("Error en odbc_exec"));
 while ($fila = odbc_fetch_object($resultado))
 {             
    $nombreBD = "nombre_bd";
    $consulta = "UPDATE producto SET pvp6 = '$fila->pvp6', 
        exisTotal = '$fila->exist5' WHERE producto.codigo = '$fila->codigo'";                    
    include("conexion.php"); 
 } 
   

Pero claro el problema es que se realiza una consulta por cada registro. Lo podría hacer con un sólo comando UPDATE usando la variable $resultado ?

[60NZ4]

no se que hace el include("conexion.php"); al final, se pone al principio

prueba con esto y luego nos cuentas

Código PHP:

include("conexion.php"); 
 
$sql = "SELECT codigo, exist5, pvp6 FROM articulo WHERE (exist5 IS NOT NULL) AND (exist5 >0)"; 
$resultado=odbc_exec($conexion,$sql)or die(exit("Error en odbc_exec"));
 while ($fila = odbc_fetch_object($resultado))
 {             
    //si no siempre se actualiza podrias meter un if aqui 
    $nombreBD = "nombre_bd";
    $consulta = "UPDATE producto SET pvp6 = '.$fila->pvp6.',   // realmente te funciona sin escribir '..$fila->pvp6'  ??
        exisTotal = '$fila->exist5' WHERE producto.codigo = '.$fila->codigo.'";                    // aqui lo mismo
    
 }
 

la variable resultado no la podrias usar deberias hacerlo asi :

Código PHP:

include("conexion.php"); 
 
$sql = "SELECT codigo, exist5, pvp6 FROM articulo WHERE (exist5 IS NOT NULL) AND (exist5 >0)"; 
$resultado=odbc_exec($conexion,$sql)or die(exit("Error en odbc_exec"));
$fila = odbc_fetch_object($resultado);
 
//usas $fila->campo
 

suerte

[alyciashape]

Muchas gracias, lo voy a probar.

El tema de conexion.php es porque tiene este aspecto:

Código PHP:

      $servidor = 'server';
    $usuario = 'user';
    $clave = 'password';
    $conn = mysql_connect($servidor, $usuario, $clave) or die( mysql_error("No se pudo conectar"));
    mysql_select_db( $nombreBD, $conn ) or die( mysql_error( $conn ) );
    $resultado = mysql_query( $consulta, $conn ) or die( mysql_error( $conn ));
    if (isset($cerrarConexion))
    {
        mysql_close();
        $unset ($cerrarConexion); 
    } 
   

Y así ya me da la variable $resultado aunque supongo que puede ser un poco confuso sobretodo si otro mira el código. Creo que lo cambiaré. Es que estoy intentando quitar todas las líneas que pueda del código porque es larguísimo.

Bueno metiendo el if para evitar ciertos campos que no vengan nulos ha bajado un poco el tiempo pero supongo que es lo que hay.

[60NZ4]

Lo siento no se me ocurre nada mas jeje

mira aqui te dejo mi conexion.php

me parece la manera mas eficiente:

Código PHP:

class DBConnection{
    function getConnection(){
     
        mysql_connect("localhost","user","pass") or
         die("Could not connect: " . mysql_error());
         
     //change to your database name
        mysql_select_db("basededatos") or 
             die("Could not select database: " . mysql_error());
    }
    
}
 
 
//para llamarlo en un archivo es :
 
$db = new DBConnection();
$db->getConnection();
 
// y la ventaja es que para hacer una consulta o un insert o lo que sea a la BD
// no necesitas volver a acordarte de la consexion, solo con esto funciona
 
$consulta = "SELECT * FROM alumnos";
$result = mysql_query($consulta);
 

espero que te sirva


un saludo