Return nic nie zwraca Opcje

[PiotrekM]

Witam,
mam dziwny problem.
Napisałem funkcję do sprawdzania sprawdzania adresów z vimeo i youtube.

[PHP] pobierz, plaintext 
function is_video($url, $n = 0)
{
    $patterns = array(
        '#http://(\w+.)?vimeo.com/(\d+)$#i', #vimeo
        '#http://(\w+.)?youtube.com/watch\?v=(\w+)$#', #youtube
    );
 
    if(preg_match($patterns[$n], $url))
    {
        return true;
        #echo 'działa';
    }
    else
    {
        if($n < (count($patterns)-1))
        {
            is_video($url, ++$n);
        }
    }
    return false;
}
 
var_dump(is_video('http://www.youtube.com/watch?v=qpff9If1J_g'));
[PHP] pobierz, plaintext 

dlaczego tak się dzieje?
funkcja powinna zwrócić true, ale zwraca false... myślałem, że coś z regexami pochrzaniłem, ale nie... gdy w pierwszego IFa wstawie echo to wiadomość jest normalnie wyświetlana...

edit:
nie wiem, dlaczego napisałem to rekurencyjnie - zamienie na pętle i dam znać, czy działa.

edit 2:
ok, teraz działa

[PHP] pobierz, plaintext 
function is_video($url)
{
    $patterns = array(
        'vimeo' => '#http://(\w+.)?vimeo.com/(\d+)$#i',
        'youtube' => '#http://(\w+.)?youtube.com/watch\?v=(\w+)$#',
    );
 
    foreach($patterns as $key => $value)
    {
        if(preg_match($value, $url))
        {
            return true;
        }
    }
    return false;
}
[PHP] pobierz, plaintext 

Ale z ciekawości, mógłby ktoś wytłumaczyć, dlaczego rekurencyjnie nie działało?

Ten post edytował PiotrekM 1.02.2013, 16:02:46

[Spawnm]

Wywołujesz is_video($url, ++$n); w 17 lini bez returna.

[PiotrekM]

Wywołujesz is_video($url, ++$n); w 17 lini bez returna.

Aaa... no jasne (IMG:style_emoticons/default/smile.gif)