Yarn: 5개의 getEntities 메소드가 있어서는 안 됩니다.

1.16-rc1+build.12 현재 ServerWorld getEntities 에 대해 5가지(!) 메서드를 사용할 수 있습니다.

• EntityView#getEntities(<strong i="10">@Nullable</strong> Entity, Box) 는 첫 번째 인수의 항목을 제외한 상자의 모든 항목을 가져옵니다.
• EntityView#getEntities(<strong i="14">@Nullable</strong> Entity, Box, Predicate) 는 술어와도 일치하는 첫 번째 인수의 항목을 제외 하고 상자의 모든 항목을 가져옵니다.
• ServerWorld#getEntities(<strong i="18">@Nullable</strong> EntityType, Predicate) 는 첫 번째 인수의 유형 및 술어와 일치하는 전 세계의 모든 엔티티를 가져옵니다.
• World#getEntities(<strong i="22">@Nullable</strong> EntityType, Box, Predicate) 는 첫 번째 인수의 유형 및 술어와 일치 하는 상자의 모든 엔티티를 가져옵니다.
• EntityView#getEntities(Class, Box, Predicate) 는 첫 번째 인수의 클래스 및 술어와 일치 하는 상자의 모든 엔티티를 가져옵니다.

이것이 왜 문제입니까?

• 그 중 2개는 첫 번째 인수를 제외 하고 3개는 첫 번째 인수로 필터링합니다 . IMO 이것은 이름을 변경하기에 충분한 이유입니다.
• null 를 첫 번째 매개변수로 사용하여 세 개의 인수 형식을 호출할 때 호출이 모호하기 때문에 호출 사이트에서 null 를 캐스팅해야 합니다.
  
  (^ IntelliJ의 조언을 받아 Class 캐스팅하면 런타임에 NPE가 됩니다. btw, 그 방법은 null-safe하지 않기 때문입니다.)

제안 된 해법:

• 모호한 구문 분석을 지우려면:
  
  
  
  • 받아 두 방법 바꾸기 Entity 하는 제 파라미터로서 getEntitiesExcept .
  
  
  • getEntities(Class, ...) 을 getEntitiesOfClass 또는 이와 유사한 것으로 변경합니다.
  
  
• 어쩌면하지 않는 하나의 이름을 변경 Box , 난 몰라,로를 getAllEntities .
• 이러한 이름 바꾸기를 WorldChunk 메서드에 전파하십시오. 이 모든 것이 위임됩니다.