JavaScript　ifでの改行を複数行いたいが、一度しか改行されない。

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

★☆★☆★
☆★☆★☆★☆★☆★☆★☆★☆★☆★☆

該当のソースコード

JavaScript
1for(var i = 0; i< 24; i++){
2  if(i % 2 == 0){
3    document.write("★");
4  }else{
5    document.write("☆");
6  }if(i == 4){
7    document.write("<br>");
8  }

HTML
1<!DOCTYPE html>
2 <html lang="ja">
3     <head>
4         <meta charset="UTF-8">
5         <title>JavaScript</title>
6     </head>
7
8     <body>
9         <script src="script.js"></script>
10     </body>
11
12
13</html>

実現したいこと

★☆★☆★
☆★☆★☆
★☆★☆★
☆★☆★☆
★☆★☆★

↑のように表示をしたいのですが、最初の５しか改行されず、その後が改行されないのでどうすればいいのかわかりません。
こちらの原因を教えていただきたく思います。よろしくお願いします。

行動規範の内容に同意します

回答4件

if(i == 4)では、iが4のときの1度しかtrueになりません。

i % 2 == 0がループ中で2回に1回trueになることを踏まえて、5回に1回処理するようにすれば良いかと思います。

投稿2021/08/17 14:34

ishina_yum

総合スコア509

Yusedesu

2021/08/17 14:57

解決しました！ありがとうございました！

行動規範の内容に同意します

ベストアンサー

こちらの原因

i == 4
↑ループカウンタ戻したりしない限り一度しかこれを満たす状況は起きませんよ。

★と☆の切り替えが「２の倍数」なのですから、よく考えれば明白です。

実現したいこと

★☆★☆★
☆★☆★☆
★☆★☆★
☆★☆★☆
★☆★☆★

i=0で<24ならこうなるはず。
★☆★☆★
☆★☆★☆
★☆★☆★
☆★☆★☆
★☆★

１つ１つ意味を理解しましょう。

投稿2021/08/17 14:38

編集2021/08/17 14:40

m.ts10806

総合スコア80875

Yusedesu

2021/08/17 14:57

解決しました！ありがとうございました！

m.ts10806

2021/08/17 21:54

最後の一行が伝わってる気がしないので https://teratail.com/help/question-tips#questionTips4-2

行動規範の内容に同意します

参考までに。
小さなプログラムでは気にしないのですが、
大きくなってくるとデータ集めと出力は分けるとなにかといいことがあります。

javascript
1const ml = 24;
2const w = 5;
3let star = "";
4let x;
5let i;
6
7for (i=0; i<ml; i++) {
8  if (i % 2 == 0) {
9    x = "★";
10  } else {
11    x = "☆";
12  }
13  star = star + x;
14}
15
16const a = star.length % w;
17if (a > 0) {
18  star = star + "　".repeat(w - a);
19}
20for (i=0; i<(star.length / w); i++) {
21  document.write(star.substr(i * w, w) + "<br>");
22}

投稿2021/08/17 21:36

takasima20

総合スコア7464

JavaScript
1for(var i = 0; i< 25; i++){
2  if(i % 2 == 0){
3    document.write("★");
4  }else{
5    document.write("☆");
6  }if(i % 5 == 4){
7    document.write("<br>")
8  }
9}