dotnet/roslyn

Deconstructed variables code fix should always camelCase variables

Open

#23,716 创建于 2017年12月11日

在 GitHub 查看
 (9 评论) (1 反应) (0 负责人)C# (4,257 fork)batch import
Area-IDEFeature - Tupleshelp wanted

仓库指标

Star
 (20,414 star)
PR 合并指标
 (平均合并 6天 17小时) (30 天内合并 256 个 PR)

描述

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.

贡献者指南