0

ループでは、最初の5つのアイテムをでラップしli、その後は6つのアイテムごとにをラップしliます。次のコードを試していますが、正しく機能していません。最初の5つのアイテムをaliでラップし、その後、liの6つおきのアイテムのみをラップしますが、liなしで7〜12番目をラップします。

$i = 0;
while ($i < 19){
    $i++;
    if($i == 1 ){ echo '<li>'; }
    if (($i > 5) AND ( $i % 6 == 0)){echo "<li>";}
    echo "<div>item " . $i . "</div>"; 
    if( $i == 5 ){ echo '</li>'; }
    if(($i > 5) AND ( $i % 6 == 0)) { echo "</li>"; }
}
if ($i < 5 ) { echo "</li>"; }
if (($i > 5) AND ($i % 6 != 0)) { echo "</li>"; }
4

3 に答える 3

2

ここでは、よりクリーンで読みやすいソリューションです。

$i = 0;
$group = 0;
$group_by = 5;
while ($i < 19){
    $i++;
    if ($group == 0) {//new group start
        echo "<li>";
    }
    echo "<div>item " . $i . "</div>"; 
    $group++;
    if ($group == $group_by) {//end of group
       echo "</li>\n";
       $group = 0;
       $group_by = 6;
    }
}
if ($group != 0) { echo "</li>"; }//end group if it was not closed

http://codepad.org/oxJloL03

于 2012-11-07T08:52:23.357 に答える
1

これは、条件が「6アイテムごとではなく、 6アイテム$i % 6 == 0ごと」を意味すると言っているためです。

6つの連続する要素、つまり7から12をラップするには、条件を$i <= 12に書き直して、コードが次のようになるようにする必要があります。

$i = 0;
for (; $i < 19; $i++) {

    if($i == 1 ){ echo '<li>'; }
    if (($i > 5) AND ( $i <= 12)){echo "<li>";}
    echo "<div>item " . $i . "</div>"; 
    if( $i == 5 ){ echo '</li>'; }
    if(($i > 5) AND ( $i <= 12)) { echo "</li>"; }
}
if ($i < 5 ) { echo "</li>"; }
if (($i > 5) AND ($i <= 12)) { echo "</li>"; }
于 2012-11-07T08:48:31.120 に答える
1

このソリューションを試してください：

for ($i = 1; $i < 20; $i++) {
    echo ($i === 1 || $i % 6 === 0) ? "<li>" : null,
        "<div>item {$i}</div>",
        ($i % 6 === 5) ? "</li>" : null;
}
if ($i % 6 !== 0) echo "</li>";

私はそれをできるだけシンプルに保つように努めました。

コード例

于 2012-11-07T08:55:38.240 に答える