3

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

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

它在 99.8% 的时间里都很好用。我无法真正确定异常，但我相信它与另一端的服务器为每次访问生成一个新的随机 URL 有关。因此，该脚本陷入无限循环。

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

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

问题

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

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

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;
}

上面的例子只是许多失败尝试中的 2 次。我准备好寻求帮助了。提前致谢。

4

3 回答 3

1

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

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

为什么计时器不起作用，我不知道。但是通过将其放在循环的顶部来修复增量$i++;应该至少可以改善您的情况。

于 2013-03-13T08:59:20.387 回答
1

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

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

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

我可以打赌，通过取消该分配/条件或任何您想调用的名称，您的代码将变得更具可读性和可维护性，并且很高兴有机会解决您所看到的问题。

于 2013-03-13T08:54:32.650 回答
0

期间有一个问题：

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

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

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

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

于 2013-03-13T08:59:20.507 回答