How To Add A MouseListener To A Frame
Asked Answered
B

1

10

I want to add a mouselistener to mt JFrame frame but when i do frame.addMouseListener(this) i get an error that i cannot use this in a static method

I am making an application that detects a click of the mouse then displays it in int clicks

code

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.FlowLayout;
import java.awt.Font;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import javax.swing.BorderFactory;
import javax.swing.JFrame;
import javax.swing.JTextField;

public class numberOfClicks implements MouseListener{

    static int clicks = 0;

    @Override
    public void mouseClicked(MouseEvent e) {
        clicks++;
    }

    static JTextField text = new JTextField();
    static String string = clicks+" Clicks";

    static JFrame frame = new JFrame("Click Counter");
    public static void frame(){
        Font f = new Font("Engravers MT", Font.BOLD, 23);
        text.setEditable(false);
        text.setBackground(Color.BLUE);
        text.setFont(f);
        text.setForeground(Color.GREEN);
        text.setBorder(BorderFactory.createLineBorder(Color.BLUE));
        text.setText(string);

        frame.add(text, BorderLayout.SOUTH);
        frame.setResizable(false);
        frame.setSize(300, 300);
        frame.getContentPane().setBackground(Color.BLUE);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setVisible(true);
        frame.addMouseListener(this);
    }

    public static void main(String[] args){
        frame();
    }

    @Override
    public void mousePressed(MouseEvent e) {}
    @Override
    public void mouseEntered(MouseEvent e) {}
    @Override
    public void mouseExited(MouseEvent e) {}
    @Override
    public void mouseReleased(MouseEvent e) {}
}
Bodnar answered 24/6, 2013 at 2:4 Comment(2)
You have been given links to the Swing tutorial in the past. Look at the table of contents. There is a section on How Write a Mouse Listener. Look at the example code and how they structure the class. You should not be using static variaibles. Do some basic reading first BEFORE posting questions. That is why I have referred you to the tutorial in the past. The tutorial contains the basics to get you started.Microcosm
1) A single blank line of white space in source code is always enough. 2) It will probably work better to add the listener to a panel, then add the panel to the frame. 3) Don't set the size of top level containers. Instead layout the content & call pack(). 4) Swing GUIs should be created & updated on the EDT.Constrict
A
9

this doesn't exist in a static method since a static method is a method of the class, not of the object (the owner of this). Solution: get rid of all statics from your code above. None of your fields or methods above should be static other than the main method.


Edit
And as Andrew Thompson correctly states, add the MouseListener to a JPanel that is added to the JFrame's contentPane.


Edit 2

  • You will want to learn and use Java naming conventions. Class names (i.e., NumberOfClicks) should start with an upper case letter. Method and variable names with a lower-case letter.
  • You are better off using the mousePressed(...) method rather than the mouseClicked(...) since the former is less persnickety about accepting presses.
  • You will also want to set your JTextField's text in your mousePressed(...) method since just changing the clicks value isn't enough to change the display.
  • I try to avoid having my GUI (or "view") classes implement my listeners. I prefer to use anonymous inner classes or stand-alone classes where possible.

e.g.,

  JPanel mainPanel = new JPanel();
  mainPanel.addMouseListener(new MouseAdapter() {
     @Override
     public void mousePressed(MouseEvent e) {
        clicks++;
        text.setText(clicks + " Clicks");
     }
  });
  // add mainPanel to the JFrame...
Argo answered 24/6, 2013 at 2:7 Comment(3)
This works, but when I used this my whole game turned white and I couldn't play it.Churr
@Churr then you've got some other problem in your codeArgo
Yes, I can see that now I just fixed it.Churr

© 2022 - 2024 — McMap. All rights reserved.