if文でラップされた3値を悪用して呼び出す条件付きメソッド？

Question

は最近、私はコードレビューを行うと、この男に出くわした：

if(!sharePermission.isExpired() ? activePermissions.add(sharePermission) : expiredPermissions.add(sharePermission)); 

は、基本的にはブール値を返し、割り当ての要件を満たすためにif(...)文でそれを包むメソッドを呼び出すif文三元を使用します。これは多かれ少なかれ有効ですか

if(!sharePermission.isExpired()) 
    activePermissions.add(sharePermission); 
else 
    expiredPermissions.add(sharePermission); 

本当にコードを1行に凝縮する必要がある場合は、代入が実際に発生している場合、if(...)に割り当てられるスペースはありますか？

私はそれらのいずれかのファンではなく、ちょうど興味があります。

Answer 1

に相当します。評価の結果に余分なメモリーは割り当てられません。三元声明をエミュレートするために三元表現を使用して

は、しかし、著しく異例です。追加の利益をもたらすことなく、コードの可読性を低下させます。したがって、普通のifにelseを使用する方が良い方法です。もしactivePermissionsとexpiredPermissionsが同じタイプであることを

注意次のように、あなたは、add呼び出しのターゲット間で決定する三元表現を使用することができます。

(sharePermission.isExpired() ? expiredPermissions : activePermissions).add(sharePermission); 

Answer 2

あなたが探しているif..elseための三元相当のようなものさ -

(!sharePermission.isExpired() ? activePermissions : expiredPermissions).add(sharePermission); // no if here 

は何の割り当て起こって、boolean条件の唯一の評価はありません

if(!sharePermission.isExpired()) { 
    activePermissions.add(sharePermission); 
} 
else { 
    expiredPermissions.add(sharePermission); 
} 

Answer 3

それはif声明への虐待ですそれをして、条件式を気にしないでください。

どちらかのフルif文を書くために、またはに追加するリストを選択するための条件演算子を使用するようにきれいになります：

List<Permission> list = isExpired() ? expiredPermission : activePermission; 
list.add(sharePermission);