回答編集履歴

追記

2018/05/28 03:29

投稿

LouiS0616

スコア35660

test CHANGED Viewed

@@ -163,3 +163,137 @@
 時／分／秒について、それぞれ入力を求めるように書き直せば良いんじゃないでしょうか。
+---
+もし例外を使うならこんな感じです。
+```Java
+class Clock {
+    ...
+    public void setHour(int hour) {
+         if(hour > 0 && hour <= 23){
+             this.hour = hour;
+         } else {
+             throw new IllegalArgumentException();
+         }
+    }
+    ...
+}
+```
+```Java
+while(true){
+    try{
+        System.out.print("hourの数値を入力してください＞" + " ");
+        String str = br.readLine();
+        clock.setHour(Integer.parseInt(str));
+    }catch(IllegalArgumentException e){
+        System.err.println("[ERROR] 数値を再入力してください");
+    }
+}
+```
+---
+もし返り値を使うならこんな感じです。
+```Java
+class Clock {
+    ...
+    public boolean setHour(int hour) {
+         if(hour > 0 && hour <= 23){
+             this.hour = hour;
+             return true;
+         } else {
+             return false;
+         }
+    }
+    ...
+}
+```
+```Java
+while(true){
+    try{
+        System.out.print("hourの数値を入力してください＞" + " ");
+        String str = br.readLine();
+        boolean result = clock.setHour(Integer.parseInt(str));
+        if(result) {
+            break;
+        } else {
+            System.err.println("[ERROR] 数値を再入力してください");
+        }
+    }catch(NumberFormatException e){
+         System.err.println("[ERROR] 数値を再入力してください");
+    }
+}
+```

追記を受けて

2018/05/28 03:29

投稿

LouiS0616

スコア35660

test CHANGED Viewed

@@ -131,3 +131,35 @@
 これはとりあえずコンパイルエラーを排除してから考えましょう。
 いちおうヒントとしては、『セッターの中で値をチェックすれば良い』とだけ言っておきます。
+質問追記を受けて
+---
+> 一度入力した数値が全ての値の範囲内であれば、全て入力されてしまいます…。
+単に入力を一度しか受け付けていないからかと。
+> ```Java
+System.out.print("数値を入力してください＞" + " ");
+String str = br.readLine();
+> clock.setHour(Integer.parseInt(str));
+clock.setMinute(Integer.parseInt(str));
+clock.setSecond(Integer.parseInt(str));
+> ```
+時／分／秒について、それぞれ入力を求めるように書き直せば良いんじゃないでしょうか。

回答書き込み

2018/05/27 14:09

投稿

LouiS0616

スコア35660

test CHANGED Viewed

@@ -1 +1,133 @@
+質問する際には、**必ず**実際のエラーメッセージを載せるようにしてください。
+インスタンス化ができませんというエラー
+---
+Clockをインスタンス化するときに適切な引数を与えていないのが原因です。
+**Clockのコンストラクタの定義**
+> ```Java
+Clock(int hour, int minute, int second){
+    this.hour = hour;
+    this.minute = minute;
+    this.second = second;
+}
+> ```
+こちらでは、int型の引数をみっつ受け取るようなインターフェースになっています。
+**実際にインスタンス化を試みている箇所**
+> ```Java
+Clock clock = new Clock();
+> ```
+しかし、実際には引数が与えられていません。これが問題です。
+#####解消するためには
+次の二つのアプローチが考えられます。
+- インスタンス生成時に、適切な引数を渡すようにすること
+- 引数なしのコンストラクタを用意し、各フィールドを適切に初期化すること
+clock.setHourに引数がありませんというエラー
+---
+同じように、メソッドのインターフェースに沿った呼び出しがされていないからです。
+**Clock#setHourの定義**
+> ```Java
+public void setHour(int hour) {
+    this.hour = hour;
+}
+> ```
+int型の引数をひとつだけ受け取るように書かれています。
-誤って投稿してしまいました。しばしお待ちください。
+**実際にメソッドを呼び出そうとしている箇所**
+> ```Java
+clock.setHour() = Integer.parseInt(str);
+> ```
+これは完全に使い方を誤っていますね。
+セッターと言えど、その実態はふつうのメソッドと一緒です。
+#####解消するためには
+Integer.parseInt(str)の結果を、メソッドの**引数として**渡してやればいいです。
+```Java
+clock.setHour(Integer.parseInt(str));
+```
+冗長な感がありますが、二行に分けて書くことも出来ます。
+```Java
+int hour = Integer.parseInt(str);
+clock.setHour(hour);
+```
+hourは0~23,minuteは0~59,secondは0~59の間の値しか設定できないように制限すること。
+---
+これはとりあえずコンパイルエラーを排除してから考えましょう。
+いちおうヒントとしては、『セッターの中で値をチェックすれば良い』とだけ言っておきます。

修正

2018/05/22 15:52

投稿

LouiS0616

スコア35660

test CHANGED Viewed

@@ -1,5 +1 @@
-質問にエラーがそのまま記載されていませんので、文言は違うかもしれません。
+誤って投稿してしまいました。しばしお待ちください。
->