字符串包含 vs List<String> 包含

Question

假设您想测试输入是否是几个常量字符串之一，并且忽略性能，它是否是一种反模式：

if ("yes oui ja da".contains(answer)) {
    // answer was in the affirmative
}

而不是更传统的：

private static List<String> affirmativeAnswers = Arrays.asList("yes", "oui", "ja", "da");

if (affirmativeAnswers.contains(answer)) {
    // answer was in the affirmative
}

它的代码少了很多，也更容易阅读，但它是“黑客”吗？

编辑：

为了更安全，如果担心部分匹配，可以这样编码：

if (",yes,oui,ja,da,".contains(',' + answer + ',')) 

还是少了很多代码（虽然越来越丑）

Answer 1

出于各种原因，我认为这是一个丑陋的 hack：

• 它对意外输入不可靠。例如("yes oui ja da".contains(" ")) 将返回 true - 可能不是您想要的。这是我认为最大的问题。即使您开始添加更多技巧（例如最近编辑中的逗号），您仍然需要考虑令人讨厌的极端情况。
• 在一般情况下它的性能不高 - 如果您有中等/大量的可能性进行测试，您可能最好检查 HashMap 或 HashSet 是 O(1) 而不是O(n) 用于扫描连接的字符串。
• 这可能会使没有经验的编码人员/未来的维护人员感到困惑。如果您希望代码可维护，请不要使用巧妙的技巧。
• 它不适合未来的重构（例如字符串的国际化？在运行时动态更改可能性列表？）

Answer 2

另外，AFAIK，String contains 方法使用正则表达式来检查子字符串是否是字符串的一部分。与搜索列表（AFAIK）相比，此操作的成本很高。所以IMO，现在最好使用列表，不是吗？ ;)

Answer 3

尽可能删除依赖项

在我看来，这不是最佳做法。最佳实践将允许消除对接受字符串的依赖。在这种情况下，依赖项是这个("yes", "oui", "ja", "da") 字符串列表。但是，如果要将字符串列表的依赖项移至服务或数据库，则将其集成到接受 List<string> 的方法中将非常容易，因为这很明显，而在尝试集成时可能会出现问题一个只需要 string 的方法。

牢记范围

与往常一样，这取决于范围。如果你只需要一次做这件小事，而且它是一个非常小的项目的一部分，那么它真的没什么区别。但是，如果这是一个不小的项目的一部分，那么如果没有遵循最佳实践，则可能需要在未来进行重构。

让编译器优化代码

关于它需要的代码量，您不必太担心。更重要的是它的可读性。编译器将使代码尽可能高效，因为这是它的职责。不要尝试编写类似于编译器的代码，因为随着更好的方法出现，编译器将使用它们，而模拟代码将保持不变。

Answer 4

如果要检查句子中是否存在字符串，则
string.contains(string) 是可行的。

否则，如果您想检查整个字符串是否相等，那么，
list.contains(string) 是可行的。

Answer 5

...这是对语言的滥用...

“Java - 他不在乎！！”。但是“哇！！讨厌！！”¹.

但是说真的，您的目标应该是让您的代码易于阅读和维护。 （或者高性能......如果这很重要。）

使用最少的击键次数来表达某些东西的棘手、晦涩的代码不是上述的。这是一种糟糕的风格......即使你的代码在功能上是正确的。

---

^{1 - 如果你在过去一年左右一直生活在岩石下......我是在暗示这个 - http://knowyourmeme.com/memes/honey-badger。}