.NET coding guidelines и потенциальные баги

[metaclass]

.NET
По правилам написания кода класс с полем, инициализирующемся в конструкторе и свойством, которое дает доступ для чтения этого поля надо писать так:

public class SomeClass
{
private readonly string configMetaID;
public string ConfigMetaID{get{return configMetaID;}}

public SomeClass(string configMetaID)
{
this.configMetaID = configMetaID;
}
}

Достаточно вместо return configMetaID; написать return ConfigMetaID; и получим Stack overflow. А сделать это очень легко, так как используется автодополнение кода. Кроме того, можно забыть написать this. в конструкторе и поле останется не проинициализированным.
Правила кодирования заимствованны из Java, где свойств нету, а есть геттеры и сеттеры, префиксы get и set которых там не дадут устроить переполнение стека из-за ошибки.
Есть вариант обхода - сделать поле public, а свойство выкинуть. Оно все равно readonly, поэтому испортить снаружи его никто не сможет. Но это в чем-то противоречит правилам кодирования - поля делать public - плохой тон.

Comments

[orloffm.livejournal.com]

VB.Net же вроде не различает регистр, разве в guidelines не указано, что нельзя делать переменные с отличающимися только регистром символов именами?

[metaclass.livejournal.com]

Это C#, он различает. По логике вещей идентификаторы различающиеся только регистром делать не стоило бы, но в данном случае получается противоречие - поле и свойство означают одно и тоже, соответственно делать сильно различающиеся имена нельзя.
В венгерской нотации это бы обходилось префиксом m_ для поля, но ее использовать в .NET не рекомендуется. Как и прочие вариации имен с подчеркиваниями в начале и тому подобное.