From b56a70ff34391372506ad1c083074d584828c268 Mon Sep 17 00:00:00 2001
From: Michael Bayne <mdb@threerings.net>
Date: Mon, 23 Jul 2001 21:13:56 +0000
Subject: [PATCH] Allow configuration to use derived class for Client and
 ClientObject.

git-svn-id: svn+ssh://src.earth.threerings.net/narya/trunk@108 542714f4-19e9-0310-aa3c-eee0fc999fb1
---
 .../presents/server/ClientManager.java        | 71 ++++++++++++++++++-
 .../presents/server/PresentsClient.java       | 11 +--
 2 files changed, 74 insertions(+), 8 deletions(-)

diff --git a/src/java/com/threerings/presents/server/ClientManager.java b/src/java/com/threerings/presents/server/ClientManager.java
index 2ddfb0daa..438390f76 100644
--- a/src/java/com/threerings/presents/server/ClientManager.java
+++ b/src/java/com/threerings/presents/server/ClientManager.java
@@ -1,5 +1,5 @@
 //
-// $Id: ClientManager.java,v 1.5 2001/06/09 23:39:04 mdb Exp $
+// $Id: ClientManager.java,v 1.6 2001/07/23 21:13:56 mdb Exp $
 
 package com.threerings.cocktail.cher.server;
 
@@ -7,6 +7,7 @@ import java.io.IOException;
 import java.util.HashMap;
 
 import com.threerings.cocktail.cher.Log;
+import com.threerings.cocktail.cher.data.ClientObject;
 import com.threerings.cocktail.cher.net.Credentials;
 import com.threerings.cocktail.cher.server.net.*;
 
@@ -29,6 +30,59 @@ public class ClientManager implements ConnectionObserver
         conmgr.addConnectionObserver(this);
     }
 
+    /**
+     * Instructs the client manager to construct instances of this derived
+     * class of <code>Client</code> to managed newly accepted client
+     * connections.
+     *
+     * @see Client
+     */
+    public void setClientClass (Class clientClass)
+    {
+        // sanity check
+        if (!Client.class.isAssignableFrom(clientClass)) {
+            Log.warning("Requested to use client class that does not " +
+                        "derive from Client " +
+                        "[class=" + clientClass.getName() + "].");
+            return;
+        }
+
+        // make a note of it
+        _clientClass = clientClass;
+    }
+
+    /**
+     * Instructs the client to create an instance of this
+     * <code>ClientObject</code> derived class when creating the
+     * distributed object that corresponds to a particular client session.
+     *
+     * @see com.threerings.cocktail.cher.data.ClientObject
+     */
+    public void setClientObjectClass (Class clobjClass)
+    {
+        // sanity check
+        if (!ClientObject.class.isAssignableFrom(clobjClass)) {
+            Log.warning("Requested to use client object class that does " +
+                        "not derive from ClientObject " +
+                        "[class=" + clobjClass.getName() + "].");
+            return;
+        }
+
+        // make a note of it
+        _clobjClass = clobjClass;
+    }
+
+    /**
+     * Returns the class that should be used when creating a distributed
+     * object to accompany a particular client session. In general, this
+     * is only used by the <code>Client</code> object when it is setting
+     * up a client's session for the first time.
+     */
+    public Class getClientObjectClass ()
+    {
+        return _clobjClass;
+    }
+
     /**
      * Called when a new connection is established with the connection
      * manager. Only fully authenticated connections will be passed on to
@@ -53,8 +107,16 @@ public class ClientManager implements ConnectionObserver
             Log.info("Session initiated [username=" + username +
                      ", conn=" + conn + "].");
             // create a new client and stick'em in the table
-            client = new Client(this, username, conn);
-            _usermap.put(username, conn);
+            try {
+                client = (Client)_clientClass.newInstance();
+                client.init(this, username, conn);
+                _usermap.put(username, conn);
+            } catch (Exception e) {
+                Log.warning("Failed to instantiate client instance to " +
+                            "manage new client connection " +
+                            "[conn=" + conn + "].");
+                Log.logStackTrace(e);
+            }
         }
 
         // map this connection to this client
@@ -129,4 +191,7 @@ public class ClientManager implements ConnectionObserver
 
     protected HashMap _usermap = new HashMap();
     protected HashMap _conmap = new HashMap();
+
+    protected Class _clientClass = Client.class;
+    protected Class _clobjClass = ClientObject.class;
 }
diff --git a/src/java/com/threerings/presents/server/PresentsClient.java b/src/java/com/threerings/presents/server/PresentsClient.java
index 2a25dd9d4..cc6eeee20 100644
--- a/src/java/com/threerings/presents/server/PresentsClient.java
+++ b/src/java/com/threerings/presents/server/PresentsClient.java
@@ -1,5 +1,5 @@
 //
-// $Id: PresentsClient.java,v 1.9 2001/07/19 19:30:14 mdb Exp $
+// $Id: PresentsClient.java,v 1.10 2001/07/23 21:13:56 mdb Exp $
 
 package com.threerings.cocktail.cher.server;
 
@@ -29,10 +29,10 @@ import com.threerings.cocktail.cher.server.net.*;
 public class Client implements Subscriber, MessageHandler
 {
     /**
-     * Constructs a new client instance bound to the specified username
-     * and initially associated with the specified connection instance.
+     * Initializes this client instance with the specified username and
+     * connection instance.
      */
-    public Client (ClientManager cmgr, String username, Connection conn)
+    public void init (ClientManager cmgr, String username, Connection conn)
     {
         _cmgr = cmgr;
         _username = username;
@@ -60,7 +60,8 @@ public class Client implements Subscriber, MessageHandler
                 return false;
             }
         };
-        CherServer.omgr.createObject(ClientObject.class, sub, false);
+        Class clobjClass = _cmgr.getClientObjectClass();
+        CherServer.omgr.createObject(clobjClass, sub, false);
     }
 
     /**