截断以逗号分隔的字符串中的最后一项的最干净的方法

本文关键字:一项 方法 最后 分隔 字符串 | 更新日期: 2023-09-27 18:06:03

用逗号分隔的字符串,如" a,B,C,D,E,F,G",只返回列表中的前x个项目,你知道的最好/最干净的方法是什么?所以如果x = 5,那么结果就是"A,B,C,D,E"。

我知道有不同的方法来做到这一点:"for循环"计数逗号,然后截断字符串;将字符串拆分为数组或列表,然后删除最后x个项目。

有没有我不知道的更干净、更有效的选择?你会怎么做呢?

提前感谢!!格雷格

截断以逗号分隔的字符串中的最后一项的最干净的方法

我会把任务分成两部分:

  • 用逗号分隔字符串
  • 只取前N个值

幸运的是,c#使这两个都非常简单,String.Split处理第一个,LINQ Take方法处理第二个:

var items = text.Split(',')
                .Take(itemLimit);

或者如果你想创建一个列表:

var items = text.Split(',')
                .Take(itemLimit)
                .ToList();
除非真的需要,否则我不会将其转换回逗号分隔的字符串。尽可能长时间地保持数据最自然的表示(例如List<string>)。如果您需要,只需使用String.Join

你可以潜在的使Split部分更有效,通过编写一个"懒惰的拆分器"-但它将是非常小的增益工作,除非你希望得到一个很长的字符串,只想保留几个项目。它看起来像这样

public static IEnumerable<string> LazySplit(this string text, string separator)
{
    int start = 0;
    while (true)
    {
        int end = text.IndexOf(separator, start);
        if (end == -1)
        {
            // Note: if the string ends with the separator, this will yield
            // an empty string
            yield return text.Substring(start);
            yield break; // This will terminate the otherwise-infinite loop
        }
        yield return text.Substring(start, end - start);
        start = end + separator.Length;
    }
}

则使用代码与前面类似:

var items = text.LazySplit(",")
                .Take(itemLimit)
                .ToList();

或者,如果你真的,真的需要把它保存在字符串中,你可以写一些东西来找到第n个逗号,然后使用Substring来取字符串的第一部分:

// TODO: Improve the name :)
public static string TruncateAfterSeparatorCount(string text,
                                                 string separator,
                                                 int count)
{
    // We pretend that the string "starts" with a separator before index 0.
    int index = -separator.Length;
    for (int i = 0; i < count; i++)
    {
        int nextIndex = text.IndexOf(separator, index + separator.Length);
        // Not enough separators. Return the whole string. Could throw instead.
        if (nextIndex == -1)
        {
            return text;
        }
        index = nextIndex;
    }
    // We need to handle the count == 0 case, where index will be negative...
    return text.Substring(0, Math.Max(index, 0));
}

但正如我所说,如果可能的话,我个人会尝试使用List<string>方法。上面的代码显然比Split/Take/ToList要复杂得多,尽管它更有效。只有在证明有必要时才使用更高效但更复杂的代码。

试试这个:

string.Join("," , str.Split(",").Take(5));

或者,如果您经常这样做,您可以为此编写一个扩展方法

string[] words = s.Split(',').Take(5);
    string[] List = SubList(5);
    string Output = string.Join(",", List);
private string[] SubList(int p)
{
    string[] List = new string[] { "A", "B", "C", "D", "E", "F" };
    string[] List2 = new string[p];
    for (int i = 0; i < p; i++)
        List2[i] = List[i];
    return List2;
}

可能是SplitTake

试试这个:

string yourString = "A,B,C,D,E,F,G";
List<string> items = yourString.Split(',')
    .Take(5)
    .ToList();
string output = string.Join(",", items);

只是为了好玩-只使用Regex/String方法(我不会使用Regex来做这个现实世界-然后我有两个问题):

string.SubString(0,Regex.Matches(string,",")[x-1].Index);

如果您只想使用String方法(而不是Take()):

string.Join(",", s.Split(","), 0, 5);

如果你知道每个元素只有一个字符,你可以:

s.Substring(0, 2*x - 1);