1

我正在尝试将旧的 vbscript 函数转换为 php 函数并运行我运行它，它似乎陷入了无限循环。这是脚本：

<?php
function pagination($str,$max_pg){
    $pg = (int)$str;
    $max_pg = (int)$max_pg;
    if($max_pg <= 1){
        exit();
    }

    $pstrout = '<table cellpadding="0" cellspacing="0" border="0">\r';
    $pstrout .= '<tr class="f5">\r';
    $pstrout .= '<td align="center" class="npagecounter" style="height:20px;color:#0000FF">\r'; 

    if ($pg >= 2){
        $pstrout .= '<a href="pg=1" id="Page1Link">&laquo;&nbsp;First</a>&nbsp;\r';
    }

    if (pg >=2){
        $pstrout .= '<a href="pg='.($pg-1).'" id="Page'.($pg-    1).'Link">&laquo;&nbsp;Previous</a>&nbsp;\r';
    }

    if (max_pg > 5){
        $npage_t = $pg + 4;
        $npage_count_diff = $max_pg - $pg;
        if ($npage_count_diff == 0){
            $npage_count_diff2 = 4;
        }elseif ($npage_count_diff == 1){
            $npage_count_diff2 = 3;
        }elseif ($npage_count_diff == 2){
            $npage_count_diff2 = 2;
        }else{
            $npage_count_diff2 = 1;
        }
        if ($npage_t > $max_pg){
            if ($npage_count_diff <= 3){
                for ($ipages=$pg-$npage_count_diff2;$ipages=$max_pg;$ipages++){
                    if($ipages == $pg){
                        $pstrout .= '<span style="padding:2px 6px" class="fb">'.$ipages.'</span>';
                    }else{
                        $pstrout .= '<a href="?pg='.$ipages.'" id="Page'.$ipages.'Link">'.$ipages.'</a>&nbsp;';
                    }
                }
            }else{
                for ($ipages=$pg;$ipages=$max_pg;$ipages++){
                    if ($ipages == $pg){
                        $pstrout .= '<span style="padding:2px 6px" class="fb">'.$ipages.'</span>';
                    }else{
                        $pstrout .= '<a href="?pg='.$ipages.'" id="Page'.$ipages.'Link">'.$ipages.'</a>&nbsp;';
                    }
                }
            }
        }else{
            if ($pg == 1){
                for ($ipages=$pg;$ipages=($pg+4);$ipages++){
                    if ($ipages == $pg){
                        $pstrout .= '<span style="padding:2px 6px" class="fb">'.$ipages.'</span>';
                    }else{
                        $pstrout .= '<a href="?pg='.$ipages.'" id="Page'.$ipages.'Link">'.$ipages.'</a>&nbsp;';
                    }
                }
            }elseif ($pg == 2){
                for ($ipages=($pg-1);$ipages=($pg+3);$ipages++){
                    if ($ipages == $pg){
                        $pstrout .= '<span style="padding:2px 6px" class="fb">'.$ipages.'</span>';
                    }else{
                        $pstrout .= '<a href="?pg='.$ipages.'" id="Page'.$ipages.'Link">'.$ipages.'</a>&nbsp;';
                    }
                }
            }else{
                for ($ipages=($pg-2);$ipages=($pg+2);$ipages++){
                    if ($ipages == $pg){
                        $pstrout .= '<span style="padding:2px 6px" class="fb">'.$ipages.'</span>';
                    }else{
                        $pstrout .= '<a href="?pg='.$ipages.'" id="Page'.$ipages.'Link">'.$ipages.'</a>&nbsp;';
                    }
                }
            }
        }
    }else{
        for ($ipages=1;$ipages=$max_pg;$ipages++){
            if ($ipages == $pg){
                $pstrout .= '<span style="padding:2px 6px" class="fb">'.$ipages.'</span>&nbsp;';
            }else{
                $pstrout .= '<a href="?pg='.$ipages.'" id="Page'.$ipages.'Link">'.$ipages.'</a>&nbsp;';
            }
        }
    }
    if ($max_pg > 1){
        if ($pg <> $max_pg){
            $pstrout .= '<a href="?pg='.($pg+1).'" id="Page'.($pg+1).'Link">Next&nbsp;&raquo;</a>&nbsp;';
        }
    }
    if ($max_pg > 1){
        if ($pg <> $max_pg){
            $pstrout .= '<a href="?pg='.$max_pg.'" id="Page'.$max_pg.'Link">Last&nbsp;&raquo;</a>';
        }
    }

    $pstrout .= '</td>\r';
    $pstrout .= '</tr>\r';
    $pstrout .= '<tr>\r';
    $pstrout .= '</table>\r'; 
    return $pstrout;
}

$current_page = 2;
$total_page = 24;
echo pagination($current_page,$total_page);
?>

我可能有一个支架不合适。请你帮我解决这个问题，无限循环。

非常感谢！

4

4 回答 4

1

你的 for 循环是错误的。

改变这个：

for ($ipages=$pg;$ipages=$max_pg;$ipages++){

对此：

for ($ipages=$pg;$ipages<=$max_pg;$ipages++){

您需要对所有 for 循环执行此操作，并根据对您问题的评论进行更改。代码中也可能存在其他错误，但这是一个很好的起点。

我对您的代码进行了一些其他更改并将其添加到 PHP Fiddle：http: //phpfiddle.org/api/raw/qdt-r7t逻辑似乎仍然无法正常工作，但该页面应该是一个很好的起点.

此外，像这样的问题更适合代码审查网站：https ://codereview.stackexchange.com/?as=1

于 2013-07-24T20:42:22.457 回答
1

您错误地使用了比较运算符，几次，您正在使用赋值操作

for ($ipages=$pg-$npage_count_diff2;$ipages=$max_pg;$ipages++)

$ipages=$max_pg;应该$ipages==$max_pg;

可能会更好 $ipages<=$max_pg;

与只是 == 一样，它仅在达到 $max_pg 时才会停止，可能会有一个实例跳过 $max_pg 并且永远不会达到它。

于 2013-07-24T20:32:08.860 回答
1

第二个循环

 }else{
                for ($ipages=$pg;HERE --> $ipages=$max_pg   <---HERE;$ipages++){
                    if ($ipages == $pg){
                        $pstrout .= '<span style="padding:2px 6px" class="fb">'.$ipages.'</span>';
                    }else{
                        $pstrout .= '<a href="?pg='.$ipages.'" id="Page'.$ipages.'Link">'.$ipages.'</a>&nbsp;';
                    }
                }
            }
于 2013-07-24T20:34:06.417 回答
1

我在您的代码中添加了一些回声以运行快速测试：

if ($pg == 1){
    for ($ipages=$pg;$ipages=($pg+4);$ipages++){
        echo "case 0 ".$ipages." ... ";
        [....]
    }
} else if ($pg == 2){
    for ($ipages=($pg-1);$ipages=($pg+3);$ipages++){
        echo "case 1 ".$ipages." ... ";
        [...]
    }
} else {
    for ($ipages=($pg-2);$ipages=($pg+2);$ipages++){
        echo "case 2 ".$ipages." ... ";
        [...]
    }
}

我得到以下结果：

case 1 5 ... case 1 5 ... case 1 5 ... case 1 5 ... case 1 5 ... case 1 5 ...

您应该像其他人提到的那样替换您的运营商

于 2013-07-24T20:52:11.507 回答