[SOLUCIONADO] Como recorrer un array y eliminar valores?

[zerobathory]

Les explico, estoy haciendo un sistema de inventarios con php, sql, html, javascript, etc...., en el cual va a haber una seccion en la que se hace un pedido de inventario, es decir, va a ser una pantalla en la que se van a mostrar todos los productos que hay, cuantas existencias y un recuadro en donde va a ponerse el numero de piezas que se necaesita pedir de ese prodcuto, como esto:



En cuanto al codigo para saber que numero de piezas son las que piden y el nombre del inventario, el formulario lo tengo asi...

Código PHP:

$consul="SELECT * FROM ".$prefijo."_inventario ORDER BY idinventario ASC";
                $query=$con->query($consul);
                for($x=1; $x<=$query->rowcount(); $x++){
                        $array=$query->fetch(PDO::FETCH_ASSOC);
                        echo"<tr>";
                        echo "<td>".$array['nombreinventario']."<input type='hidden' name='contr[$x][nombreinv]' value='".$array['nombreinventario']."'></td>";
                        echo "<td>".$array['piezas']."</td>";
                        echo "<td><input type='text' class='form-control' name='contr[$x][npiezas]' value='0'></td>";
                        echo"</tr>";
                }
 

Para insertarlo en la base de datos pues se envia todo como array, y lo que quiero es recorrer el array y eliminar los nombres de inventario que el numero de piezas sea igual a 0... El array sin eliminar lo que quiero, me lo manda asi:

Código:

Array ( [1] => Array ( [nombreinv] => Motor Lavadora Koblenz [npiezas] => 10 ) [2] => Array ( [nombreinv] => Empaque olla 4" [npiezas] => 5 ) [3] => Array ( [nombreinv] => REVISION ASPIRADORAS [npiezas] => 0 ) [4] => Array ( [nombreinv] => camara [npiezas] => 0 ) [5] => Array ( [nombreinv] => Motor Raul 1 [npiezas] => 0 ) [6] => Array ( [nombreinv] => SIN REPARACION [npiezas] => 0 ) )

Y al recorrer el array y eliminar las piezas que estan en cero, quiero que aparezca asi...

Código:

Array ( [1] => Array ( [nombreinv] => Motor Lavadora Koblenz [npiezas] => 10 ) [2] => Array ( [nombreinv] => Empaque olla 4" [npiezas] => 5 ) )

Pero no se muy bien como recorrer el array y haga lo que quiero, se que se puede hacer algo asi:

Código PHP:

$arraytodo=$_POST['contr'];
foreach($arraytodo as $key => $val){
    if($val['npiezas'] == 0 ){
        unset($key);
    }
}
 

Pero no pasa nada y me sigue imprimiendo todo... Alguien me ayuda a saber como puedo hacerlo?

[pateketrueke]

Lee bien el manual de unset() porque parece que no lo has hecho:

Código PHP:

$foo = ['x' => 'y'];
 
// MAL (esto sería equivalente a lo que haces)
$key = 'x';
unset($key);
 
// BIEN
unset($foo['x']);
 

Otra forma es usar array_filter() para filtrar precisamente el array dada una función propia, etc.

[gonzaherrera77]

Buenas aporto mi pequeño código que hice recien, no se si es optimo por que recien estoy empezando

Recuerda que tenes un array de bidimensional.

Pruebalo

Código PHP:

<?php
 
   //Array bidimencional
   $nombreProd = Array (
                        Array ('nombreinv' => 'Motor Lavadero', 'npiezas' => 15),
                        Array ('nombreinv' => 'Empaque olla 4', 'npiezas' => 5 ),
                        Array ('nombreinv' => 'REVISION ASPIRADORAS', 'npiezas' => 0 ),
                        Array ('nombreinv' => 'camara', 'npiezas' => 0 ),
                        Array ('nombreinv' => 'Motor Raul', 'npiezas' => 0 ),
                        Array ('nombreinv' => 'SIN REPARACION', 'npiezas' => 0 )
              );
//Variable para concatenar 
$indiceArray = "";
 
for($i = 0; $i < count($nombreProd); $i++) {
         if($nombreProd[$i]['npiezas'] === 0) {
            //Concateno la posicion del arreglo cuyo valor de npiezas es 0
            $indiceArray .= $i;
         }
   }
//Recorro el string $indiceArray, que va a obtener la posicion del array cuyo valor de npiezas es 0.
for($i = 0; $i < strlen($indiceArray); $i++) {
   //Borro el array
   unset($nombreProd[$indiceArray[$i]]);
}
 
print_r($nombreProd);
 
 
?>
 

¡Saludos!

[zerobathory]

Iniciado por gonzaherrera77

Buenas aporto mi pequeño código que hice recien, no se si es optimo por que recien estoy empezando

Recuerda que tenes un array de bidimensional.

Pruebalo

Código PHP:

<?php
 
   //Array bidimencional
   $nombreProd = Array (
                        Array ('nombreinv' => 'Motor Lavadero', 'npiezas' => 15),
                        Array ('nombreinv' => 'Empaque olla 4', 'npiezas' => 5 ),
                        Array ('nombreinv' => 'REVISION ASPIRADORAS', 'npiezas' => 0 ),
                        Array ('nombreinv' => 'camara', 'npiezas' => 0 ),
                        Array ('nombreinv' => 'Motor Raul', 'npiezas' => 0 ),
                        Array ('nombreinv' => 'SIN REPARACION', 'npiezas' => 0 )
              );
//Variable para concatenar 
$indiceArray = "";
 
for($i = 0; $i < count($nombreProd); $i++) {
         if($nombreProd[$i]['npiezas'] === 0) {
            //Concateno la posicion del arreglo cuyo valor de npiezas es 0
            $indiceArray .= $i;
         }
   }
//Recorro el string $indiceArray, que va a obtener la posicion del array cuyo valor de npiezas es 0.
for($i = 0; $i < strlen($indiceArray); $i++) {
   //Borro el array
   unset($nombreProd[$indiceArray[$i]]);
}
 
print_r($nombreProd);
 
 
?>
 

¡Saludos!

Hola!! Te agradezco mucho tu respuesta, cuando la copio tal cual como la pones, funciona a la perfeccion, mas sin embargo, el array que hay que recorrer lo recibo por un formulario y el metodo $_POST... Entonces al cambiar:

Código PHP:

  $nombreProd = Array ( 
                        Array ('nombreinv' => 'Motor Lavadero', 'npiezas' => 0), 
                        Array ('nombreinv' => 'Empaque olla 4', 'npiezas' => 0 ), 
                        Array ('nombreinv' => 'REVISION ASPIRADORAS', 'npiezas' => 0 ), 
                        Array ('nombreinv' => 'camara', 'npiezas' => 2 ), 
                        Array ('nombreinv' => 'Motor Raul', 'npiezas' => 3 ), 
                        Array ('nombreinv' => 'SIN REPARACION', 'npiezas' => 0 ) 
              );  
   

Lo cambio por:

Código PHP:

  $nombreProd = $_POST['contr'];  
   

Deja de funcionar y me marca lo mismo que al principio, no se si hay que recibir de alguna forma especial el $_POST, inclusive le puse:

Código PHP:

  $nombreProd =array( $_POST['contr']);  
   

Y nada

[pateketrueke]

Bueno, de hecho en tu código original tenías que hacer esto:

Código PHP:

// MAL
unset($key);
 
// BIEN
unset($arraytodo[$key]);
 

Lo digo porque analizando la solución de gonzaherrera77 veo que tiene fallas graves.

Por ejemplo, concatena todas las claves a borrar en una cadena de texto quedando "375" por ejemplo.

Y al recorrer de nuevo la lista (innecesariamente) lo hace carácter a carácter ¿que sucedería si el índice que intentamos borrar es 99?

Pues quedaría una secuencia así "37599" pero se recorrería así "3, 7, 5, 9, 9" lo cual en definitiva es un error de concepto muy grave si se usa dicho código.

Igual quedaría mejor con un sólo loop así:

Código PHP:

// MAL
$indiceArray .= $i;
 
// BIEN
unset($nombreProd[$i]);
 

¡Bingo!

[Alexis88]

Solo tienes que recorrer al array principal y, en cada iteración, verificar si la propiedad "npiezas" de cada array contenido, tiene como valor cero. De ser así, eliminas al array hijo del array padre a partir del índice que lo identifica en este último.

Código PHP:

foreach ($padre as $indice => $hijo){
    if (!$hijo['npiezas']) unset($padre[$indice]);
}
 

Un saludo

EDITO: Recién veo que ya te dieron la solución.

[gonzaherrera77]

Iniciado por pateketrueke

Bueno, de hecho en tu código original tenías que hacer esto:

Código PHP:

// MAL
unset($key);
 
// BIEN
unset($arraytodo[$key]);
 

Lo digo porque analizando la solución de gonzaherrera77 veo que tiene fallas graves.

Por ejemplo, concatena todas las claves a borrar en una cadena de texto quedando "375" por ejemplo.

Y al recorrer de nuevo la lista (innecesariamente) lo hace carácter a carácter ¿que sucedería si el índice que intentamos borrar es 99?

Pues quedaría una secuencia así "37599" pero se recorrería así "3, 7, 5, 9, 9" lo cual en definitiva es un error de concepto muy grave si se usa dicho código.

Igual quedaría mejor con un sólo loop así:

Código PHP:

// MAL
$indiceArray .= $i;
 
// BIEN
unset($nombreProd[$i]);
 

¡Bingo!

Asi es tenes razón, yo concatene la posicion de del array por que haciendo un unset($nombreProd[$i]); solo me tomaba dos posciones solo hace la 2 y 3 no se por que motivo muestro.

Código PHP:

<?php
 
   //Array bidimencional
   $nombreProd = Array (
                        Array ('nombreinv' => 'Motor Lavadero', 'npiezas' => 15),
                        Array ('nombreinv' => 'Empaque olla 4', 'npiezas' => 5 ),
                        Array ('nombreinv' => 'REVISION ASPIRADORAS', 'npiezas' => 0 ),
                        Array ('nombreinv' => 'camara', 'npiezas' => 0 ),
                        Array ('nombreinv' => 'Motor Raul', 'npiezas' => 0 ),
                        Array ('nombreinv' => 'SIN REPARACION', 'npiezas' => 0 )
              );
 
for($i = 0; $i < count($nombreProd); $i++) {
         if($nombreProd[$i]['npiezas'] === 0) {
 
            unset($nombreProd[$i]);
            echo $i.' ';
         }
   }
echo '<br/>';
 
print_r($nombreProd);
 
 
?>
 

zerobathory ahora lo encapsule en una función probalo

Código PHP:

<?php
   if(count($_POST)) {
 
 
function limpiarArray($vector) {
   $i = 0;
         foreach($vector as $key) {
            foreach($key as $valor) {
               if($valor == '0') {
                   array_splice($vector, $i, 1);
                   $i--;
               }
            }
            $i++;
         }
            return $vector;
      }
//Invoco a la funcion.
$c  = limpiarArray($_POST['c']);
 
print_r($c);
}
?>
<!DOCTYPE html>
<html>
   <head>
      <meta charset="utf-8">
      <title>Test</title>
   </head>
   <body>
      <form action="limpiarArray.php" method="post">
            <input type="text" name="c[0]['nombreinv']" value="Motor Lavadora Koblenz">
            <input type="text" name="c[0]['npieza']" value="10"><br />
            <input type="text" name="c[1]['nombreinv']" value="Empaque olla 4">
            <input type="text" name="c[1]['npieza']" value="5"><br />
            <input type="text" name="c[2]['nombreinv']" value="REVISION ASPIRADORAS">
            <input type="text" name="c[2]['npieza']" value="0"><br />
            <input type="text" name="c[3]['nombreinv']" value="camara">
            <input type="text" name="c[3]['npieza']" value="0"><br />
            <input type="text" name="c[4]['nombreinv']" value="SIN REPARACION">
            <input type="text" name="c[4]['npieza']" value="0"><br />
 
            <input type="submit" value="Enviar" />
         </form>
   </body>
</html>
 

Posiblemente se pueda mejorar, hasta aca llegan mis conocimiento hasta hora.

¡Saludos!

[zerobathory]

Iniciado por pateketrueke

Bueno, de hecho en tu código original tenías que hacer esto:

Código PHP:

// MAL
unset($key);
 
// BIEN
unset($arraytodo[$key]);
 

[/HIGHLIGHT]

¡Bingo!

Tienes toda la razon, era muy sencillo de corregir en el codigo original que puse, lo que pasa es que la primera vez que respondiste si lo intente pero intente otra cosa, lo habia entendido mal y no funciono, entonces por eso no habia respondido, lo siento ejeje... Pero con esto que me dices, quedo a la perfeccion

Iniciado por Alexis88

Solo tienes que recorrer al array principal y, en cada iteración, verificar si la propiedad "npiezas" de cada array contenido, tiene como valor cero. De ser así, eliminas al array hijo del array padre a partir del índice que lo identifica en este último.

Código PHP:

foreach ($padre as $indice => $hijo){
    if (!$hijo['npiezas']) unset($padre[$indice]);
}
 

Un saludo

EDITO: Recién veo que ya te dieron la solución.

Iniciado por gonzaherrera77

Asi es tenes razón, yo concatene la posicion de del array por que haciendo un unset($nombreProd[$i]); solo me tomaba dos posciones solo ....

Te agradezco mucho que hayas respondido, ya quedo totalmente solucionado, muchos saludos :)