1. ホーム
  2. java

[解決済み] マトリックス「条件付きロジックは削除可能」チェックスタイル

2022-02-07 16:42:31

質問

public static boolean isCompatibleForMultiplcation(final Matrix a, final Matrix b)  
    {
        if (a == null)
        {
            throw new IllegalArgumentException("a cannot be null");
        }
        if (b == null)
        {
            throw new IllegalArgumentException("b cannot be null");
        }

        if(!(a.getNumberofColumns()== b.getNumberOfRows()))
        {
            return false;
        }
        else
        {
            return true;
        }

    }

以下のメソッドのcheckstyleで、「Conditional Logic can be removed argument」が表示されます。原因がわからないようです...。誰かポインタを教えてください。

解決方法を教えてください。

この部分に不満があるようです。

    if(a.getNumberofColumns() != b.getNumberOfRows())
    {
        return false;
    }
    else
    {
        return true;
    }

このようなコードを書いていても、if文の条件を返すだけで簡単に1行で置き換えることができます。

return a.getNumberofColumns() == b.getNumberOfRows();

このステートメントが返すのは true は、a の列と b の行の数が等しい場合、そして false である。