使用LINQ分组字符串并生成指定格式序列的问题求助

你的代码存在两个核心问题：

1. 总长度计算错误：你用x.ToString().Length是把分组对象转成字符串后取长度，这完全不是分组内所有字符串的总长度，应该用x.Sum(s => s.Length)来计算总和。
2. 排序逻辑错误：你对生成的"S-C"字符串的长度排序，这会导致数值大小判断错误（比如"10-A"长度4，"9-B"长度3，但10的数值更大），而且首字母排序也不应该取生成字符串的第一个字符，应该直接用分组的首字母键。

正确的一行LINQ实现如下：

return stringList.GroupBy(s => s[0])
                 .OrderByDescending(g => g.Sum(s => s.Length))
                 .ThenBy(g => g.Key)
                 .Select(g => $"{g.Sum(s => s.Length)}-{g.Key}")
                 .ToList();

如果想避免重复计算总长度以提升效率，可以用Select先封装总长度和首字母，再排序、生成结果：

return stringList.GroupBy(s => s[0])
                 .Select(g => new { TotalLength = g.Sum(s => s.Length), Char = g.Key })
                 .OrderByDescending(item => item.TotalLength)
                 .ThenBy(item => item.Char)
                 .Select(item => $"{item.TotalLength}-{item.Char}")
                 .ToList();

说明： #

• 题目明确字符串序列非空且每个字符串仅含大写拉丁字母，所以直接用s[0]获取首字母比FirstOrDefault更严谨，不会出现默认值。
• 先按分组的总长度降序排序，再按首字母的字符编码升序排序，完全符合需求。
• 最后格式化为"S-C"的字符串输出。

内容的提问来源于stack exchange，提问作者Mr. Kolia