Help with Event handling

Hi ... I'm hoping someone can explain what I've done wrong in the following:
The example code is from "Java Certification Exam Guide" by Barry Boone. The original code works fine, it creates an applet and adds two custom canvas components as seperate threads. It then updates the canvas items each time the user clicks on the mouse, displaying the number of mouse-clicks.
The code uses the JDK 1.0 event handling model. I thought I could re-write it to use the new model by simply adding a MouseListener; however, the canvas threads are not being updated when I click on the applet.

/** Synchronized example based on pg 266 of
    'Java Certification Exam Guide for Programmers and Developers'  
    by Barry Boone, McGraw-Hill, 1997
    Modified to remove deprecated methods
 */
 
 import java.awt.*;
 import java.awt.event.*;
 import java.applet.*;
 
 public class ClickApplet extends Applet {
    boolean clicked;
    int counter;
    
    public void init() {
        // add a mouse handler to the applet
        addMouseListener(new MyMouseHandler());
        add(new ClickCanvas(this));
        add(new ClickCanvas(this));
    }
    
    /* original mouse handler     
    public boolean mouseDown(Event e, int x, int y) {
        synchronized(this) {
            clicked = true;
            notifyAll();
        }
        
        counter++;
        
        Thread.currentThread().yield();
        clicked = false;
        return super.mouseDown(e, x, y);
    }
    */
    
    /* Add inner class to create a mouse handler */
    class MyMouseHandler extends MouseAdapter {
        public void mouseClicked(MouseEvent e) {
            synchronized(this) {
                clicked = true;
                notifyAll();
            }
            
            counter++;
            System.out.println("Count: " + counter);
            Thread.currentThread().yield();
            clicked = false;
        }
    }    
        
} // end of ClickApplet
class ClickCanvas extends Canvas implements Runnable {
    ClickApplet applet;
    
    ClickCanvas(ClickApplet applet) {
        this.applet = applet;
        setSize(30,30);                     // original resize() deprecated
        new Thread(this).start();
    } 
    
    // override run() in Runnable
    public void run() {
        while(true) {
            synchronized(applet) {
                while( !applet.clicked) {
                    try {
                        applet.wait();
                    } catch(InterruptedException e) {
                    }
                } // end of while
            } // end of synchronized block
            repaint();
            
        } // end of while
    } // end of run()
    
    public void paint(Graphics g) {
        g.drawString(new Integer(applet.counter).toString(), 10,20);
    }
} // end of ClickCanvas

Thanks in advance for any help.
------------------
Jane
The cure for boredom is curiosity.
There is no cure for curiousity.
-- Dorothy Parker

Jane,
I think the problem stems from the use of the inner class to employ the MouseAdapter. If you try this, it works fine.
public class MyApplet extends Applet implements MouseListener{
boolean clicked;
int counter;

public void init() {
// add the applet as the MouseListener
addMouseListener(this);
add(new ClickCanvas( this ));
add(new ClickCanvas( this ));
}
public void mouseClicked(MouseEvent e) {
synchronized(this) {
clicked = true;
notifyAll();
}
counter++;
Thread.currentThread().yield();
clicked = false;
System.out.println("Count: " + counter);
}
public void mouseEntered(MouseEvent e) {}
public void mouseExited(MouseEvent e) {}
public void mousePressed(MouseEvent e) {
public void mouseReleased(MouseEvent e) {}
etc...
Notice, the applet itself becomes the MouseListener that you pass to the canvases. Hope this helps.
Sean

Thanks Sean. Still not sure why the inner class doesn't work. Any ideas?
------------------
Jane
The cure for boredom is curiosity.
There is no cure for curiosity.
-- Dorothy Parker

Java 1.0 doesn't suppot inner class. It that the reason?

That's not it for me. I was using 1.2. Good call though.
Sean

Hi Alan, Sean ...
I'm using JDK 1.3 so don't think the inner class 'per se' is the problem.
I think, possibly, it's the way I'm registering the MouseListener. Most of the example code I've been able to find with seperate handlers defined, are assigning them to a component ie they create a button b then use b.addMouseListener(new MyHandler()). Other code, which doesn't use a seperate handler class, will add the listener to the applet itself with <code>addMouseListener(this)</code>.
I think using the code <code>addMouseListener(new MyMouseHandler())</code> doesn't properly register the listener with the applet and I can't figure out how to do this directly.
If you happen to trip over any similar examples, I'd appreciate a link back.
Thanks
------------------
Jane
The cure for boredom is curiosity.
There is no cure for curiosity.
-- Dorothy Parker