15 Apr
2009
15 Apr
'09
7:20 p.m.
Florin Ciubotaru wrote:
Sergiu Dumitriu wrote:
I was just asking about the fact that, AFAIK, the C# naming convention
is to start things with upper case, like CurrentPagePublished instead of
currentPagePublished. As about more/less files for the same class, I'd
say it depends on whether the separation makes sense and is obvious, and
it actually improves the readability of the code. I can't say unless I
know what each file does.
Shouldn't properties start with uppercase? I
thought that's the convention.
That is a field. While generally speaking public fields are bad practice
they seem to be used by VSTO devs in the add-in class because:
- COM and .NET interoperability is done by using ref parameters. Fields
can be passed as ref, properties cannot.
- They want to keep the addin class files shorter. Properties
significantly increase the length of the file
- Some use public fields and only when a custom getter, setter or
indexer is needed then it's switched to private/protected.
- Properties are slower if not inlined at release.
I'm don't like public fields, but I would like to have the XWikiAddin.cs
file cleaner having only VSTO specific events if possible.
IMO all fields should be encapsulated so I'm thinking at moving them and
the properties in another file but keeping them in the same class.
Then, COM interop will be handled by always boxing/unboxing to local
variables in the methods that call COM, this is what I'm usually doing
anyway.
This would be the fourth file for this class. What do you think about
more files vs less but larger files? >> + public bool currentPagePublished =
false;
>> + /// <summary>
--
Sergiu Dumitriu
http://purl.org/net/sergiu/