java - 这个空字符串检查是多余的吗？

Question

在 Android 的 frameworks/base/services/core/java/com/android/server/policy/PhoneWindowManager.java 中有这样的子句:

    //if device type is tablet force enable NavigationBar and forbid NavigationBar move
    String deviceType = SystemProperties.get("sys.device.type");
    if (! "".equals(deviceType) && deviceType.equals("tablet")) {
        mNavigationBarCanMove = false;
        mHasNavigationBar = true;
    }

是! "".equals(deviceType) && 完全多余，或者它有什么意义？

Answer 1

是的，在这种情况下它是多余的，因为第二个子句涉及第一个子句(如果 deviceType 不为空)，因此您可以简化执行以下操作:

//Use constant string to perform comparison to avoid possible NPE
if("tablet".equals(deviceType)) {}