自分で例外を投げる

Question

###前提・実現したいこと
javaの例外処理についてです。
自分で例外を投げる方法が分かりません。
この問題ではidに１００未満または９９９より大きい数字が入った場合に例外を投げるという処理を行いたいです。
その後、Kadai66クラスをtry-catchを例外処理するようにしたいです。

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

自分で例外を投げる方法がわかりません。

###該当のソースコード

/**
*番号と名前を扱うMemberクラス
*/
class Member{
    private int id;
    private String name;

    
    public Member(int id , String name){
        if(id >= 100 && id <= 999){
            this.id = id;
        }else if(id < 100 && id > 999){
            throw new MyException("IDが正しくない");
        }   
        this.name = name;
    }

    public void who(){
        System.out.println(id + ":" + name);
    }
}



/**
*idの値の範囲を設定して例外を投げるKadai66クラス
*/
public class Kadai66{
    public static void main(String args[]){
        int id;
        String name;

        id = 200;
        name = "太郎";
        Member m1 = new Member(id,name);
        m1.who();

        try{
            id = 98;
            name = "二郎";
            Member m2 = new Member(id,name);
            m2.who();
        }
        catch(MyException e){
            System.err.println("IDが正しくありません.");
        }
    }
}

###試したこと
Memberクラスのコンストラクタにて、idに代入する値が100未満または999より大きい場合に例外を投げる処理を書いてみました。

###補足情報(言語/FW/ツール等のバージョンなど)
javaです。

Answer 1

ソースコード

変更箇所にはコメントいれました。

// <変更：MyExceptionクラスを追加しました>
class MyException extends Exception{
  public MyException(String msg){
    super(msg);
  }
}

/**
*番号と名前を扱うMemberクラス
*/
class Member{
    private int id;
    private String name;


    public Member(int id , String name) throws MyException { // <変更：throwsを追加しました>
        if(id >= 100 && id <= 999){
            this.id = id;
        } else { // <変更：ifを削除しました>
            throw new MyException("IDが正しくない");
        }
        this.name = name;
    }

    public void who(){
        System.out.println(id + ":" + name);
    }
}



/**
*idの値の範囲を設定して例外を投げるKadai66クラス
*/
public class Kadai66{
    public static void main(String args[]){
        int id;
        String name;



        try{
          // <変更：移動しました>
          id = 200;
          name = "太郎";
          Member m1 = new Member(id,name);
          m1.who();


          id = 98;
          name = "二郎";
          Member m2 = new Member(id,name);
          m2.who();
        } catch(MyException e){
            System.err.println("IDが正しくありません.");
        }
    }
}

実行結果

$ javac Kadai66.java && java Kadai66
200:太郎
IDが正しくありません.

変更点

• MyExceptionクラスの作成
• Memberのコンストラクタにthrowsを追加しMyExceptionを投げることを宣言
• 論理的に問題があると思われるif文を削除
• new Member()を使うものはMyExceptionを投げる可能性があるので、try節に移動した

補足

「論理的に問題があると思われるif文」とは if(id < 100 && id > 999)のことです。

「100よりも小さく、999より大きい数」多分ないです。
多分したいことは「100よりも小さい、または、999より大きい数」だと思うので、
if(id < 100 || id > 999)です。ですが、else文で問題がないと思われるので削除しました。

Comments

[KiyoshiMotoki]

横から失礼します。

　// <変更：移動しました>
部分は、何のための修正でしょうか？

「最低限、意図通りに動くようにする」
という意味では 修正してもしなくても良いように思えたため、気になりました。

[grovion]

最低限は、`Member m1 = new Member(id,name);`を`try`の中に移動させないと、コンパイルが通りません。理由は`Member`コンストラクタが`throws MyException`するからです。

id = 200;
name = "太郎";

は`try`の外でもいいのですが、あとはソースコードの見た目の問題です。

[KiyoshiMotoki]

失礼しました。

MyException は RuntimeException ではなく Exception を拡張しているため、
仰る通り "new Member(id,name)" 部分は try 句の中に移動する必要がありますね。

[grovion]

追記：
>> MyException は RuntimeException ではなく Exception を拡張しているため、
>> 仰る通り "new Member(id,name)" 部分は try 句の中に移動する必要がありますね。

下のコメントは釈迦に説法でした。すみません

>> 横から失礼します。
質問者さんじゃなかったのですね。

もう少し、雑談をします。移動なしでコードを書く方法についてです。
その際は`MyException`を変更し、

`class MyException extends RuntimeException{`

というふうに`RuntimeException`を継承させます。

Javaの正解では、`RuntimeException`を継承させた例外クラスは`try`で囲まなくてもよくなります。「非チェック例外」と呼ばれています。

この非チェック例外（RuntimeException）の実用的な使われ方は、`java.lang.NullPointerException`などです。`NullPointerException`はいつも起こりうるので、いちいち`try`を書いてられません。ですので、非チェック例外というのが存在しているのだと思います。

[wakataku1318]

わかりやすい回答ありがとうございます。
自分で例外を投げる場合は、自分でクラスを作ることでできることがよくわかりました。
その他の指摘についても自分の知識の浅さがよく感じられました。
ありがとうございました。

Answer 2

"または" は && じゃないですよ?

"||" です。

else if(id < 100 && id > 999)
の方を

else if(id < 100 || id > 999)

にする。

なぜって?

Truth-table から考えて、

どちらかを満たす場合発動するので、
tf か ft の場合 を true としたいのですから、

tt -> t
tf -> t
ft -> t
ff -> f

となるのは and か or なら "or" になるので、

|| を使います。

例外そのものは投げられています。

たぶん、ちゃんとした動きじゃないのは、

1. 例外を投げる条件の文 else if(id < 100 && id > 999) が And ではなく Or になっていること
2. 例外を処理する側 ( mainメソッド ) で、"太郎"さん用のものがtry...catch に入っていないこと

では?

なので、else if(id < 100 && id > 999) の && を || に修正し、

例外処理区域に "太郎"さん用のオブジェクトを...

とするか、

// for "Taro" 
try{
     id = 200;
     name = "太郎";
     Member m1 = new Member(id,name);
     m1.who();
}catch( MyException e ){

}

// for "Jiro"
try{
     id = 98;
     name = "二郎";
     Member m1 = new Member(id,name);
     m1.who();
}catch( MyException e ){

}

のように個別でやるか。

あと、MyExceptionってありますが、定義されていますよね？

存在しないクラスを使おうとしても無理ですから。

Comments

[wakataku1318]

回答ありがとうございます。
初歩的なミスをしていました・・・。
ご指摘感謝致します。

Answer 3

学校の課題なら自分で解くべきです、人に答えを教えてもらっては意味がないですよ。
また、参考書等にも書いてある内容です。

Answer 4

以下で、自分で例外を投げる事はできるように思えます。

class Member{
    private int id;
    private String name;


    public Member(int id , String name) throws Exception{
        if(id >= 100 && id <= 999){
            this.id = id;
        }else if(id < 100 || id > 999){
            throw new Exception("IDが正しくない");
        }   
        this.name = name;
    }

    public void who(){
        System.out.println(id + ":" + name);
    }
}



/**
*idの値の範囲を設定して例外を投げるKadai66クラス
*/
public class KON{
    public static void main(String args[]){
        int id;
        String name;

        id = 200;
        name = "太郎";


        try{
        Member m1 = new Member(id,name);
        m1.who();
            id = 98;
            name = "二郎";
            Member m2 = new Member(id,name);
            m2.who();
        }
        catch(Exception e){
            System.err.println("IDが正しくありません.");
        }
    }
}