javaでのコンストラクターの未定義エラーおよび、計算方法について

java初心者です。私の大学の教授から学習のためいただいたプログラムです。自分で調べても解決に至らなかったので、どなたかお力を貸していただけたら嬉しいです。

こちらのプログラムですが、Appleとオラクルの名前をgetSymbolで表示し、changePriceにて株価の変更計算を行い、getPriceにて、変更後の価格を表示するプログラムなのですが、new Stock("AAPL", 226.10);とnew Stock("ORCL", 48.55);の未定義のエラーが出ていまいます。価格変更メソッドでの計算方法もわかりません。

ご協力頂ける方、よろしくお願いします。
また、今回のプログラムで使われている内容が記載されていて、オススメの参考書などございましたら、教えて頂けると嬉しいです。

public class Stock {

	String symbol;
	double price;
		public static void main(String[] args) {

		Stock apple1 = new Stock("AAPL", 226.10);
        apple1.changePrice(25.0);	// Apple up 25.0%!

        System.out.println(apple1.getSymbol() + " now at: " +
                                                  apple1.getPrice());

        Stock oracle = new Stock("ORCL", 48.55);
        oracle.changePrice(-10.0);	// Oracle down 10.0%!

        System.out.println(oracle.getSymbol() + " now at: " +
                                                  oracle.getPrice());
	}
	public String getSymbol() {
		return symbol;
    }
    public double getPrice() {
    	return price;
    }
    public void changePrice(double byPercent) {
    	
    }

}

m.ts10806

2018/09/06 23:26

タイトルはなるべく質問内容の「要件」部分を簡潔にまとめたものに修正してください。このタイトルだと具体的に何を聞きたいのかがわかりません。あとは、ご自身が理解されている部分をどのような形でも結構ですので、少しでも良いので記載してください。そのうえで「何がどうわからないのか」を書くとよいです。この書き方だと何を聞きたいのかが伝わりづらいように思います。また、このコードがどのような経緯で出てきたものなのか、など背景や経緯もあれば回答も得やすくなります。

退会済みユーザー

2018/09/06 23:36

簡単なら回答する必要はないよね

Masaki21

2018/09/06 23:39

変更させていただきました。私にとっては難しいです笑

退会済みユーザー

2018/09/06 23:43

個人的かどうかは別として課題としてもらったなら課題提出者に質問するのが筋

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

回答2件

ベストアンサー

java
1public class Stock {
2
3    String symbol;
4    double price;
5    
6    // これを追加
7    private Stock(final String symbol, final double price) {
8        this.symbol = symbol;
9        this.price = price;
10    }
11    
12    public static void main(String[] args) {
13
14        Stock apple1 = new Stock("AAPL", 226.10);
15        apple1.changePrice(25.0);    // Apple up 25.0%!
16
17        System.out.println(apple1.getSymbol() + " now at: " +
18                                                  apple1.getPrice());
19
20        Stock oracle = new Stock("ORCL", 48.55);
21        oracle.changePrice(-10.0);    // Oracle down 10.0%!
22
23        System.out.println(oracle.getSymbol() + " now at: " +
24                                                  oracle.getPrice());
25    }
26    
27    public String getSymbol() {
28        return symbol;
29    }
30    
31    public double getPrice() {
32        return price;
33    }
34    
35    public void changePrice(double byPercent) {
36
37    }
38
39}

new Stock("AAPL", 226.10);と書かれているので、Stockクラスに引数を渡してインスタンスを生成しようとしています。
が、Stockクラス（自分自身）は、引数を受け取るコンストラクタが定義されていません。
コンストラクタが省略できるのは、引数なしのデフォルトコンストラクタの場合だけです。
引数受け取ったらその引数どうすりゃいいのってなりますから、どうするかこっちで教えてあげないといけないわけです。

で、クラス変数としてsymbolとpriceが定義されていたので、それにぶっこんでます。

コンストラクタのアクセス修飾子をprivateにしているので、自分自身の中からしかインスタンスを作れないようになっています。
1クラスのプログラムのようだったのでそうしてあります。
main以外の他のメソッドも、この場合ならprivateでいいと思います。

アクセス修飾子は適切に設定する癖をつけた方がいいです。
プログラムが巨大になるほど恩恵を感じられるはず。

投稿2018/09/06 23:51