使用条件函数停止PHP while循环

Question

我正在使用以下函数从一系列重定向中获取最终的URL ...

https://stackoverflow.com/a/4102293/1183476

它的工作效率很高，达到99.8％。 我无法确定异常，但我认为它与另一端的服务器有关，为每次访问生成一个新的随机URL。 因此，这个脚本陷入无限循环。

要复制该问题，请将get_redirect_url函数替换为...

function get_redirect_url($url){
    return $url.'x';
}

问题

如何设置时间或迭代限制？

我觉得我已经尝试了一切。 我尝试在while循环中设置一个基于时间的条件，寻找下一个URL，但它不起作用，我不知道为什么。 像这样...

function get_all_redirects($url){
    $redirects = array();
    $start = time();
    while ($newurl = get_redirect_url($url) && time()-$start < 10 ){
        if (in_array($newurl, $redirects)){
            break;
        }
        $redirects[] = $newurl;
        $url = $newurl;
    }
    return $redirects;
}

我也试过计算这样的迭代......

function get_all_redirects($url){
    $redirects = array();
    $i = 0;
    while ($newurl = get_redirect_url($url) && $i < 10 ){
        if (in_array($newurl, $redirects)){
            break;
        }
        $redirects[] = $newurl;
        $url = $newurl;
        $i++;
    }
    return $redirects;
}

上面的例子只是许多失败尝试中的两个。 我准备好了。 提前致谢。

Answer 1

只扫描（一秒钟）你的代码没有显示任何明显的问题，但我想提出一些建议。

每当我在控制流语句中看到使用赋值结果的条件时，它总是闻起来很腥（好吧，让我们说它不是我的风格）：

while ($newurl = get_redirect_url($url) ...

我敢打赌，通过剔除任务或条件或任何你想要它的代码，你的代码将变得更易读和可维护，并且通过一些快乐的机会解决你所看到的问题。

Answer 2

我假设，你所说的循环是一个真正的循环，因此迭代不起作用，因为如果url已经在数组中，则在递增之前中断。

if (in_array($newurl, $redirects)){
    break;
}

为什么定时器不起作用，我不知道。 但是通过放置$i++;修复递增$i++; 在循环的顶部至少应该改善你的情况。

Answer 3

有一个在当一个问题：

while ($newurl = get_redirect_url($url) && time()-$start < 10 )

并且应该在$newurl = get_redirect_url($url)周围添加一些parens

while ( ($newurl = get_redirect_url($url)) && time()-$start < 10 )

相反，因为&&优先级高于= 。 否则$newurl得到条件表达式get_redirect_url($url) && time()-$start < 10结果为1 。