2017-03-09 12 views
1

手順: enter image description herelinkedListのlinkedListメソッドですか?

私のテストでは、私はすべてのケースや可能性をカバーしていますかどうかわからない渡すように見えるが、。

私の実装が単純すぎると感じます。誰か調べてもらえますか?

私の実装:

@Override 
public int compareTo(ListNoOrder<T> otherList){ 
    int sizeCompare = otherList.sizeOfList; 

    if (this.sizeOfList > sizeCompare) { 
     return 1; 
    } 

    else if (this.sizeOfList < sizeCompare) { 
     return -1; 
    } 

    else { 
     return 0; 
    } 
} 
+1

3回目のチェックを実装しました。しかし、あなたはすべての場合に1を返します。ケースの1つが-1を返す必要があります – SilverNak

+1

これはおそらく[codereview](http://codereview.stackexchange.com/)IMOに行く必要があります。 – Moira

+0

あなたのイメージは[あまり役に立ちません](// meta.unix.stackexchange.com/q/4086)。大声で読み上げたりエディタにコピーしたりすることはできません。インデックスはあまりうまくいきません。つまり、同じ問題を抱えている他のユーザーが答えを見つけにくいということです。関連するテキストを直接組み込むには、投稿を編集してください(できれば、コピーエラーを避けるためにコピー+貼り付けを使用することをおすすめします)。 –

答えて

0

あなただけListssizeを比較しました。最初のステップは大丈夫ですが、sizesが同じ場合は、それぞれcharを比較する必要があります。