Suche im Array nach Wert

[compuboy1010]

Es soll in einem Array nach einem Artikel gesucht werden und dessen Anzahl aktualisiert werden.

Komischerweise klappt das nur beim letzten Artikel im Array. Verstehe aber nicht warum.

function updateItemCount(nItemNo:Number, nCount:Number):Void {
    _level0.tTest.text = nItemNo + " " + nCount;
    var i:Number= 0;
    while(_global.aCardItems[i].no != nItemNo) {
        i++;
    }
    if(nCount == 0) {
        deleteItem(nItemNo);
    } else {
        _global.aCardItems[i].noItems = nCount;
    }
} 

[hgseib]

was sollen wir denn aus so einem fragment auslesen können?

nur mal so:
eine artikel-nummer ist eine eindeutige feststehende bezeichnung. danach muss man nicht suchen! z.b.:

var artikel = [];
artikel['7112'] = {bez:'hose', preis:3.99, anzahl:12};
artikel['0815'] = {bez:'jacke', preis:12.99, anzahl:9};
artikel['4711'] = {bez:'immer dabei', preis:0.99, anzahl:300};
//
artikel['0815'].anzahl += 5;
trace(artikel['0815'].anzahl); 

[compuboy1010]

Ja, aber ich habe ein Array von Arrays. Wie soll ich da direkt drauf zugreifen können? In deinem Beispiel ist die Artikelnummer der Index. Ich habe aber in jedem Arrayplatz ein Item Array und in einem dieser Plätze des Item Arrays steht die Artikelnummer. So muss ich danach suchen.

Das Array wurde so angelegt:

function addItemToCard(aItem:Array, nNoItemsAdded:Number):Void {
    var nPosition:Number = _global.aCardItems.length;
    var nItemsInCard:Number = nPosition + 1;
//  _level0.tTest.text = aItem.no + " - " + nNoItemsAdded + " (" + nItemsInCard + ")";
    if (nPosition == 0) {
        // if no item in card yet
         aItem.noItems = nNoItemsAdded;
        _global.aCardItems.push(aItem);
    } else {
        // if item with same item no in card is already in card
        var inArray:Boolean = false;
        for (var i = 0; i < nPosition; i++) {
            if (_global.aCardItems[i].no == aItem.no) {
                _global.aCardItems[i].noItems += nNoItemsAdded;
                inArray = true;
            }
        }
        if (!inArray) {
            // append item
            aItem.noItems = nNoItemsAdded;
            _global.aCardItems.push(aItem);
        }
    }
    //testoutput
//  for (var i = 0; i < _global.aCardItems.length; i++) {
//    _level0.tTest.text = _global.aCardItems[i].id + " " + _global.aCardItems[i].noItems + "\n";
//  }
} 

[hgseib]

"..Ja, aber ich habe ein Array von Arrays.."
das hat mein beispiel auch

du hast deine datenstruktur falsch aufgebaut! eins der wichtigsten dinge nutzt du garnicht. anstatt das eindeutige kriterium als kennung zu nutzen 'schmeisst' du alles auf einen haufen.

mit dem code den du hier veröffentlichst kann nur derjenige was anfangen, der weiss um was es geht (das bist hoffentlich du :-)

rein optisch gesehen müsste updateItemCount gehen und addItemToCard ist ober-kompliziert aufgebaut, aber nicht direkt falsch.

also nochmals: aus den fragmenten kann man nichts erkennen. ist wie kaffeesatz lesen. hab auch keine lust deinen kompletten script durchzusehen - damit du da nicht auf falsche gedanken kommst ;-)

tipp:
mal mit dem debugger schritt für schritt durchgehen

[compuboy1010]

Also ich habe das Skript mal umgeschrieben, nun schmiert der Flash Player aber immer ab. Er sagt mein Skript würde den Player verlangsamen und frage mich,ob ich abbrechen will. Was läuft da falsch?

function addItemToCard(aItem:Array, nNoItemsAdded:Number):Void {
    var nPosition:Number = _global.aCardItems.length;
    var nItemsInCard:Number = nPosition + 1;
//  _level0.tTest.text = aItem.no + " - " + nNoItemsAdded + " (" + nItemsInCard + ")";
    if (nPosition == 0) {
        // if no item in card yet
         _global.aCardItems[aItem.no] = aItem;
    } else {
        // if item with same item no in card is already in card
        if (_global.aCardItems[aItem.no]) {
            _global.aCardItems[aItem.no].noItems += nNoItemsAdded;
        } else { // append item
            _global.aCardItems[aItem.no] = aItem;;
        }
    }
 
} 

[Janoscharlipp]

an diesem Stückchen liegt es nicht, ist ja keine Schleife drinn.
Warum nimmst du nicht endlich mal den Debugger? Der frist niemanden auf.

[compuboy1010]

Genau schau dir mal bitte die Funktion addContentToContainer() an.

Die lässt den Rechner nun abschmieren. Wie muss ich die nun abändern, damit es passt?

PS: Ich weiß nicht wie ich den Debugger benutze. Ich kann auch nicht tracen, da die Daten aus der DB/XML kommen und ohne die kann ich nicht testen.

[compuboy1010]

Zitat von hgseib

was sollen wir denn aus so einem fragment auslesen können?

nur mal so:
eine artikel-nummer ist eine eindeutige feststehende bezeichnung. danach muss man nicht suchen! z.b.:

var artikel = [];
artikel['7112'] = {bez:'hose', preis:3.99, anzahl:12};
artikel['0815'] = {bez:'jacke', preis:12.99, anzahl:9};
artikel['4711'] = {bez:'immer dabei', preis:0.99, anzahl:300};
//
artikel['0815'].anzahl += 5;
trace(artikel['0815'].anzahl); 

Kannst du mir verraten, wie man dein Array in einer Schleife ausgeben kann?
Und warum steht die Artikelnummer in Anführungszeichen? Sollte das nicht ein numerischer Index sein?

[Janoscharlipp]

Eigentlich ist das kein Array, sondern ein Objekt. Deshalt müsste man schreiben:

var artikel = new Object();
artikel['7112'] = {bez:'hose', preis:3.99, anzahl:12}; 

Nun werden als Indizes Strings erwartet, gibst du Zahlen an, so werden diese in Strings umgewandelt.
Vorteil ist, dass ein Objekt nicht 7000 Einträge haben muss, um einen Eintrag mit dem Index 7001 zu haben.
Nachteil ist, dass der Inhalt von Objekten keine eindeutige Reihenfolge hat, es ist einfach eine Menge von Werten.

Ein Schleife über alle Einträge eines Objektes schreibt man:

var i : String;
 
for (i in deinObjekt) {
trace(i + " : " + deinObjekt[i]);
} 

[compuboy1010]

Und wie würde ich das dynamisch machen können.
Du hast die Artikelnummer ja in Anführungszeichen. Aber wie mache ich das dynamisch über die Funktion? Die Artikelnummer ist in dem Array das übergeben wird: aItem.no. Wie bekomme ich die in den Index?

Wenn ich das so schreibe:

artikel[aItem.no] = {bez:'hose', preis:3.99, anzahl:12};

ist es ja numerisch
und so

artikel['aItem.no'] = {bez:'hose', preis:3.99, anzahl:12};

bringt das auch nix.

Wie greife ich auf bestimmte Attribute des Objekts zu? Also nicht alle sondern aus der Objektmenge nur Attribute mit ganz bestimmten Namen.

[Der Frager]

Tag!
Wieso soll die Artikelnummer denn nicht nummerisch sein? Macht doch irgendwie Sinn...

var nr = 7112;
var nr1 = 7113;
var artikel = new Object();
artikel[nr] = {bez:'Hose', preis:3.99, anzahl:12};
artikel[nr1] = {bez:'Jacke', preis:93.99, anzahl:2};
for (i in artikel) {
    trace("ArtNr.: "+i+", "+artikel[i].bez+", Preis: "+artikel[i].preis+", Anzahl: "+artikel[i].anzahl);
} 

[compuboy1010]

Oh mann, jetzt habe ich den ganzen Warenkorb verhunzt.
Nun spuckt der gar keine Artikel mehr in der Anzeige(2. Funktion) aus:

_global.oCardItems = new Object();
function addItemToCard(aItem:Array, nNoItemsAdded:Number):Void {
    //  _level0.tTest.text = aItem.no + " - " + nNoItemsAdded;
    if (!_global.oCardItems[aItem.no]) {
        // if no item in card yet
        _global.oCardItems[aItem.no] = aItem;
    } else {
        // if item with same item no in card is already in card
        _global.oCardItems[aItem.no].noItems += nNoItemsAdded;
    }
}
 
function addContentToContainer():Void {
    var mcCardTable:MovieClip = this.mcEmptyScrollPane.spEmpty.spContentHolder;
    var oCardItems:Object = _global.oCardItems;
    //  add items to card display
    var nTotalPrice:Number;
    var price:Number = 0;
    var j:Number = 0;
    for (var i in oCardItems) {
        var mcItem:MovieClip = mcCardTable.attachMovie("cardEntry", "mcCardEntry"+j, j, {_x:0, _y:(30*i)+5});
        mcItem.tItem.text = oCardItems[i].no;
        mcItem.tName.text = oCardItems[i].description;
        mcItem.tNumber.value = Number(oCardItems[i].noItems);
        // calculation of discounted price
        if (oCardItems[i].noItems >= 12) {
            price = oCardItems[i].ab12;
        } else if (oCardItems[i].noItems >= 6) {
            price = oCardItems[i].ab6;
        } else if (oCardItems[i].noItems >= 4) {
            price = oCardItems[i].ab4;
        } else {
            price = oCardItems[i].ab1;
        }
        mcItem.tPrice.text = price;
        mcItem.tSubtotal.text = oCardItems[i].noItems * Number(price);
        nTotalPrice += Number(mcItem.tSubtotal.text);
        // Controller for "update" button
        mcCardTable["mcCardEntry"+j].btUpdate.onRelease = function() {
            updateItemCount(_global.oCardItems[this._parent._name.substr(11)].no, mcItem.tNumber.value);
            mcCard.emptyContainerMC();
            mcCard.onEnterFrame = function() {
                delete this.onEnterFrame;
                this.addContentToContainer();
            };
        };
        // Controller for "delete item" button
        mcCardTable["mcCardEntry"+j].mcDelete.onRelease = function() {
            mcCard.deleteItem(_global.oCardItems[this._parent._name.substr(11)].no);
            mcCard.emptyContainerMC();
            mcCard.onEnterFrame = function() {
                delete this.onEnterFrame;
                this.addContentToContainer();
            };
        };
        j++;
    }
    // end of for
    // ----------------------- <add "total price" mc to display> ------------------------------------------- \\
    mcCardTable.attachMovie("cardTotalPrice", "mcCardTotalPrice", 500);
    mcCardTable.mcCardTotalPrice._y = (30 * oCardItems.length)+70;
    mcCardTable.mcCardTotalPrice._x = 0;
    mcCardTable.mcCardTotalPrice.tTotalPrice.text = "";
    mcCardTable.mcCardTotalPrice.tTotalPrice.text = nTotalPrice;
    // ------------------------ </add "total price" mc to display> ----------------------------------------- \\
} 

[compuboy1010]

ich dachte die Zuordnung wäre falsch und habe es folgerndermaßen umgeschrieben:

function addItemToCard(aItem:Array, nNoItemsAdded:Number):Void {
        _level0.tTest.text = aItem.no + " - " + nNoItemsAdded;
        // if no item in card yet
    if (!_global.oCardItems[aItem.no]) {
        _global.oCardItems[aItem.no].no = aItem.no;
        _global.oCardItems[aItem.no].ab1 = aItem.ab1;
        _global.oCardItems[aItem.no].ab4 = aItem.ab4;
        _global.oCardItems[aItem.no].ab6 = aItem.ab6;      
        _global.oCardItems[aItem.no].ab12 = aItem.ab12;  
        _global.oCardItems[aItem.no].description = aItem.description;      
        _global.oCardItems[aItem.no].noItems = aItem.noItems;
    } else {
        // if item with same item no in card is already in card
        _global.oCardItems[aItem.no].noItems += nNoItemsAdded;
    }
} 

Klappt auch net, obwohl es eigentlich müsste...

[hgseib]

hier gehts ja heiss zu ;-)

"..addContentToContainer() an.
Die lässt den Rechner nun abschmieren..
wie Janoscharlipp schon geschrieben hat, in der geänderten version ist keine schleife drinn. das kann flash nicht aufhalten.

"..weiß nicht wie ich den Debugger benutze.."
schaum mal auf meine signatur!

"..Ich kann auch nicht tracen, da die Daten aus der DB/XML kommen.."
was geht da nicht bei Movie testen?
erstelle zum testen textfiles. ist flash doch egal, ob die daten von einem file kommen, oder eben frisch aus der DB gezapft.

"..Sollte das nicht ein numerischer Index sein?.."
a) nö b) ist flash absolut egal
generelle regel: wenn man nicht damit rechnet, dann ist es text!
z.b. postleitzahlen: man addiert keine PLZs, es gibt aber welche die mit null beginnen, also lieber text als zahl
z.b. hausnummer: werden auch nicht addiert und es gibt auch 7a als hausnummer, also lieber text
dito telefon-nummern usw. sind im sprachgebrauch alles zahlen, aber ein computermensch denkt eben anders als andere.

"..Und wie würde ich das dynamisch machen können.."
wie kann den flash z.b. sowas machen: 'hallo'+7
es wandelt die werte bei bedarf einfach um. nur in extremen fällen muss man nachhelfen mit z.b. Number(xxx); oder String(xxx);


"..Oh mann, jetzt habe ich den ganzen Warenkorb verhunzt.."
:-)))
zuerst einmal: KEINE PANIK

so ein warenkorb ist keine kleinigkeit und niemand hat gesagt das du ihn umschreiben sollst! ich hatte dir empfohlen, mal dein ding mit dem debugger zu testen.
wenn du wirklich glaubts, das komplett ändern zu müssen (die datenstruktur ist das fundament des hauses, die wirft man nicht mal eben so um), dann zuerstmal auf papier planen!!!!! dann stück für stück alles NEU aufbauen und nicht 'hier mal ganz schnell' und 'dort noch ein bisschen'. so kann man nicht programmieren.

naja, bei deinem 10. warenkorb wirds dann schon besser klappen.

-------
"..Klappt auch net, obwohl es eigentlich müsste.."
zum 100.sten mal, aus teilstücken kann keiner ersehen, ob etwas geht oder nicht. wenn, dann müsstest du das schon komplett uploaden, wenn wir was dazu sagen sollen. dann verhunzen wir das ding eben zu viert ;-)

[compuboy1010]

Es kann aber nur an der ersten Funktion liegen.

derFrager hat dem Objekt die Atriitbute so zugeordnet.

artikel[nr] = {bez:'Hose', preis:3.99, anzahl:12}; 

Bei mir kommen die Attribute aber via einem Array in die Funktion:

_global.oCardItems[aItem.no] = aItem; 

Und das scheint nicht zu klappen. Aber wie?