我认为您想要这样的东西，但是由于您没有真正提出问题，所以我只是在猜测：
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->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;
仅提取您感兴趣的扩展，仅此而已。