[SOLUCIONADO] doble foreach dentro de while

[gmyb]

Hola amigos, estoy con un problema que no logro resolver, quizas alguien pueda echarme una mano.

Necesito abrir todos los archivos de un directorio, y a su vez para cada archivo de esos necesito mostrar las lineas que contiene. (Esta ultima parte de mostrar la linea ya funciona, la probe por separado y esta ok, el problema es cuando le agregué la parte de buscar todos los archivos del directorio).

Código PHP:

  <?php


$directorio = opendir("/dir33/"); //ruta actual
while ($archivo = readdir($directorio)) //obtenemos un archivo y luego otro sucesivamente
{
//para cada archivo guardo muestro las lineas
    foreach ($archivo as $sarchivo){
    
        $filename=$archivo[0];
        $vlineas = file("/dir33/".$filename);
    
                foreach ($vlineas as $sLinea){

                    echo "<br/>";
                    $from = $vlineas[0];
                    $enviado = $vlineas[3];
                    $recibido = $vlineas[4];
                    $imsi = $vlineas[7];
                    $mensaje = $vlineas[12];
                    $from_ok = ereg_replace("[^0-9]", "", $from);
                    $enviado_ok = ereg_replace("[^0-9]", "", $enviado);
                    $recibido_ok = ereg_replace("[^0-9]", "", $recibido);
                    $imsi_ok = ereg_replace("[^0-9]", "", $imsi);

                    echo $from_ok;
                    echo "<br>";
                    echo $enviado_ok;
                    echo "<br>";
                    echo $recibido_ok;
                    echo "<br>";
                    echo $imsi_ok;
                    echo "<br>";
                    echo $mensaje; 
                } //cierro foreach linea

    } //cierro foreach archivo

} //cierra while   

Creo que el problema está en la linea 9 (primer foreach) segun el logphp pero no me doy cuenta...

Gracias!

[abrahamhs]

El problema son tus nombres de variable, cuando leí tu código me confundi, y eso es lo mismo que te paso. Ademas si ya tienes los archivos con el while ¿porque recorres de nuevo?

foreach ($archivo as $sarchivo){

¿No deberias ir ya a $vlineas?, ya tienes el archivo o por lo menos el nombre.
Y repito, tus nombres de variables son muy malos $archivo as $sarchivo
Recomiendo que pongas algunos 'echo's para que veas que traen las variables o usa el PDT con eclipse para que puedas debugear

[xfxstudios]

bueno esta parte de aqui

$filename=$archivo[0];

no deberia de ser

$filename=$sarchivo[0];

con la letra s asi como la llamaste en el foreach???

[abrahamhs]

Tambien medio la impresión de algo así como lo que comenta xfxstudios, por eso digo que sus nombres de variable son muy malos

[gmyb]

Muchas gracias a todos, como decía abrahams, el while o un foreach estaba de más.

Y si, los nombres de variables no eran los mejores.

Ya quedo funcionando asi:

Código PHP:

  <?php


$directorio = opendir("/var/spool/sms/incoming/"); //ruta actual
while ($archivo = readdir($directorio)) //obtenemos un archivo y luego otro sucesivamente
{
        $vlineas = file("/var/spool/sms/incoming/".$archivo);
    
                foreach ($vlineas as $sLinea){

                    echo "<br/>";
                    $from =  $vlineas[0];
                    $enviado =  $vlineas[3];
                    $recibido =  $vlineas[4];
                    $imsi =  $vlineas[7];
                    $mensaje=  $vlineas[12];
                    
                    $from_ok = ereg_replace("[^0-9]", "", $from);
                    $enviado_ok = ereg_replace("[^0-9]", "", $enviado);
                    $recibido_ok = ereg_replace("[^0-9]", "", $recibido);
                    $imsi_ok = ereg_replace("[^0-9]", "", $imsi);

                    echo $from_ok;
                    echo "<br>";
                    echo $enviado_ok;
                    echo "<br>";
                    echo $recibido_ok;
                    echo "<br>";
                    echo $imsi_ok;
                    echo "<br>";
                    echo $mensaje; 
                } //cierro foreach linea

} //cierra while   

[gmyb]

Mmm, olvidé un detalle...

Ahora funciona el mostrar los datos, pero me repite el mismo dato 13 veces, y en realidad hay 1 solo archivo.


Donde puede estar el problema?

Gracias