Find Static Method Code Issue: Listing 1

Full code for custom code issue

Listing 1: Find Static Method Code Issue
using System;
using System.Collections.Generic;
using System.ComponentModel.Composition;
using System.Linq;
using System.Threading;
using Roslyn.Compilers;
using Roslyn.Compilers.Common;
using Roslyn.Compilers.CSharp;
using Roslyn.Services;
using Roslyn.Services.Editor;

namespace VSMRolynCTP.DemoCodeIssue
    [ExportSyntaxNodeCodeIssueProvider("VSMRolynCTP.DemoCodeIssue", LanguageNames.CSharp,
    class CodeIssueProvider : ICodeIssueProvider
        private readonly ICodeActionEditFactory editFactory;

        public CodeIssueProvider(ICodeActionEditFactory editFactory)
            this.editFactory = editFactory;

        public IEnumerable GetIssues(IDocument document, CommonSyntaxNode node,
 CancellationToken cancellationToken)
            var methodDeclaration = (MethodDeclarationSyntax)node;
            var semanticModel = document.GetSemanticModel(cancellationToken);

            if (methodDeclaration.DescendentTokens().Any(x => x.Kind == 
                yield return null;
                IEnumerable memberIdentifiers = from m in
                                                                where m as IdentifierNameSyntax
 != null
 IsClassMember(cancellationToken,semanticModel,m as ExpressionSyntax)
                                                                select m as IdentifierNameSyntax;

                if (!memberIdentifiers.Any())
                    yield return new CodeIssue(CodeIssue.Severity.Info, methodDeclaration.Span,
                   String.Format("{0} can be made static.", methodDeclaration));

        private static bool IsClassMember(CancellationToken cancellationToken, ISemanticModel
 semanticModel, ExpressionSyntax syntax)
            bool isMember = false;

            var memberInfo = semanticModel.GetSemanticInfo(syntax, cancellationToken);
            if (memberInfo.Symbol != null && !memberInfo.Symbol.IsStatic && 
                memberInfo.Symbol.OriginalDefinition.Kind != CommonSymbolKind.Local)
                isMember = true;

            return isMember;

        #region Unimplemented ICodeIssueProvider members

        public IEnumerable GetIssues(IDocument document, CommonSyntaxToken token,
 CancellationToken cancellationToken)
            throw new NotImplementedException();

        public IEnumerable GetIssues(IDocument document, CommonSyntaxTrivia trivia,
 CancellationToken cancellationToken)
            throw new NotImplementedException();


About the Author

Eric Vogel is a Senior Software Developer for Red Cedar Solutions Group in Okemos, Michigan. He is the president of the Greater Lansing User Group for .NET. Eric enjoys learning about software architecture and craftsmanship, and is always looking for ways to create more robust and testable applications. Contact him at [email protected].

comments powered by Disqus


  • Customize Your Own Audio Test Cues in Visual Studio 2019 v16.9 Preview 3

    Yes, developers can be alerted to a failed test with a fart sound.

  • Progress Touts New Third-Party Blazor UI Components

    Third-party dev tool specialist Progress announced an update to its .NET-centric offerings, touting new controls for Blazor, Microsoft's red-hot project for creating web apps with C#.

  • Entity Framework Core 6: What Developers Want

    Microsoft outlined its plan for Entity Framework Core 6, which in November will take its place as the data access component of the landmark .NET 6, a long-term support (LTS) release that will mark Microsoft's transition from the Windows-only .NET Framework to an open source, cross-platform umbrella offering of all things .NET.

  • AWS Open Sources .NET Porting Assistant GUI

    After previously open sourcing components of its Porting Assistant for .NET, Amazon Web Services open sourced the tool's GUI.

Upcoming Events