Pourquoi mon implémentation de get / set donne-t-elle une erreur de pile ?

Question

J'essaie ce qui suit en C# :

public class Reference : AuditableTable
    {
        [Range(0, 99, ErrorMessage = "{0} must be between {1} and {2}")]
        [DisplayName("Order")]
        public int Order {
                get { return Order; }
                set {
                    if ((value < 0) || (value > 99)) {
                        throw new Exception(string.Format("{0} must be between 0 and 99", value.ToString()));
                    } else {
                        Order = value;
                    }
                }
            }

Quelqu'un peut-il m'expliquer pourquoi cela donne une erreur de dépassement de pile lors de la lecture ?

Answer 1

Votre propriété se réfère à elle-même, ce qui entraîne une boucle infinie.
Une solution évidente consiste à utiliser un champ privé et à l'exposer par le biais de votre propriété :

private int order;
public int Order
{
   get
   {
      return order; //private field
   }
   set
   {
      if ((value < 0) || (value > 99)) {
               throw new Exception(string.Format("{0} must be between 0 and 99",    
                                     value.ToString()));
      } else {
          order = value; // again accessing the private field (setting this time)
      }
   }
}

Answer 2

Le problème est cette ligne :

get { return Order; }

Cela revient à appeler la propriété de manière récursive, ce qui entraîne un dépassement de pile.

Answer 3

Le setter et le getter se réfèrent tous deux à eux-mêmes, ce qui provoque le Stack Overflow. Corrigez-le comme ceci :

private int _order;

public int Order {
    get { return _order; }
    set {
        if ((value < 0) || (value > 99)) {
            throw new Exception(string.Format("{0} must be between 0 and 99", value.ToString()));
        } else {
            _order = value;
        }
    }
}

Answer 4

int _order;
public int Order {
    get { return _order; }
    set {
        if ((value < 0) || (value > 99)) {
            throw new Exception(string.Format("{0} must be between 0 and 99", value.ToString()));
        } else {
            _order = value;
        }
    }
}