Set의 contains() 메서드를 재정의해도 되나요?

Question

자바 리플렉션을 사용하여 놀고 있었고 다른 클래스에서 선언 된 메서드에서 메커니즘 캐싱 메서드를 만들고 싶습니다. 임의의 동작을 방지하기 위해 캐시에 동일한 서명이있는 메서드를로드하는 것을 금지하고 싶습니다 (다른 클래스에 선언 된 메서드는 동일한 서명을 가질 수 있습니다).

이 작업을 수행하는 유일한 방법은 메서드를 캐시하는 집합의 contains() 메서드를 재정의하는 것입니다.

위험 할 수 있습니까? 이것을 달성하기 위해 더 좋은 생각이 있습니까?

private final Set<Method> methodsCache; 

public MyMethodCachingClass(Set<Class<?>> classes) { 
    methodsCache = new HashSet<Method>(){ 
     private static final long serialVersionUID = -1467698582662452923L; 

     /** 
     * Overwriting the contains method of this Set so that we don't allow multiple methods with the same signature, 
     * even if they are declared in different classes. 
     */ 
     @Override 
     public boolean contains(Object o) { 
      if (!(o instanceof Method)) { 
       return false; 
      } 

      Method method = (Method) o; 
      for (Method m : this) { 
       if (method.getName().equals(m.getName()) && method.getParameterTypes().equals(m.getParameterTypes())) { 
        return true; 
       } 
      } 
      return false; 
     } 

    }; 

    for (Class<?> c : classes) { 
     for (Method m : c.getDeclaredMethods()) { 
      if (methodsCache.contains(m)) { 
       throw new IllegalArgumentException("The method " + m.getName() + " exists with the same signature in two different classes."); 
      } 
      methodsCache.add(m); 
     } 
    } 
} 

고마워요!

Answer 1

은 단순히 캐시 키에 대해 다음의 조합을 사용 :

클래스 이름 + 메소드 이름 + 메소드 매개 변수이 무시할 완전히 괜찮

Answer 2

유형 "포함()"방법 - 그러나, 그것을 일반적으로 불필요합니다. contains() 메서드의 목적은 단순히 컬렉션에있는 해당 객체가 이미 있는지 확인하는 것입니다.

특정 개체 컬렉션에 대한 "같음"방법이이를 확인하는 데 사용됩니다.

그러나 객체에 포함 할 수없는 contains에 대한 사용자 정의 동작을 원한다면 contains() 메소드를 해킹하는 것이 좋습니다. 내 생각에, 자바 메소드를 캐시하려고한다는 사실을 감안할 때 이러한 메소드를 포함하는 세트에 "포함하는"논리를 붙이기를 원할 수있다.

Answer 3

나는 위의 게시물 (@ jayunit100)에 동의합니다. 그러나, 나는 이것을 위해 contains()를 재정의 (override)하지 않을 것이다. 대신 Comparator의 구현을 작성하고 SortedSet을 사용합니다. 예 :

SortedSet<Method> cachedMethods = new TreeSet<Method>(new Comparator<Method>() { 
    // compare() implementation omitted 
});