java - 比较多个字符串时切换大小写与 If, else

Question

我有一个关于解析文件的问题。以下是我解析文件时使用的一些语句:

while(scan.hasNextLine()){
    line = scan.nextLine();//this gets the whole line
    if(line.contains("SomeDataIKnow"){
        data = new Scanner(line).useDelimiter("=");//scans just the one line
        value = data.next();//sets value to the string right after the =
        this.Data1 = value;
    }
    else if(line.contains("DifferentDataIKnow"){
        data = new Scanner(line).useDelimiter("=");
        value = data.next();
        this.Data2 = value;
 }

请注意，大约有 30 行在包含语句的行中都有不同的字符串。

我知道每个“=”之前的单词会说什么，但不知道等于之后的单词。因此，我需要通过查找特定数据进行解析，然后在等号后获取这些值。我知道当有超过 2 或 3 个语句时，通常不应该使用 if 语句。所以我很好奇是否有人有其他方法的想法？我正在考虑可能 Switch 语句，但不确定这是否对 30 行有帮助。另外，如果它有助于可视化文件，总共大约有 400 行，但我只需要其中大约 30 行。我无法发布该文件，但文件中整行的示例如下:

设备。一堆发生变化的其他信息。我所知道的数据 = 我需要的值(value)

因此，我解析多个文件，数据发生变化，但我知道的数据部分除外，它永远不会改变，我用它来查找具有我需要的值的行。抱歉，如果这听起来有点令人困惑。

Answer 1

你不应该这样编码，DRY 是编码中最重要的规则——当你查看 switch 语句时，你会发现很多冗余。不应该使用这样的 if/elseif 链的原因也适用于 switch 语句——它们是一种糟糕的代码味道，因为它们会导致冗余代码。

所以要做的第一件事就是看看每个 block 有什么不同。这是您的冗余代码:

if(line.contains("SomeDataIKnow"){
    data = new Scanner(line).useDelimiter("=");//scans just the one line
    value = data.next();//sets value to the string right after the =
    this.Data1 = value;
}
else if(line.contains("DifferentDataIKnow"){
    data = new Scanner(line).useDelimiter("=");
    value = data.next();
    this.Data2 = value;
}

差异是 [SomeDataIKnow, Data1] 与 [DifferentDataIKnow, Data2]

标量不是存储这样的数据的好方法 - 它们通常会让您创建像上面这样的代码，因为没有明显更简单的方法，所以通常我使用 map 。

因此，如果我在代码库中看到这段代码，我的第一次重构将类似于:

String[] strings=new String[]{"SomeDataIKnow","DifferentDataIKnow"};
Map results=new HashMap<String, String>()

for(String sub:strings)
{
    if(line.contains(sub){
        data = new Scanner(line).useDelimiter("=");
        value = data.next();
        results.put(sub, value);
}

这有一些问题但有效。当有新的字符串需要检查时，您不必添加任何代码，您的代码减少了 30 倍，并且不需要在 30 个不同的位置重复对 if 语句/代码进行任何修补。

最大的问题是查找值不再是小事。您需要字符串“SomeDataIKnow”作为 key ，这真是太痛苦了...您曾经有一个变量“Data1”。

如果您想保留“Data1”，您可以将字符串设置为如下所示:

String[] strings=new String[]{"SomeDataIKnow:Data1","DifferentDataIKnow:Data2"};

然后执行 String[] split=sub.split(":") 并通过 split[0] 进行查找，但通过 split[1] 将其输入到映射中

在此之后，您将使用 results.get("Data1") 而不是使用 this.Data1，这是合理的。

还有很多其他方法可以为这样的常见操作提供元数据..只要保持干燥，大多数都可以正常工作。我使用字符串数组构造只是因为它很容易初始化和使用...拆分也是如此。

作为另一种方法的示例，如果您确实想使用标量，您可以像这样设置您的类:

class MyClass{
    @SearchStr("SomeDataIKnow")
    String Data1;
    @SearchStr("DifferentDataIKnow")
    String Data2;
...

如今，注释非常容易处理——您只需遍历类中的所有 @SearchStr 注释即可构建搜索词，以与上面相同的方式循环它们，并将结果值填充到相应的变量中。

像这样的重构模式的技巧就是简单地识别差异并将它们分组——任何让您感觉舒服并且不会增加太多复杂性的方式。

顺便说一句，一旦您在 map 中获得结果，请查看它们的使用位置。您经常会在那里发现其他可以重构的冗余代码——像这样的重构可以串联起来，从而在您的代码库中节省大量成本。