-Test nur, wenn die Variable nicht null ist, wenn Anweisung

Question

Ich habe die folgende Methode, wo ich nur die event.status Eigenschaft testen will, ob status hat bestanden:

def findEvent(String desc, String status = null, Collection events) { 
     return events.find { 
      it.description == desc && \\If status is not null: it.status == status 
     } 

     throw new Exception("Review Event Record Not Found: ${desc}") 
} 

Ich dachte, es wie dies geschehen könnte, aber es scheint nicht zu funktionieren:

def findEvent(String desc, String status = null, Collection events) { 
     return events.find { 
      it.description == desc && (status != null ?: {it.status == status}) 
     } 

     throw new Exception("Review Event Record Not Found: ${desc}") 
} 

Gibt es eine Möglichkeit, dies zu tun? Oder muss ich zurück zu etwas wie diesem gehen:

if (status != null) { 
    return events.find { 
     it.description == desc && it.status == status 
    } 
} else if (status == null) { 
    return events.find { 
     it.description == desc 
    } 
} 

Gibt es eine Art von Best Practice?

Answer 1

Ich glaube nicht, dass der Ausdruck so sensibel ist wie es ist.

Elvis bedeutet "wenn truthy, verwenden Sie den Wert, sonst verwenden Sie diese andere Sache."

Ihre "andere Sache" ist eine Schließung, und der Wert ist status != null, von denen keiner scheint, was Sie wollen. Wenn status null ist, sagt Elvis true. Ist dies nicht der Fall, erhalten Sie eine zusätzliche Verschlussschicht.

Warum können Sie nicht nur verwenden:

(it.description == desc) && ((status == null) || (it.status == status)) 

Auch wenn die tat nicht Arbeit, alles, was Sie brauchen, ist die Schließung des entsprechenden Wert zurück, nicht wahr? Es ist nicht notwendig, zwei separate find Aufrufe zu erstellen, verwenden Sie einfach eine Zwischenvariable.