0 out of 1 rated this helpful - Rate this topic

CA1052: Static holder types should be sealed

Visual Studio 2012

TypeName	StaticHolderTypesShouldBeSealed
CheckId	CA1052
Category	Microsoft.Design
Breaking Change	Breaking

Cause

A public or protected type contains only static members and is not declared with the sealed (C# Reference) (NotInheritable (Visual Basic)) modifier.

Rule Description

This rule assumes that a type that contains only static members is not designed to be inherited, because the type does not provide any functionality that can be overridden in a derived type. A type that is not meant to be inherited should be marked with the sealed modifier to prohibit its use as a base type.

How to Fix Violations

To fix a violation of this rule, mark the type as sealed. If you are targeting .NET Framework 2.0 or earlier, a better approach is to mark the type as static. In this manner, you avoid having to declare a private constructor to prevent the class from being created.

When to Suppress Warnings

Suppress a warning from this rule only if the type is designed to be inherited. The absence of the sealed modifier suggests that the type is useful as a base type.

Example of a Violation

Description

The following example shows a type that violates the rule.

Code

C++

using System;

namespace DesignLibrary
{
    public class StaticMembers
    {
        static int someField;

        public static int SomeProperty
        {
            get
            {
                return someField;
            }
            set
            {
                someField = value;
            }
        }

        StaticMembers() {}

        public static void SomeMethod() {}
    }
}

Fix with the Static Modifier

Description

The following example shows how to fix a violation of this rule by marking the type with the static modifier.

Code

using System; 

namespace DesignLibrary
{    
    public static class StaticMembers    
    {        
        private static int someField;     

        public static int SomeProperty        
        {            
            get { return someField; }            
            set { someField = value; }        
        }                

        public static void SomeMethod()         
        {        
        }         

        public static event SomeDelegate SomeEvent;    
    }     

    public delegate void SomeDelegate();
}

Did you find this helpful? Yes No

Not accurate

Not enough depth

Need more code examples

(1500 characters remaining)