PHP OO Undefined offset, Array en Php

[misterpoloy]

mi codigo es este:

Este es el de una pagina:
<?
include('functions.php');
$variables = array("listo!", "tambien", "perfectos", "ok");
VerificadorDeContenido($variables);
exit;
?>

Este es el de las funciones:
function VerificadorDeContenido($contenido){
for ($i=0; $i<=count($contenido); $i++) {
echo $contenido[$i]."<br />";;
}
}

la funcion debera de imprimirme los elementos de mi arreglo, y lo hace pero lo que pasa es que me debuelve este error:

listo!
tambien
perfectos
ok

Notice: Undefined offset: 4 in \functions.php on line 4


Que puedo hacer?
Gracias!

[andresdzphp]

En vez de $i<=count tiene que ser $i<count, ya que el array empieza con índice cero (0 a 3) y count te dice que hay 4 elementos (No puede ser igual a 4).

Saludos.

[skiper0125]

Hola que tal.

Tienes un error al momento de generar el for

Código PHP:

  function VerificadorDeContenido($contenido){ 
      for ($i=0; $i<=count($contenido)-1; $i++) 
      { 
            echo $contenido[$i]."<br />";; 
      } 
}  
   

Le coloque el -1 ya que recuerda que en los array el indice inicial es 0

Saludos

me ganaron el post

[andresdzphp]

De todas maneras, no se debe calcular el total de elementos de un array de esa forma. Calcularlo en cada iteración del for, puede hacer el código más lento. Puedes optimizarlo así:

Código PHP:

function VerificadorDeContenido($contenido) {
    for ($i = 0, $size = count($contenido); $i < $size; $i++) {
        echo $contenido[$i] . "<br />";
    }
}
 

o calculando la cantidad de elementos por fuera del for.

Creo que usar foreach es mucho más cómodo:

Código PHP:

function VerificadorDeContenido($contenido) {
    foreach ($contenido as $con) {
        echo "$con <br />";
    }
}
 

http://php.net/manual/es/control-structures.for.php
http://php.net/manual/es/control-structures.foreach.php

Saludos.

[skiper0125]

Iniciado por andresdzphp

Creo que usar foreach es mucho más cómodo:

Código PHP:

function VerificadorDeContenido($contenido) {
    foreach ($contenido as $con) {
        echo "$con <br />";
    }
}
 

http://php.net/manual/es/control-structures.for.php
http://php.net/manual/es/control-structures.foreach.php

Saludos.

Efectivamente tienes razón si lo que deseas hacer es optimizar el tiempo y el recursos para que de esta forma el código sea más eficiente.

Saludos

[misterpoloy]

Muchas Gracias, si esa es la solucion!