Attribute/Base Class Naming

[chris.bahns]

Hello All,
I am fairly new to creating custom attributes, and want to get a
better feel for how they should be named.

We have a class hierarchy, with a base class called "SequenceComman d".
This class has a couple static functions and an embedded class, all of
which are referenced in code like "SequenceComman d.Create()" or
"SequenceComman d.ArgumentDef". There is also a fairly rich hierarchy
of classes under it (maybe 10-15 intermediate classes and about 30
concrete/instantiable classes).

We recently created a custom attribute, which we would apply to the
concrete classes in the hierarchy that we intend to "use" (allowing
them to be used in reflection). This makes our reflection code, which
finds usable classes by name, quite a bit cleaner. Classes without
this attribute are not used, even if they are otherwise capable of
being instantiated, etc.

We want to call this custom attribute class
"SequenceComman dAttribute". The fact that you are applying this
attribute to a class implies that it is "usable", "instantiab le", or
"executable ", so I think it is slightly redundant to include these
terms in the name of the attribute class.

This approach leads to code like...

[SequenceCommand ("MY_COMMAND ")]
public class MySequenceComma nd : SequenceCommand
{
// implementation

}

There is no conflict here, but it just looks odd that you see
"SequenceComman d" in two places -- one of which is actually referring
to the SequenceCommand Attribute class.

Question: Is this bad practice?
I have found two similar classes in .NET:
System.Runtime. Remoting.Activa tion.UrlAttribu te and
System.Security Policy.Url. These are in two different namespaces, but
perhaps, with the right combination of "using" statements, they could
lead to the same syntax I am describing.

Two alternatives:
1. Change the base class to "SequenceComman dBase"
I don't like this because of the references to the embedded class
"ArgumentDe f" and the static functions. ("SequenceComma ndBase" just
does not feel right in those cases)

2. Change the attribute class to "ExecutableSequ enceCommandAttr ibute"
I don't like this because it is a little bit verbose, and the
attribute will be applied in many places.

Is there an official position on this sort of thing?
TIA for your help!
- Chris

[Nicholas Paldino [.NET/C# MVP]]

Chris,

I wouldn't say that it is bad practice, but perhaps you can rename the
SequenceCommand to be more reflective of what you are trying to do. For
example, "MY_COMMAND " is some sort of key, or command text, right? In that
case, you can be more specific with the attribute name, naming it something
like "SequenceComman dText" or "SequenceComman dKey". Of course, this
attribute serves double-duty, in that your framework can only use the class
if that attribute is on it.
--
- Nicholas Paldino [.NET/C# MVP]
- mv*@spam.guard. caspershouse.co m

"chris.bahn s" <ch***@bahns.co mwrote in message
news:11******** **************@ n2g2000hse.goog legroups.com...

Hello All,
I am fairly new to creating custom attributes, and want to get a
better feel for how they should be named.

We have a class hierarchy, with a base class called "SequenceComman d".
This class has a couple static functions and an embedded class, all of
which are referenced in code like "SequenceComman d.Create()" or
"SequenceComman d.ArgumentDef". There is also a fairly rich hierarchy
of classes under it (maybe 10-15 intermediate classes and about 30
concrete/instantiable classes).

We recently created a custom attribute, which we would apply to the
concrete classes in the hierarchy that we intend to "use" (allowing
them to be used in reflection). This makes our reflection code, which
finds usable classes by name, quite a bit cleaner. Classes without
this attribute are not used, even if they are otherwise capable of
being instantiated, etc.

We want to call this custom attribute class
"SequenceComman dAttribute". The fact that you are applying this
attribute to a class implies that it is "usable", "instantiab le", or
"executable ", so I think it is slightly redundant to include these
terms in the name of the attribute class.

This approach leads to code like...

[SequenceCommand ("MY_COMMAND ")]
public class MySequenceComma nd : SequenceCommand
{
// implementation

}

There is no conflict here, but it just looks odd that you see
"SequenceComman d" in two places -- one of which is actually referring
to the SequenceCommand Attribute class.

Question: Is this bad practice?
I have found two similar classes in .NET:
System.Runtime. Remoting.Activa tion.UrlAttribu te and
System.Security Policy.Url. These are in two different namespaces, but
perhaps, with the right combination of "using" statements, they could
lead to the same syntax I am describing.

Two alternatives:
1. Change the base class to "SequenceComman dBase"
I don't like this because of the references to the embedded class
"ArgumentDe f" and the static functions. ("SequenceComma ndBase" just
does not feel right in those cases)

2. Change the attribute class to "ExecutableSequ enceCommandAttr ibute"
I don't like this because it is a little bit verbose, and the
attribute will be applied in many places.

Is there an official position on this sort of thing?
TIA for your help!
- Chris