ongelmia valikkotapahtumankäsittelyssä

Eli olisi ongelmia valikkotapahtumankäsittelyssä.

Ohjelma toimii eli ikkuna aukeaa. Ikkunassa valikko "Tiedosto", jossa alivalikkoina "Avaa", "Uusi", "Lopeta".

Nyt pitäisi tehdä noille alavalikoille tapahtumankäsittelyä eli esim. "Sulje" sammuttaa ikkunan. En saa tuota tehtyä kun en ymmärrä miten saan tuon ikkunan sulkemistoiminnon tuon "Sulje" alavalikkkoon. Olen saanut toimimaan vain siten, että toimii mistä tahansa "Tiedosto" valikon toiminnosta.

Liitän kirjoitetun koodin tarvittaessa myöhemmin.
9548
Äänestä

Muut lukivat myös

Vastaukset 9

Vanhimmat

---------------------------
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Ä

//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);

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


//lisätään valikoihin komennot, menu-itemit
valikko1.add(new MenuItem("Avaa"));
valikko1.add(new MenuItem("Tallenna"));
valikko1.add(new MenuItem("Uusi"));
//valikko1.add(new MenuItem("Lopeta"));
//valikko1.add(new MenuItem("Lopeta")).addActionListener(this);
MenuItem lopetamenuitem=valikko1.add(new MenuItem("Lopeta"));
lopetamenuitem.addActionListener(this);
//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()==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) {}

}
-----------------

Eli miten tuon Tiedosto valikon toimintoihin saa "yhteyden"?

Kuinka voi tämän osuuden tehdä muutoin
------
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
}
--------------

Helppinkivä?

Äänestä
  • 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.

    Äänestä

  • 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.

    Äänestä

  • ö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.

    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...

    Äänestä

  • 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...

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

    Äänestä

  • 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...

    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) {
    }
    }
    ---

    Äänestä

  • 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...

    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.

    Äänestä

  • 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) {
    }
    }
    ---

    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).

    Äänestä

  • ö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.

    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.

    Äänestä

Kommentoi aloitusta


en saa toimimaan kirjoitti:

Eli olisi ongelmia valikkotapahtumankäsittelyssä.

Ohjelma toimii eli ikkuna aukeaa. Ikkunassa valikko "Tiedosto", jossa alivalikkoina "Avaa", "Uusi", "Lopeta".

Nyt pitäisi tehdä noille alavalikoille tapahtumankäsittelyä eli esim. "Sulje" sammuttaa ikkunan. En saa tuota tehtyä kun en ymmärrä miten saan tuon ikkunan sulkemistoiminnon tuon "Sulje" alavalikkkoon. Olen saanut toimimaan vain siten, että toimii mistä tahansa "Tiedosto" valikon toiminnosta.

Liitän kirjoitetun koodin tarvittaessa myöhemmin.

Anonyymi
5000

Ilmoita asiaton sisältö

  • Sisältää materiaalia, joka on loukkaavaa, herjaavaa, rasistista, uhkailevaa tai ahdistelevaa.

  • Materiaali sisältää nimiä, yhteystietoja tai muita henkilökohtaisia tietoja. Julkisuuden henkilöistä, julkisissa viroissa toimivista ihmisistä sekä yritysten vastuuhenkilöistä saa keskustella työhön liittyen. Myös yksityiselämään liittyvistä asioista voi keskustella siltä osin, kuin niistä on julkisesti kerrottu.

  • Sisältää lapsille haitallista tai heiltä kiellettyä materiaalia. Seksuaalinen sisältö on sallittua aiheeseen tarkoitetuilla palstoilla, joiden ikäraja on 18 vuotta.

  • Lainvastaista sisältöä voi olla mm. yksityiselämää loukkaavan tiedon levitys, kiihottaminen kansanryhmää vastaan, laiton uhkaaminen, alaikäisiin liittyvä seksuaalinen sisältö/grooming, petokset, identiteettivarkaudet, kunnianloukkaukset ja tekijänoikeusrikkomukset.

  • Sisältö ei liity palstan tai keskusteluketjun aiheeseen.

  • Viesti ei ole suomea tai ruotsia (pl. International Forums), se sisältää mainontaa, se sisältää tekijänoikeuksin suojattua materiaalia, se on massapostitus tai jokin muu syy.

Jaa keskustelu