[SOLUCIONADO] continue y/o brake en foreach

[ArturoGallegos]

Hola compañeros, hace tiempo que no me toca preguntar.


Veamos, hasta ahora he utilizado la función continue o brake para saltar un elemento dentro de un bucle,



tipo

Código PHP:

foreach ($xxx as $value) {
      if($value == 'xxxx'){
        continue;
      }
      // aqui mis acciones para el caso contrario
    }
 

Mi problema viene en que necesito hacer un foreach (o cualquier otra función recursiva) para llamar a una función que evalué el valor enviado.
por ejemplo:

Código PHP:

public function evalXXX($valor){
    if($valor == 'xxxx'){
      continue;
    }
    // aquí mis acciones
  }
  public function xxx(){
    foreach ($xxx as $value) {
      $this->evalXXX($value);
    }
  }
 

Y como resultado me produce un error ya que continue no se puede utilizar de esa forma dentro de una función.


Estoy utilizando PHP 7.1 , alguien conoce alguna alternativa para hacer que el bucle se salte ese item y pase a la siguiente evaluación.

[alvaro_trewhela]

Antes que nada el signo igual (=) es de asignación, el doble signo igual (==) es de comparación, cabe destacar que tanto break y continue no son funciones, son sentencias, en cuanto a la tuyo no te compliques la vida:

Código PHP:

foreach ($xxx as $value) {
    if($value != "xxxx"){
    // aqui mis acciones
    }
}
 

Saludos.

[ArturoGallegos]

jaja sorry, fue un ejemplo elaborado al momento ahorita lo corrijo, fuera de eso el problema va mas aya de eso... ya que el valor no lo puedo evaluar directamente en el foreach

la función que se manda llamar hace una consulta sql que es procesada para obtener mas datos y en base a estos decidir si saltarse el elemento o continuar al procesamiento de lo obtenido.

Aunque vaya al contradicción, el interés de saltarme el item desde esa función no es porque no pueda hacerlo directamente en el foreach, sino para evitar duplicar código, que es el fin de usar POO en el proyecto.

En todo caso tienes razón para salir del paso duplicare código para hacer una validación previa a llamar la función, pero sera una chapuza ya que no debería.

Si alguien mas tiene otra idea se agradece.

[alvaro_trewhela]

A lo que voy yo es que

Código PHP:

public function evalXXX($valor){
    if($valor == 'xxxx'){
    continue;
    }
// aquí mis acciones
}
 

Es equivalente a

Código PHP:

public function evalXXX($valor){
    if($valor != "xxxx"){
    // aquí mis acciones
    }
}
 

Saludos

[xerifandtomas]

Por que no hacer que la funcion retorne un boleano para evaluar si se continua o no con el ciclo

Código PHP:

  function evalXXX($valor)
{
    if($valor == 'xxxx')
    {
        return true;
    }
    else
    {
        return false;
    }
}

foreach ($xxx as $value)
{
        if( evalXXX($value) )
        {
            continue;
        }
 }  
   

[ArturoGallegos]

@xerifandtomas tienes toda toda la razón, un return me detiene todo el ciclo, lo descarte en su momento porque extrañamente un scope que utilizo en laravel mediante terminal al retornar null o false me retorna un bucle que me saturaba la memoria.

Pero después de tantas vueltas caí en cuenta que no tiene porque ser false o null, le retorne un -1 y me funciona.


@alvaro_trewhela entiendo tu punto y tienes toda la razón, para el ejemplo expuesto funciona y es valido, creo que minimice demasiado el caso y el ejemplo, debiendo explicar que usaba laravel y sus scope mediante cronjobs.

Por si a alguien le sirve, al ejecutar un cronjob en laravel (5.4) para enviar correos tenia esto

Scope: aquí es donde no podía retornar null, false o 0 porque entra en un bucle muy extenso que satura la memoria del servidor. y al utilizar continue o break arrojaba un error que detenía todo.
PHP Fatal error: 'continue' not in the 'loop' or 'switch' context in

Código PHP:

public function scopeTemplate($query, $type, $version, $data){
        $result = $query->where('type', $type)->where('version', $version)->first();
        if(is_null($result)){
// este return es el que no me dejaba avanzar
            return -1; 
        }
 
        $parsed = preg_replace_callback('/{{(.*?)}}/', function ($matches) use ($data) {
            list($shortCode, $index) = $matches;
            if( isset($data[trim($index)]) ) {
                return $data[trim($index)];
            } else {
                throw new Exception("Shortcode {$shortCode} not found in template", 1);
            }
 
        }, $result->template);
 
        return ['content' => $parsed, 'from' => $result->from, 'from_name' => $result->from_name, 'subject' => $result->subject];
    }
 

y este es el controlador que lo llama:

Código PHP:

public function SendEmail($type, $version, $data){
        $datos = User::where('id', $data['xxxx'])->first();
        $xxxx_data = array(
            'xxx_nombre' => $datos ->name,
            'xxx_info' => $datos->firma
        );
        $email_data = emails_template::Template($type, $version, array_merge($data, $xxxx_data));
        if($email_data == -1){
            return false;
        }
        
        Mail::send([], [], function($message) use ($email_data, $data)
        {
            $message
                ->to('xxxxx', 'xxxx')
                ->from('xxxxx', 'xxxx')
                ->subject($email_data['subject'])
                ->setBody($email_data['content'], 'text/html');
        });
    }
 

Dado que el scope entraba en un bucle, al validar $email_data no conseguía una respuesta o bien contenía los datos producidos durante el bucle, que no tenían relación con el valor esperado.... cualquiera que fuera el caso continuaba hasta llegar a la función Mail que hacia tronar todo el sistema de envió, de un controlador principal arriba de este.

[hhs]

Solo como nota al margen el problema pudo haberse evitado si en lugar de hacer la condición hubieras dejado que se generará la excepción usando firstOrFail en lugar de first, de esa forma esto

Código PHP:

$result = $query->where('type', $type)->where('version', $version)->first();
        if(is_null($result)){
// este return es el que no me dejaba avanzar
            return -1; 
        }
 

Hubiera quedado como

Código PHP:

$result = $query->where('type', $type)->where('version', $version)->firstOrFail();
 

si no te regresa ningún valor se genera una excepción del tipo ModelNotFoundException y no requiere el if dentro de la función ni el if que tienes en el controlador el cual desaparece y solo queda la llamada a la consulta

Código PHP:

$email_data = emails_template::Template($type, $version, array_merge($data, $xxxx_data));