Sonar提示移除无用局部变量赋值，如何优化列表初始化代码？

嘿，我来帮你搞定这个Sonar的提示！首先得明确：Sonar说的“无用赋值”不是不让你用new初始化列表，而是你的代码写法太冗余了——手动循环添加元素的操作完全可以用LINQ一步到位，所以原来的初始化加循环的组合就显得多余啦。

优化方案：用LINQ直接生成列表 #

你根本不需要手动初始化列表再逐个Add元素，直接通过LINQ的Select投影出目标对象，再转成List<Comment>就行，代码简洁又符合Sonar的要求：

IList<Comment> listComments = paxComments
    .Where(x => x.Id == paxID)
    .Select(comment => new Comment 
    { 
        CommentID = comment.CommentId, 
        Text = comment.Comment 
    })
    .ToList();

为什么原代码会被Sonar警告？ #

你的原代码先初始化了一个空列表，然后循环筛选后的元素逐个添加，这和直接用ToList()生成列表的效果完全一样，但写法更啰嗦。Sonar能识别这种冗余操作，所以提示你去掉无用的初始化+手动添加的步骤。

关于“不用new初始化列表添加元素”的补充 #

其实列表必须先被实例化才能添加元素，要么通过new List<Comment>()手动初始化，要么通过ToList()这类方法间接实例化。如果你的场景确实需要逐步添加元素（不是一次性从现有集合转换），那初始化列表是必要的，Sonar也不会报错——但你的代码明显是从现有集合筛选转换，所以用LINQ的方式是最优解。

内容的提问来源于stack exchange，提问作者Ash18