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)。
我正在试用 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)。