質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.51%
Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

Q&A

解決済

1回答

10331閲覧

コンストラクタは指定された方に適応されませんというエラーについて

catcatcat527

総合スコア8

Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

0グッド

0クリップ

投稿2018/05/18 06:39

編集2018/05/18 06:45

以下のコードをコンパイルすると
クラス Itemのコンストラクタ Itemは指定された型に適用できません。
public class Cart extends Item{
^
期待値: String,int
検出値: 引数がありません
理由: 実引数リストと仮引数リストの長さが異なります
エラー1個
といったエラーが出てしまいます。

どこを直せば良いのか教えていただきたいです。

Java

1```ここに言語を入力 2public class Item{ 3 String name; 4 int price; 5 static int number = 0; 6 7 Item(String name, int price){ 8 this.name = name; 9 this.price = price; 10 number ++; 11 } 12 13 static int getNumberOfInstances(){ 14 return number; 15 } 16 17 public String toString(){ 18 return "商品名" +name+ "価格" +price ; 19 } 20} 21 22 23public class Cart extends Item{ 24 Item[] items = new Item[10]; 25 26 Item addItem(Item i){ 27 this.items[i.number - 1] = i; 28 return items[i.number-1]; 29 } 30 31 void info(){ 32 for(int i = 0; i < 10; i++){ 33 System.out.println(items[i].toString()); 34 } 35 } 36 37 public static void main(String[] args) { 38 System.out.println(Item.getNumberOfInstances()); 39 Cart cart = new Cart(); 40 Item i1 = new Item("PC", 98000); 41 Item i2 = new Item("Display", 40000); 42 cart.addItem(i1); 43 cart.addItem(i2); 44 cart.info(); 45 System.out.println(Item.getNumberOfInstances()); 46 } 47}

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

クラスItemには引数なしのコンストラクタがありませんから、
クラスCartのコンストラクタで適切にどのコンストラクタを呼ぶか指定してやらないといけません。

ただし。CartがItemを継承しているのがそもそも変です。
たとえばItemのリストをフィールドとして保持するのが良いでしょう。

追記: 配列を保持しているのですね。それなら継承関係だけ切れば良いです。

その他

気になることがいくつか。

getNumberOfInstanceがstaticメソッドであること

Java

static int getNumberOfInstances(){
return number;
}

numberを通し番号として利用すると同時に、インスタンスそれぞれの番号も無いとだめですね。


常にCartに10要素アイテムがあることを仮定していること

Java

for(int i = 0; i < 10; i++){
System.out.println(items[i].toString());
}

まだアイテムが置かれていない領域があるので、ヌルポが発生します。
むしろ先の『通し番号』的なものは、Cartインスタンスが持つべきなのではないでしょうか。

Cart#addItemが呼ばれた際に、『今保持しているアイテム数』をインクリメントすれば良いです。

コードの書き方について

teratailには、コードを見やすく表示する機能があります。
質問編集画面を開き、コードを選択した状態で<code>ボタンを押してください。
Java

投稿2018/05/18 06:43

編集2018/05/18 07:07
LouiS0616

総合スコア35658

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.51%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問