for-løkke problem.

levelItem skal lave som til et array så !

Item levelItem[] = new Item[levels.length];

for (int i = 0; i < levels.length; i++) {
levelItem[i].addActionListener(ML);
}

Så har du den som fx levelItem[0], osv. (hvis klassen ellers hedder Item)

Så kan du bruge:

for (int i = 0; i < levels.length; i++) {
levelItem[i].addActionListener(ML);
}

Og du tilføjer passende objekter på pladserne i levelItem[]

Du kan fx, hvis du allereded har dem:

  levelItem[0]=levelItem0;

...osv

Hey... Det vil ikke rigtigt virke... Det er til en menu hvor man skal kunne vælge level til et spil - level-navnene er så i et array som varierer i længde fra spil til spil... mvh Mads

Ok, så en HashMap måske - kender du den?

øhh - nææ - kigger lidt på suns website da...

Arrays bør virke i den beskrevne problem stilling.

Hvis du viser hvordan du har forsøgt at bruge arrays, så kan vi måske
hjælpe.

HashMap h = new HashMap();

//..der skal puttes noget i den:  h.put("levelItem1",etellerandet);

for (int i = 0; i < levels.length; i++) {
((Item)h.get("levelItem"+i)).addActionListener(ML);
}

Men jeg egentlig enig med Arne - hvorfor vil array ikke virke?

Hey.. Koden ser således ud:

JMenu levelMenu = new JMenu("Level");
menu.add(levelMenu);
for (int i = 0; i < levels.length; i++) {
JMenuItem levelItem = new JMenuItem("Level: " + i);
}

...hvor det så er meningen af levelItem også skal tilføjes tallet i...
levels er et array med forskellige navne på levels...
Kan godt være det bare er mig der er en spasser - men kan ikke få det til at virke

mvh
Mads

Måske noget a la:

JMenu levelMenu = new JMenu("Level");
menu.add(levelMenu);
JMenuItem[] levelItem = new JMenuItem[levels.length];
for (int i = 0; i < levels.length; i++) {
    levelItem[i] = new JMenuItem("Level: " + i);
}

Ahh - bonus!

Tak skal i ha - smid lige et svar i to!

mvh
Mads

Tak, jeg skal ikke have point.

hehe - fair nok - det syns jeg aldrig du skal ha - en beskeden herre...

Jeg er lige knap så beskeden ...

:-)

Hey igen

Hvis man lige kan nå at stille en hurtig udfordring mere:
Jeg har nogle actionlisteners der lytter på de forskellige menuting man kan vælge - men hvordan skal den lige omformuleres?

for (int i = 1; i <= levels.length; i++) {
else if (menuClicked.equals("Level: " +i)) {
  //kode til at skifte level
}
}

Det er en else if, og det kan man jo ikke ha' med if... mvh Mads

uden if skulle der vist have stået...

Jeg kan ikke helt gennemskue konteksten, men jeg har flere gange brugt en
løsning a la:

  public void actionPerformed(ActionEvent ev) {
      for (int i = 0; i < x.length; i++) {
        if (ev.getSource() == x[i]) {
            // brugeren klikkede på x[i]
        }
      }
  }

Hey - nææ var vist ikke for tydelig der... Problemet er at der er flere forskellige menupunkter og at de ikke hedder noget med level allesammen.... Men hvis man bare kan smide flere "public void actionevent"'s ind så er de jo en herlig løsning! mvh Mads

Så skal den vel bare udvides:

  public void actionPerformed(ActionEvent ev) {
      for (int i = 0; i < x.length; i++) {
        if (ev.getSource() == x[i]) {
            // brugeren klikkede på x[i]
        }
      }
      for (int i = 0; i < y.length; i++) {
        if (ev.getSource() == y[i]) {
            // brugeren klikkede på y[i]
        }
      }
      for (int i = 0; i < z.length; i++) {
        if (ev.getSource() == z[i]) {
            // brugeren klikkede på z[i]
        }
      }
  }

Hey... Det er kun level der er et array... De andre er bare normale statiske menupunkter:

public void actionPerformed(ActionEvent menuClick) {
  if (menuClick.getSource() instanceof JMenuItem) {
  String menuClicked = menuClick.getActionCommand();

  if (menuClick.equals("Restart")) {
    //kode til at genstarte spil
  }

  else if (menuClicked.equals("Open")) {
  //kode til at åbne spil
  }

  ?????????
  for (int i = 1; i <= levels.length; i++) {
    if (menuClicked.equals("Level: " +i)) {
      //kode til at skifte level
  ?????????
    }
  }
}
}

Hov - du skulle jo ikke snydes for de andre point!

Tja - det skal sikkert nok virke.

Jeg plejer bare altid at teste på getSource().

k - prøver noget - takker!