Ich bin in Endlosschleife im property-setter

public int Position
{
    get
    {
        if (Session["Position"] != null)
        {
            Position = Convert.ToInt32(Session["Position"]);
        }
        else
        {
            Position = 5;
        }
        return Position;
    }
    set
    {
        Position = value;
    }
}

mein Programm ruft die get-und geht in die if-Schleife und dann läuft infitely in den code einstellen, der

  • hat der korrigierte code, aber weiter erklären, warum: Denken Sie an diese Getter und setter als Funktionen, weil das ist, was Sie hinter den kulissen. Wenn eine Funktion sich selbst aufruft und innerhalb, ohne ein Mittel zu beenden, werden Sie in eine unendliche Schleife.
InformationsquelleAutor Ben | 2013-04-22
  1. 13

    Der Fehler ist, da in Ihr set {} werden Sie den Aufruf der gleichen Set-rekursiv.

    Richtige code wäre

    private int _position;
public int Position
{
    get
    {
        if (Session["Position"] != null)
        {
            this._position = Convert.ToInt32(Session["Position"]);
        }
        else
        {
            this._position = 5;
        }
        return this._position;
    }
    set
    {
        this._position = value;
    }
}
    • Sie nicht wirklich brauchen, den Unterstrich und die this zur gleichen Zeit. Die meisten Leute werden erkennen, dass der Unterstrich bedeutet, dass Sie in Bezug auf ein Mitglied vor Ort, und es gibt nichts, das muss disambiguated.
    • Der Unterstrich ist wirklich nur eine Frage der Gewohnheit für mich (und viele andere), aber meiner Meinung nach (und auch StyleCop stackoverflow.com/questions/1562540/...) this sollte zwingend immer.
    • Ja, StyleCop ist falsch. 🙂 Und das Stack-Overflow-Frage hat abweichende Meinungen.
    InformationsquelleAutor Knaģis
  2. 4

    Verwenden Sie eine member-variable oder vielleicht speichern Sie es in der Sitzung.

    private int _position;
public int Position
{
    get
    {
        if (Session["Position"] != null)
        {
            _position= Convert.ToInt32(Session["Position"]);
        }
        else
        {
            _position= 5;
        }
        return _position;
    }
    set
    {
        _position = value;
    }
}
    InformationsquelleAutor Daniel A. White
  3. 2

    Es gibt nichts besonders-string-wie etwa session-state Elemente.

    Warum würden Sie nicht einfach Folgen Sie dem KISS-Prinzip und etwas tun, wie

    public int Position
{
  get { return (int) ( Session["Position"] ?? 5 ) ; }
  set { Session["Position"] = value ;               }
}

    oder (abhängig von Ihren tatsächlichen Anforderungen/Spezifikationen:

    public int Position
{
  get { return Session["Pointer"] as int? ?? position ?? 5 ; }
  set { position = value ; }
}
private int? position ; //backing store
    • als int? int ist-Wert geben, wie kann es sein as?
    • ist syntaktischer Zucker für Nullable<int>, ein Referenz-Typ (und spezielle, zu Booten). Eine gerade int erhalten boxed in der session zu speichern; eine int? ist für alle Absichten und Zwecke, schon boxed. Siehe die Skillung, ISO 23270 § 14.9.11: Der as-operator wird verwendet, um explizit konvertiert einen Wert in einen bestimmten Referenz-Typ oder null-Werte zulässt Typ...Bei der Konvertierung in einen nullable-Typ, der auch als operator verwendet eine Verpackung Konvertierung, ein unboxing-Konvertierung, oder ein null-Typ-Umwandlung (§13.7.1)...wenn die angegebene Umwandlung ist nicht möglich, der resultierende Wert ist null.
    • Sorry .. nur der Lesbarkeit problem ..
    InformationsquelleAutor Nicholas Carey
  4. 1

    Eine automatisch implementierte Eigenschaft Anwesen besteht aus einem getter einen setter und ein Feld sichern. Wenn Sie den code selbst schreiben, ein Feld kann nicht notwendig sein.

    Ihre getter ruft setter und setter aufruft setter; das wäre eine unendliche Rekursion. Müssen Sie möglicherweise ein Feld für die Speicherung Position.

    Allerdings, wenn wir ändern es mit der Speicherung auf einem Feld, und der setter in der Tat nicht wirksam. So konnte der code geändert werden: 

    public int Position {
    set {
    }

    get {
        int x;
        return (x=Convert.ToInt32(Session["Position"]))>0?x:5;
    }
}

    Müssen Sie nicht auf null prüfen, Convert.ToInt32(null) null ist.

    InformationsquelleAutor Ken Kin
Schreibe einen Kommentar