IDE0059 Incorrectly reported for out arguments

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 Test
{
		public static void Foo ()
		{
			int i;
			OutFoo (out i, 9); // error IDE0059: Unnecessary assignment of a value to 'i'
			OutFoo (out i, i);
            Console.WriteLine (i);
		}

		public static void OutFoo (out int i, int arg)
		{
			i = ++arg;
		}
}

Guide contributeur

Direction de recherche

Le problème est un faux positif dans IDE0059 (affectation inutile) pour les arguments out. Commencez par localiser l'analyseur IDE0059 dans le dépôt Roslyn, probablement sous src/Analyzers/CSharp. Examinez comment l'analyseur suit les affectations et identifie l'utilisation des variables out. Le code fourni montre que la première affectation à 'i' avant OutFoo est incorrectement signalée, mais la variable est ensuite réaffectée à l'intérieur de OutFoo via out. Examinez la logique du graphe de flux de contrôle pour les arguments out et vérifiez si l'analyseur reconnaît correctement qu'un paramètre out écrit dans la variable. Recherchez les tests existants pour IDE0059 et ajoutez un cas de test qui reproduit ce scénario.

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.

2

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

Une demi journée

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#out parameters

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

70