在一行代码中将string.contains和string.replace

Question

我目前正在编写一些软件，必须从外部文件中加载很多列名。 通常，我会使用一些JSON进行此操作，但是出于用户友好的原因，我现在无法执行此操作。 我需要使用一个用户可读的文本文件，其中包含很多注释。

因此，我创建了自己的文件来保存所有这些值。

现在，当我在软件中导入这些值时，实际上我逐行运行了configfile，并检查每行是否与我随后解析的参数匹配。 但是这样一来，我最终得到了一个包含大量重复代码的大代码块，而我想知道是否无法以某种方式简化它，以使每项检查都只能在一行中完成。

这是我当前正在使用的代码：

if (line.Contains("[myValue]"))
{
   myParameter = line.Replace("[myValue]", string.Empty).Trim();                        
}

我知道使用Linq可以简单地将事情放在一行中，我不确定在这种情况下是否行得通？

谢谢你的帮助！ 肯尼斯

Answer 1

如果这段代码经常重复，为什么不创建一个方法：

void SetParameter(string line, string name, ref string parameter)
{
    if (line.Contains(name))
    {
       parameter = line.Replace(name, string.Empty).Trim();                        
    }
}
SetParameter(line, "[myValue]", ref myParameter);

如果您想避免同时调用Replace和Contains ，这可能是一个好主意，则也可以只调用Replace ：

void SetParameter(string line, string name, ref string parameter)
{
    var replaced = line.Replace(name, string.Empty);
    if (line != replaced)
    {
       parameter = replaced.Trim();                        
    }
}

Answer 2

尝试这种方式（三元）：

myParameter = line.Contains("[myValue]")?line.Replace("[myValue]", string.Empty).Trim():myParameter;

其实，

line.IndexOf应该更快。

从您的代码中，看起来好像您只是用空文本替换，所以为什么不取出整个字符串（由多行组成）并一次替换，而不是一次检查一行。

Answer 3

您可以使用RegEx。 这可能会减轻您的重复代码

        string line = "[myvalue1] some string [someotherstring] [myvalue2]";
        // All your Keys stored at a single place
        string[] keylist = new string[] {  @"\[myvalue1]", @"\[myvalue2]" };

        var newString = Regex.Replace(line, string.Join("|", keylist), string.Empty);

希望能帮助到你。