Roslyn 的“.WithFooToken()”调用是否多余?

Are Roslyn's ".WithFooToken()" calls superfluous?

我正在试用 Roslyn 的代码生成功能,使用 LinqPad to 运行 片段。 LinqPad 的 .Dump() 扩展方法将对象的格式化视图呈现到结果窗格。

http://roslynquoter.azurewebsites.net/ 生成的代码包含许多代码,除了增加膨胀之外似乎没有太多作用。以下代码输出 return null; 无论 .WithFooToken(...) 调用是否被注释掉。

using Microsoft.CodeAnalysis;
using Microsoft.CodeAnalysis.CSharp;
using Microsoft.CodeAnalysis.CSharp.Syntax;

var syn =
    SyntaxFactory.ReturnStatement(
        SyntaxFactory.LiteralExpression(SyntaxKind.NullLiteralExpression)
//          .WithToken(SyntaxFactory.Token(SyntaxKind.NullKeyword))
    )
//  .WithReturnKeyword(
//      SyntaxFactory.Token(SyntaxKind.ReturnKeyword)
//  )
//  .WithSemicolonToken(
//      SyntaxFactory.Token(SyntaxKind.SemicolonToken)
//  )
;
syn.NormalizeWhitespace().ToFullString().Dump();

这样的调用是对所有语言都是可选的,还是仅对在其 Roslyn 支持库中内置了合理默认值的语言是可选的?

是的,在简单的情况下,这些调用是不必要的。 The code of RoslynQuoter has the RemoveRedundantModifyingCalls option, which I believe is supposed to do exactly what you want. But it's not implemented, the only usage被注释掉:

private void AddModifyingCall(ApiCall apiCall, MethodCall methodCall)
{
    // TODO: this needs scripting
    ////if (RemoveRedundantModifyingCalls)
    ////{
    ////    var before = Evaluate(apiCall, UseDefaultFormatting);
    ////    apiCall.Add(methodCall);
    ////    var after = Evaluate(apiCall, UseDefaultFormatting);
    ////    if (before == after)
    ////    {
    ////        apiCall.Remove(methodCall);
    ////    }
    ////}

    apiCall.Add(methodCall);
    return;
}

@svick 的回答是正确的。我只是忘记重新启用该代码,因为 Roslyn 脚本不可用。

现在脚本是 public,我已重新启用该代码并更新了网站: http://roslynquoter.azurewebsites.net/

立即试用该网站,您上面提到的冗余调用应该会消失(它们在需要修改时仍会出现 trivia/comments/etc)。