PHP OO reemplazar foreach

[rufoqe]

hola a todos, tengo el siguiente codigo

$posts = array();
$i = 0;
foreach($a as $key => $value) {

preg_match("#php\?sid=([0-9]+)#", $c[$i],$id);
$id = $id[1];

$img = (strlen($b[$i])<20?'no':photo($b[$i]));
$img = preg_replace("#c([0-9]+).([0-9]+).([0-9]+).([0-9]+)#", '',$img);


sin embargo cuando lo ejecuto me sale lo siguiente

Warning: Invalid argument supplied for foreach() in /home/pagina/public_html/pagina.php on line 47


hay alguna forma de reemplazarlo, me han dicho que es por la versión de php, es cierto? gracias

[Alexis88]

En este caso, la versión de PHP no tiene nada que ver, probablemente $a no sea un array. Deja como comentarios el código que muestras y pega este para ver qué contenido tiene $a.

Código PHP:

var_dump($a);
 

Y luego muéstranos el resultado.

Saludos

[rufoqe]

Hola, si el codigo completo es el siguiente:

Código PHP:

  <?php 
include "header.php"; 
include "simple_html_dom.php"; 
 
$c = getFanpage($fanpage); 
 
 
libxml_use_internal_errors(true); 
$dom = new domDocument; 
$dom->loadHTML($c); 
$dom->preserveWhiteSpace = false; 
 
 
function text($a){ 
    return (strip_tags(str_replace("<br>", "\n", $a))); 
} 
 
 
function photo($a){ 
    preg_match("#src=\"(.*?)\"#",$a,$a); 
    return $a[1]; 
} 
 
function listDivsByClass($class){ 
    global $dom; 
    $xpath = new DomXpath($dom); 
    $div = $xpath->query('//*[@class="'.$class.'"]'); 
 
    foreach ($div as $element) { 
        $nodes = $element->childNodes; 
        foreach ($nodes as $node) { 
          $a[] = $node->ownerDocument->saveHTML( $node ); 
        } 
    } 
 
    return $a; 
} 
 
 
$a = listDivsByClass('tlAboveUnit'); 
$b = listDivsByClass('tlUnitContent'); 
$c = listDivsByClass('tlBelowUnit'); 
 
 
$posts     = array(); 
$i         = 0; 
 foreach($a as $key => $value) { 
 
    preg_match("#php\?sid=([0-9]+)#", $c[$i],$id); 
    $id = $id[1]; 
 
    $img = (strlen($b[$i])<20?'no':photo($b[$i])); 
    $img = preg_replace("#c([0-9]+).([0-9]+).([0-9]+).([0-9]+)#", '',$img); 
 
 
    $titleee = ($value==''?':O':text($value)); 
    $titleee = preg_replace('@(https?://([-\w\.]+)+(:\d+)?(/([\w/_\.]*(\?\S+)?)?)?)@','',$titleee); 
 
 
    $posts[] = array( 
                        'title' => $titleee, 
                        'img'     => $img, 
                        'id'    => $id 
                        ); 
    $i++; 
 
    unset($img); 
    unset($id); 
    unset($titleee); 
 
} 
 
 
 
include("save.php");   

yo decía o me quedaba la duda que si pudiera ser la versión de php por que , yo tengo un hosting de pago en hostgator Cpanel y me sale este error, he probado en varios hosting VPS Webuzo y también sale el mismo error , pero he probado en un hosting gratuito y si funciona , otras personas me han dicho que si funciona en sus hosting pero no quiero estas gastando inutilmente si se puede arreglar,

[Alexis88]

¿Hiciste lo que te indiqué en mi anterior respuesta?

[rufoqe]

hola, si sale null

[Alexis88]

Ahí está el problema, como $a es null, no puedes tratarlo como un array, por eso aparece el mensaje de error. Al parecer, está ocurriendo un problema en la función listDivsByClass, revisa esa sección del código.

[Italico76]

En caso de que pudieras recibir NULL porque no hubiera datos que mostrar pero no fuera por error de programacion:

Código PHP:

foreach( (array) $a as $key => $value) {
   // etc
}
 

Con eso te libras de que el array si viene vacio (null) tengas un error

Ej:

Código PHP:

<?php
$a = [];  // supongamos viene de una DB y no hay resultados de tu consulta
 
foreach((array) $a as $key => $val) {
    echo "$key : $val\n<br/>";
    // ...
}
 
# No produce ningun error :)
 

[rufoqe]

Iniciado por Italico76

En caso de que pudieras recibir NULL porque no hubiera datos que mostrar pero no fuera por error de programacion:

Código PHP:

foreach( (array) $a as $key => $value) {
   // etc
}
 

Con eso te libras de que el array si viene vacio (null) tengas un error

Ej:

Código PHP:

<?php
$a = [];  // supongamos viene de una DB y no hay resultados de tu consulta
 
foreach((array) $a as $key => $val) {
    echo "$key : $val\n<br/>";
    // ...
}
 
# No produce ningun error :)
 

muchas gracias, si me funciona,

[Italico76]

jeje... me alegro.. lo que haces con eso se llama casting: conviertes NULL en array()

[Alexis88]

Creo que lo más conveniente sería verificar que si el array es nulo, no realizar el bucle, pues sería innecesario. La función count te puede ayudar para saber eso.

[rufoqe]

Iniciado por Alexis88

Creo que lo más conveniente sería verificar que si el array es nulo, no realizar el bucle, pues sería innecesario. La función [URL="http://www.php.net/manual/es/function.count.php"][B]count[/B][/URL] te puede ayudar para saber eso.

Gracias, lo hare

[Italico76]

Iniciado por rufoqe

Gracias, lo hare

Logicamente que no hay ciclado del foreach si el array esta vacio.... pues es muy parecido a un un for en su implmentacion:

Código PHP:

  for ($i=0;$i<count($array);$i++)
{
    # ...
}  
   

El unico problema es que NULL no pertenece a ningun tipo en particular y por eso foreach queda todo confundido cuando encuentra NULL y el casting lo resuelve.

Por lo dicho... excepto que las keys sean no-numericas es lo mismo que...

Código PHP:

<?php
$a = [];  // supongamos viene de una DB y no hay resultados de tu consulta
 
for ($key=0;$key<count($a);$key++)
    echo "$key : ".$a[$key]."\n<br/>";
    // ...
}
 
# No produce ningun error :)
 

Y la misma vaina se puede hacer con while ()

[Alexis88]

Claro, la solución para este caso es la conversión de tipo de null a un array vacío, pero ya en términos generales, es preferible verificar el tamaño del array antes de iterarlo. Es una buena práctica pues he visto muchos (pero muchos) casos de personas que no entendían el porque se producía un error al momento de iterar un array luego de una búsqueda de datos en la BD, cuando dicha búsqueda solía no devolver dato alguno. Es más por seguir una buena práctica, porque, como dije, la solución para este caso, es la conversión de tipo.

Saludos