Encontrar "cualquiera de estas" palabras en una cadena

[Mannuel]

Hola a todos.

Lo que quiero hacer es que si en una cadena existe cualquier palabra de las que tengo en un array, me devuelva un booleano. Ya he encontrado varios temas similares, y créanme que intento hacer lo que dicen, pero no me funciona bien.

Este es el código que llevo hasta ahora:

Código PHP:

  function block_post () { 
$hosts = array('megaupload', 'rapidshare', 'mediafire'); 
$text = 'Descarga aquí: rapidshare.com'; 
 
for ($i=0; $i < count($hosts); $i++) {  
        if (preg_match('/' .$hosts[$i]. '/i', $text)) 
            return TRUE; 
        else 
            return FALSE; 
    }     
}     
 
if (block_post()) 
    echo "El post contiene links no permitidos!"; 
else 
    echo "Bienvenido al post.";  
   

Cuando en la variable $text escribo "megaupload", funciona correctamente, pero cuando escribo "rapidshare" o "mediafire" es como si no los encontrara...

¿Qué es lo que he hecho mal?
Muchísimas gracias.

[pateketrueke]

Eso es normal, ya que sin importar el resultado de la comparación estas usando return, y eso detiene la ejecución misma del resto del bucle y la función.

En dado caso, solo conviene tener el return positivo dentro del bucle, y el return negativo hacía el final del mismo.

[Mannuel]

Podrías explicarme mejor cuando dices "Solo conviene tener el return positivo dentro del bucle, y el return negativo hacía el final del mismo."

¿Cuál es la forma correcta de hacerlo?
Un saludo.

[Sourcegeek]

Te explicaré con manzanas

Código PHP:

  
for ($i=0; $i < count($hosts); $i++) { //Un bucle
    //Dentro del bucle
}
//Fuera del bucle

return true; // Return positivo
return false; //Return negativo  
   

Ahora, asociando... Return positivo dentro del bucle y return negativo fuera...

Saludos!

[Mannuel]

¡Oh! pero qué despistado soy jajaja. Y pensar que era algo tan simple, funciona perfectamente ahora.

¡MIL GRACIAS!

[oso96_2000]

Como comentario, tambien podrías aprovechar mas las expresiones regulares y usar algo como:

Código PHP:

<?php
function block_post() {
    $hosts = array('megaupload', 'rapidshare', 'mediafire');
    $text = 'Descarga aquí: rapidshare.com';
 
    if (preg_match('@(' . implode('|', $hosts) . ')@i', $text) !== false) {
        return false;
    }
 
    return true;
}
 

No lo probe, pero en teoria debería funcionar :P

[masterpuppet]

Tal cual lo tienes oso96_2000 siempre te va a devolver false y no creo que sea necesario el if, podria ser algo así:

Código PHP:

...
return preg_match('@(' . implode('|', $hosts) . ')@i', $text);
 

o

Código PHP:

...
return !!preg_match('@(' . implode('|', $hosts) . ')@i', $text);
 

Saludos.