为什么我的Codingbat String-2 EndOther代码不起作用？

Question

public boolean endOther(String a, String b) {
    a = a.toLowerCase();
    b = b.toLowerCase();
    if (a.contains(b) && a.indexOf(b) == a.length() - b.length()) {
        return true;
    } else if (b.contains(a) && b.indexOf(a) == b.length() - a.length()) {
        return true;
    } else {
        return false; 
    }
}

Answer 1

我的直觉是，他们希望您使用子字符串。原因很简单：模式出现在任何一个字符串中的其他位置都无关紧要，只要它位于该字符串的最后一个位置。

您也不应该（在实践中）操作传入的参数，因为这可能会在以后绊倒您。将final添加到param列表中会阻止您这样做，以防您意外地认为它在做您没有做的事情。

最后，您根本不需要将两个字符串都转换为小写；有一个名为EqualSignorecase的方法可以很好地处理这种情况。

public boolean endOther(final String a, final String b) {
    if (a.length() >= b.length()) {
        return endOtherHelper(a, b);
    } else {
        return endOtherHelper(b, a);
    }
}

private boolean endOtherHelper(final String longStr, final String shortStr) {
    return longStr.substring(longStr.length() - shortStr.length()).equalsIgnoreCase(shortStr);
}