入力した値を合算して返すメソッドを作成したい。

Question

###前提・実現したいこと
入力した値を合算して返すメソッドを作成したい。
このコードの間違っているところを教えていただけますでしょうか。
よろしくお願いいたします。

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

public final class NumberExercise{
 public static Long addition(final integer a, final integer b)throws IllegalArgumentException{
  return (long)a + (long)b;
 }
}

Comments

[LouiS0616]

エラーは出ませんが。どうして間違っていると考えているのでしょう？？？設計に関する話でしょうか？

[b_b_b]

すみません。コードに間違いがありました。修正させていただきます。

Answer 1

プログラミングでは、値の型を変えることをキャストと呼びます。
調べてみたらすぐ出ましたよ。目的は逆向きですが。
テックノート：【Java】LongオブジェクトからIntegerオブジェクトへ変換する方法

このページに従うならば、次のように書けばいいです。(コメントも見てね。)

public static Long addition(final Integer a, final Integer b) {
    return new Long(a) + new Long(b);
}

...integerをIntegerに直したりもしましたが。

---

そもそもプリミティブ型でいいんじゃなかろうか。

public static long addition(final int a, final int b) {
    return (long)a + (long)b;
}

Comments

[swordone]

無駄が多いです。

[退会済みユーザー]

そもそもナンバー型の型変換は
https://docs.oracle.com/javase/8/docs/api/java/lang/Number.html#longValue--
じゃろ　文字列挟むなｗ・）

[LouiS0616]

桁あふれ起きないですもんね。

[LouiS0616]

いや、return a.longValue() + b.longValue();の方がいいのかな...？

[swordone]

そもそもラッパークラス使うならvalueOfメソッドが推奨されてます。

[LouiS0616]

If a new Long instance is not required, this method should generally be used in preference to the constructor Long(long). https://docs.oracle.com/javase/7/docs/api/java/lang/Long.html#valueOf(long)
これですね。

[LouiS0616]

勉強になります、コメントありがとうございます。

[swordone]

long→intの桁あふれチェックも今やMath.toIntExact()がありますし

[LouiS0616]

1.8で追加ですか、進化してますね。

Answer 2

public class Sum00 {
    public static void main(String[] args) {
        System.out.println(addition(1, 2));
        System.out.println(addition(new Integer(1), new Integer(2)));
        System.out.println(addition(Integer.MAX_VALUE, Integer.MAX_VALUE));
        System.out.println(addition(Integer.MIN_VALUE, Integer.MIN_VALUE));
        try {
            System.out.println(addition(1, null));
        } catch(Exception e) {
            System.out.println(e);
        }
    }

    public static Long addition(final Integer a, final Integer b) throws NullPointerException {
        if (a == null) {
            throw new NullPointerException("a: null");
        }
        if (b == null) {
            throw new NullPointerException("b: null");
        }
        return (long)a + b;
    }
}

実行例

Answer 3

引数型に使っているintegerという型は存在しません。Integer(一文字目が大文字)ならありますが。