課題1（homework1-1.js）

Question

課題：0から23までのいずれかの値である整数hourを考える。このときhourが0から11までのときは「午前xx時です」、12のときは「正午です」、13から23までのときは「午後xx時です」と表示するプログラムを作成せよ。また、hourの値を様々に変えてプログラムが正しく動作していることを確かめよ。

私の回答:
let hour = 1;
if (hour<=11) {
console.log(午前${hour}時です);
} else if (hour = 12) {
console.log(正午です)}
else (hour>=13) ;{
console.log(午後${hour}時です);
}
実行すると、consoleに'午前1時です'
'午後1時です'と表示されてしまいます。
どこをどのように直せばよいのか、ご教示下さい。宜しくお願い申し上げます。

Comments

[m.ts10806]

課題でしたら出題者に聞きましょう。
いずれにしてもコード提示するときはマークダウンのcode機能を利用してご提示ください。
https://teratail.com/questions/238564

[BeatStar]

同じ質問はしないように!!!!
( https://teratail.com/questions/345896 )

質問者は質問を編集することが出来ます。「編集」でやりましょう。

[kotosato]

ご免なさい。次回から気をつけます！

[YT0014]

編集可能ですので、次回からではなく、今回も、コードのマークダウンのcode機能の利用と、重複元の削除はご対応ください。

Answer 1

整数hourは0～23までの整数であるということを前提とします。

let hour = 1;
if (hour <= 11) {
    console.log(`午前${ hour }時です`);
} else if (hour === 12) {
    console.log("正午です")
} else {
    console.log(`午後${ hour }時です`);
}

Comments

[Zuishin]

> 0から23までのいずれかの値である整数hourを考える。

0 から 23 までの整数という条件があります。
最後の else は不要でしょう。
例外をスローするべきかもしれませんが、課題なのでそれも無用だと思います。

[root_jp]

おっしゃる通り、確かにそこも前提仕様ですね。
削除します。

[Zuishin]

また、11.5 は 0 ～ 23 の数値なので、この回答のエラーメッセージでは不適切だと思います。

[root_jp]

11.5は少数であり、整数を前提としていたためなのですが、そもそも0～23という前提が漏れてました。

[kotosato]

回答していただいた皆様方、ありがとうございました!
大きな力になっています。

Answer 2

else (hour>=13) ; の末尾のセミコロン(;)を削除すれば、正しく動くと思われます。

まず、見づらいので整形しましょう。

let hour = 1;
if (hour<=11) {
    console.log(午前${hour}時です);
} else if (hour = 12) {
    console.log(正午です)
} else (hour>=13) ; {
    console.log(午後${hour}時です);
}

セミコロンは行の区切りと見なされるので、条件文がそこで終わってしまいます。

最後の console.log(午後${hour}時です); が条件文に関係なく実行されてしまうため、
'午前1時です' '午後1時です'と表示されてしまうのです。

また、以下の問題点もあります。

① (hour = 12) としていますが、=は代入を意味しています。
(hour === 12) にしましょう。

②else (hour>=13) という書き方がおかしいです。
elseはそれまでの条件分岐以外という意味になるので、(hour>=13) は不要です。

以下のようなコードにするのが正しいのではないでしょうか。

let hour = 1;
if (hour<=11) {
    console.log(午前${hour}時です);
} else if (hour === 12) {
    console.log(正午です)
} else {
    console.log(午後${hour}時です);
}

Comments

[root_jp]

これでは「午前-1時です」と出たり、「午後25時です」のように表示されてしまい

・0から11までのときは「午前xx時です」
・12のときは「正午です」
・13から23までのときは「午後xx時です」

上記の仕様通りではなくなってしまうので、範囲を絞る必要がありますね。

[kotosato]

早速のご指導ありがとうございました。もっと掘り下げて考えてみます。

[root_jp]

> so9999さん
仕様の中に、整数hourは0～23までと決められていたので、これで問題ありませんでした。
失礼しました。

[kotosato]

とんでもございません。こちらこそありがとうございました！！

Answer 3

回答にならないかもしれませんが・・・

else (hour>=13)
こういう構文大丈夫なのですかね？

else ifにするかelseだけでカッコの部分は不要なのではないかなと。

Comments

[kotosato]

ご指導ありがとうございました。仰る通りでした。全くの初心者の私にとっては十分な回答でした。

[YT0014]

このミスがあるため、
else {
(hour>=13)
}
と解釈され、セミコロンが必要というエラーメッセージが発生したのではないでしょうか？
エラーメッセージが示した箇所にエラーがあるとは限らないので、周辺のチェックもするようにしてください。

Answer 4

コード
```let hour = 23;
if (hour <=23) {
if (hour <= 11) {
  console.log(`午前${hour}時です`);
} else if (hour === 12) {
  console.log('正午です');
} else if (13 <= hour <= 23) {
          (hour = hour - 12);
  console.log(`午後${hour}時です`);
} else {
  console.log(null);
}
}

Answer 5

const times=Array(24).fill(null).map((_,x)=>x<12?`午前${x}時です`:(x,x>12?`午後${x}時です`:'正午です'));
console.log(times);