элегантный способ построить строку в C # - PullRequest
10 голосов
/ 06 января 2010
Строка

для построения с использованием ключевой пары выглядит следующим образом: "name1 = v1 & name2 = v2 & name3 = v3"

что я делаю:

var sb = new StringBuilder();

foreach (var name in nameValues)
{
            sb.AppendFormat("{0}={1}&", name.Key, name.Value);
 } 

//remove last '&' sign, this is what i think is ugly
sb.ToString().Remove(lastIndex);

какой-нибудь элегантный способ избежать последнего оператора удаления знака '&'?

Ответы [ 7 ]

18 голосов
/ 06 января 2010
var joined =
    String.Join("&", nameValues.Select(n => n.Key + "=" + n.Value).ToArray());

Учитывая, что мы не конкатенируем одну большую строку (мы производим много маленьких строк), конкатенация в этом случае не имеет штрафов за производительность. А в .NET строки в любом случае имеют префикс длины, поэтому проблема производительности всей конкатенации менее актуальна, чем в C. String.Join () также очень быстр, быстрее, чем StringBuilder.

TLDR: Использование String.Join()

5 голосов
/ 06 января 2010

Взгляните сюда: Как создать строку запроса для URL в C #? ; Цитирование:

private string ToQueryString(NameValueCollection nvc)
{
    return "?" + 
        string.Join("&", 
            Array.ConvertAll(
                nvc.AllKeys, 
                key => String.Format("{0}={1}", HttpUtility.UrlEncode(key),
                HttpUtility.UrlEncode(nvc[key]))));
}
4 голосов
/ 06 января 2010
foreach (var name in nameValues)
    {
        if (sb.Length > 0) sb.Append("&");
                sb.AppendFormat("{0}={1}", name.Key, name.Value);
     }

Просто добавьте «&» при необходимости, не удаляйте его с конца.

3 голосов
/ 06 января 2010

Я склонен использовать это, используя тот факт, что вы можете обрезать построитель строк с уменьшением свойства length:

var sb = new StringBuilder();

foreach (var name in nameValues)
{
    sb.AppendFormat("{0}={1}&", name.Key, name.Value);
} 

if (sb.Length > 0) sb.Length--;
3 голосов
/ 06 января 2010

Вот еще один подход, который я иногда использовал:

var sb = new StringBuilder();
string prefix = "";
foreach (var name in nameValues)
{
    sb.Append(prefix);
    prefix = "&";
    sb.AppendFormat("{0}={1}", name.Key, name.Value);
}

Это просто способ добавления и перед каждой парой, отличной от первой, без использования условного теста.

Если вы хотите использовать свою первоначальную идею обрезки StringBuilder, кстати, я бы предложил следующий код:

sb.Length--; // Remove the last character
return sb.ToString();
1 голос
/ 06 января 2010

Ну, по крайней мере, вы можете удалить знак & перед вызовом ToString(), выполнив --sb.Length;

0 голосов
/ 06 января 2010
var sb = new StringBuilder();

sb.AppendFormat("{0}={1}", nameValues[0].Key, nameValues[0].Value);

for (int i = 1; i < nameValues.Count; i++)
{
        sb.AppendFormat("&{0}={1}", nameValues[i].Key, nameValues[i].Value);
} 
...