[SOLUCIONADO] Update Mysql en PHP

[AnGuisi]

Tengo el siguiente formulario y el script que procesa tal formulario.

Formulario:

Código HTML:

<table width="100%">
 
<form id="busq" name="busq" action="controlador_modificar_hardware.php" method="POST">
 
    <tr>
        <td>HARDWARE</td>
        <td>MARCA</td>
        <td>MODELO</td>
        <td>TIPO</td>
         <td>SERIAL</td>
        <td>OBSERVACIONES</td>
        
      </tr>
 
      <?
      
foreach ($row as $datos) {
       ?>
       
 
      <tr>
 
 
      
    
        
     <td><input type="text" name="tipo[]" value="<? echo $datos['tipo_h']; ?>" disabled="disabled" /> </td>
<td><input type="text" class="form" name="marca[]" value="<? echo $datos['marca']; ?>" disabled="disabled" /></td>
        
        <td><input type="text" class="form" name="modelo[]" value="<? echo $datos['modelo']; ?>" disabled="disabled" /></td>
        <td><input type="text" class="form" name="tipo_nombre[]" value="<? echo $datos['tipo_nombre']; ?>" disabled="disabled" /></td>
        <td><input type="text" class="form" name="serial[]" value="<? echo $datos['serial']; ?>" disabled="disabled" /></td>
        <td><input type="text" class="form" name="observaciones_h[]" value="<? echo $datos['observaciones_h']; ?>" disabled="disabled" /></td>
        <input type="text" class="form" name="id_hardware[]" value="<? echo $datos['id_hardware']; ?>" disabled="disabled" />               
        
   </tr>
   
      
 
      <? 
      
       } 
       ?>
  
    
       <input type="text" id="codigo" class="form" name="codigo" value="<? echo $datos['codigo']; ?>" disabled="disabled" /> 
  
 </form>  
    </table>
  </div>
 </br>
     <div align="center"><button id="activar" type="button" style="display:block;" onclick="mostrar_ocultar3('activar','modificar')">Modificar Registro</button></div>
     </br>
     
     
     <div align="center"><input type="submit" id="modificar" name="modificar" style="display:none;" value="Incluir Modificacion" /></div>
     
     </br></br>
 

Function:

Código PHP:

public function editar($datos = array()) {
         
          
          while (true) {
          
          $tipo_nombre = current($tipo_nombre);
          $serial = current($serial);
          $observaciones_h = current($observaciones_h);
          
 
               
          if ($codigo != '') {
              $query = "UPDATE `soptecvnzla`.`caracteristicas_hardware` SET serial = '$serial', observaciones_h = '$observaciones_h', tipo_nombre = '$tipo_nombre' WHERE codigo = '$codigo' AND id_hardware = '$id_hardware'";
              
      
          $tipo_nombre = current($tipo_nombre);
          $serial = current($serial);
          $observaciones_h = current($observaciones_h);
          $id_hardware = current($id_hardware);
              
        if($tipo_nombre === false && $serial === false && $observaciones_h === false && $id_hardware === false) 
        break; 
              
              $mysqli = $this->abrir_conexion();
              $result = $mysqli->query($query);
          }
           
              if (!$result) {
                echo "error".$mysqli->error;
            }
            return $result;
          }
 

El problema esta en que no me modifica, me causa error. Como pueden observar intento hacer un ciclo de UPDATE mientras reciba los valores de los arrays en el formulario.

Creo que el error esta tambien en la funcion extract puesto que esta recibe otros arrays. Agradezco de antemano cualquier ayuda.

[pateketrueke]

Todos tus campos están disabled, por esa razón no se enviarán jamás.

Si tu intención es evitar que los modifiquen debes usar el atributo readonly y no disabled.

Además no se ve por ningún lugar como accedes a las variables de $_POST como debería ser, consulta el manual: http://www.php.net/variables.external

También toma en cuenta que tus comparaciones estrictas tampoco funcionarán si no las conviertes apropiadamente, la razón es simple: todo valor recibido por formulario o URL (método GET) es de tipo string, y ningún otro tipo más.

[AnGuisi]

Las variables las recibo en otro archivo y las paso en forma de array que extraigo con extract Mientras los campos se activan por un boton y una funcion javascript; el problema esta en lo que engloba el while

[Erick_MD9]

hola, es la primera vez que veo que alguien intenta recorrer un array con current(); desde mi punto de vista seris mas fácil utilizar foreach().

No veo en donde avanzas el puntero para trabajar con el siguiente elemento del array. Asi como lo tienes unicamente trabajas con un elemento; encerio veo medio retorcida la lógica.

Mi punto de vista perdonsal, a lo mejor me equivoco.
Saludos.

[pateketrueke]

Iniciado por AnGuisi

Las variables las recibo en otro archivo y las paso en forma de array que extraigo con extract Mientras los campos se activan por un boton y una funcion javascript; el problema esta en lo que engloba el while

¿Y cómo esperabas que adivináramos todo eso que dices?

Lo que es no saberse explicar, mal ahí.

Por otro lado, el problema, tal vez, sea el uso de current(), seguramente la estás usando de una forma (que no vemos si acaso haces algo más y no lo dices) correcta:

Código PHP:

// suponiendo que tienes un array de datos
$transport = array('foot', 'bike', 'car', 'plane');
 
// MAL (así lo haces)
$foot = current($foot); // Warning: ...
$bike = current($bike); // Warning: ...
 
// BIEN
$mode = current($transport); // $mode = 'foot';
 

¿Se entiende?

Edito: definitivamente un foreach() es la mejor solución si $data es un array asociativo o un objeto.

[AnGuisi]

Si pues he fallado en no saber explicarme.
En el name de los campos del formulario como pueden ver esta por ejemplo: name="marca[]" puesto que recibire mas de una marca por lo que lo trato como arreglo.

Esta es la parte del script donde recibo estos datos:

Código PHP:

$datos = array();
    
    foreach ($_POST as $clave => $valor) {
        $datos[$clave]=$valor;
                                         }
 

Puesto a que recibo muchas marca ejemplo:
marca[0], marca[1]...

Luego el array $datos lo paso como parametro al script anteriormente señalado. Luego la idea de recorrerlo de esa forma la tomo de este script posteado por el compañero Gatorv

Código PHP:

Ver original
<?php
$items1 = array( 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15 );
$items2 = array( 2, 4, 6 );
$items3 = array( 10, 20, 30, 40 );
$items4 = array( 100, 200, 300, 400, 500, 600, 700 );
 
echo "<table border=1><tr>";
while(true) {
    // Fetch Values
    $item1 = current($items1);
    $item2 = current($items2);
    $item3 = current($items3);
    $item4 = current($items4);
    
    // Print Values
    echo "<td>" . (( $item1 !== false) ? $item1 : "&nbsp;") . "</td>";
    echo "<td>" . (( $item2 !== false) ? $item2 : "&nbsp;") . "</td>";
    echo "<td>" . (( $item3 !== false) ? $item3 : "&nbsp;") . "</td>";
    echo "<td>" . (( $item4 !== false) ? $item4 : "&nbsp;") . "</td>";
    
    // Up! Next Value
    $item1 = next( $items1 );
    $item2 = next( $items2 );
    $item3 = next( $items3 );
    $item4 = next( $items4 );
    
    // Check terminator
    if($item1 === false && $item2 === false && $item3 === false && $item4 === false) break;
    echo "</tr><tr>";
}
echo "</tr></table>";
 
echo sprintf("Done! Max array size is %s.", get_max_count($items1, $items2, $items3, $items4));
 
function get_max_count() {
    $nMax = 0;
    foreach(func_get_args() as $param ) {
        if(is_array($param)) {
            $nSize = count($param);
            if($nSize > $nMax) {
                $nMax = $nSize;
            }
        }
    }
    
    return $nMax;
}
 

El cual señala como recorrer varios arrays, puesto a que los formularios en mi caso son arrays como bien señado en el name.

Creo que ahora si podrian ayudarme mas en que es lo que esta fallando en mi script.

[pateketrueke]

Bien, al menos tienes una base clara, sólo que con un detalle mínimo.

Código PHP:

// GatorV
$item1 = current($items1);
 
// tú
$tipo_nombre = current($tipo_nombre);
 

¿Notas la diferencias?

(y no, no es el nombre de la variable)

En el ejemplo inicial se usa $items1 (plural) e $item1 (singular) para denotar que se extrae el elemento actual de la colección anterior.

Por otro lado tampoco estás avanzando el cursor con next() como ya te han dicho.

No te queda mas que leer bien y repasar el ejercicio.

[Erick_MD9]

Hola, te dejo un pequeño ejemplo de como lo podrias realizar.
No esta completo, es para que lo tu lo complementes.

NO USAR extract(). Hay una enorme advertencia en el manual. Revisalo.

Código PHP:

//a la funcion le envias tu conjunto de variables recibidas en forma de array.
    public function editar($id_s,$marca_s,$tipo_s) {
        foreach($id_s as $key => $id){
            //utilizamos la %key de $id, ya que es una fila y al generarse se otorga la mima $key a los elementos de la fila, "para el caso específico que planteas".
            $marca=$marca_s[$key];
            $tipo=$tipo_s[$key];
            //los datos deben ser escapados antes de introducirse a la sentencia sql, para evitar inyecciones. 
            //Por lo menos mysql_real_scape_string()
            
            $query = "UPDATE `soptecvnzla`.`caracteristicas_hardware` 
            SET serial = '$serial', 
            observaciones_h = '$observaciones_h', 
            tipo_nombre = '$tipo_nombre' 
            WHERE codigo = '$codigo' 
            AND id_hardware = '$id_hardware'";
            
            //ejecucion de consulta
        }
    }
 

Saludos

[AnGuisi]

Pff que poco detallista he sido.
Ya he solucionado ese detalle y me reconoce todos los valores de los arrays.
Ahora solo me hace UPDATE al primer conjunto de valores, he probado el next() y funciona perfectamente. El problema creo que esta en la ejecucion de la sentencia sql. Solo se ejecuta una vez. Creo que es cuestión de colocacion del mysqli->query o quizas deba utilizar mysqli->multi_query? Aunque lo probé e igual no me ha funcionado.


Muchas gracias amigo Erick_MD9 muy bueno lo que me planteas y mas sencillo, aunque ya mi script funciono y pues tomare en cuenta lo que dices.
Pero como digo el problema ahora lo tengo a la hora de la ejecucion de la sentencia. Pues esta al tomar varios valores tambien debe ejecutarse varias veces.

[GatorV]

El Script que diseñe, es para listar arrays que no tienen la misma longitud, en una tabla, es solo un concepto, para lo que quieres hacer, no lo deberías de utilizar ya que tus tablas son lineales, para eso sí debes de ciclar cada campo con un valor común, lo recomendable es hacer algo así:

Código PHP:

// suponiendo que tienes múltiples arrays con datos $codigos, $nombres, $seriales, etc
$query = "UPDATE foo SET `nombre`=?, `serial`=? WHERE `codigo`=?";
$stmt = $link->prepare($query);
$stmt->bind_param('sss', $nombre, $serial, $codigo);
 
foreach ($codigos as $n => $codigo) {
      $nombre = $nombres[$n];
      $serial = $seriales[$n];
 
      $stmt->execute();
}
$stmt->close();
 

Saludos.

[AnGuisi]

Lo hice de esta forma y el Update me sigue dando error.


He aquí el código.

Código PHP:

public function editar($datos = array()) {
         
         
          extract($datos);
             
          if ($codigo != '') {
              $query = "UPDATE `soptecvnzla`.`caracteristicas_hardware` SET serial = ?, observaciones_h = ?, tipo_nombre = ? WHERE codigo = ?";
             
              $mysqli = $this->abrir_conexion();
              $stmt = $mysqli->prepare($query);
              
              $stmt->bind_param("ssss", $serialx, $observaciones_hx, $tipo_nombrex, $codigo);
 
foreach ($serial as $n => $serialx) {
      $observaciones_hx = $observaciones_h[$n];
      $tipo_nombrex = $tipo_nombre[$n];
      $id_hardwarex = $id_hardware[$n];
 
      $stmt->execute();
}
$stmt->close();
              
             
              
          }
          
              
          }
 

[AnGuisi]

Llevado por la emoción del momento me he equivocado a la hora de decir que mi tema está solucionado y luego de varias pruebas me he dado cuenta de que aun no he encontrado solución a mi problema.

Resulta que ahora solo me modifica el primer conjunto de resultados que recibo y sabiendo que el ciclo foreach si recoje todos los resultados pues ya he probado eso.

He aqui mi script:

Código PHP:

public function editar($datos = array()) {
         
         
          extract($datos);
            
          if ($codigo != '') {
              $query = "UPDATE `soptecvnzla`.`caracteristicas_hardware` SET serial = ?, observaciones_h = ?, tipo_nombre = ? WHERE id_hardware = ? AND codigo = ?";
              
              $mysqli = $this->abrir_conexion();
              $stmt = $mysqli->prepare($query);
              if (!$stmt) {
              echo "Falló la preparación: (" . $mysqli->errno . ") " . $mysqli->error;
              }
              
              if(!$stmt->bind_param("sssss", $serialx, $observaciones_hx, $tipo_nombrex,$id_hardwarex,$codigox)) {
                   echo "Falló la vinculación de parámetros: (" . $mysqli->errno . ") " . $mysqli->error;
                                                                                                   }
              
foreach ($codigo as $n => $codigox) {
      $observaciones_hx = $observaciones_h[$n];
      $tipo_nombrex = $tipo_nombre[$n];
      $id_hardwarex = $id_hardware[$n];
      $serialx = $serial[$n];
     
 
      if (!$stmt->execute()) {
    echo "Falló la ejecución: (" . $mysqli->errno . ") " . $mysqli->error;
}
else {
    return true;
     }
}
 
      
          }
          $stmt->close();
      }
 

Creo que el problema puede estar reflejado en la sentencia mysql, pero no estoy seguro. Lo cierto es que solo me modifica el primer campo y los otros no.
Recuerden lo que antes señale sobre que el id_hardware me indica a que componente pertenecen las caracteristicas : serial, observaciones_h y tipo_nombre mientras el codigo engloba todos los componentes en general. Es un mismo codigo para todas las caracteristicas pero cada una de ella tiene id_hardware diferentes, esto podria ayudar a entender mejor mi problema fijense en la sentencia mysql, ¿Es la adecuada para el caso?

Agradecería a algún moderador pudiera cambiar mi tema y quitarle el marcado de solucionado para que otros usuarios puedan seguir comentando. Es un proyecto de la universidad que realmente me tiene casi loco

[GatorV]

¿Que datos tienes en $datos?

[AnGuisi]

$codigo que no es un array ya que es general para todos los componentes.

$id_hardware que es un array con todos los $id_hardware
$serial todos los seriales
$observaciones las observaciones todas y
$tipo_nombre array tambien.

Lo que si note es que haciendo echo a algunos de estos arrays dentro del foreach solo me muestra una clave y un valor y no todos los q trae como deberia de ser.

[AnGuisi]

Ahora si lo he solucionado haciendolo de esta forma:

Código PHP:

public function editar($datos = array()) {
         
         
          extract($datos);
            
          if ($codigo != '') {
              $query = "UPDATE `soptecvnzla`.`caracteristicas_hardware` SET serial = ?, observaciones_h = ?, tipo_nombre = ? WHERE id_hardware = ?";
              
              $mysqli = $this->abrir_conexion();
              $stmt = $mysqli->prepare($query);
              if (!$stmt) {
              echo "Falló la preparación: (" . $mysqli->errno . ") " . $mysqli->error;
              }
              
              if(!$stmt->bind_param("ssss", $serialx, $observaciones_hx, $tipo_nombrex,$id_hardwarex)) {
                   echo "Falló la vinculación de parámetros: (" . $mysqli->errno . ") " . $mysqli->error;
                                                                                                   }
                                                                                                  
              
foreach ($id_hardware as $n => $id_hardwarex) {
    
      $observaciones_hx = $observaciones_h[$n];
      $tipo_nombrex = $tipo_nombre[$n];
      $serialx = $serial[$n];
      echo $n;
    echo $id_hardwarex;
 
      $stmt->execute();
      
     
 
      
          }
          $stmt->close();
      }
      }
 

El problema al parecer estaba en la parte del script anterior:

Código PHP:

  if (!$stmt->execute()) {
    echo "Falló la ejecución: (" . $mysqli->errno . ") " . $mysqli->error;
}
else {
    return true;
     }  
   

Pero no entiendo el por que eso afectaba y solo me hacia update el primer conjunto de datos.

[GatorV]

Eso es porque al hacer el return true; terminabas la ejecución de tu método, el return siempre va a terminar la ejecución de la función actual, por eso solamente pasaba una vez por el foreach.

Saludos.