Es valido romper un FOREACH con un return?

[Chico3001]

Hola... tengo el siguiente codigo dentro de un metodo:

Código PHP:

          // Busca en todo el array almenos un numero telefonico 
        foreach ($this->telefono AS $t1) { 
            if (!empty($t1['telefono'])) { 
                $this->error[0]= 2; 
                $this->error[4]="Por Favor ingrese almenos un numero telefonico"; 
                return FALSE; 
            } 
        }  
   

Pero es valido hacer esto? o tedria que romperlo con un break y despues por fuera del ciclo evualuar y colocar el return?

[SetheR]

Es válido, es una cuestión de diseño de código. Sacar el return dejará tu código mas limpio y mantenible en caso de que el método crezca.

Incuso hay gente que prefiere que cada método/función tenga una unica salida (return).

Saludos

[eits]

creo que lo mejor sería un continue

saludos.

Editado: perdón creo que ando perdido.

[Chico3001]

No, por que el exit terminaria con el programa y lo que quiero es que el metodo regrese un error para que se muestre en la plantilla...

[orochies]

claro es cuestion de diseño pero se te olvida un pequeño detalle a explicar que si usas break continua la ejecucion de la funcion despues del bucle y con return se corta en esa linea y retorna el control a la linea de codigo donde fue invocada esta. He leido que siempre es preferible salir cuanto antes de una funcion que seguir hasta el final solo para hacer un return. Mi opinion saludos...

[eits]

sip, tienes razón, pero creo que el que si te serviría es el break, con este creo que no estoy tan perdido ^_^"

saludos.

[Chico3001]

Tienen razon... Si rompo el ciclo con el return el metodo termina y no continua buscando mas errores en otros campos... lo que provocaria que solo se muestre un error a la vez, en vez de mostrar de una sola vez todos los errores encontrados...

[orochies]

bueno con break igual te funciona pero si hay mas codigo despues del bucle se ejecutara y con return ya no. Si ese codigo depende que del bucle alla al menos un telefono es mejor usar un return false. Asi no ejecutas codigo que sabes que de todas formas retornara false.