Question

我有一个接收字符串的网络服务。

此字符串包含多个键=＆gt;与字符'+'连接的值。

我必须验证每个值（“必需”，“非空”），并将每个值分配给具有相同名称的变量。

以下是我如何从字符串构建Dictionary：

string firstname;
string lastname;
string amount;

string request = "firstname=foo+lastname=bar+amout=100.58";

Dictionary<string, string> arguments = new Dictionary<string, string>();

request.Split('+').ToList<string>().ForEach(p =>
{
    string[] tmp = p.Split('=');

    if (tmp.Length == 2)
        arguments.Add(tmp[0], tmp[1]);
});

// Validate and assign : How I do with one value : (I must find a better way)
bool isValid = true;

// check "firstname"
if(arguments.ContainsKey("firstname") && string.IsNullOrWhiteSpace(arguments["firstname"]) == false)
{
    firstname = arguments["firstname"];
}
else
{
    isValid = false;
    Logger.Write("Invalid argument : firstname");
}

// Do this for about 20 arguments, it becomes huge...

if(isValid)
{
    Console.WriteLine(firstname); // Displays foo
    Console.WriteLine(lastname); // Displays bar
    Console.WriteLine(amout); // Displays 100.58
}

谢谢，对不起拼写错误，我是法国人。

Answer 1

我想你想要这样的东西，但是因为你没有实际问一个我只是猜测的问题：

request.Split('+').ToList<string>().ForEach(p =>
{
    string[] tmp = p.Split('=');

    if (tmp.Length == 2 && !string.IsNullOrWhiteSpace(tmp[1]))
    {
        // edit - if your string can have duplicates, use
        // Dictionary<U,K>.ContainsKey(U) to check before adding
        var key = tmp[0];
        var value = tmp[1];

        if(!arguments.ContainsKey(key))
        {
            arguments.Add(key, value);
        }
        else
        {
            //overwrite with new value
            //could also maybe throw on duplicate or some other behavior.
            arguents[key]=value; 
        }
    }
    else
        throw InvalidOperationException("Bad dictionary string value");
});

另外，如果在代码审查中我在我面前，我会质疑ToList-＆gt; ForEach的使用。你想避免在Linq中出现副作用，我会用传统的foreach来写它：

var itemValues = request.Split('+');
foreach(var item in itemValues)
{
    string[] tmp = item.Split('=');

    if (tmp.Length == 2 && !string.IsNullOrWhiteSpace(tmp[1]))
        arguments.Add(tmp[0], tmp[1]);
    else
        throw InvalidOperationException("Bad dictionary string value");
});



// Validate and assign
//read values from the dictionary
//use ContainsKey to check for exist first if needed

Console.WriteLine(arguments["firstname"]); // Displays foo
Console.WriteLine(arguments["lastname"]); // Displays foo
Console.WriteLine(arguments["amout"]); // Displays 100.58

编辑2 - 您应该将逻辑封装在方法中：

private string TryGetValue(IDictionary<string,string> dict,string key)
{
    string value = null;
    if(dict.ContainsKey(key) && !string.IsNullOrWhiteSpace(dict[key]))
    {
        value = dict[key];
    }
    else
    {
        Logger.Write("Invalid argument : " + key);
    }
    return value;
}

现在你可以说：

string firstName = TryGetValue(arguments,"firstname");
string lastName= TryGetValue(arguments,"lastName");
string amount = TryGetValue(arguments,"amount");

bool isValid = firstName!=null && lastName != null && amount != null;

if(isValid)
{
    Console.WriteLine(firstName ); // Displays foo
    Console.WriteLine(lastName); // Displays bar
    Console.WriteLine(amout); // Displays 100.58
}

TryGetValue会成为一种出色的扩展方法：

public static class Extensions
{
    public static string TryGetValue(this IDictionary<string,string> dict, string key)
    {
        string value = null;
        if(dict.ContainsKey(key) && !string.IsNullOrWhiteSpace(dict[key]))
        {
            value = dict[key];
        }
        else
        {
            Logger.Write("Invalid argument : " + key);
        }
        return value;
    }

}

现在调用代码如下：

string firstName = arguments.TryGetValue("firstname");
string lastName= arguments.TryGetValue("lastname");
string amount = arguments.TryGetValue("amount");

上次修改 关于扩展方法的说明 - 是的，它们很整洁，但是在使用它们时也很容易意外地陷入困境。阅读有关它们的msdn和博客，请遵循指南。避免使用object，string等通用类型的扩展名。

在我的项目中，我总是根据它们与之交互的类型在不同的命名空间中布置扩展方法，这迫使希望使用它们的类非常明确地表达它：

namespace Extensions.IDictionary { ... }
namespace Extensions.string { ... }
namespace Extensions.SomeType { ... }
namespace Extensions.IList { ... }

并且使用代码将using子句匹配：

using Extensions.IDictionary;

将拉入只是您感兴趣的扩展程序，不再需要。

Answer 2

这只是猜测，因为你的问题不明确。

// Validate and assign

foreach(KeyValuePair<string,string> pair in arguments)
{
    if(!String.IsNullOrEmpty(pair.Value))
    {
        Console.WriteLine(pair.Value);
    }
}

Answer 3

如果你只有一个firstnames和lastnames，那么你可以使用List for Dictionary of Dictionary。这样，您只需为firstname，lastname和其他值添加其他值。

要验证，您可以在将值添加到字典中之间执行此操作：

 string request = "firstname=foo+lastname=bar+amout=100.58+firstname=+lastname=bar2+amout=100.59+firstname=foo3+lastname3=bar3+amout=100.60";

        Dictionary<string, List<string>> arguments = new Dictionary<string, List<string>>();
        request.Split('+').ToList<string>().ForEach(f =>
            {
                string[] data = f.Split('=');
                if (data.Length == 2)
                {
                    if (!arguments.ContainsKey(data[0]))
                    {
                        if (data[1] != "")
                            arguments.Add(data[0], new List<string> { data[1] });
                        else
                            arguments.Add(data[0], new List<string> { "no firstname" });
                    }
                    else
                    {
                        if (data[1] != "")
                            arguments[data[0]].Add(data[1]);
                        else
                            arguments[data[0]].Add("no firstname");
                    }
                }
            });

希望它有所帮助，再见

检查存储在Dictionary中的参数是否有效

3 个答案: