dotnet/roslyn
Voir sur GitHubDeconstructed variables code fix should always camelCase variables
Open
#23 716 ouverte le 11 déc. 2017
Area-IDEFeature - Tupleshelp wanted
Métriques du dépôt
- Stars
- (20 414 stars)
- Métriques de merge PR
- (Merge moyen 6j 17h) (256 PRs mergées en 30 j)
Description
class Program
{
private static readonly (string Factory, string EvaluationRuleName) Foo;
static void Main(string[] args)
{
var foo = Foo; // Deconstruct tuple here
}
}
Expected:
class Program
{
private static readonly (string Factory, string EvaluationRuleName) Foo;
static void Main(string[] args)
{
var (factory, evaluationRuleName) = Foo;
}
}
Actual:
class Program
{
private static readonly (string Factory, string EvaluationRuleName) Foo;
static void Main(string[] args)
{
var (Factory, EvaluationRuleName) = Foo;
}
}
Because tuple names appear as public fields when you access them via the tuple itself, it's common practice to PascalCase them in a declaration. However, when they are deconstructed into variables - you want them them to be camelCase to match the naming style for variables.
Either add an option here that I can switch or change the behavior. As it stands, I can't use this code fix.