Jeśli ostateczny obiekt jest przekazywany, powinno być nadal zaznaczone zero?

Question

Sprawdź poniższy kod:

@Override 
    public int compareTo(final Intersection o) { 
     if (o == null) 
      return 0; 

     double distance = t; 
     double distance2 = o.t; 

     if (distance == distance2) 
      return 0; 


     return distance2 > distance ? -1 : 1; 
    } 

---

Wszystko wydaje się dobrze jednak fakt, że 0 mogą zostać zwrócone na dwóch różnych okazjach warunkowych ma nieco przeszkadza mi. Jeśli ja jednak przenieść zmienne przyporządkowania distance i distance2 do góry, mój IDE ostrzega mnie, że

  if (o == null) 
       return 0; 

byłby wtedy „martwy” kod. Jeśli tak, to czy w tym scenariuszu powinna być sprawdzana wartość null?

---

Chodzi mi o to:

@Override 
    public int compareTo(final Intersection o) { 

     double distance = t; 
     double distance2 = o.t; 

     if (o == null) 
      return 0; 

     if (distance == distance2) 
      return 0; 


     return distance2 > distance ? -1 : 1; 
    } 

Answer 1

końcowy może być null, to po prostu oznacza, że ​​nie można przypisać i jest widoczny w anonimowych klas wewnętrznych itp

w Twoim przypadku, problem jest inny: (patrz komentarze)

public int compareTo(final Intersection o) { 

    double distance = t; 
    double distance2 = o.t; // here you use it, potentially causing NPE 

    if (o == null) // if o was null, this is never reached 
     return 0; 

    if (distance == distance2) 
     return 0; 


    return distance2 > distance ? -1 : 1; 
} 

Answer 2

Nie jestem do końca pewien, dlaczego uważasz, że nie jest to legalne, albo że to nie może wystąpić, ponieważ parametr jest final:

yourClass.compareTo(null); 

należy zawsze sprawdzić null w scenariuszach, które opierają się na przykład będąc obecny, korzystniej przed użyciem I nstance.

Oznaczenie parametru final uniemożliwia aktywną zmianę referencji lub mutację wartości podczas korzystania z metody; jest to sposób na wyrażenie, że ten kod jest wolny od efektów ubocznych od wartości przekazanej.

Ponadto, zauważam problem z twoją metodą compareTo; jeśli obiektem, który porównujesz jest null, to komparator oznacza go jako odpowiednik. Będziesz także napadał na przypadki o skrajnym poziomie z NaN i ==, ponieważ dwie wartości double mogą nie być równoważne z ,.

Prawdopodobnie chcesz coś takiego zamiast:

@Override 
public int compareTo(final Intersection o) { 

    double distance = t; 
    double distance2 = o == null ? 0 : o.t; 

    return Double.compare(distance, distance2); 
} 

Answer 3

Jest to odniesienia do obiektu, która jest przekazywana i jako taki może być zerowy lub niezerowy niezależnie od pogody to jest ostateczna, czy też nie.

Answer 4

Twój ide ostrzega, ponieważ ostateczna obiekt może być null i można uzyskać wyjątku null pointer na

double distance2 = o.t; 

stąd oświadczenie zwrot lub jakiekolwiek oświadczenie wewnątrz jeśli o == null nigdy nie osiągnie/nie wykona

if (o == null) 
    return 0;