ongelmia valikkotapahtumankäsittelyssä

Olet jo itse tehty tuon sulkemis homman. et samanlailla vaan muut.
Siis:
MenuItem foobar = new MenuItem();
valikko1.add(foobar);
foobar.addActionListener(this);

public void actionPerformed(ActionEvent e){
if(e.getSource()==foobar){ //ja tahan mita foobar tekee}
}
Kun tuo saa toimimaan ni voi alkaa kattomaan setActionCommand-komentoa tai alkaa miettiin et tekis oman actionlistenerin jokaselle toiminnolle.

Koodissasi on se vika että olet kommentoinut luokan sisäisen "lopetamenuitem" -muuttujan pois ja lisännyt vastaavan nimisen muuttujan muodostimeen.

--- ALKAA ---

import java.awt.*;
import java.awt.event.*;

public class Harjoitus4kutistettu extends Frame implements WindowListener, ActionListener {
   Button      suljenappi;
   MenuBar      palkki;
   Menu      valikko1, valikko2, valikko3, valikko4;
   MenuItem   lopetamenuitem;
   
   private Font textFont;

   public static void main(String[] args) {
      Harjoitus4kutistettu myWindow = new Harjoitus4kutistettu("Sovelluksen_nimi");
      myWindow.setSize(300,400);
      myWindow.setVisible(true);
   }
   
   public Harjoitus4kutistettu(String title) {
      super(title);
      setLayout(new FlowLayout());
      addWindowListener(this);
      suljenappi = new Button(" X Sulje");
      add(suljenappi);
      suljenappi.addActionListener(this);

      palkki = new MenuBar();
      palkki.add(valikko1=new Menu("Tiedosto"));

      valikko1.add(new MenuItem("Avaa"));
      valikko1.add(new MenuItem("Tallenna"));
      valikko1.add(new MenuItem("Uusi"));

      lopetamenuitem=valikko1.add(new MenuItem("Lopeta"));
      lopetamenuitem.addActionListener(this);
      
      setMenuBar(palkki);
   }

   public void actionPerformed(ActionEvent e) {
      if(e.getSource() == suljenappi) {
         dispose();
         System.exit(0);
      }

      if(e.getSource() == lopetamenuitem) {
         dispose();
         System.exit(0);
      }
}

   public void windowClosing(WindowEvent e) {
      dispose();
      System.exit(0);
   }

   public void paint(Graphics g) {
      textFont = new Font("Arial", Font.BOLD, 20);
      g.setFont(textFont);
      g.drawString("", 70,100);
   }

   public void windowOpened(WindowEvent e) {}
   public void windowActivated(WindowEvent e) {}
   public void windowIconified(WindowEvent e) {}
   public void windowDeiconified(WindowEvent e) {}
   public void windowDeactivated(WindowEvent e) {}
   public void windowClosed(WindowEvent e) {}
}

--- LOPPUU ---


Mutta, kuten joku jo sanoi, niin setActionCommand käskyn käyttö olisi varmaankin näppärämpää kuin luoda jokaista välitettävää tapahtumaa varten oma olio.

ökjaölksjfdöaklsdf kirjoitti:

Koodissasi on se vika että olet kommentoinut luokan sisäisen "lopetamenuitem" -muuttujan pois ja lisännyt vastaavan nimisen muuttujan muodostimeen.

--- ALKAA ---

import java.awt.*;
import java.awt.event.*;

public class Harjoitus4kutistettu extends Frame implements WindowListener, ActionListener {
   Button      suljenappi;
   MenuBar      palkki;
   Menu      valikko1, valikko2, valikko3, valikko4;
   MenuItem   lopetamenuitem;
   
   private Font textFont;

   public static void main(String[] args) {
      Harjoitus4kutistettu myWindow = new Harjoitus4kutistettu("Sovelluksen_nimi");
      myWindow.setSize(300,400);
      myWindow.setVisible(true);
   }
   
   public Harjoitus4kutistettu(String title) {
      super(title);
      setLayout(new FlowLayout());
      addWindowListener(this);
      suljenappi = new Button(" X Sulje");
      add(suljenappi);
      suljenappi.addActionListener(this);

      palkki = new MenuBar();
      palkki.add(valikko1=new Menu("Tiedosto"));

      valikko1.add(new MenuItem("Avaa"));
      valikko1.add(new MenuItem("Tallenna"));
      valikko1.add(new MenuItem("Uusi"));

      lopetamenuitem=valikko1.add(new MenuItem("Lopeta"));
      lopetamenuitem.addActionListener(this);
      
      setMenuBar(palkki);
   }

   public void actionPerformed(ActionEvent e) {
      if(e.getSource() == suljenappi) {
         dispose();
         System.exit(0);
      }

      if(e.getSource() == lopetamenuitem) {
         dispose();
         System.exit(0);
      }
}

   public void windowClosing(WindowEvent e) {
      dispose();
      System.exit(0);
   }

   public void paint(Graphics g) {
      textFont = new Font("Arial", Font.BOLD, 20);
      g.setFont(textFont);
      g.drawString("", 70,100);
   }

   public void windowOpened(WindowEvent e) {}
   public void windowActivated(WindowEvent e) {}
   public void windowIconified(WindowEvent e) {}
   public void windowDeiconified(WindowEvent e) {}
   public void windowDeactivated(WindowEvent e) {}
   public void windowClosed(WindowEvent e) {}
}

--- LOPPUU ---


Mutta, kuten joku jo sanoi, niin setActionCommand käskyn käyttö olisi varmaankin näppärämpää kuin luoda jokaista välitettävää tapahtumaa varten oma olio.

Lue lisää

Ovat siis kommentissa koska eivät ainakaan minulla toimineet. Ei tuosta Tiedosto valikon lopeta kohdasta tapahtunut mitään.

Siksi yleesäkin kysyn koko asiaa, koska minusta tuon olisi pitänyt toimia noin.

Ja näettepähän tässä ettei kaikki kysele heti vaan sitten kun on jo tehty ja yritetty...

koska eivät toimineet kirjoitti:

Ovat siis kommentissa koska eivät ainakaan minulla toimineet. Ei tuosta Tiedosto valikon lopeta kohdasta tapahtunut mitään.

Siksi yleesäkin kysyn koko asiaa, koska minusta tuon olisi pitänyt toimia noin.

Ja näettepähän tässä ettei kaikki kysele heti vaan sitten kun on jo tehty ja yritetty...

Lue lisää

enpä ole tömännyt SetActionCommand käskyn käyttöön. Pitää yrittää ihmtelellä asiaa jos vain aika riittää.

koska eivät toimineet kirjoitti:

Ovat siis kommentissa koska eivät ainakaan minulla toimineet. Ei tuosta Tiedosto valikon lopeta kohdasta tapahtunut mitään.

Siksi yleesäkin kysyn koko asiaa, koska minusta tuon olisi pitänyt toimia noin.

Ja näettepähän tässä ettei kaikki kysele heti vaan sitten kun on jo tehty ja yritetty...

Lue lisää

Alla on nyt toimiva esimerkki. Muutoksia: Lisäsin yhden label:n näkymään, jonka teksti muuttuu kun valitaa avaa tai tallenna valikosta. Avaa on toteutettu vertaimella olioita ja tallenna setActionCommand:lla. Tehty java 1.6.03
---
import java.awt.*;
import java.awt.event.*;

public class Harjoitus4kutistettu extends Frame implements WindowListener, ActionListener {

Button suljenappi;

MenuBar palkki; //TÄMÄ
Menu valikko1, valikko2, valikko3, valikko4; //TÄMÄ

//Lisäykset
Label testi = new Label("Teksti");
MenuItem avaa = new MenuItem("Avaa");
MenuItem tallenna = new MenuItem("Tallenna");

//MenuItem lopetamenuitem;
private Font textFont; // Tekstin fontti.

public static void main(String[] args) {
Harjoitus4kutistettu myWindow = new Harjoitus4kutistettu("Sovelluksen_nimi"); //ikkunan otsikko
myWindow.setSize(300, 400);
myWindow.setVisible(true);

}

public Harjoitus4kutistettu(String title) {
super(title);
setLayout(new FlowLayout());
addWindowListener(this);
suljenappi = new Button(" X Sulje");
add(suljenappi);
suljenappi.addActionListener(this);
add(testi);
palkki = new MenuBar();
palkki.add(valikko1 = new Menu("Tiedosto"));
//palkki.add(valikko1=new Menu("Tiedosto")).addActionListener(this);

//MenuItem avaa = new MenuItem("Avaa");
//lisätään valikoihin komennot, menu-itemit

valikko1.add(avaa);
valikko1.add(tallenna);
valikko1.add(new MenuItem("Uusi"));
//valikko1.add(new MenuItem("Avaa"));
//valikko1.add(new MenuItem("Lopeta"));
//valikko1.add(new MenuItem("Lopeta")).addActionListener(this);

MenuItem lopetamenuitem = valikko1.add(new MenuItem("Lopeta"));
lopetamenuitem.addActionListener(this);
avaa.addActionListener(this);
tallenna.addActionListener(this);

tallenna.setActionCommand("tallenna");

//lopetamenuitem.addWindowListener(this); //miksi ei kelpaa

//valikko näkyviin
setMenuBar(palkki);
}

public void actionPerformed(ActionEvent e) //(ActionEvent evt)
{
if (e.getSource() == suljenappi) //if(evt.getSource()==suljenappi)
{
dispose(); //Tämä on ikkunaan tehdyllä Sulje-napilla sulkeminen
System.exit(0);
}
if (e.getSource() == (Object) avaa) {
testi.setText("Avaa");
}
if(e.getActionCommand().equals("tallenna")){
testi.setText("Tallenna");
}

/*
if(e.getSource()==lopetamenuitem)
{
dispose(); //Tämä on ikkunaan tehdyllä Sulje-napilla sulkeminen
System.exit(0);
} */
}

public void windowClosing(WindowEvent e) {
dispose(); //Tämä on ikkunan oikean yläkulman Sulje-toiminto
System.exit(0);
}

public void paint(Graphics g) {
// Järjestelmä kutsuu tätä metodia aina, kun
// sovelluksen ikkuna piirretään näytölle.
textFont = new Font("Arial", Font.BOLD, 20); //tässä fontin muotoilu
g.setFont(textFont); // Asetetaan fontti.
g.drawString("", 70, 100); //tässä ikkunan teksti ja sijainti ikkunassa
}

public void windowOpened(WindowEvent e) {
}

public void windowActivated(WindowEvent e) {
}

public void windowIconified(WindowEvent e) {
}

public void windowDeiconified(WindowEvent e) {
}

public void windowDeactivated(WindowEvent e) {
}

public void windowClosed(WindowEvent e) {
}
}
---

koska eivät toimineet kirjoitti:

Ovat siis kommentissa koska eivät ainakaan minulla toimineet. Ei tuosta Tiedosto valikon lopeta kohdasta tapahtunut mitään.

Siksi yleesäkin kysyn koko asiaa, koska minusta tuon olisi pitänyt toimia noin.

Ja näettepähän tässä ettei kaikki kysele heti vaan sitten kun on jo tehty ja yritetty...

Lue lisää

Ei se avauksen koodisi toimi vaikka kommentit olisi poistettu, koska siinä lopetamenuitem oli "esitelty" kahdessa erissä paikkaa, (vrt. tuohon toimivaan koodiin jonka postitin jossa se on vain yhdessä paikkaa).

Edellisen lisäksi siinä koodissasi oli "lopetamenuitem.addWindowListener(this)" rivi joka jää jo kääntäjään nalkkiin. Mutta sen varmaan olit jo ennen osannut poistaa, koska muutenhan koodisi ei olisi edes lähtenyt käyntiin.

toimiva esimerkki kirjoitti:

Alla on nyt toimiva esimerkki. Muutoksia: Lisäsin yhden label:n näkymään, jonka teksti muuttuu kun valitaa avaa tai tallenna valikosta. Avaa on toteutettu vertaimella olioita ja tallenna setActionCommand:lla. Tehty java 1.6.03
---
import java.awt.*;
import java.awt.event.*;

public class Harjoitus4kutistettu extends Frame implements WindowListener, ActionListener {

Button suljenappi;

MenuBar palkki; //TÄMÄ
Menu valikko1, valikko2, valikko3, valikko4; //TÄMÄ

//Lisäykset
Label testi = new Label("Teksti");
MenuItem avaa = new MenuItem("Avaa");
MenuItem tallenna = new MenuItem("Tallenna");

//MenuItem lopetamenuitem;
private Font textFont; // Tekstin fontti.

public static void main(String[] args) {
Harjoitus4kutistettu myWindow = new Harjoitus4kutistettu("Sovelluksen_nimi"); //ikkunan otsikko
myWindow.setSize(300, 400);
myWindow.setVisible(true);

}

public Harjoitus4kutistettu(String title) {
super(title);
setLayout(new FlowLayout());
addWindowListener(this);
suljenappi = new Button(" X Sulje");
add(suljenappi);
suljenappi.addActionListener(this);
add(testi);
palkki = new MenuBar();
palkki.add(valikko1 = new Menu("Tiedosto"));
//palkki.add(valikko1=new Menu("Tiedosto")).addActionListener(this);

//MenuItem avaa = new MenuItem("Avaa");
//lisätään valikoihin komennot, menu-itemit

valikko1.add(avaa);
valikko1.add(tallenna);
valikko1.add(new MenuItem("Uusi"));
//valikko1.add(new MenuItem("Avaa"));
//valikko1.add(new MenuItem("Lopeta"));
//valikko1.add(new MenuItem("Lopeta")).addActionListener(this);

MenuItem lopetamenuitem = valikko1.add(new MenuItem("Lopeta"));
lopetamenuitem.addActionListener(this);
avaa.addActionListener(this);
tallenna.addActionListener(this);

tallenna.setActionCommand("tallenna");

//lopetamenuitem.addWindowListener(this); //miksi ei kelpaa

//valikko näkyviin
setMenuBar(palkki);
}

public void actionPerformed(ActionEvent e) //(ActionEvent evt)
{
if (e.getSource() == suljenappi) //if(evt.getSource()==suljenappi)
{
dispose(); //Tämä on ikkunaan tehdyllä Sulje-napilla sulkeminen
System.exit(0);
}
if (e.getSource() == (Object) avaa) {
testi.setText("Avaa");
}
if(e.getActionCommand().equals("tallenna")){
testi.setText("Tallenna");
}

/*
if(e.getSource()==lopetamenuitem)
{
dispose(); //Tämä on ikkunaan tehdyllä Sulje-napilla sulkeminen
System.exit(0);
} */
}

public void windowClosing(WindowEvent e) {
dispose(); //Tämä on ikkunan oikean yläkulman Sulje-toiminto
System.exit(0);
}

public void paint(Graphics g) {
// Järjestelmä kutsuu tätä metodia aina, kun
// sovelluksen ikkuna piirretään näytölle.
textFont = new Font("Arial", Font.BOLD, 20); //tässä fontin muotoilu
g.setFont(textFont); // Asetetaan fontti.
g.drawString("", 70, 100); //tässä ikkunan teksti ja sijainti ikkunassa
}

public void windowOpened(WindowEvent e) {
}

public void windowActivated(WindowEvent e) {
}

public void windowIconified(WindowEvent e) {
}

public void windowDeiconified(WindowEvent e) {
}

public void windowDeactivated(WindowEvent e) {
}

public void windowClosed(WindowEvent e) {
}
}
---

Lue lisää

Tuo ohjelma käyttää niin vanhoja kirjastoja ja metodeja, että ne löytyvät vanhimmistakin kääntäjistä, eikä ohjelma käytä muitakaan kielen uusia ominaisuuksia joka estäisi kääntämisen vanhalla kääntäjällä tai vanhaan versioon.

Työpöytäsovelluksisa Java ohjelmien käyttöliittymät tehdään nykyään usein Swing:llä AWT:n sijasta, (eli: JFrame, JMenuItem, JMenu, JMenuBar). Swing komponenteissa on se hyvä puoli että ne näyttävät samanlaisilta eri käyttöjärjestelmissä, eikä niissä ole käyttöliittymäkohtaisia rajoituksia. (Apletteihin swing ei taida soveltua kovin hyvin).

ökjaölksjfdöaklsdf kirjoitti:

Ei se avauksen koodisi toimi vaikka kommentit olisi poistettu, koska siinä lopetamenuitem oli "esitelty" kahdessa erissä paikkaa, (vrt. tuohon toimivaan koodiin jonka postitin jossa se on vain yhdessä paikkaa).

Edellisen lisäksi siinä koodissasi oli "lopetamenuitem.addWindowListener(this)" rivi joka jää jo kääntäjään nalkkiin. Mutta sen varmaan olit jo ennen osannut poistaa, koska muutenhan koodisi ei olisi edes lähtenyt käyntiin.

Lue lisää

se oli kahdessa paikkaa koska testasin kummassa sen pitäisi olla. Minulla on noissa koodeissa tekovaiheeessa ihan perkeleellinen määrä tavaraa kommenteissa erilaisten kokeilujen ja vaihtoehtojen takia...

Mutta noiden esimerkkien ja omasta takaa löytyvien esimerkkien saan taas jatkettua eteenpäin noita hommia. Kiitoksia.