如何编写更简洁/更清晰的代码？

Question

newRow("OrderReference") = line.Substring(line.IndexOf("*1003") + 5, line.IndexOf("*", line.IndexOf("*1003") + 5) - line.IndexOf("*1003") - 5)

你有它。很长很难看。我在想这个：

Dim indexPlus = line.IndexOf("*1003") + 5
Dim indexMinus = line.IndexOf("*1003") - 5
newRow("OrderReference") = line.Substring(indexPlus, line.IndexOf("*", indexPlus) - indexMinus)

但是这引入了新的无意义的变种。不令人满意的。

也许RegEx是这里的救世主？

不幸的是，我不能更改输入数据:-(

输入数据包含BWA格式（受书籍欢迎）。在这里你可以看到有问题的部分：

此示例集中的所有代码都是必需的。只有相应的值会发生变化。

Answer 1

我甚至认为你的第二个代码无效。看起来更像是这样。

Dim index = line.IndexOf("*1003") + 5
newRow("OrderReference") = line.Substring(index, line.IndexOf("*", indexPlus) - index)

10 - 5 - 2与10 - （5 - 2）相同，但它与10 - （5 + 2）相同。

下次，查看代码回放堆栈交换。

Answer 2

鉴于您的数据始终是恒定的，并且您要查找的内容始终以“* 1003”开头，您不需要使用正则表达式（即使您可以）。只需使用您已经使用的内容，但需要进行一些更正。

using System;

public class Program
{
    public static void Main()
    {
        string input = "L10113540   VD44444     VD2002100234949     000116161       04201261\r\n";
        input += "  KN00010000000129000LPEUR003000001*1003A.Muller-Schulz*1017Bastei\r\n";
        input += "Lubbe.61204 Laund.Meine Schuld*1019KL*102990300*1030NO*1032EUR*1131KT";

        int start = input.IndexOf("*1003");
        int end = input.IndexOf("*", start + 1);
        string result = input.Substring(start + 5, end - start - 5);

        Console.WriteLine(result);

        // Your code
        start = input.IndexOf("*1003") + 5;
        end = input.IndexOf("*1003") - 5;
        result = input.Substring(start, input.IndexOf("*", start) - end);

        Console.WriteLine(result);

    }
}

结果

A.Muller-Schulz
A.Muller-Schulz*1017Baste

您可以看到您在问题中发布的内容未提供您想要的结果。所有你真正想要的只是第一个“* 1003”之后的下一个星号。您可以看到代码与我给出的代码之间的差异。

.NET Fiddle Example