Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Where should the line between property and method be? [duplicate]

Possible Duplicate:
Properties vs Methods

For many situations it is obvious whether something should be a property or a method however there are items that might be considered ambiguous.

Obvious Properties:

  • "name"
  • "length"

Obvious Methods:

  • "SendMessage"
  • "Print"

Ambiguous:

  • "Valid" / "IsValid" / "Validate"
  • "InBounds" / "IsInBounds" / "CheckBounds"
  • "AverageChildValue" / "CalcAverageChildValue"
  • "ColorSaturation" / "SetColorSaturation"

I suppose I would lean towards methods for the ambiguous, but does anyone know of a rule or convention that helps decide this? E.g. should all properties be O(1)? Should a property not be able to change other data (ColorSaturation might change R,G,B values)? Should it not be a property if there is calculation or aggregation?

Just from an academic perspective, (and not because I think it's a good idea) is there a reason not to go crazy with properties and just make everything that is an interrogation of the class without taking an argument, and everything that can be changed about the class with a single argument and cant fail, a property?

like image 640
Catskul Avatar asked Mar 12 '10 22:03

Catskul


4 Answers

I typically convert a property to a function if it has one of the following behaviors

  • Causes a side effect (other than setting the backing field)
  • Implementation is expensive when compared to say a field access
  • Implementation has higher complexity than Log(N)
  • Can throw an exception
like image 65
JaredPar Avatar answered Nov 13 '22 01:11

JaredPar


I´ve found some interesting text about this

MSDN | Properties vs Methods

EDIT

It says things like:

Use a property when

  • The member is a logical data member

Use a method when

  • The operation is a conversion, such as Object.ToString.
  • The operation is expensive enough that you want to communicate to the user that they should consider caching the result.
  • Obtaining a property value using the get accessor would have an observable side effect.
  • Calling the member twice in succession produces different results.
  • The order of execution is important. Note that a type's properties should be able to be set and retrieved in any order.
  • The member is static but returns a value that can be changed.
  • The member returns an array. Properties that return arrays can be very misleading.
  • Usually it is necessary to return a copy of the internal array so that the user cannot change internal state. This, coupled with the fact that a user can easily assume it is an indexed property, leads to inefficient code.
like image 27
Javier Avatar answered Nov 13 '22 01:11

Javier


Another consideration is bindability. Most frameworks can bind only to properties. For example, if you want IsValid to be usable in binding (say, as a binding sourcee for the OK button's IsEnabled property), then it has to be a property rather than a method.

like image 4
itowlson Avatar answered Nov 13 '22 00:11

itowlson


In deciding whether to use a property or a method you should also consider the ammount of work the method involves. I.e. if it is relatively cheap to retrieve the value make it a property, if it costly make it a method.

like image 1
AxelEckenberger Avatar answered Nov 13 '22 00:11

AxelEckenberger