エラーが出て動かない。

Question

**public class Item {**にエラーが出てしまいうまく動きません。
どうすればエラーが出ずに動くようになりますか？

Itemクラス
商品名： itemName String型
単価 ： cost int型
数量 ： quantity int型
itemListメソッド

(商品名，単価，数量，価格)
MP3,12000円,2,24000円
(商品名，単価，数量，価格)
USB,1980円,3,5940円

import java.util.Scanner;
public class CQuest01 {
public static void main(String args[]) {
	
public class Item {	
String itemName;
int cost;
int quantity;
public void itemList(){
System.out.println("(商品名，単価，数量，価格)");
System.out.println(itemName+","+cost+"円,"+quantity+","+(cost*quantity)+"円");
}
}


System.out.println("商品の数を入力してください");
Scanner sc = new Scanner(System.in);
int n = sc.nextInt();
Item im[] = new Item[n];
for(int i=0;i<n;i++){
im[i] = new Item();
System.out.println("商品"+(i+1));
System.out.println("商品名");
im[i].itemName=sc.next();
System.out.println("単価");
im[i].cost=sc.nextInt();
System.out.println("数量");
im[i].quantity=sc.nextInt();
}
for(int i=0;i<n;i++){
im[i].itemList();
}
}
}

エラー内容は以下の通りです。

Exception in thread "main" java.lang.Error: Unresolved compilation problem: 
	ローカル・クラス Item の修飾子が正しくありません。abstract または final だけが許可されています

	at CQuest01.main(CQuest01.java:5)

Comments

[y_waiwai]

エラーが出るなら、エラーメッセージをコピペでそのまま提示しましょう

Answer 1

修正がいまいちですが、これで一応通ります。
インデントはちゃんと付けましょうｄ＾＾

import java.util.Scanner;

public class Main {
    /**
     */
    public static class Item {

        String itemName;
        int cost;
        int quantity;

        public void itemList() {
            System.out.println("(商品名，単価，数量，価格)");
            System.out.println(itemName + "," + cost + "円," + quantity + "," + (cost * quantity) + "円");
        }
    }

    public static void main(String args[]) {

        System.out.println("商品の数を入力してください");
        Scanner sc = new Scanner(System.in);
        int n = sc.nextInt();
        Item im[] = new Item[n];
        for (int i = 0; i < n; i++) {
            im[i] = new Item();
            System.out.println("商品" + (i + 1));
            System.out.println("商品名");
            im[i].itemName = sc.next();
            System.out.println("単価");
            im[i].cost = sc.nextInt();
            System.out.println("数量");
            im[i].quantity = sc.nextInt();
        }
        for (int i = 0; i < n; i++) {
            im[i].itemList();
        }
    }

}

商品の数を入力してください
2
商品1
商品名
aaa
単価
123
数量
5
商品2
商品名
bbb
単価
234
数量
3
(商品名，単価，数量，価格)
aaa,123円,5,615円
(商品名，単価，数量，価格)
bbb,234円,3,702円

Process finished with exit code 0

Javaのstaticメソッドを丁寧に解説! 活用例や考え方も一緒に学ぼう!

Comments

[退会済みユーザー]

すいません...
public class Main {
　のところでエラーが出てしまいます...

[cateye]

クラス名をCQuest01に変えましょう。・・・クラス名＝ファイル名なことはわかりますよね？
既存のソースにソース貼り付けただけなんで、クラス名変わっています。

[退会済みユーザー]

横からすみませんが、何というエラーでしょう？

[退会済みユーザー]

すみません・・・かぶってしまいました。

[cateye]

Error:(13, 8) java: クラス Mainはpublicであり、ファイルMain.javaで宣言する必要があります
・・・とかでますね

[退会済みユーザー]

すみません、質問者さんに向けてのつもりでした。
エラーがでたといいつつ書かれてなかったので。

[cateye]

(`Д´)ゞﾗｼﾞｬｰ!!

[退会済みユーザー]

エラー内容を書いていなくてすいません...

Answer 2

単純に動くようにするのであれば、おそらくItemクラスについているpublicを消せば動くと思います。

現状、Itemクラスがmainメソッド内で宣言されているローカルクラスという扱いになっています。ローカルクラスはローカル変数と同様に、宣言したメソッド内でのみ有効で、そのメソッドの外側からはアクセスできません。外からアクセスできないものに対してpublicもへったくれもないので、そんな変な修飾子がついたローカルクラスはコンパイルエラーとなります。

ただ、通常ローカルクラスが有用だという場面は非常に少ないと考えられるので、通常であればcateyeさんのようにメソッドの外にstaticクラスとして宣言するか、別のクラスファイルとして作成するべきでしょうね。

Answer 3

そのエラーメッセージにでているとおりです。
main関数の中でクラスを定義するなら、示された修飾子を付加する必要があります

Comments

[退会済みユーザー]

すいません...
このエラーは初めて出たので調べてもいまいちわかりません...
示された修飾子を付加するとはどこにどう付加すればよいのでしょうか...

[y_waiwai]

そのクラスをmain関数の中で定義する必要はあるんでしょうか？
なんかわざわざややこしい方へ持っていこうとしている感じがしますが。

[swordone]

"付加できる修飾子"がabstractとfinalの2種だけだということなので、それを付加しなければならないというのは誤りです。
ローカルクラスにpublic修飾子がついていることがそもそもの原因です。