C# propiedades de solo lectura calculadas, ¿deberían ser métodos?

Question

Tengo varias entidades que tienen campos calculados en ellas, como TotalCost. En este momento los tengo a todos como propiedades, pero me pregunto si realmente deberían ser métodos. ¿Hay un estándar de C# para esto?

public class WorkOrder 
{ 
    public int LaborHours { get; set; } 
    public decimal LaborRate { get; set; } 

    // Should this be LaborCost()? 
    public decimal LaborCost 
    { 
     get 
     { 
      return LaborHours * LaborRate; 
     } 
    } 
} 

Answer 1

Está bien utilizar propiedades calculadas en lugar de métodos, siempre que el cálculo no tome un tiempo notable

Ver Property usage guidelines

Answer 2

En mi opinión, es una preferencia; es lo que quieres hacer Hago propretios en la mayoría de los casos, a menos que haya lógica involucrada. Además, si necesita pasar parámetros para cambiar la funcionalidad, obviamente se aplicaría un método ...

Answer 3

Los dejaría como propiedades. Pero no hay una razón "estándar" para hacer las cosas de una manera u otra. Si estás solo, haz lo que más te guste. Si está en un equipo, siga las convenciones que el resto de su equipo está siguiendo.

Answer 4

Creo que los métodos deben realizar acciones en el objeto, generalmente cambian el estado del objeto. Las propiedades deben reflejar el estado actual del objeto incluso si se calcula la propiedad. Entonces deberías mantener tus propiedades IMO.

Answer 5

Depende, si sus "propiedades" se convierten en mamuts y requieren una gran cantidad de lógica comercial, no deberían ser propiedades, debería haber un método. El ejemplo que ha publicado se ve como una propiedad. No hay una forma estándar de hacerlo, vaya con su instinto visceral; si parece que tiene que hacer mucho, probablemente necesites un método.

Answer 6

Si son a) livianos yb) no tienen efectos secundarios, los convertiría en Propiedades.

Ligero es un poco confuso, por supuesto, pero la regla general es: si alguna vez me tengo que preocupar de llamar a una propiedad (ya sea en un bucle o en cualquier otro lugar), posiblemente sea un método.

Answer 7

Creo que todas deberían ser propiedades. Mientras no cambie el estado del objeto, estoy de acuerdo con él como una propiedad.

Además, si estoy utilizando su clase para el enlace de datos (WPF, etc.), entonces puedo enlazar directamente a su propiedad sin tener que modificar/extender la clase.

Answer 8

Si una propiedad es particularmente costosa de calcular, podría cambiarla por un método GetWhatever(). Esto sirve como una pista para quien utiliza mi clase de que este valor requiere un trabajo importante para llegar, y la persona que llama debe almacenar en caché el valor en lugar de llamar al método varias veces.

Los cálculos triviales son perfectamente apropiados dentro de las propiedades.

Answer 9

De todos modos, se trata de azúcar sintáctico, así que quiero que sea convencional en su equipo, o lo que prefiera, siempre y cuando solo devuelva información sobre el objeto y no lo cambie ni interactúe con otros objetos.

Answer 10

MSDN contiene información sobre este here

diseñadores biblioteca de clases a menudo deben decidir entre la implementación de un miembro de la clase como una propiedad o un método. En en general, los métodos representan acciones y las propiedades representan datos.

¿Cuál crees que es? Una acción calcular/obtenerLaborCost o datos?

WorkOrder workOrder = new WorkOrder(); 
workOrder.LaborHours = 8; 
workOrder.LaborRate = 20; 

decimal cost = workOrder.LaborCost; // This is OK here 

pero si usted va a hacer esto para el mismo objeto también:

worOrder.LaborHours = 18; 
decimal newCost = workOrder.LaborCost 

Ahora bien, esto puede no ser una propiedad. Sería mucho mejor ser un método.

Answer 11

A veces, también debe tener en cuenta lo que está modelando ... En algunos dominios, los valores calculados a menudo o se espera que sean un atributo del modelo: una propiedad. Si este fuera el caso, entonces escríbalo como una propiedad aunque el cálculo no sea del todo trivial o un poco costoso de computar. Solo documente en su API o implemente algún mecanismo de almacenamiento en caché para minimizar el recálculo de esta propiedad.