actualizar diferentes registros ejecutando una consulta a una tabla

midpunto · #1 (**permalink**) 21/06/2011, 21:58

Hola, me he quedado medio trabado con este tema y despues de varias pero que varias horas de busqueda , lectura e infinidad de pruebas recurro al foro a ver si me pueden hechar una mano el tema es el siguiente: tengo que actualizar el campo cantidad de diferentes artículos pero no logro hacer que el bucle funcione, este es el código que tengo:

while(mysql_num_rows>0){
mysql_select_db($database_conecta, $conecta);
$query_artics = "SELECT * FROM ventas_lin WHERE contrato_venlin = 1 ";
$artics = mysql_query($query_artics, $conecta) or die(mysql_error());
$row_artics = mysql_fetch_assoc($artics);
$totalRows_artics = mysql_num_rows($artics);
mysql_select_db($database_conecta, $conecta);
$updateSQL = "UPDATE aticulos set stock_art = stock_art + 1 where Id_art= '".$row_artics['artic_venlin']."'";
mysql_select_db($database_conecta, $conecta);
$Result1 = mysql_query($updateSQL, $conecta) or die(mysql_error());

}
¿Que estoy haciendo mal?

agradezco cualquier ayuda un saludo.

repara2 · #2 (**permalink**) 22/06/2011, 04:05

Hola midpunto, la verdad es que no se entienda nada tu codigo, te paso un version comentada a ver si te ayudo, saludos:

Código PHP:

Ver original<?php
while(mysql_num_rows>0){ //?? es una variablo o una funci´on?
    mysql_select_db($database_conecta, $conecta); //Seleccionas db en cada vuelta del loop??
    $query_artics = "SELECT * FROM ventas_lin WHERE contrato_venlin = 1 "; //Esta select es estatica, siempre es la misma, no tiene sentido que este en un bucle
    $artics = mysql_query($query_artics, $conecta) or die(mysql_error());
    $row_artics = mysql_fetch_assoc($artics);
    $totalRows_artics = mysql_num_rows($artics);
    mysql_select_db($database_conecta, $conecta); //otra conexion para hacer un update?
    $updateSQL = "UPDATE aticulos set stock_art = stock_art + 1 where Id_art= '".$row_artics['artic_venlin']."'";
    mysql_select_db($database_conecta, $conecta);
    $Result1 = mysql_query($updateSQL, $conecta) or die(mysql_error());
 
}
 
 
//<----------------------------
//1. Conectarse a la db...
//2. Seleccionar db, no entiendo por que necesitas dos pasos, pero ok
    if(!mysql_select_db($database_conecta, $conecta))die(mysql_error());
    $query_artics = "SELECT * FROM ventas_lin WHERE contrato_venlin = 1 ";
    if(!$query = mysql_query($query_artics))die(mysql_error()." en linea 18");
    //Mientras haya resultados, actualizar la otra tabla
    while($f = mysql_fetch_assoc($query))
    {
    $updateSQL = "UPDATE aticulos set stock_art = stock_art + 1 where Id_art= '".$row_artics['artic_venlin']."'";
    if(!$query2 = mysql_query($updateSQL))die(mysql_error()." en linea 25");
    }
    //Liberar resultados
    mysql_free_result($query);
 
?>

midpunto · #3 (**permalink**) 22/06/2011, 11:48

Gracias por contestar, si a esas horas lo que publique estaba bastante mal, por otra parte he seguido tu código de ejemplo pero no hay forma de hacerlo funcionar de esta forma directamente no me actualiza ninguno de los registros, de esta otra manera me actualiza solamente uno:

Código PHP:

  //conexion y consulta
mysql_select_db($database_conecta, $conecta);
$query_artics = "SELECT * FROM ventas_lin WHERE contrato_venlin = $id_ven ";
$artics = mysql_query($query_artics, $conecta) or die(mysql_error());
$row_artics = mysql_fetch_assoc($artics);
$totalRows_artics = mysql_num_rows($artics);
//bucle para actualizar todos los registros que correspondan a esta id obtenida de la select
while ($row=mysql_fetch_assoc($artics)){ 
mysql_select_db($database_conecta, $conecta);
$updateSQL = "UPDATE aticulos set stock_art = $stock_udp + '".$row_artics['cant_venlin']."' where Id_art= '".$row_artics['artic_venlin']."'";
mysql_select_db($database_conecta, $conecta);
  $Result1 = mysql_query($updateSQL, $conecta) or die(mysql_error());
 }

En que me estoy equivocando? Gracias y saludos.

midpunto · #4 (**permalink**) 22/06/2011, 15:48

Bueno es una lastima la poca colaboración aunque tambien agradecido a "repara2" por brindarme su ayuda, finalmente pude solucionar mi problema y aqui les dejo el código por si a alguien le sirve:

Código PHP:

   //conexion y consulta
$maxRows_artics = 10;
$pageNum_artics = 0;
if (isset($_GET['pageNum_artics'])) {
  $pageNum_artics = $_GET['pageNum_artics'];
}
$startRow_artics = $pageNum_artics * $maxRows_artics;

mysql_select_db($database_conecta, $conecta);
$query_artics = "SELECT * FROM ventas_lin WHERE contrato_venlin = 1 ";
$query_limit_artics = sprintf("%s LIMIT %d, %d", $query_artics, $startRow_artics, $maxRows_artics);
$artics = mysql_query($query_limit_artics, $conecta) or die(mysql_error());
$row_artics = mysql_fetch_assoc($artics);

if (isset($_GET['totalRows_artics'])) {
  $totalRows_artics = $_GET['totalRows_artics'];
} else {
  $all_artics = mysql_query($query_artics);
  $totalRows_artics = mysql_num_rows($all_artics);
}
$totalPages_artics = ceil($totalRows_artics/$maxRows_artics)-1;

echo "<span class=aviso>Actualizando  el stok de los siguientes artículos:</span><BR>
<table border=0 cellspacing=2 width=40%>
  <tr>
    <td class=txtablec><strong>Artículo ID</strong></td>
    <td class=txtablec><strong>Nombre</strong></td>
    <td class=txtablec><strong>Cantidad</strong></td>
    <td class=txtablec><strong>PVP</strong></td>
    <td class=txtablec><strong>Nº de contrato</strong></td>
  </tr>";
   do { 
    echo "<tr>"; 
//bucle para actualizar todos los registros que correspondan a esta id obtenida de la select

mysql_select_db($database_conecta, $conecta);
$updateSQL = "UPDATE aticulos set stock_art = stock_art + '".$row_artics['cant_venlin']."' where id_art = '".$row_artics['artic_venlin']."'";
mysql_select_db($database_conecta, $conecta);
$Result1 = mysql_query($updateSQL, $conecta) or die(mysql_error());
//muestro los artículos que seran actualizados
     echo "<td class=txtablec>".$row_artics['artic_venlin']."</td>";
     echo "<td class=txtablec>".$row_artics['nomart_venlin']."</td>";
     echo "<td class=txtablec>".$row_artics['cant_venlin']."</td>";
     echo "<td class=txtablec>".$row_artics['pvp_venlin']."</td>";
     echo "<td class=txtablec>".$row_artics['contrato_venlin']."</td>";
    echo "</tr>";
  } while ($row_artics = mysql_fetch_assoc($artics));
echo "</table>";
echo "<BR><BR><span class=txtablec> stok Actualizando correctamente, anulando contrato...</span>";
}

Si se observa he solucionado con while y do para poder armar el bucle, recorrer , mostrar y actualizar cada uno de los registros que cumplen con la condicion "where" en la select.
Un saludo.

repara2 · #5 (**permalink**) 24/06/2011, 00:20

ok, aun no entiendo porque haces tantos mysql_select_db(), pero si funciona ok, salu2

mayid · #6 (**permalink**) 25/06/2011, 09:54

Cita:

Iniciado por repara2

ok, aun no entiendo porque haces tantos mysql_select_db(), pero si funciona ok, salu2

Lo mismo digo. Generalmente en proyectos normales se selecciona la db tras hacer la coneccion. Y no mas. Acá estas teniendo un error de concepto grosso.

midpunto · #7 (**permalink**) 05/07/2011, 11:47

Cita:

Iniciado por mayid

Lo mismo digo. Generalmente en proyectos normales se selecciona la db tras hacer la coneccion. Y no mas. Acá estas teniendo un error de concepto grosso.

vale... VALE!! tanto como error de concepto no me parece,.. pero bueno esto es solamente una serie de pruebas que realizo y lógicamente el código lo limpio,.. pero creo que aquí lo importante era aportar una solución y es lo que acabo de hacer y funciona perfectamente. Si alguien me puede mostrar un camino distinto o mejor pero con el mismo resultado bienvenido sea.. la cuestión es aprender al menos para mi.
Saludos.

gnzsoloyo · #8 (**permalink**) 05/07/2011, 12:15

Cita:

tanto como error de concepto no me parece

Es un error conceptual y procedimental.
Cada vez que haces eso le estás agregando "ruido" a la aplicación porque le estás enviando, via Internet, una petición a la base de datos para que haga algo que no necesita hacer. En otras palabras, solamente le estás agregando overhead sin ninguna necesidad.
Imaginate que eso lo hicieras en un While, una vez por registro, y tuvieses 5000 registros para procesar. ¿Te imaginas el efecto a largo plazo?

O sea: No hagas cosas que ya hiciste. Es más fácil verificar si la conexión está activa y la base ya seleccionada, porque ambas cosas se resuelven en el PHP.

mayid · #9 (**permalink**) 05/07/2011, 13:55

Cita:

Imaginate que eso lo hicieras en un While

Es que ademas lo estaba haciendo en un while!

Código PHP:

Ver originalwhile(mysql_num_rows>0){ 
 mysql_select_db($database_conecta, $conecta);
 $query_artics = "SELECT * FROM ventas_lin WHERE contrato_venlin = 1 ";
 $artics = mysql_query($query_artics, $conecta) or die(mysql_error());
 $row_artics = mysql_fetch_assoc($artics);
 $totalRows_artics = mysql_num_rows($artics);
 mysql_select_db($database_conecta, $conecta);
 $updateSQL = "UPDATE aticulos set stock_art = stock_art + 1 where Id_art= '".$row_artics['artic_venlin']."'";
 mysql_select_db($database_conecta, $conecta);
 $Result1 = mysql_query($updateSQL, $conecta) or die(mysql_error());
 
 }

victor_aguilar · #10 (**permalink**) 05/07/2011, 16:15

no tienes una variable que le diga al while que avance y cuando terminar el proceso, la condición actual siempre se cumple con un solo registro
Salu2

mayid · #11 (**permalink**) 05/07/2011, 16:21

Cierto. Evalua en true.

midpunto · #12 (**permalink**) 06/07/2011, 02:42

Se ve que cuando me expreso lo hago mal.. el codigo es de "muestra" por si a alguien le sirve, y si, correcto.. hay conexiones de mas eso lo reconozco es solamente quitarlas y listo

eso no significa que no sirva o que este tan equivocado conceptualmente hablando, por otra parte el codigo me actualiza perfectamente los registros que cumplen con la condicion y aparte me los imprime funcionando perfectamente. Si alguien tiene una solución mejor o por otra parte mi solución esta equivocada pues muestrennos a los lectores un código mejor, se agradece. A mi este códgo me funciona y con toda la mejor intención y como no encontre en todo el foro una solución a este tipo de problema lo publico.

gnzsoloyo · #13 (**permalink**) 06/07/2011, 07:21

Bueno, mira, a mi entender esto hace lo mismo que tu quieres hacer, pero con un código más simple y fácil de entender:

Código PHP:

Ver originalmysql_select_db($database_conecta, $conecta);
$query_artics = "SELECT * FROM ventas_lin WHERE contrato_venlin = 1";
$artics = mysql_query($query_artics, $conecta) or die(mysql_error());
if($artics)
    {
    while ($row_artics = mysql_fetch_assoc($artics)) 
        {
        $updateSQL = "UPDATE aticulos set stock_art = stock_art + 1 where Id_art= '" . $row_artics['artic_venlin'] . "'";
        $result1 = mysql_query($updateSQL, $conecta) or die(mysql_error());
        }
    }

Pero por otro lado, aprovechando los recursos de ambos lenguajes (PHP y SQL), esto otro hace exactamente lo mismo pero en una sola operación:

Código PHP:

Ver originalmysql_select_db($database_conecta, $conecta);
$query_artics = "UPDATE aticulos SET stock_art = stock_art + 1 WHERE Id_art IN(SELECT artic_venlin FROM ventas_lin WHERE contrato_venlin = 1)";
$artics = mysql_query($query_artics, $conecta) or die(mysql_error());

midpunto · #14 (**permalink**) 06/07/2011, 12:48

Hola y gracias por contestar, muy buena tu respuesta y la verdad es que lo haces algo mas simple, lo probare. De momento mi codigo"limpio" completo ejecutando la consulta, actualizando e imprimiendo los resultados, es este:

Código PHP:

  //conexion y consulta

mysql_select_db($database_conecta, $conecta);

$query_artics = "SELECT * FROM ventas_lin WHERE contrato_venlin = 1 ";
$artics = mysql_query($query_limit_artics, $conecta) or die(mysql_error());
$row_artics = mysql_fetch_assoc($artics);


echo "<span class=aviso>Actualizando  el stok de los siguientes artículos:</span><BR>
<table border=0 cellspacing=2 width=40%>
  <tr>
    <td class=txtablec><strong>Artículo ID</strong></td>
    <td class=txtablec><strong>Nombre</strong></td>
    <td class=txtablec><strong>Cantidad</strong></td>
    <td class=txtablec><strong>PVP</strong></td>
    <td class=txtablec><strong>Nº de contrato</strong></td>
  </tr>";
   do { 
    echo "<tr>"; 
//bucle para actualizar todos los registros que correspondan a esta id obtenida de la select

$updateSQL = "UPDATE aticulos set stock_art = stock_art + '".$row_artics['cant_venlin']."' where id_art = '".$row_artics['artic_venlin']."'";
mysql_select_db($database_conecta, $conecta);
$Result1 = mysql_query($updateSQL, $conecta) or die(mysql_error());
//muestro los artículos que seran actualizados
     echo "<td class=txtablec>".$row_artics['artic_venlin']."</td>";
     echo "<td class=txtablec>".$row_artics['nomart_venlin']."</td>";
     echo "<td class=txtablec>".$row_artics['cant_venlin']."</td>";
     echo "<td class=txtablec>".$row_artics['pvp_venlin']."</td>";
     echo "<td class=txtablec>".$row_artics['contrato_venlin']."</td>";
    echo "</tr>";
  } while ($row_artics = mysql_fetch_assoc($artics));
echo "</table>";
echo "<BR><BR><span class=txtablec> stok Actualizando correctamente, anulando contrato...</span>";
}

Saludos y gracias nuevamente por la respuesta, para mi esta todo más que solucionado.