I\'ve read some discussions about this subject, and there\'s something I just don\'t understand.
The most common answer seems to be this: use a ReadOnly Property to
Reason #1 = Properties can be used in data-binding. Methods cannot.
Reason #2 = When debugging the watch window will automatically expose the properties/expand the object. Methods do not get automatically watched this way.
I remember reading someplace that the read property should not change the object state. I think that is a good practice to follow especially considering the reason #2. As soon as you watch an object and expand it in the watch the objects state could be changed thus making debugging more difficult.
Also if a expensive property is bound by accident it can introduce serious performance problems. (Attributes can "hide" properties from data binding but just making them methods means you don't have to worry about it.)
Properties are conveniences for window development. They are "used" differently in designers etc.
(I sometimes just expose some variables. But I name them as I would a property. So instead of
Integer mCounter = 0;
I just do
Integer Counter = 0;.
If I have to do "extra stuff" at some point in the future I create the Property with the same name and rename the variable to mCounter. Admittedly this is laziness on my part and I don't really recomend it. Just wrap it up in a property.)
A good reason for read only properties are values that are calculated. In this case there is no variable to export.
For example
public class Person {
private readonly DateTime _birthday;
public int Age { get { return (DateTime.Now - _birthday).TotalYears; } }
...
}
In this case whether to expose _birthday as a property or a field is certainly debatable. But for other calculated values like Age, the only way to expose them as a variable is to store them in the object. In this case having an extra variable for Age is essentially storing redundant information. Exposing it as calculated property has minor overhead and avoids storing redundant data
Properties are a convenient substitute for method-based operations. There's no functional difference between a property-getter-setter and a method that performs the same actions; in fact, if you look at IL you'll see the property accessors replaced by "get" and/or "set" methods.
The strongest reason to use properties over just allowing access to the variables is encapsulation. Let's say you're writing a library and you expose an IsBlue
variable. You distribute the library, everyone loves and uses it. Now, it's time for version 2, and you want to do something when the user sets IsBlue
- maybe perform a check, maybe cache something. To do that, you'll have to convert the variable into a property or method, and do the check in there. Now, you've broken all your client code - the variable they had accessed isn't there anymore. If you had implemented it originally as a property, you could have just modified the property's code, and retained binary compatibility.
There is more to properties than auto properties. If you do:
public int MyProp { public get; public set; }
Then it's really no different from
public int MyProp;
But the big advantage of the former is that you can later change it to:
public int MyProp {
get {
// Do some processing
return someValue;
}
set {
// Do some processing
DoMyProcess(value);
}
}
And other code that uses your object would work without recompiling. If, instead, you had used a public field, client code would need to be recompiled if you ever want to change it from a field to a property.
"Cached data -> Public variable" - bad idea. This would allow another class to modify the cached data. Also, computing the data for caching might be expensive: using a read-only property allows you to defer calculation until the property is accessed.
It is considered bad practice to expose your variable from your class. Plus if you change the structure of your object but keep your properties intact, it won't affect the code that use your class.