Q&A
（質問文は編集できます）質問文のコードはコードブロックで囲んでいただけませんか？ ```（バッククオート3つ）で囲み、前後に改行をいれるか、コードを選択して「<code>」ボタンを押すとコードブロックになります。
「エラーが出ていた」では無く、エラー文を具体的にコードブロックで質問文に追記ください。
javascriptの教科書にある問題を学習していたのですが、結果がでません。
また、「検証」でエラーが出ていたのですが、理由がわかりませんでした。
どこを直したらいいか、ご教示お願いします。
合計値から平均値を出す問題です。
<!DOCTYPE html> <html lang="ja"> <head> <meta charset="UTF-8"> <title>課題</title> </head> <body> <script> var data1 = [59, 39, 100, 2, 15, 40, 84, 97]; var data2 = [63, 18, 64, 97, 50, 98]; for(i=0; i < data1.length; i++){ sum1 += data[i]; } document.write(sum1); for(j=0; j <= data2.length; j++){ sum2 += data2[j]; } document.write(sum2); ave = (sum1 + sum2)/(data1.length+data2.length); document.write(ave); </script> </body> </html>
修正したコード
<script> const data1 = [59, 39, 100, 2, 15, 40, 84, 97]; const data2 = [63, 18, 64, 97, 50, 98]; /* 以下に課題処理を追加 */ let sum1; for(i=0; i < data1.length; i++){ sum1 += data1[i]; } document.write(sum1); let sum2; for(j=0; j < data2.length; j++){ sum2 += data2[j]; } document.write(sum2); ave = (sum1 + sum2)/(data1.length+data2.length); document.write(ave); </script> </body> </html>
回答2件
0
ベストアンサー
何点か修正点がありそうです
１．
sum1、sum2の変数はfor文の外で宣言しておく。
２．
data1のループ内で、data[i] ではなく data1[i]
３．
data2のループで j <= data2.length ではなく j < data2.length
このままだと配列要素数を越えてしまって計算できません。
あとエラーになるモノではありませんが、変数宣言では「var」ではなく、
固定値のdata1、data2は「const」、変数のsum1、sum2は「let」で宣言するのがいいと思います。
投稿2019/09/16 10:12
総合スコア610
ご指摘ありがとうございました！
頂いた内容を修正したのですが、今度は、NaNがでてしまいまして...
他は、どこに間違いがあるのでしょうか？
このスクリプトでNaNになりそうなところというとdata2のループの判定（上記、３．の内容）が直っていないパターンがまず考えられますがそこはどうでしょうか？
そうでなければ、質問に現在のHTMLを追記して頂けると何かわかるかもしれません。
追記させていただきました。
お手数ですが、よろしくお願いします。
sum1、sum2は宣言時に0で初期化が必要と思います。
ありがとうございます！
できました！
また、わからないところがありましたら、教えてください。
今回は、ループをforを使ってやっていますが、JavaScriptの配列にはいろいろ関数があり、
例えばループ処理なら『forEach』、また今回のように加算していくには『reduce』などが使えたりします。
検索すると情報がたくさんあると思いますので、ゆくゆくはそちらも使用していくといいと思います。
はい！とても参考になります。
0
for(i=0; i < data1.length; i++){
sum1 += data[i];
}
の部分
data[i]がdata1[i]ではないでしょうか？
投稿2019/09/16 10:01
退会済みユーザー
総合スコア0
あなたの回答
tips
プレビュー
