teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

1

質問文編集を受けて回答追記

2019/01/16 09:53

投稿

dice142
dice142

スコア5158

answer CHANGED
@@ -6,4 +6,20 @@
6
6
  > 残り時間が1時間を切ったら1秒ごとの呼び出しに変更するというプログラムを作成したい
7
7
 
8
8
  なので、上記の判定をして、1時間以上あれば実装中の表示を、
9
- 1時間を下回っていれば残り時間を秒に変換した値を表示すれば良いですね。
9
+ 1時間を下回っていれば残り時間を秒に変換した値を表示すれば良いですね。
10
+
11
+ ---
12
+
13
+ > 問題❶
14
+ goalの時間を1時間未満にすると表示されない。
15
+ どこが問題か教えていただけますでしょうか。
16
+
17
+ ブラウザの開発者ツールを表示するとコンソールにエラーが表示されています。
18
+ `recalc`関数の定義が、呼び出し部分より後に書かれていることが原因かと思います。
19
+
20
+ > 問題❷
21
+ JSは本当に始めたばかりですが、このコードは不細工すぎると思うので、少し改善点を教えてください。
22
+
23
+ 同じ処理は関数化するとかいろいろできるとは思いますが、まずは動作するものを作ることが優先かと思います。
24
+ 動作できたらコードを見直して冗長なところとかを直していきましょう。
25
+ (どういう処理を作れば良いのか計画を練ってから作ると割とはじめからきれいに書けたりもしますが。)