No implementing listener interfaces in the controller. Bad bad bad.

Unfortunately PlaceController already implements AttributeChangeListener
and various things take advantage of that. At some point I'll go in and
clean that all up, but for now I'll limit my fiddling to PuzzleController
on down.


git-svn-id: svn+ssh://src.earth.threerings.net/narya/trunk@2893 542714f4-19e9-0310-aa3c-eee0fc999fb1

src/java/com/threerings/puzzle/client/PuzzleController.java

@@ -1,5 +1,5 @@
 //
-// $Id: PuzzleController.java,v 1.1 2003/11/26 01:42:34 mdb Exp $
+// $Id: PuzzleController.java,v 1.2 2003/12/05 02:00:48 mdb Exp $
 
 package com.threerings.puzzle.client;
 
@@ -56,7 +56,7 @@ import com.threerings.puzzle.util.PuzzleContext;
  * The puzzle game controller handles logical actions for a puzzle game.
  */
 public abstract class PuzzleController extends GameController
-    implements PuzzleCodes, ElementUpdateListener
+    implements PuzzleCodes
 {
     /** The action command to toggle chatting mode. */
     public static final String TOGGLE_CHATTING = "toggle_chat";
@@ -259,6 +259,8 @@ public abstract class PuzzleController extends GameController
 
         // get a casted reference to our puzzle object
         _puzobj = (PuzzleObject)plobj;
+        _puzobj.addListener(_kolist);
+        _puzobj.addListener(_mlist);
 
         // listen to key events..
         _pctx.getKeyDispatcher().addGlobalKeyListener(_globalKeyListener);
@@ -302,7 +304,11 @@ public abstract class PuzzleController extends GameController
         _pctx.getKeyDispatcher().removeGlobalKeyListener(_globalKeyListener);
 
         // clear out the puzzle object
-        _puzobj = null;
+        if (_puzobj != null) {
+            _puzobj.removeListener(_mlist);
+            _puzobj.removeListener(_kolist);
+            _puzobj = null;
+        }
     }
 
     /**
@@ -882,17 +888,6 @@ public abstract class PuzzleController extends GameController
         });
     }
 
-    // documentation inherited
-    public void elementUpdated (ElementUpdatedEvent event)
-    {
-        String name = event.getName();
-        if (name.equals(PuzzleObject.PLAYER_STATUS)) {
-            if (event.getIntValue() == PuzzleObject.PLAYER_KNOCKED_OUT) {
-                playerKnockedOut(event.getIndex());
-            }
-        }
-    }
-
     /**
      * Called when a player is knocked out of the game to give the puzzle
      * a chance to perform any post-knockout actions that may be desired.
@@ -942,6 +937,31 @@ public abstract class PuzzleController extends GameController
         protected long _lastProgressTick;
     };
 
+    /** Listens for players being knocked out. */
+    protected ElementUpdateListener _kolist = new ElementUpdateListener() {
+        public void elementUpdated (ElementUpdatedEvent event) {
+            String name = event.getName();
+            if (name.equals(PuzzleObject.PLAYER_STATUS)) {
+                if (event.getIntValue() == PuzzleObject.PLAYER_KNOCKED_OUT) {
+                    playerKnockedOut(event.getIndex());
+                }
+            }
+        }
+    };
+
+    /** Listens for various attribute changes. */
+    protected AttributeChangeListener _mlist = new AttributeChangeListener() {
+        public void attributeChanged (AttributeChangedEvent event) {
+            String name = event.getName();
+            if (name.equals(PuzzleObject.DIFFICULTY)) {
+                difficultyChanged(_puzobj.difficulty);
+
+            } else if (name.equals(PuzzleObject.SEED)) {
+                generateNewBoard();
+            }
+        }
+    };
+
     /** A casted reference to the client context. */
     protected PuzzleContext _pctx;