最もエレガントなisEmpty（）バイナリツリーのメソッド

Question

私はLinkedBinaryTreeクラスのために最もエレガントなisEmpty（）メソッドを作成しようとしています。

コンパイラがifステートメントと対話するときは、ブール値またはfalseのいずれかを返す必要があります。しかし、の外にの値を返す必要があるので、私はfalseの値を返しました。私のロジックはif（count == 0）で、isEmpty（）はtrueを返し、2行目で気​​にしません（falseを返します）。これは正しいです？

public class LinkedBinaryTree<T> implements BinaryTreeADT<T> 
{ 
    protected int count; 
    protected BinaryTreeNode<T> root; 

    /** 
    * Creates an empty binary tree. 
    */ 
    public LinkedBinaryTree() 
    { 
     count = 0; 
     root = null; 
    } 

    /** 
    * Creates a binary tree with the specified element as its root. 
    * 
    * @param element the element that will become the root of the new binary tree 
    */ 
    public LinkedBinaryTree (T element) 
    { 
     count = 1; 
     root = new BinaryTreeNode<T> (element); 
    } 

    /** 
    * Returns true if this binary tree is empty and false otherwise. 
    * 
    * @return true if this binary tree is empty 
    */ 
    public boolean isEmpty() 
    { 
     if (count == 0); 
     return false; 
    } 

Answer 1

「これは正しいですか？」あなたはそれをテストすることで見つけ出すことができます。理想的には、jUnitテストでこれを行うことをお勧めします。

@Test 
public void isEmpty_returns_true_for_empty_list() { 
    LinkedBinaryTree tree = new LinkedBinaryTree(); 
    assertTrue(tree.isEmpty()); 
} 

...非空のツリーについても同様のテストを行います。

「最も優雅」？

Iのいずれかのために行くだろう。おそらく

public boolean isEmpty() { 
    return count == 0; 
} 

...か...

public boolean isEmpty() { 
    return root == null; 
} 

は（あなたは必ず、あなたの実装がcount変数を必要としていますか？）

---

Javaでは、Cと同様に、ifステートメントから中カッコを省略することができます。共通の推奨事項は、混乱の原因であるため、中括弧を省略しないことです。

何が書かれていることに相当します。

if(count == 0) { 
} 
return false; 

だから、常にfalseを返します。

あなたは書くことができます：

if(count == 0) { 
    return true; 
} 
return false; 

または中カッコなし（推奨しませんが）：

if(count == 0) 
    return true; 
return false; 

しかし、これはreturn count == 0と等価であるから、それはifステートメントを使用して洗練ます。

Answer 2

あなたはいいえ、それはない "あれば" 文

public boolean isEmpty() { 
    return (count == 0); 

}