shuffle関数について

Question

PHP初心者です。
JQueryを使った簡単なスライドショーで一度に３枚ずつ表示するスライドショーを作成しました。
そこでアクセスする度に同じ画像だと面白くないので、
アクセスする度に画像を切り替えるように設定しようと思いました。

【menu.php】(クラス指定をするためのファイル)

<?php

class Menu {
    protected $name;
    protected $img;

    public function __construct($name,$img)
    {
        $this -> name = $name;
        $this -> img = $img;
    }

    public function getName() {
        return $this -> name;
    }

    public function getImage() {
        return $this -> img;
    }

    
}

【data.php】(連想配列を格納するためのファイル)

$slide1 = new Menu('画像1','common/img/img01.png');
$slide2 = new Menu('画像2','common/img/img02.png');
$slide3 = new Menu('画像3','common/img/img03.png');

【index.php】

<?php
                shuffle($menus);
?>
            <?php foreach($menus as $menu) :?>
                <div class="swiper-slide scale">
                    <a title=" <?php echo $menu -> getName() ?>" href="#">
                        <img src="<?php echo $menu -> getImage() ?>" alt="">
                    </a>
                </div>
            <?php endforeach ?>

まず１番最初に試したのが上記です。
とりあえずshuffle関数を使ってみたという感じです。
次に

$menus = array($slide1,$slide2,$slide3);
$menusKey = array_keys($menus); //キー取り出し
shuffle($menusKey);//キーを並び替え
$newMenus = array();//結果を入れる配列
foreach($menusKey as $key) { //配列並び替え
    $newMenus[$key] = $menus[$key];
    echo $newMenus;　//結果Array
    echo '<br/>';
}

と上記のようにキーを取り出し、shuffle関数でキーを並び替えをし、空の配列を用意し、foreach文で配列を並び替えるという処理をしてみましたが、結果はArrayでした。

考えれば考えるほど、ドツボにハマり質問させていただきました。
なんとかスライドショーをシャッフルで表示させる方法はないでしょうか？
よろしくお願いいたします。

Comments

[kei344]

「１番最初に試した」ものでもできなかった、エラーが出た、ということでしょうか。

[SayakaxxSakura]

少し訂正します。エラーがでたと言うよりは何も変わらなかった。というのが正しいです。

Answer 1

んーと、PHPの場合、配列をechoすると、"Array"っていうstringが表示されるだけです。なので、

print_r($newMenus);

で、配列の中身が確認できます。

で、PHPの場合の特殊仕様として、配列の代入は参照渡しではなくcopyになるのです。
そして、shuffle()メソッドは単純に配列要素をシャッフルできるので、

$newMenus = $menus;    // ← copy

shuffle($newMenus);

するだけで良いのです。キーがどうしたとか、余計なこと考えなくて大丈夫です。

あとは、foreachで1つづつ要素を確認したい時は

foreach($newMenus as $val) 
{
    echo $val->getName() . "<br>";
}

こんなんでいいです。（要素がオブジェクトな事忘れてたのでちょっと修正しました。さらに、マジックメソッドじゃなくてゲッタ・セッタだった事も忘れてたので修正しました。）

＠kei344 2017/10/05 02:36
「１番最初に試した」ものでもできなかった、エラーが出た、ということでしょうか。

これ、何が聞きたかったのかな？　質問文、特に編集してある様子もないし、聞いたらなんか回答が変わる可能性があんのかな？　どこにも「エラーが出た」とか書いてないみたいだし、よくわかんねぇなぁ…

Comments

[kei344]

To: luckerさん
【index.php】でluckerさんが回答されている使い方をされているので、なぜ「キーを並び替え」をする必要があるのかわからないので、確認してみました。エラーについては出ていることを記載されない質問者が結構いるためです。

[退会済みユーザー]

> と上記のようにキーを取り出し、shuffle関数でキーを並び替えをし、空の配列を用意し、foreach文で配列を並び替えるという処理をしてみましたが、結果はArrayでした。

って、嵌まってる所ズバリ書いてあるのに？

[kei344]

To: luckerさん
最初の方法で不都合があるから別の方法を試したのでは、と推測しました。質問者も「何も変わらなかった」と回答しています。

[m.ts10806]

回答やそのコメントででわざわざ書くのではなく、「質問への追記・修正、ベストアンサー選択の依頼」の方でするやり取りと思いますが。

[SayakaxxSakura]

返信おそくなってしまい申し訳ございません。
非常にわかりやすい説明ありがとうございます。
初心者なばかりに曖昧な質問をしてしまい申し訳ございません。

[MasahikoHirata]

これでマイナス評価？やっかみが多いサイトですね。ってかこれ書くと、俺も多くのマイナス。なんだかなぁ。

[MasahikoHirata]

luckerさんより的確な回答を先にすれば良いのに。”思考過程”が重要と言う点を完全に無視した評価。まあ回答が気に入らなければ（自分の考えと違う）程度で”相手の意図を理解しない？”（要求仕様も作れない）程度の評価ですね。