0

私は正常に動作しているPHPコード（ワードプレス）を持っています。ある種の配列によって構文全体を短縮するよりスマートな方法があると感じています。誰かが助けてくれます。コードのすっきりが最初に来ます。

<?php
    $user = wp_get_current_user();
    // is there someone logged?
    if ($user->ID) {
        echo '<ul class="availableindex nav nav-pills nav-stacked">';
        $n1 = cimy_uef_sanitize_content(get_cimyFieldValue($user->ID, 'NUMBER1'));
        $n2 = cimy_uef_sanitize_content(get_cimyFieldValue($user->ID, 'NUMBER2'));
        $n3 = cimy_uef_sanitize_content(get_cimyFieldValue($user->ID, 'NUMBER3'));
        $n4 = cimy_uef_sanitize_content(get_cimyFieldValue($user->ID, 'NUMBER4'));
        $n5 = cimy_uef_sanitize_content(get_cimyFieldValue($user->ID, 'NUMBER5'));
        $n6 = cimy_uef_sanitize_content(get_cimyFieldValue($user->ID, 'NUMBER6'));
        $n7 = cimy_uef_sanitize_content(get_cimyFieldValue($user->ID, 'NUMBER7'));
        $n8 = cimy_uef_sanitize_content(get_cimyFieldValue($user->ID, 'NUMBER8'));
        $n9 = cimy_uef_sanitize_content(get_cimyFieldValue($user->ID, 'NUMBER9'));
        $n10 = cimy_uef_sanitize_content(get_cimyFieldValue($user->ID, 'NUMBER10'));

        echo '<li><a href=# class=dir>' . $n1 . '</a></li>';
        echo '<li><a href=# class=dir>' . $n2 . '</a></li>';
        echo '<li><a href=# class=dir>' . $n3 . '</a></li>';
        echo '<li><a href=# class=dir>' . $n4 . '</a></li>';
        echo '<li><a href=# class=dir>' . $n5 . '</a></li>';
        echo '<li><a href=# class=dir>' . $n6 . '</a></li>';
        echo '<li><a href=# class=dir>' . $n7 . '</a></li>';
        echo '<li><a href=# class=dir>' . $n8 . '</a></li>';
        echo '<li><a href=# class=dir>' . $n9 . '</a></li>';
        echo '<li><a href=# class=dir>' . $n10 . '</a></li>';
        echo '</ul>';
    }

?>
4

4 に答える 4

4

短縮方法は

$numberArray = array('NUMBER1', 'NUMBER2',...);

foreach ($numberArray as $number)
{
   echo '<li><a href=# class=dir>' . cimy_uef_sanitize_content(get_cimyFieldValue($user->ID, $number)) . '</a></li>';

}
于 2012-06-09T08:14:19.763 に答える
2 
for($i = 1; $i <= 10; $i++) {
    echo '<li><a href=# class=dir>' .
    cimy_uef_sanitize_content(get_cimyFieldValue($user->ID, 'NUMBER' . $i))
   '</a></li>';
}
于 2012-06-09T08:15:37.700 に答える
1

ループを使用して配列を作成し$n、次に別のループを使用して配列を出力できます。

$count = 10;
for($i = 0; $i < $count; $i++) {
    $n[$i] = cimy_uef_sanitize_content(get_cimyFieldValue($user->ID, 'NUMBER' . $i+1));
}
for($i = 0; $i < $count; $i++) {
    echo '<li><a href=# class=dir>' . $n[$i] . '</a></li>';
}

または、個別に実行する必要がない場合は、配列をまったく必要とせずに、すべてを1つのループで実行できます。

$count = 10;
for($i = 0; $i < $count; $i++) {
    echo '<li><a href=# class=dir>' . cimy_uef_sanitize_content(get_cimyFieldValue($user->ID, 'NUMBER' . $i)); . '</a></li>';
}
于 2012-06-09T08:16:21.863 に答える
0

。これを試して

$user = wp_get_current_user();
// is there someone logged?
if ($user->ID) {
    echo '<ul class="availableindex nav nav-pills nav-stacked">';

    for($i=1; $i<=10; $i+=1){
      $t = cimy_uef_sanitize_content(get_cimyFieldValue($user->ID, 'NUMBER' . $i));
      echo '<li><a href=# class=dir>' . $n[$i] . '</a></li>';
    }
    echo '</ul>';
}
于 2012-06-09T08:20:45.193 に答える