Ne pas déclarer les membres protégés dans les types sealed
Mise à jour : novembre 2007
TypeName |
DoNotDeclareProtectedMembersInSealedTypes |
CheckId |
CA1047 |
Catégorie |
Microsoft.CSharp |
Modification avec rupture |
Modification sans rupture |
Cause
Un type public est sealed (NotInheritable en Visual basic) et déclare un membre protégé ou un type imbriqué protégé. Cette règle ne rapporte pas de violations pour les méthodes Finalize qui doivent suivre ce schéma.
Description de la règle
Les types déclarent des membres protégés afin que des types qui héritent puissent accéder au membre ou le substituer. Par définition, vous ne pouvez pas hériter d'un type sealed (scellé) ; en d'autres termes, les méthodes protégées sur les types sealed ne peuvent pas être appelées.
Le compilateur C# émet un avertissement pour cette erreur.
Comment corriger les violations
Pour corriger une violation de cette règle, changez le niveau d'accès du membre en privé ou rendez le type héritable.
Quand supprimer les avertissements
Ne supprimez aucun avertissement de cette règle. Laisser le type dans son état actuel peut induire des problèmes de maintenance et ne fournit aucun avantage.
Exemple
L'exemple suivant affiche un type qui ne respecte pas cette règle.
Imports System
Namespace DesignLibrary
Public NotInheritable Class BadSealedType
Protected Sub MyMethod
End Sub
End Class
End Namespace
using System;
namespace DesignLibrary
{
public sealed class SealedClass
{
protected void ProtectedMethod(){}
}
}
Le type sealed ci-dessus déclare un membre protégé, qui ne peut pas être appelé à l'extérieur de la classe qui le déclare.
Si la méthode a été conçue pour être appelée par d'autres types, augmentez son accessibilité publique, sinon, réduisez son accessibilité privée.
L'exemple suivant résout la violation précitée en augmentant l'accessibilité de la méthode au public.
Imports System
Namespace Samples
Public NotInheritable Class Book
Protected Sub Read
End Sub
End Class
End Namespace
using System;
namespace Samples
{
public sealed class Book
{
protected void Read()
{
}
}
}