Vorhandensein eines Objektes prüfen...

[Jarell]

Kann gut sein, dass ich gerade den Wald vor Bäumen nicht mehr sehe,… AAaaaaber ich habe folgendes Problem:

Ich habe ein value object A, welches ein weiteres value object B enthält

objectA.objectB

Nun möchte ich in einer Komponente eine Fallunterscheidung machen, die sich danach richtet, ob bereits ein objectB existiert (bzw zugeordnet wurde) oder nicht.

Ich habe also total blauäugig folgende Prüfung durchgeführt:

if(objectA.objectB != null)

Und hier knallt es auch gleich:

TypeError: Error #1009: Der Zugriff auf eine Eigenschaft oder eine Methode eines null-Objektverweises ist nicht möglich.

If(objectA.objectB) habe ich auch noch probiert,… sonst habe ich aber keine zündende Idee mehr…

Kann mit da eventuell jemand aus der Patsche helfen?

[Martin Kraft]

Der Fehler tritt bereits in der if-Bedingung auf, weil bereits das erste Objekt nicht existiert. Du solltest deshalb jedes Objekt einzeln prüfen:

PHP-Code:

if(objectA && objectA.objectB){
   //...
} 


[hgseib]

u.U. auch
if (objectA && objectA!=null && objectA.objectB){..

und natürlich
try {
} catch..

möglich: die variable objectA wäre vorhanden, besitzt aber kein objectB. damit greifst du auch auf etwas zu, das es garnicht gibt.

[Martin Kraft]

Zitat von hgseib

if (objectA && objectA!=null [...]

Warum das? IMHO ist das doppelt gemoppelt: Wenn objectA==null ist, wäre auch Boolean(objectA)==false?!

[Jarell]

@ Martin Kraft

Danke für den Hinweis! Daran hat es tatsächlich gelegen. Habe die Komponente im Testbetrieb versucht zu starten und dabei vergessen das result-object zuzuweisen.

... den Wald vor Bäumen und so... ^^

[hgseib]

Zitat von Martin Kraft

Warum das?

hatte schon mehrmals den fall, das die browser fehler melden, die in flash niemals geworfen werden. da hat ein zusätzlicher test auf null immer geholfen.

[sytch]

hi

und natürlich
try {
} catch..

die verwendung von try/catch ist in meinen augen nur das umgehen von unsauberer programmierung. eine try/catch ist eine zuviel...

ich würde den vorschlag von martin kraft vorziehen...

[hgseib]

ja, vielleicht
schon der titel: 'Vorhandensein eines Objektes prüfen...' lässt auf unsauberes programmieren schliessen. weil: wer kennt da seine eigenen objekte nicht? aber, es gibt nunmal solche situationen.

und warum nur gibt es dann überhaupt try/catch ?
alles hat irgendwo seine berechtigung. und gerade in catch kann man sich dinge anzeigen lassen, wenn etwas nicht so ist wie es sein soll. also ja: besonders in der entwicklungsphase geeignet.

[Omega Psi]

@H.G.: Nicht jedes Objekt ist zwangsläufig zu initialisiert, dass alle Properties schon Werte besitzen oder andere Object referenzieren (sie werden lazy gebaut).

try {} catch {} sind ja dann doch eher für "richtige" Ausnahmen gedacht wie zum Beispiel fehlgeschlagene IO Operationen. try { } catch {} ist in ActionScript imo aber auch eher das größte Geschütz, da vieles über ErrorEvents verarbeitet werden kann und auch sollte, da alles asynchron ist.

Da in Flex vieles, obwohl asynchron, durch eine synchron wirkende Facade versteckt wird, sind null Checks die saubere Variante. Das ganze mal illustriert an folgendem Beispiel und gerade in der UI Entwicklung können die Modelle/Daten für die Views auch leer sein:

package fs.components
{
    import mx.core.IDataRenderer;
    import mx.core.UIComponent;

    public class Component extends UIComponent implements IDataRenderer
    {

        private var _data:Object;

        private var _dataChanged:Boolean;

        public function get data():Object
        {
            return _data;
        }

        public function set data(value:Object):void
        {
            if (_data === value)
            {
                return;
            }
            _data = value;
            _dataChanged = true;
            invalidateProperties();
            invalidateSize();
            invalidateDisplayList();
        }

        public function Component()
        {
        }

        override protected function commitProperties():void
        {
            super.commitProperties();

            if (_dataChanged)
            {
                if (_data)
                {
                    // process data
                }
                else
                {
                    // process null
                }
            }
        }
    }
}

Man sollte sich halt darüber im Klaren sein, welche Verantwortlichkeiten ein Objekt hat und wann der Zustand als valide gilt und wann nicht.