解决循环[关闭]答案

【问题标题】：Working around for loop [closed]解决循环[关闭]
【发布时间】：2013-08-23 00:42:58
【问题描述】：

我注意到类似的重复，并尝试使用单个 for 循环来解决此问题，如果我可以最小化代码长度：

如果我可以形成一个循环，我就不需要使用开关盒了吗？

$returnNo 变量从 5 开始，每个 case 乘以 2 然后减 1。
在显示“$a
if() 语句从 if($matchno == 7) 开始，每个 case 乘以 2 然后再加 1。
最后的 if() 语句从 if($matchno == 8) 开始，每个 case 乘以 2。
我已经完成了案例 64，它实际上会上升到 512。我知道代码在重复，我希望有人可以帮助我为此生成一个循环？

非常感谢！

    switch($max) {

    case 80 :
        $returnNO = 5;
        for($a = 1; $a<=5; $a++) {
            if($matchno == $a || $matchno == ($a+1)){
                $matchinfo['matchno'] = $returnNO;
                $matchinfo['place'] = ($matchno == $a ? 'clan1' : 'clan2');
                return $matchinfo;
            }
            $returnNO++;
            $a++;
        }
        if($matchno == 7){
            $matchinfo['winner'] = true;
            return $matchinfo;
        }elseif($matchno == 8){
            $matchinfo['third_winner'] = true;
            return $matchinfo;
        }
    break;

    case 160 :
        $returnNO = 9;
        for($a = 1; $a<=13; $a++) {
            if($matchno == $a || $matchno == ($a+1)){
                $matchinfo['matchno'] = $returnNO;
                $matchinfo['place'] = ($matchno == $a ? 'clan1' : 'clan2');
                return $matchinfo;
            }
            $returnNO++;
            $a++;
        }
        if($matchno == 15){
            $matchinfo['winner'] = true;
            return $matchinfo;
        }elseif($matchno == 16){ 
            $matchinfo['third_winner'] = true;
            return $matchinfo;
        }
    break;

    case 320 :
        $returnNO = 17;
        for($a = 1; $a<=29; $a++) {
            if($matchno == $a || $matchno == ($a+1)){
                $matchinfo['matchno'] = $returnNO;
                $matchinfo['place'] = ($matchno == $a ? 'clan1' : 'clan2');
                return $matchinfo;
            }
            $returnNO++;
            $a++;
        }

        if($matchno == 31){
            $matchinfo['winner'] = true;
            return $matchinfo;
        } elseif($matchno == 32){
            $matchinfo['third_winner'] = true;
            return $matchinfo;
        }
    break;
    case 640 :              
        $returnNO = 33;
        for($a = 1; $a<=61; $a++) {
            if($matchno == $a || $matchno == ($a+1)){
                $matchinfo['matchno'] = $returnNO;
                $matchinfo['place'] = ($matchno == $a ? 'clan1' : 'clan2');
                return $matchinfo;
            }
            $returnNO++;
            $a++;
        }   
        if($matchno == 63){
            $matchinfo['winner'] = true;
            return $matchinfo;
        }elseif($matchno == 64){
            $matchinfo['third_winner'] = true;
            return $matchinfo;
        }               
    break;      

    }
}

【问题讨论】：

你已经发布了工作代码，要求重构。这不是 Stack Overflow 的用途。 codereview.stackexchange.com 祝你好运
使用此 switch 语句可以达到的最大情况是多少？
最大大小写为 512 或添加零 5120

标签： php loops if-statement for-loop

【解决方案1】：

如果我理解得很好，这里有一个替代解决方案，我认为它适用于您指定的所有情况，不使用 switch case。

$div10 = $max / 10;
$maxLoop = $div10 - 3;
$returnNO = $div10 / 2 + 1;

for($a = 1; $a<=$maxLoop; $a++) {
    if($matchno == $a || $matchno == ($a+1)){
        $matchinfo['matchno'] = $returnNO;
        $matchinfo['place'] = ($matchno == $a ? 'clan1' : 'clan2');
        return $matchinfo;
    }
    $returnNO++;
    $a++;
}
if($matchno == ($div10-1)){
    $matchinfo['winner'] = true;
    return $matchinfo;
}elseif($matchno == $div10){
    $matchinfo['third_winner'] = true;
    return $matchinfo;
}

【讨论】：

非常感谢我正在寻找的东西 :)
这假设你的 $max 是一个偶数 >= 40，否则调用这个 for 循环不会有多大用处
如果这解决了您最初的问题，您可以将此答案标记为已接受吗？

【解决方案2】：

我以前两种情况为例：

switch ($max) {
case 80:
        $returnNO = 5;
        $loopCount = 5;
        $winner = 7;
        $thirdWinner = 8;
        break;

    case 160:
        $returnNO = 9;
        $loopCount = 13;
        $winner = 15;
        $thirdWinner = 16;
        break;
    ...
}

for ($a = 1; $a <= $loopCount; $a++) {
    if ($matchno == $a || $matchno == ($a + 1)) {
        $matchinfo['matchno'] = $returnNO;
        $matchinfo['place'] = ($matchno == $a ? 'clan1' : 'clan2');
        return $matchinfo;
    }
}

if ($matchno == $winner) {
    $matchinfo['winner'] = true;
    return $matchinfo;
} else if ($matchno == $thirdWinner) {
    $matchinfo['third_winner'] = true;
    return $matchinfo;
}

简单解释一下，删除一直重复的代码并尝试对其进行参数化（通过创建函数或将所有重复的代码放在其他地方......在这个例子中，我把重复的代码放在 switch 语句之后和参数化它。

【讨论】：