-2

この foreach 配列に array_reverse 関数を実装しようとしていますが、3 行目 (まあ、4) で間違った変数を呼び出していると思います。

<?php
    $j=1;
    $skill_slugs = array_reverse( $skill_slugs );

    foreach ($skills as $skill) {               
        $skill = get_term_by( 'slug', trim(htmlentities($skill)), 'skill');
        if($skill) {
            $skill_slug = $skill->slug;             
            $skill_slugs .= $skill_slug . ",";
            $a = '<li><a href="#" data-filter=".'.$skill_slug.'">';
            $a .= $skill->name;                 
            $a .= '</a></li>';
            echo $a;
            echo "\n";
            $j++;
        }         
    }
?>
4

1 に答える 1

1

新しく反転した配列を循環させたい場合...

foreach ($skills as $skill)

する必要があります

foreach ($skill_slugs as $skill)

アップデート

// This array is populated within the if-block below
$slugs = array();

// Foreach item in the reversed array, assign a handle of "skill_slug"
foreach ( array_reverse( $skill_slugs ) as $skill_slug ) {

  // Attempt to get Term Data from WordPress
  $s = get_term_by( 'slug', trim(htmlentities($skill_slug)), 'skill' );

  // If Term Data was returned, increment $j and proceed
  if ( $s && $j++ ) {

    // Push Term slug into our $slugs array
    array_push( $slugs, $s->slug );

    // Output a new list-item for this Term
    printf("<li><a href='#' data-filter='%s'>%s</a></li>", $s->slug, $s->name);

  }

}

ここには、各変数に選択した名前に混乱を招くようなあいまいさがたくさんあります。私はここでそれを少し分解しようとしています。さらに、それほど多くのステートメントは必要ないためecho、書式設定された 1 つの文字列に減らしました。最後に、スラッグを文字列に連結するのではなく、配列にプッシュしています。間にコンマを入れてリストしたい場合は、 juse join():

$slugs = array( "PHP", "JavaScript", "C#" );

// PHP, JavaScript, C#
echo join( ", ", $slugs );
于 2012-05-27T22:24:07.773 に答える