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

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

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

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

Q&A

解決済

1回答

11220閲覧

setAttributeメソッドのエラーについて

Suffenjoying

総合スコア10

JavaScript

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

0グッド

0クリップ

投稿2021/02/28 01:52

前提・実現したいこと

簡単な神経衰弱のアプリをjavascript で作っています。

setAttributeメソッドを使った際にエラーが発生しました。

発生している問題・エラーメッセージ

エラーメッセージ Uncaught TypeError: Cannot read property 'setAttribute' of undefined at checkForMatch (app.js:86)

該当のソースコード

ソースコード html <!DOCTYPE html> <html lang="en"> <head> <meta charset="UTF-8"> <meta http-equiv="X-UA-Compatible" content="IE=edge"> <meta name="viewport" content="width=device-width, initial-scale=1.0"> <link rel="stylesheet" href="styles.css"> <title>Memory Game</title> </head> <body> <h3>Score:<span id="result"></span></h3> <div class="grid"> </div> <script src="app.js"></script> </body> </html>
Javascript document.addEventListener('DOMContentLoaded', () => { const cardArray = [ { name: 'apple', img: './images/apple.jpeg' }, { name: 'apple', img: './images/apple.jpeg' }, { name: 'banana', img: './images/banana.jpeg' }, { name: 'banana', img: './images/banana.jpeg' }, { name: 'kiwi', img: './images/kiwi.jpeg' }, { name: 'kiwi', img: './images/kiwi.jpeg' }, { name: 'strawberry', img: './images/strawberry.jpeg' }, { name: 'strawberry', img: './images/strawberry.jpeg' }, { name: 'watermelon', img: './images/watermelon.jpeg' }, { name: 'watermelon', img: './images/watermelon.jpeg' }, { name: 'orange', img: './images/orange.jpeg' }, { name: 'orange', img: './images/orange.jpeg' }, ] cardArray.sort(() => 0.5 - Math.random()) const grid = document.querySelector('.grid') const resultDisplay = document.getElementById('result') var cardChosen = [] var cardChosenID = [] var cardsWon = [] function createBoard() { for (let i = 0; i<cardArray.length; i++) { var card = document.createElement('img') card.setAttribute('src', 'images/blank.jpeg') card.setAttribute('data-id', i) card.addEventListener('click', flipCard) grid.appendChild(card) } } //check for matches function checkForMatch () { var cards = document.querySelector('img') const optionOneId = cardChosenID[0] const optionTwoId = cardChosenID[1] if (cardChosen[0] === cardChosen[1]) { alert('You found a match') cards[optionOneId].setAttribute('src', './images/clown.jpg') cards[optionTwoId].setAttribute('src', './images/clown.jpg') cardsWon.push(cardChosen) 85 } else { 86 cards[optionOneId].setAttribute('src', 'images/blank.jpeg') 87 cards[optionTwoId].setAttribute('src', 'images/blank.jpeg') alert('Sorry, try again!') } cardChosen = [] cardChosenID = [] resultDisplay.textContent = cardsWon.length if (cardsWon.length === cardArray.length/2) { resultDisplay.textContent = 'Congratulations! You found them all!' } } //flip your card function flipCard() { var cardId = this.getAttribute('data-id') cardChosen.push(cardArray[cardId].name) cardChosenID.push(cardId) this.setAttribute('src', cardArray[cardId].img) if (cardChosen.length === 2){ setTimeout(checkForMatch, 500) } } createBoard() })

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

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

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

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

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

guest

回答1

0

ベストアンサー

cardsをquerySelectorで取っているので、ここで取れるのは1エレメントだけです。

querySelectorAllが適切なのではないでしょうか。

投稿2021/02/28 03:12

maisumakun

総合スコア145183

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

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

Suffenjoying

2021/02/28 09:17

ALLに修正してエラー解決できました! ありがとうございました!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問