这些解决方案中哪一个具有更好的风格/性能？ [关闭]

Question

我发现了一个关于codingBat 的有趣练习，现在我有一个关于解决方案的问题。任务是：

给定一个字符串，如果字符串中x 的第一个实例紧跟着另一个x，则返回true。

我写的是：

boolean doubleX(String str) {
      return str.contains("x") ? str.indexOf('x') == str.length() - 1 ? false : str.charAt(str.indexOf('x')) == str.charAt(str.indexOf('x') + 1) : false;
}

他们在页面上的解决方案是：

boolean doubleX(String str) {
   int i = str.indexOf("x");
   if (i == -1) return false; // no "x" at all

   // Is char at i+1 also an "x"?
   if (i+1 >= str.length()) return false; // check i+1 in bounds?
      return str.substring(i+1, i+2).equals("x");

所以现在我的问题是哪种解决方案具有更好的编码风格？哪个解决方案更美观甚至更高效？

感谢所有回答

Answer 1

我可能会这样编码：

boolean doubleX(String str) 
{
    int index = str.indexOf("x");
    return (index >= 0 && index == str.indexOf("xx", index));
}

可能不是最高效的，但它可以处理所有可能性。

如果indexOf("x") 和indexOf("xx") 相距很远（不必要，长时间搜索"xx"）或者如果两个索引都非常高（长时间搜索完成了两次），它可能会很慢。

因此，另一个稍微不那么直观但性能更高的解决方案可能是：

boolean doubleX(String str)
{
    int index = str.indexOf("x");
    return (index >= 0 && index < str.length() - 1 && str.charAt(index + 1) == 'x');
}