Model - UpdateEvent

[grizu]

Hallo zusammen,

steh grad - nicht vor einem Problem, aber einer Frage:

Mein Model beinhaltet ein Objekt "List", das als Container für einzelne Unterobjekte dient.
Der Controller greift im Model über getter auf diese List zu, also:

model.list.unterobjekt.id = 2;

Jetzt hab ich folgende Frage:
Auch der View greift ja über diesen getter auf das Model zu, also kann ich nicht in dieser Funktion ein "update"-Event dispatchen.

Jetzt hab ich 2 Möglichkeiten:
Entweder ich schreib für den View eine eigene Funktion, über die er auf das model zugreifen kann, oder ich dispatche den "update"-Event im model.list.unterobjekt und leite den über "list" ans Model weiter, was ich irgendwie eleganter finde.

Ich mein, ist jetzt nur eine Kleinigkeit, aber mich würds interessieren, wie ihr das machen würdet.

Danke!

[mildesign]

Der Controller greift im Model über getter auf diese List zu, also:

das ist ein setter du setzt/veränderst ja werte.

Edit:
gerade erst gesehen
Benutz du etwa ein Object für dein Datenmodel?

[sebastian]

Hi,
wie wäre sowas:

var list : XYZ = model.list;
 
list.unterobjekt.id = 2;
 
model.list = list; 

Also du holst dir quasi eine Vorlage für deine neue Liste und setzt diese dann.
(Der erste Schritt dient nur zur Verdeutlichung. Natürlich ist das nur eine
Referenz auf das Objekt im Model und wird direkt verändert.)

Durch das erneute Setzen wird das Update ausgelöst.

[Omega Psi]

Ich würde dir raten, das analog zum DataBinding in Flex zu machen:

private var _value:Object;
 
[Bindable("valueChanged")]
public function get value():Object {
    return this._value;
}
 
public function set value(value:Object):void {
    if (this._value == value) {
        return;
    }
    this._value = value;
    this.dispatchEvent(new Event("valueChanged"));
} 

Dort, wo du dann auf Änderungen des Models (in dem obigen Beispiel nur Änderungen des Wertes value) bekommt das Model einen EventListener und in dem updatest du dann die View.

[grizu]

@mildesign: was meinst du mit object?

@sebastian:
Danke, das is es!

Trotzdem, rein interessehalber: was haltet ihr von dieser Event-Weiterleitung?

edit: omega, genauso schauen meine setter und getter aus, das problem is nur, das eben view und controller über den getter zugreifen, weil sie nicht direkt das model updaten, sondern das list-Objekt im model.

[Omega Psi]

Dein Model ist ein EventDispatcher, oder?

[grizu]

jop

[Omega Psi]

Hole dir nur die Referenz auf die Liste und setze einen EventListener. Das sollte so passen. Wenn sich die Liste ändert, update die View... das passt schon so. Ich arbeite nicht anders.

[sebastian]

Psi: Siehe Grizus Anmerkung. Wenn im Object was verändert wird und es
nicht erneut gesetzt wird, kommt das Update Event nicht.

Finde übrigens this._property doppelt gemoppelt. Der Unterstrich ist ja
schon für die Eigenschaften und damit ist das this überflüssig.

lg

[grizu]

meinst du jetzt die Variante von sebastian? oder den event in der liste dispatchen und ans model weiterleiten?

edit: ich steh auch auf this

[Omega Psi]

@Sebastian... oder den Underscore kriegst du einen Fehler, da Property und Accessor Sonst gleich benannt sind. Das muss so sein oder du benennst die Property ganz anders... was wenig Sinn macht.

Thema Update: dann liegt es aber an der Registrierung des EventListeners, nicht an der Implementierung des des Accessors.

[sebastian]

Zitat von Omega Psi

@Sebastian... oder den Underscore kriegst du einen Fehler, da Property und Accessor Sonst gleich benannt sind. Das muss so sein oder du benennst die Property ganz anders... was wenig Sinn macht.

Ohne den Underscore natürlich. Macht auch Sinn - dann am besten aber
einheitlich bei allen Eigenschaften.

Aber das "this" ist dann überflüssig.

[Omega Psi]

this ist funktional immer überflüssig, wenn man nicht gerade eine Selbstreferenzierung durchführst. Für mich erhöht es aber die Lesbarkeit des Codes -> persönliche Preferenz.

Aber wie gesagt:

private var value:Object;
 
[Bindable("valueChanged")]
public function get value():Object {
    return this.value;
}
 
public function set value(value:Object):void {
    if (this.value == value) {
        return;
    }
    this.value = value;
    this.dispatchEvent(new Event("valueChanged"));
} 

Wird nicht kompilieren: doppelter Verweis auf Value + Endlosschleife.

Kann auch sein sein, dass ich dich falsch verstanden hatte.

[grizu]

um jetzt nochmal zu fragen...

model.list im controller updaten, oder event aus der list ans model dispatchen?

[sebastian]

Ja, "will" ja das du genau das andere weglässt. Also Unterstriche beibehalten
und this weglassen.

this ist z.B. wichtig, wenn du gleichnamige lokale Variablen hast.

Aber wenn man sich daran hält, dass man bei allen(!) Eigenschaften den
Unterstrich nutzt, dann hat man eine schöne Trennung zu den lokalen
Variablen und auch keine Probleme bei Settern/Gettern.

In deinem Fall war das "this" ja nur zur Verdeutlichung, dass du auf eine
Eigenschaft zugreifst. Aber wenn diese Funktion der Unterstrich übernimmt,
dann kannst du das this auch weglassen.

(Habe es früher auch anders gemacht, aber mittlerweile finde ich es
so deutlich besser.)

Zurück zu Grizu, ist sein Thread, sorry