For-Schleife schreibt nicht in richtig ins Array

[Reehna]

Hallo Gemeinde,

kann mir vielleicht jemand einen Tipp geben, wo hier mein Fehler liegt?

Ich möchte per Schleife abfragen, ob der Eintrag im Array schon vorhanden ist und wenn ja, nur zwei dazugehörende Elemente ändern.

Der gesuchte Begriff wird auch gefunden, aber meine Änderungen werden immer in den letzten Eintrag des Arrays geschrieben.

Da hab ich sicher noch einen Denkfehler. Aber wo?

for (p=1; p<=_root.posit.length; p++)
        {
            if (_root.posit[p-1].bez == _root.Bauk1[_root.Bezeich]._name)
            {
                _root.posit[p-1].apos = _root.Bauk1[_root.Bezeich]._x;
                _root.posit[p-1].epos = _root.Bauk1[_root.Bezeich]._x + 17.4;
            }
        } 

[hgseib]

in diesem code-schnippsel ist kein fehler zu sehen.

benütz doch bitte mal den debugger von flash. da kannst du dein programm schrittweisse abarbeiten und in jede variable reingucken!

[Reehna]

Danke für dein Interesse.

Ich hab den Code schon durch den Debugger gejagt, komme aber mit dem schrittweise Abarbeiten nicht so zurecht.

Per trace hab ich mir den Wert von p-1 anzeigen lassen.
Auch der gesuchte Begriff wird richtig gefunden.

Das Schreiben ins Array funktioniert auch - nur ins falsche Element.

Vielelicht trägt ja der Aufbau des Arrays zur Klärung bei:

0 -
bez = name
apos = Zahl
epos = Zahl
1 -
bez = name
apos = Zahl
epos = Zahl
.
.
.
usw.

Ist teilweise assoziativ. Kann das vielleicht die Ursache sein?

Wie gesagt: es wird auch was ins Array geschrieben. So ganz falsch kann es also nicht sein.

Leider kann ich auch nicht die ganze Fla hochstellen, da das Teil mittlerweile ziemlich umfangreich ist.

[pokepika]

abgesehen davon, dass da kein fehler ist!?

warum machst du das nicht anstat:

for (p=1; p<=_root.posit.length; p++)
{
    if (_root.posit[p-1].bez == _root.Bauk1[_root.Bezeich]._name)
    // ...
} 

einfach so:

for (p=0; p < _root.posit.length; p++)
{
    if (_root.posit[p].bez == _root.Bauk1[_root.Bezeich]._name)
    // ...
} 

dann sparst du dir das p-1..

[Utopian]

Zeig mal wie du das Array definiert hast!

meinArr = Array;

geht z.B. nicht, verursacht auch keinen Fehler.

meinArr = Array();

wäre korrekt.

Gruß Christian

[Reehna]

Ich hab noch eine weitere Information:

Wenn p-1 = 0 ist, wird der veränderte Wert in
0 und in 2 geschrieben.

Wenn p-1 = 1 ist, wird der veränderte Wert in
1 und in 2 geschrieben.

Also immer das geforderte Element und der letzte Eintrag des Arrays.

[Reehna]

p=0 hab ich auch schon ausprobiert. Und dann natürlich nur p gesetzt, also ohne -1. Dummerweise bekomme ich genau das gleiche Ergebnis.


Das Array ist auf der root definiert:

_root.posit = new Array(); 

[Reehna]

Ich seh schon, ihr braucht mehr Input:

Das ist der ganze Code von diesem Button. Natürlich nicht der ganze Code vom Film. das wäre wirklich zu umfangreich.

Aber das Array _root.posit wird nur in diesem Teil angesprochen.
Definiert sind alle Variablen, die ein _root vorstehen haben, auch auf der root.

on(rollOver)
{
    this.createTextField(tooltip, 1, _xmouse, _ymouse, 50, 40);    
    this[tooltip].border = true;
    this[tooltip].background = true;
    this[tooltip].html = true;
    this[tooltip].backgroundColor = 0xFFFFCC;
    this[tooltip].selectable = false;
    this[tooltip].autoSize = true;
    this[tooltip].multiline = true;
    this[tooltip].wordWrap = false;
    this[tooltip].htmlText = _root.S190Bezeichnung;
    
    fontEigenschaften = new TextFormat();
    fontEigenschaften.bold = false;
    fontEigenschaften.size = 12;
    fontEigenschaften.font = "Arial";
    fontEigenschaften.align = "center";
    this[tooltip].setTextFormat(fontEigenschaften);
}
on(rollOut)
{
    this[tooltip].removeTextField();
}
 
on (press)
{
    var Teil={}; 
    Teil.onRollOver=function()
    { 
         _root.Bezeich = this._name;
    }
    Teil.onPress=function()
    {
        startDrag(this,false,36,84,507,84);
    }
    Teil.onRelease=function()
    {
        stopDrag();
        for (p=1; p<=_root.posit.length; p++)
        {
            if (_root.posit[p-1].bez == _root.Bauk1[_root.Bezeich]._name)
            {
                trace(p-1 + _root.posit[p-1].bez + _root.Bauk1[_root.Bezeich]._name);            
                _root.posit[p-1].apos = _root.Bauk1[_root.Bezeich]._x;
                _root.posit[p-1].epos = _root.Bauk1[_root.Bezeich]._x + 17.4;
                trace(_root.posit[p-1].apos);            
            }
        }
        
        _root.posit[_root.nr - 1] = {bez:_root.Bauk1[_root.Bezeich]._name, apos:_root.Bauk1[_root.Bezeich]._x, epos:_root.Bauk1[_root.Bezeich]._x + 17.4};
        _root.poslista = _root.posit.sortOn("apos", 16 | 8);
        _root.posliste = _root.posit.sortOn("apos", 18 | 8);
        _root.anfpos = _root.posit[_root.poslista[0]].apos;
        _root.endpos = _root.posit[_root.posliste[0]].epos;
        _root.breit = Math.round((_root.endpos - _root.anfpos) / 1.43);
        
    }
    var menu_cm = new ContextMenu();
    menu_cm.hideBuiltInItems();
    menu_cm.customItems.push(new ContextMenuItem("Loeschen", loeschen));
    function loeschen(menu, obj)
    {
        _root.Bauk1[_root.Bezeich].removeMovieClip();
        _root.preis = Number(_root.preis) - Number(_root.S190preis);
        _root.S190Z = _root.S190Z - 1;
        _root.posit.splice(_root.Bauk1[_root.Bezeich]);  
    }
    i = i + 1;
    _root.S190Z = _root.S190Z + 1;
    _root.preis = Number(_root.preis) + Number(_root.S190preis);
    if (_root.preis < _root.RabattStufe1)
    {
        _root.rabBetr = _root.RabattStufe1 - _root.preis;
        _root.rabProz = _root.RabattProz1;
    }
    if (_root.preis > _root.RabattStufe1)
    {
        _root.ProzBetr = (_root.preis * _root.RabattProz1)/100;
        _root.Nettopreis = _root.preis - _root.ProzBetr;
        _root.rabBetr = _root.RabattStufe2 - _root.preis;
        _root.rabProz = _root.RabattProz2;
    }
    if (_root.preis > _root.RabattStufe2)
    {
        _root.ProzBetr = (_root.preis * _root.RabattProz2)/100;
        _root.Nettopreis = _root.preis - _root.ProzBetr;        
        _root.rabBetr = 0;
        _root.rabProz = _root.RabattProz2;
    }   
    _root.Bauk1.attachMovie("St_190","St190"+i,i,Teil);
    (_root.Bauk1["St190"+i]).menu = menu_cm;
    _root.nr = _root.nr +1; 
    startDrag(_root.Bauk1["St190"+i],true,537,84,537,84);
}
on (release, releaseOutside)
{
    stopDrag();
} 

[Reehna]

Hat noch irgendjemand einen Tipp für mich?
Das Problem ist nach wie vor aktuell.

*verschämtnachobenschubs*

[hgseib]

ja, ich hab noch einen tipp:
mach bitte ein funktionierendes beispiel und lad das hoch. danke.

müssen WIR erst eine fla aus deinem programm bauen, damit wir DEINE fragen beantworten können?

------------------
var Teil={};
Teil.onRollOver=function()
{ ...

teil ist "garnichts." also kann das auch kein onRoll.. haben
und wo steht, wie du "posit" zu einem array machst?
und die da _root.posit[p-1] müssen ja auch irgendwann einmal zu einem array gemacht worden sein?

----------------
"..komme aber mit dem schrittweise Abarbeiten nicht so zurecht.."
- an der zu prüfenden stelle vor den code klicken (z.b. den skript in deinem button) - es entsteht ein roter punkt
- menüpunkt debuggen aufrufen
- einmal auf den weiter button klicken
- jetzt auf deinen button im swf klicken
- der code erscheint und hält am roten punkt.
- mit den pfeilen kannst du schrittweise weiter klicken.
- links im debugger das gewünschte objekt anklicken, variable anklicken ..