Change signature adds unnecessary argument names

Métriques du dépôt

Stars

20 414 étoiles (20 414 étoiles)

Métriques de merge PR

Merge moyen 6j 17h (Merge moyen 6j 17h)256 PRs mergées en 30 j (256 PRs mergées en 30 j)

Description

class Program
{
    static void F(int a, int b, int c)
    {
    }

    static void Main()
    {
        F(1, b: 2, 3);
    }
}

class Program
{
    static void F(int b, int c, int a)
    {
    }

    static void Main()
    {
        F(b: 2, 3, 1);
    }
}

class Program
{
    static void F(int b, int c, int a)
    {
    }

    static void Main()
    {
        F(b: 2, c: 3, a: 1);
    }
}

Guide contributeur

Direction de recherche

Inspectez le code de refactoring ChangeSignature, en particulier la partie qui décide quels arguments afficher avec des arguments nommés sur le site d'appel. Recherchez la logique qui ajoute des noms d'arguments même lorsque ce n'est pas nécessaire, comme dans le cas du déplacement d'un paramètre qui avait initialement un argument nommé après avoir changé l'ordre de la signature.

Stack technique

csharp

Domaine

developer experience

Type d'issue

Bug

DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.

3

Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.

1-2 jours

Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.

Active

ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.

Claire

Prérequis

C#Git

Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.

70