エラーの解決法が分かりません

Question

前提・実現したいこと

Javaで修正方法が分かりません。

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

ShowingPrimeNumber.java:7: エラー: setNumに適切なメソッドが見つかりません(引数がありません)

ShowingPrimeNumber.java:7: エラー: setNumに適切なメソッドが見つかりません(引数がありません)
        System.out.println("pn1にセットされている数値："+pn1.setNum());
                                                ^
    メソッド NaturalNumber.setNum(int)は使用できません
      (実引数リストと仮引数リストの長さが異なります)
    メソッド PrimeNumber.setNum(int)は使用できません
      (実引数リストと仮引数リストの長さが異なります)
エラー1個

該当のソースコード

class NaturalNumber{
	private int num = 1;
	public void setNum(int n){
	if (n > 0){
		num = n;
	}
	}
	public int getNum(){
		return num;
	}
	public NaturalNumber(){
	}
	public NaturalNumber(int n){
		setNum(n);
	}
	}

class PrimeNumber extends NaturalNumber{

	@Override
	public void setNum(int n){
	for (int i=2; i<n; i++){
	if ( n % i == 0){
	setNum(n);
	}
	}
	}
	public PrimeNumber(){
		super();
		setNum(2);
	}
	public PrimeNumber(int x1){
		super();
		setNum(x1);
	}
}

class ShowingPrimeNumber{
	public static void main(String[] args){
	
	PrimeNumber pn1 = new PrimeNumber();
	PrimeNumber pn2 = new PrimeNumber(37);
	
	System.out.println("pn1にセットされている数値："+pn1.setNum());
	
	pn1.setNum(10);
	System.out.println("pn1にセットされている数値："+pn1.getNum());
	
	pn1.setNum(19);
	System.out.println("pn1にセットされている数値："+pn1.getNum());
	
	System.out.println("pn2にセットされている数値："+pn2.getNum());
	}
}

試したこと

検索し、似たようなエラーの解決法がないか調べましたが見つかりませんでした。
自身でも修正したが解決できませんでした。

Comments

[dodox86]

エラーが出ている箇所で、本当にsetNum()を使いたかったのでしょうか？　単純に、getNum()としたかったのをsetNum()と書いてしまってハマッているだけでは？　ご自身がやろうとしていることを理解されていますか？
※しかし、それを直すと新たな問題に直面します。setNum()の中でまたsetNum()を呼んでます。

Answer 1

そもそも基礎自体が出来ていないです。

とりあえずエラーメッセージを読みましょう。エラーメッセージにヒントが隠されています。

ShowingPrimeNumber.java:7: エラー: setNumに適切なメソッドが見つかりません(引数がありません)
        System.out.println("pn1にセットされている数値："+pn1.setNum());
                                                ^
    メソッド NaturalNumber.setNum(int)は使用できません
      (実引数リストと仮引数リストの長さが異なります)
    メソッド PrimeNumber.setNum(int)は使用できません
      (実引数リストと仮引数リストの長さが異なります)
エラー1個

メッセージを読むと、「なあ、引数が足りないんだけど?」と言われていますね。

問題のメソッドは pn1.setNum() です。

では、このpn1 とはなんでしょうか。
PrimeNumberですね。

つまり、PrimeNumberのsetNumメソッドを動かそうとしています。
では、そのsetNumメソッドの定義はなんでしょうか。

public void setNum(int n) ですね。

では、呼び出し側はどうなっているでしょうか。

System.out.println("pn1にセットされている数値："+pn1.setNum());

ですね。

…あれ？　なんか引数が足りませんよね？
定義上では int型のデータを引き受けていますが、呼び出すときは何も指定していません。

それに、意味もおかしいです。

setNumってどういう意味でしょうか。

set (オブジェクトのフィールドとして追加する) + Num (数字を)

ですよね？
でも質問にあるコードではなんか、getNum的な使い方をしている。

それに、setNumの型はvoidです。
ということは『戻り値無し』です。それなのに、戻り値を受け取ろうとしています。

class PrimeNumber extends NaturalNumber{

    @Override
    public void setNum(int n){
    for (int i=2; i<n; i++){
    if ( n % i == 0){
    setNum(n);
    }
    }
    }
    public PrimeNumber(){
        super();
        setNum(2);
    }
    public PrimeNumber(int x1){
        super();
        setNum(x1);
    }
}

のPrimeNumberクラスを見てもおかしいです。

そもそも何がしたいのでしょうか。
このクラスの役割・責務はなんでしょうか。

コードから考えられるのは、

1. 100以下の素数を保持する系
2. 素数な数字だけ持つ(ただし一つだけ)

が思いつきますね。

2の場合は、たとえば n = 100 なら素数ではないので『データなし』、
n = 3なら素数なので、『3』を持つ。

という風に。でもこの可能性はなさそう。

それなら1ということに。
でもこのコードだとそんな処理はできませんよ。

まず、ご自分のコードを読んでください。

コードを読むコツは、『一行レベルで、その行が何をしているかを考えながら読む』です。

// NutralNumberを継承してPrimeNumberを定義
class PrimeNumber extends NaturalNumber{

     // オーバーライドしたものである
    @Override
     // setNumメソッドを定義
    public void setNum(int n){
    // i の初期値: 2, i が n未満の間, +1しながらループ
    for (int i=2; i<n; i++){
    // n が 2で割り切れるなら
    if ( n % i == 0){
     // 自身を再起的に呼び出す
    setNum(n);
    }
    }
    }
    // 空のコンストラクタ？
    public PrimeNumber(){
        // 親のコンストラクタを呼び出す
        super();
        // 自身のsetNumメソッドを呼び出す
        setNum(2);
    }
    // 引数が一個あるコンストラクタ？
    public PrimeNumber(int x1){
        // 親のコンストラクタを呼び出す
        super();
        // 自身のsetNumを呼び出す
        setNum(x1);
    }
}

これをmainメソッドとかでもやってみてください。

そうすると、このコンストラクタ内で、setNumを呼び出しています。

ですが、その自身のsetNumメソッドでは、『自身を再帰的に呼び出しているだけ』です。そうなると無限ループに陥るはずです。

関数、再起メソッド？がわかっていればありえない問題です。

言いすぎましたね…。

再帰メソッドは簡単に言えば、『自身を呼び出している状態のメソッド』です。
詳しくは再帰メソッドで調べてください。

仮に『親のsetNumを呼び出したい』のなら、super.setNum(2); のようにsuperをつけるはずです。

同様に、NaturalNumberクラスも同じように読みます。

すると、『どこでデータを持っているのか』が疑問に思いませんか？

確かにNaturalNumberが保持しています。
ですが、それは『一個だけ』です。

データを複数保持したいなら配列かArrayList, LinkedListといったデータ構造？を利用するはずです。
変数では一個しか保持できません。

それと、インデントは揃えましょうです。

NaturalNumberクラスでの{と} が合っていません。個数自体は合っていますが、読みづらいです。

setNumメソッド内でもfor文内のif文もインデントがないため、読みづらい。

この程度のコードなら問題ありませんが、普通、自分用のCLIとかを作るだけでも割と複雑な構造になりやすいです。
(特に初心者の頃は)

それを『コードレビューしてくれ』とか言われたらもう、キレると思いますよ。

日本語の小説でも、改行なし、ひらがなのみ、句読点無しみたいなものです。

読めなくはないが、読む気失せますね。
(子どもが書いたものだったらわからないではないが…)

それと一緒です。

まずは基礎を理解して、今までに述べたものすべてやりましょう。それからです。

質問者さんがやるべき範囲:

■ 変数、配列
■ クラス
■ メソッド、引数、戻り値
■ プログラミングとは
■ エラーメッセージを読むこと
■ コードを読むこと
■ ロジックの考え方
■ インデントを入れて読みやすくすること
■ 可読性を上げる方法
■ デバッグ方法

Comments

[dodox86]

> 再起関数、再起メソッド？がわかっていればありえない問題です。

再"帰"です。また、質問者さんは初心の方であることは明らかなのですから、「再帰処理に関して理解している」ことが前提というのは少々無理がある、と言いますか酷ではないでしょうか。

[BeatStar]

再帰の件は…単純な変換ミスですね。修正します。

> 「再帰処理に関して理解している」ことが前提というのは少々無理がある

あー、確かに…

Answer 2

System.out.println("pn1にセットされている数値："+pn1.setNum());

setNumを引数なしで呼び出しています。

まずエラーメッセージを読みましょう。２行目にエラー箇所が出力されています。

ShowingPrimeNumber.java:7: エラー: setNumに適切なメソッドが見つかりません(引数がありません)
        System.out.println("pn1にセットされている数値："+pn1.setNum());
                                                ^
    メソッド NaturalNumber.setNum(int)は使用できません
      (実引数リストと仮引数リストの長さが異なります)
    メソッド PrimeNumber.setNum(int)は使用できません
      (実引数リストと仮引数リストの長さが異なります)
エラー1個

Answer 3

set〇〇はセッターで、返却値もvoidですし出力に使うのは間違っていると思います。
本当は pn1.setNum(10);のようにセットしてから使いたかったか、初期値を確認したかったのでは？