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

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

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

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

Q&A

解決済

2回答

1269閲覧

リスト内のオブジェクトと引数を比較し、リストに存在しなければオブジェクトを追加、存在する時は別の処理をする。というプログラムを書いています。

Neko_ni_koban

総合スコア7

Java

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

0グッド

0クリップ

投稿2020/04/17 02:05

編集2020/04/17 03:03

前提・実現したいこと

持っている乗り物をリスト化し、新しく買おうとする乗り物を引数で指定しているとします。
引数で指定した乗り物とコレクション内の乗り物を比較し、リストにそれが存在しなければそのまま追加、存在すれば台数をふやす。というプログラムを組みたいです。

ここに質問の内容を詳しく書いてください。

Personクラスにおいて、find~メソッドを使って引数に指定した乗り物がリストにあるかどうかを判定し、ない場合はnullを返し、ある場合はその乗り物を返すようにしたいです。
Testでリストへの追加等は問題なく行えているのですが、同じ乗り物オブジェクトを作成してもfindメソッドでnullが返ってきてしまうので、その原因、解決法をご教授願いたいです。

該当のソースコード

java

1package vehicles; 2import java.util.*; 3 4abstract class Vehicle 5{ 6 private String color; 7 private String name; 8 9 10 public Vehicle(String color,String name) 11 { 12 if(name == null || color == null) 13 { 14 throw new IllegalArgumentException(); 15 } 16 this.color = color; 17 this.name = name; 18 } 19 20 public String getColor() 21 { 22 return this.color; 23 } 24 public String getName() 25 { 26 return this.name; 27 } 28} 29 30

java

1package vehicles; 2import java.util.*; 3 4public class Car extends Vehicle 5{ 6 private int number; 7 8 public Car(String color,String name,int number) 9 { 10 super(color,name); 11 this.number = number; 12 } 13 14 public int getNumber() 15 { 16 return this.number; 17 } 18 public String toString() 19 { 20 return super.getColor() +""+ super.getName() +""+ this.number; 21 } 22 23 public boolean equals(Car car) 24 { 25 if(car == null) 26 { 27 return false; 28 } 29 else if(!(car instanceof Car)) 30 { 31 return false; 32 } 33 else 34 { 35 if(this.number == car.getNumber() && this.getName().equals(car.getName())) 36 { 37 return true; 38 } 39 return false; 40 } 41 } 42 public int hasCode() 43 { 44 int hash = 17; 45 hash *= 31; 46 hash += this.number; 47 hash *= 31; 48 hash += this.getName().hashCode(); 49 return hash; 50 } 51}

java

1package vehicles; 2 3public class Collection 4{ 5 private Vehicle vehicle; 6 private int amount = 0; 7 8 public Vehicle getVehicle() 9 { 10 return this.vehicle; 11 } 12 public int getAmount() 13 { 14 return this.amount; 15 } 16 17 public void setVehicle(Vehicle vehicle) 18 { 19 this.vehicle = vehicle; 20 } 21 public void setAmount(int Amount) 22 { 23 this.amount += amount; 24 } 25}

java

1package vehicles; 2import java.util.*; 3 4public class Person 5{ 6 List<Collection> collectionList = new ArrayList<Collection>(); 7 8 public Collection findCollection(Vehicle vehicle) 9 { 10 int num = this.collectionList.size(); 11 if(num == 0) 12 { 13 return null; 14 } 15 for(int i = 0; i < num; i ++) 16 { 17 if(vehicle.equals(this.collectionList.get(i).getVehicle())) 18 { 19 return collectionList.get(i); 20 } 21 } 22 return null; 23 } 24 25 public void addCollection(Collection collection) 26 { 27 if(this.findCollection(collection.getVehicle()) == null) 28 { 29 this.collectionList.add(collection); 30 } 31 else 32 { 33 this.findCollection(collection.getVehicle()).setAmount(1); 34 } 35 } 36}

java

1package vehicles; 2import java.util.*; 3 4class Test 5{ 6 public static void main(String args[]) 7 { 8 Car car1 = new Car("赤","プリウス",1234); 9 Car car2 = new Car("茶","タント",1235); 10 Car car3 = new Car("青","ヤリス",1054); 11 12 Collection collection1 = new Collection(); 13 collection1.setVehicle(car1); 14 Collection collection2 = new Collection(); 15 collection2.setVehicle(car2); 16 Collection collection3 = new Collection(); 17 collection3.setVehicle(car3); 18 19 Person person1 = new Person(); 20 person1.addCollection(collection1); 21 person1.addCollection(collection2); 22 person1.addCollection(collection3); 23 System.out.println(person1.collectionList); 24 25 Car car4 = new Car("赤","プリウス",1234); 26 System.out.println(person1.findCollection(car4));null 27 System.out.println(person1.collectionList.get(0).getVehicle());→赤プリウス1234 28 System.out.println((Vehicle)car4);→赤プリウス1234 29 } 30}

試したこと

上記のようにSystem.out.printで出力しても同様の内容が出るので困ってしまっています。

補足情報(FW/ツールのバージョンなど)

初学者のため、質問もわかりづらくコードも冗長になってしまい申し訳ございません。
コードをすっきりとさせるためのアドバイス等もいただけると幸いです。
ここにより詳細な情報を記載してください。

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

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

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

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

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

guest

回答2

0

ベストアンサー

おそらくObject#equalsのオーバーライドに失敗しているのが原因でしょう。
equalsメソッドの引数の型は常にObjectにします。

Python

1public class Car extends Vehicle 2{ 3 ... 4 @Override // <= 付けておくと失敗したときエラーがでる 5 public boolean equals(Object obj) 6 { 7 if(obj == null) 8 { 9 return false; 10 } 11 else if(!(obj instanceof Car)) 12 { 13 return false; 14 } 15 16 Car car = (Car)obj; 17 ...

投稿2020/04/17 02:22

LouiS0616

総合スコア35658

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

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

Neko_ni_koban

2020/04/17 03:08

そうだったのですね!ただいま書き換えたところ思い通りの動作をしてくれているようです。ご回答ありがとうございました!また機会がありましたらよろしくお願いいたします。
guest

0

ループで見てるだけなので、hashCode関係なかったですね。

以下は、今回の件(equalsでfalseになる)に関しては直接関係ありません。

public int hasCode()

hashCodeの実装ができていません。

@Overrideアノテーションをつけるとスペルミスに気づけます。

投稿2020/04/17 02:28

編集2020/04/17 02:35
momon-ga

総合スコア4820

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

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

Neko_ni_koban

2020/04/17 03:02

すみません。findで返ってくるのはnullでした。誤字です。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問