質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.60%

Javascriptで画像のランダム表示

解決済

回答 1

投稿

  • 評価
  • クリップ 1
  • VIEW 994

score 10

全部で20枚の画像を、
画面サイズ1600以上では横に8つ×2列の16枚(※1)、
画面サイズ1600以下では横に6つ×2列の12枚(※2)を並べようと、以下のように記述しました。
(知識が浅いので、もっと良い書き方があると思いますがご了承ください)

※1(1600以上)
■■■■■■■■
■■■■■■■■

※2(1600以下)
■■■■■■
■■■■■■

無事ランダムで並べることに成功しましたが、
画面サイズを横に引き延ばすと、ランダムですので画像がチカチカと入れ替わり、いくつかの画像の場所に「undefined」という文字が出力されてしまいます。

i < 16(あるいはi < 12)としてしまったため、読み込まれなかった画像が画面サイズを伸ばしたことにより出現してしまったのかと思い、
全ての画像を読み込もうとしてi < 16(あるいはi < 12)の部分をarr.lengthにしてしまうと、20枚全部が読み込まれてリストの形が崩れてしまいます。

「undefined」という文字を出さないため、全ての画像を読み込んだ上でリストを崩さずに表示するには、どのように記述したらよいでしょうか。
(cssでoverflowをhiddenにするとかではなく、Javascriptで解決したいです)

勉強ですので、この場所をこんな風にしたら、というヒントだけでも良いので教えていただけたら嬉しいです。
よろしくお願いいたします。

<ul id="photo">
<li><img src="../assets/img/photo001.jpg" alt=""></li>
<li><img src="../assets/img/photo002.jpg" alt=""></li>
<li><img src="../assets/img/photo003.jpg" alt=""></li>
<li><img src="../assets/img/photo004.jpg" alt=""></li>
<li><img src="../assets/img/photo005.jpg" alt=""></li>
<li><img src="../assets/img/photo006.jpg" alt=""></li>
<li><img src="../assets/img/photo007.jpg" alt=""></li>
<li><img src="../assets/img/photo008.jpg" alt=""></li>
<li><img src="../assets/img/photo009.jpg" alt=""></li>
<li><img src="../assets/img/photo010.jpg" alt=""></li>
<li><img src="../assets/img/photo011.jpg" alt=""></li>
<li><img src="../assets/img/photo012.jpg" alt=""></li>
<li><img src="../assets/img/photo013.jpg" alt=""></li>
<li><img src="../assets/img/photo014.jpg" alt=""></li>
<li><img src="../assets/img/photo015.jpg" alt=""></li>
<li><img src="../assets/img/photo016.jpg" alt=""></li>
<li><img src="../assets/img/photo017.jpg" alt=""></li>
<li><img src="../assets/img/photo018.jpg" alt=""></li>
<li><img src="../assets/img/photo019.jpg" alt=""></li>
<li><img src="../assets/img/photo020.jpg" alt=""></li>
</ul>
var $win = $(window);

$win.on('load resize', function() {
  var windowWidth = window.innerWidth;

  if (windowWidth > 1600) {
$(function() {
    var arr = [];
    $("#photo li").each(function() {
        arr.push($(this).html());
    });
    arr.sort(function() {
        return Math.random() - Math.random();
    });
    $("#photo").empty();
    for(i=0; i < 16; i++) {
        $("#photo").append('<li>' + arr[i] + '</li>');
    }
});
  } else {
$(function() {
    var arr = [];
    $("#photo li").each(function() {
        arr.push($(this).html());
    });
    arr.sort(function() {
        return Math.random() - Math.random();
    });
    $("#photo").empty();
    for(i=0; i < 12; i++) {
        $("#photo").append('<li>' + arr[i] + '</li>');
    }
});
}
});
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

読み込まれるたびに画像が並び替わるのを止めたいのですね。
原因は、「$win.on('load resize', function() {」がブラウザのサイズを替えるたびに実行され、その中で無条件で配列の並び替えを実行しているためです。
元となる配列arrを外で一度だけ作るようにすれば良いとかと思います。

var $win = $(window);
var arr = [];
$("#photo li").each(function() {
    arr.push($(this).html());
});
arr.sort(function() {
    return Math.random() - Math.random();
});

$win.on('load resize', function() {
  var windowWidth = window.innerWidth;
  $("#photo").empty();
  if (windowWidth > 1600) {
    $(function() {
      for(i=0; i < 16; i++) {
        $("#photo").append('<li>' + arr[i] + '</li>');
    });
  } else {
    $(function() {
      for(i=0; i < 12; i++) {
        $("#photo").append('<li>' + arr[i] + '</li>');
      }
    });
  }
});

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/04/20 09:43

    ありがとうございます!
    返答遅くなり申し訳ありません。

    しかも記述までしてくださって恐縮です。
    これをそのまま差し替えてみたところランダムの動作がしませんでしたので、else前にカッコを足してみたところ、ランダムの動作はしましたが、今度は全ての画像が「undefined」になってしまいました。

    ただ、考え方として画面サイズを変えるたびに実行されてしまうのを、一度にしてあげれば願いが叶う!というのがわかりましたので、ちょっと時間が掛かってしまうと思いますが、またトライしたいと思います。
    ありがとうございました!

    キャンセル

  • 2019/07/04 13:13

    ベストアンサー遅くなり申し訳ありません!
    正解例まで丁寧に書いてくださり恐縮です。
    結果から言いますと先日表示させることに無事成功しました。
    時間があるときに後学のためコード記述したいと思います。
    ありがとうございました。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.60%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る