Javaを用いた素数判定について

前提・実現したいこと

はじめまして。Java初心者です。
ｎが素数か否かを判定するJavaプログラムを、for文かwhile文を使って作りたいのですが、コマンドラインから入力するとエラーメッセージがでてしまいます。調べてつくってみたのですが、ここからどうしていいのか分かりません。御手数おかけしますが、教えていただけると嬉しいです。よろしくお願いいたします。

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

javac -encoding utf-8 keisan.java 10
javac: 10は無効なフラグです
使用方法: javac <options> <source files>
使用可能なオプションのリストについては、-helpを使用します

該当のソースコード

☆Java

class keisan {
public static void main(String args[]){

int i, n;

String num1 = args[0];
n = Integer.parseInt(num1);

for (i = 2; i <= n - 1; i++){
if (n % i == 0) {
System.out.println(n + " は素数ではありません。");
}
}
System.out.println(n + " は素数です。");
}
}

行動規範の内容に同意します

回答2件

javacはJavaプログラムのソースファイルをコンパイルするツールです

javac -encoding utf-8 keisan.java

を実行してkeisan.classを生成した後、

java keisan 10

と実行すればプログラムが実行されると思います。

投稿2018/11/24 12:14

Simb

総合スコア118

nanaminnnn

2018/11/24 13:04

ご回答ありがとうございます。無事実行できました。ですが、ソースコードが間違っているようで、１は素数です。と表示されてしまいます。素数判定について教えて頂くことは出来ますか？何度も申し訳ありません。

行動規範の内容に同意します

ベストアンサー

コマンドライン引数は実行するときに渡してください。

$ javac -encoding utf-8 keisan.java

$ java keisan 10

コメントを受けて

ソースコードが間違っているようで、１は素数です。と表示されてしまいます。

インデントを揃えて良く観察すれば理由は自然と分かります。

Java

for (i = 2; i <= n - 1; i++){
if (n % i == 0) {
System.out.println(n + " は素数ではありません。");
}
}
System.out.println(n + " は素数です。");

これだとfor文内部の処理に全く関係なく、最終行が実行されます。
修正の方針は主に二つです。
0. フラグを使う方法
合成数であることが分かった時にフラグを立てる方法です。
安直ですが、何ともレガシーで残念な感じになります。
0. メソッドを早期に脱出する方法
合成数であることが分かった時点で、returnする方法です。
素数判定自体を別のメソッドに切り出せば、さらに見通しの良いコードになります。