ジェネリック型コレクションでは、オーバーライドされた等価メソッドでgetClassをチェックする必要がありますか？

Question

ジェネリックコレクションでは、同じタイプのオブジェクトを使用できます。したがって、等しいメソッドをオーバーライドする場合は、同じタイプのオブジェクトをチェックするためにgetClassを使用する必要があります。

私の質問は(obj.getClass()!=this.getClass())です。このコードは、コレクションの汎用タイプとして使用されるクラスと同じメソッドで必要ですか。 私たちは汎用型のコレクションを知っている、我々はは同じ種類のオブジェクトを追加することができます。したがって、等価メソッドからこのコードを削除すると、問題はありますか？

@Override 
    public boolean equals(Object obj) { 
     if(obj==null){ 
      return false; 
     } 
     if(this == obj){ 
      return true; 
     } 
     Student std = (Student)obj; 
     return name.equals(std.name) && rollNum == std.rollNum; 
    } 

の新オーバーライド同じ方法で、我々は一般的なコレクションでこのクラスを使用している場合、私の質問は、我々は同じ方法で使用するのはgetClass（）を必要とするんですが、私は、そのコード行が関連はないと思います一般的なオブジェクトの追加異なるタイプに許可されませんので、

import java.util.*; 
public class HashSetDemo { 
    public static void main(String [] args){ 
      HashSet<Student> hashSet = new HashSet<Student>(); 
      hashSet.clear(); 
      Student std1 = new Student(11, "qaa"); 
      Student std2 = new Student(21, "aa"); 
      Student std3 = new Student(31, "saa"); 
      Student std4 = new Student(51, "aad"); 
      Student std5 = new Student(71, "aad"); 
      hashSet.add(std1); 
      hashSet.add(std2); 
      hashSet.add(std3); 
      hashSet.add(std4); 
      hashSet.add(std5); 
    } 
} 

class Student{ 
    int rollNum; 
    String name; 
    public Student(int rollNum, String name){ 
     this.rollNum = rollNum; 
     this.name = name; 
    } 

    @Override 
    public int hashCode() { 
     //some code 
    } 
    @Override 
    public boolean equals(Object obj) { 
     if(this == obj){ 
      return true; 
     } 
     if(obj == null || (obj.getClass()!=this.getClass())){ 
      return false; 
     } 
     Student std = (Student)obj; 
     return name.equals(std.name) && rollNum == std.rollNum; 
    } 
} 

Answer 1

Khelwoodが提供するコメントに同意し、また、我々はそれができるチェックヌルと&&条件で検証されている||objがある場合理由nullの場合、NullpointerExceptionが発生します。

if(obj == null || (obj.getClass()!=this.getClass())){ 
     return false; 
    } 

Answer 2

@khelwoodはrigthです。そして、あなたのコード内で「場合は、」間違っている：

if(obj == null && (obj.getClass()!=this.getClass())){ 
     return false; 
    } 

それが何をするか：objがnullの場合は、次のobj.getClassでNPEを取得します（）。 nullにはクラスがありません。 :-)

if (!(obj instanceof Student)) { return false;} 

で十分です。 nullはどのクラスのインスタンスでもありません。

Answer 3

equalsは、一貫性のある対称、推移、再帰可能とnullためfalseを返さなければなりません。

obj instanceof Studentの問題は対称です。 MasterStudent extends Studentがあり、 "instanceof CurrentClassのパターンに従ってCurrentClassのフィールドを比較する"と仮定します。これはStudentに実装されているのと同じです。この場合、簡単にstudent.equals(masterStudent) == trueとmasterStudent.equals(student) == falseが得られます。

あなたは、あなたの質問を編集し、この部分を削除しました：

ジェネリックコレクション、同じメソッドをオーバーライドしながら、我々はサブクラスをチェックするためのinstanceofかのgetClassを使用する必要がないので、オブジェクトの同じタイプを許可または同じ種類のオブジェクト。

基本的には、クラスがequalsであることを確認する必要があるかどうかを尋ねるだけで、そのタイプがコレクションによって保証されることを示唆しています。

短い回答はもちろんです。equalsでクラスをチェックする必要があります。使用可能な用途に関係なくequalsを実装する必要があります。