addMouseMotionListener

[Fireking]

As mentioned in my previous post i following a tutorial on 2d graphics and im stuck :(
the tutorial is java.sun.com/docs/books/tutorial/uiswing/painting/step3.html
Im adding in code about addMouseMotionListener and it is red lining (im using netbeans)

package movement;
import java.awt.*;
import javax.swing.*;
import java.awt.event.*;


public class mypanel extends JPanel{
    
    redSquare redSquare = new redSquare();
    
    public mypanel() {
        //set border size
        setBorder(BorderFactory.createLineBorder(Color.black));
        
        addMouseListener(
                new MouseAdapter(){
                    public void mousePressed(MouseEvent e){
                        moveSquare(e.getX(), e.getY());
                }
        });
        
        addMouseMotionListener(
                new MouseAdapter(){ //*redline*
                    public void mouseDragged(MouseEvent e){ //*redline*
                        moveSquare(e.getX(), e.getY()); //*redline*
                    }
        })
        
    }// constructor
    
    
public void paintComponent(Graphics g) {
        super.paintComponent(g);       

        // Draw Text
        g.drawString("This is my custom Panel!",10,20);
        
        //Draw Red Square
        redSquare.paintSquare(g);
        
    }  


    public Dimension getPreferredSize() {
        return new Dimension(500,400);
    }
    
    private void moveSquare(int x, int y){
        final int CURR_X = redSquare.getX();
        final int CURR_Y = redSquare.getY();
        final int CURR_W = redSquare.getWidth();
        final int CURR_H = redSquare.getHeight();
        final int OFFSET = 1;
        
        if ((CURR_X != x) || (CURR_Y != y)){ //checks if square has moved
            
            //repaints the background over the old square
            repaint(CURR_X, CURR_Y, CURR_W + OFFSET, CURR_H + OFFSET); 
            
            //update new coordiantes
            redSquare.setX(x);
            redSquare.setY(y);
            
            //repaint square at new location
            repaint(redSquare.getX(), redSquare.getY(), 
                    redSquare.getWidth() + OFFSET, 
                    redSquare.getHeight() + OFFSET);
            
            
        }
    }
    


}

the error it gives me is "addMouseMotionListener(java.awt.event.MouseMotion Listener) in java.awt.Component cannot be applied to (<anonymous java.awt.event.MouseAdapter>)"

any help with why it would be doing this would be appreciated.

Thx

Fireking

[Norm]

Only error I get is for a missing ; near end of constructor
Can you post the full text of the error message?

[Fireking]

sorry i did forget the ; when i rewrote it.
i added the ; back in and i still get the 3 red lines and when i run it it gives

"init:
deps-jar:
Compiling 2 source files to /Users/david/NetBeansProjects/movement/build/classes
/Users/david/NetBeansProjects/movement/src/movement/mypanel.java:29: addMouseMotionListener(java.awt.event.MouseMotionL istener) in java.awt.Component cannot be applied to (<anonymous java.awt.event.MouseAdapter>)
addMouseMotionListener(
1 error
BUILD FAILED (total time: 0 seconds)"

[Norm]

As I said, I didn't get that error when I compiled the program. I did get undefined symbols for missing classes.
How are you compiling the program? What version of the JDK do you have?
Have you tried compiling it from the command prompt?

[Supamagier]

He's probably using an older version of the JDK, try downloading a new one. :)

[hardwired]

        addMouseMotionListener(
                new MouseAdapter(){ //*redline*

You cannot use a MouseAdapter for/as a MouseMoionAdapter.
Change the above to:

        addMouseMotionListener(
                new MouseMotionAdapter(){

[Fubarable]

You cannot use a MouseAdapter for/as a MouseMotionAdapter.

Why not? The API tells me that I can as MouseAdapter implements the MouseMotionListener (which is what I think you meant) interface.

[Fubarable]

I do see an unrelated problem here:

 redSquare redSquare = new redSquare();

There's some naming confusion here where your object name matches the class name. I strongly recommend that you don't do this, that you make the class name start with an upper-case letter and the object remain lower-case.

[hardwired]

Why not? The API tells me that I can as MouseAdapter implements the MouseMotionListener
You're right. I hadn't noticed the addition of the MouseMotionListener and MouseWheelListener interfaces to MouseAdapter in j2se 1.6
which is what I think you meant
Yes, that is what I meant: I thought they were still mutually exclusive.
Please disregard post #6.

[Fireking]

Right ok i think i may definatly be out of date, because what hardwired suggested worked (thank you very much) :) so i will look at updating.

Thx Fubarable I will do this in the future :)

Thx for everybodys help, much appreciated :D

Fireking