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

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

新規登録して質問してみよう
ただいま回答率
85.50%
HTML5

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

HTML

HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

CSS

CSSはXMLやHTMLで表現した色・レイアウト・フォントなどの要素を指示する仕様の1つです。

Q&A

解決済

2回答

441閲覧

スライドショーが1周したあと止まってしまいます

samepanda

総合スコア14

HTML5

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

HTML

HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

CSS

CSSはXMLやHTMLで表現した色・レイアウト・フォントなどの要素を指示する仕様の1つです。

0グッド

0クリップ

投稿2020/10/23 01:55

現在JavaScriptでスライドショーを作成しているのですが、なぜか1周目で止まってしまいます。

以下コード
HTML

<div class="slideshow"> <div> <img alt="読込中" id="mainImage"> </div> <nav> <ul> <li id="next">&gt;</li> </ul> </nav> </div>

JavaScript

const images = [ 'img/sample1.jpg', 'img/sample2.jpg', 'img/sample3.jpg', 'img/sample4.jpg', 'img/sample5.jpg', ] let currentIndex = 0; const mainImage = document.getElementById('mainImage'); mainImage.src = images[currentIndex]; const next = document.getElementById('next'); next.addEventListener('click', () => { let target = currentIndex++; if (target === images.length) { target = 0; } mainImage.src = images[target]; });

上記コード中のこの部分↓でクリックイベントが起きるとcurrentIndexが1ずつ増え、if文によって画像の順番が初期化されるはずなのですが・・・

     let target = currentIndex++; if (target === images.length) target = 0;

どうかご教授お願いいたします。

気になる質問をクリップする

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

これだとtargetは0にしてますがcurrentIndexは増え続ける処理になってると思います。

投稿2020/10/23 02:06

q_sane_q

総合スコア610

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

samepanda

2020/10/23 02:49

ご回答ありがとうございます。 確かにcurrentIndex自体を初期化しているわけではなかったです。
guest

0

ベストアンサー

currentIndexがずっとインクリメントされ続けていて、そのcurrentIndexがtargetに代入されているので、増え続けてます。

js

1let target = currentIndex++; 2console.log(target);

最初に戻すだけならtarget不要ではないでしょうか。

js

1 currentIndex++; 2 if (currentIndex === images.length) { 3 currentIndex = 0; 4 } 5 mainImage.src = images[currentIndex];

投稿2020/10/23 02:08

m.ts10806

総合スコア80765

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

samepanda

2020/10/23 02:46

ご回答ありがとうございます。 確かによく見返すと、targetがただ単に0に再代入されているだけでした。 初学者ゆえ簡単なミスに気づかず質問してしまいましたが、丁寧な御解答ありがとうございました。
m.ts10806

2020/10/23 03:06

回答の1つ目のコードのようにconsole.log()などで変数や分岐を確認していく癖をつけていくと良いと思います。 「デバッグ」といって、これを的確にできるかどうかで自己解決能力と成長の度合いが違ってきます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問