JMapViewer, MouseListener вызывается 2 раза

Question

При работе с JMapViewer было распознано странное поведение компонента. Я использую DefaultMapController, чтобы получить положение карты (широта, долгота).

import java.awt.Graphics;
import java.awt.Point;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import javax.swing.JFrame;
import org.openstreetmap.gui.jmapviewer.DefaultMapController;
import org.openstreetmap.gui.jmapviewer.JMapViewer;

public class Test extends JMapViewer{

public Test() 
{
    addMouseListener(new DefaultMapController(this) {
            public void mouseClicked(MouseEvent e){
            Point  start = e.getPoint();
            System.out.println(e.getPoint());
            }            
       });
  }

protected void paintComponent(Graphics g){super.paintComponent(g);}  

public static void main (String [] args){
            JFrame jf = new JFrame();
            jf.setSize(800, 600);
            Test t= new Test();
            jf.add(t);
            jf.setVisible(true);
    }
}

После запуска кода после нажатия левой кнопки мыши метод mouseClicked() вызывается несколько раз (2x). После замены

    addMouseListener(new DefaultMapController(this) {

с

    addMouseListener(new MouseAdapter() {

код работает правильно, метод вызывается только 1x. В чем проблема? Это ошибка внутри библиотеки или синтаксис неправильный или небезопасный? Как избежать этой проблемы? Спасибо за вашу помощь.

Answer 1

Ваш Test продолжается JMapViewer, добавив MouseListener в блоке инициализатора экземпляра. Как следствие, "конструктор по умолчанию вызовет конструктор суперкласса без аргументов". Суперкласс, JMapController, добавляет ваш MouseListener- как вы уже догадались - во второй раз.

public JMapController(JMapViewer map) {
    this.map = map;
    if (this instanceof MouseListener)
        map.addMouseListener((MouseListener) this);
    …
}

Вместо этого создайте новый JMapController или же DefaultMapController, как показано здесь, и используйте его для построения JMapViewer,

import java.awt.EventQueue;
import java.awt.event.MouseEvent;
import javax.swing.JFrame;
import org.openstreetmap.gui.jmapviewer.DefaultMapController;
import org.openstreetmap.gui.jmapviewer.JMapViewer;

/**
 * @see https://stackru.com/a/39461854/230513
 */
public class TestMapController {

    private void display() {
        JFrame f = new JFrame("TestMapController");
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        JMapViewer map = new JMapViewer();
        new DefaultMapController(map) {
            @Override
            public void mouseClicked(MouseEvent e) {
                System.out.println(e.getPoint());
            }
        };
        f.add(map);
        f.pack();
        f.setLocationRelativeTo(null);
        f.setVisible(true);
    }

    public static void main(String[] args) {
        EventQueue.invokeLater(new TestMapController()::display);
    }
}