Updated docs and logging for the picking api.
[vamsas.git] / src / org / vamsas / client / picking / PickManager.java
index e4f4445..c313e25 100644 (file)
@@ -2,9 +2,18 @@ package org.vamsas.client.picking;
 \r
 import java.net.*;\r
 import java.util.*;\r
+import java.util.logging.*;\r
 \r
+/**\r
+ * Manager class that maintains a list of connected clients in addition to\r
+ * attempting to run a server to listen for client connections. If the server\r
+ * initialization fails, then an attempt to connect (as a client) to another JVM\r
+ * that is already a server happens instead.\r
+ */\r
 public class PickManager\r
 {\r
+       private static Logger logger = Logger.getLogger("org.vamsas.client.picking");\r
+       \r
        // Maintains a list of client communication objects - each object represents\r
        // a way of talking to either:\r
        //  the server - if this is client side (and in which case, the list will only contain one element\r
@@ -13,6 +22,11 @@ public class PickManager
        \r
        private PickServer server;\r
        \r
+       /**\r
+        * Constructs a new PickManager. This method will return immediately, while\r
+        * a looping thread runs that attempts to run the server or connect to an\r
+        * existing server.\r
+        */\r
        public PickManager()\r
        {\r
                server = new PickServer(this);\r
@@ -21,29 +35,45 @@ public class PickManager
                new InitializeThread().start();\r
        }\r
        \r
-       // Called whenver we try to make a new endpoint<->endpoint connection\r
-       boolean addClientConnection(Socket socket)\r
+       /**\r
+        * Attempts to establish a connection between two client endpoints. This\r
+        * method is called in two ways: 1) by the server when it receives a remote\r
+        * request (in which case the socket will already be established) and 2) by\r
+        * a client that is attempting to connect *to* the server.\r
+        * @param socket a socket endpoint for the connection\r
+        * @return true if the connection is successfully, false otherwise\r
+        */\r
+       boolean addEndPoint(Socket socket)\r
        {\r
                PickEndPoint client = new PickEndPoint(this, socket);\r
                \r
                if (client.openConnection())\r
                {\r
                        clients.add(client);\r
-                       System.out.println("List now contains " + clients.size() + " client(s)");\r
+                       logger.fine("List now contains " + clients.size() + " client(s)");\r
                        return true;\r
                }\r
                \r
                return false;\r
        }\r
        \r
+       /**\r
+        * Sends a message to other clients.\r
+        * @param str the message to send\r
+        */\r
        public void sendMessage(String str)\r
        {\r
                forwardMessage(null, str);\r
        }\r
        \r
-       // Forwards (or sends) a message\r
-       // When the server receives a message, it will be resent to all clients,\r
-       // but *not* to the client that sent it out in the first place!\r
+       /**\r
+        * Forwards (or sends) a message. When the server (client A) receives a\r
+        * message from client B, it must also forward it to clients C and D (etc),\r
+        * but mustn't forward it *back* to client B.\r
+        * @param origin the client endpoint that received the message (will be null\r
+        * if the message originates from this instance\r
+        * @param str the message to send\r
+        */\r
        private void forwardMessage(PickEndPoint origin, String str)\r
        {\r
                ListIterator itor = clients.listIterator();\r
@@ -56,10 +86,12 @@ public class PickManager
                }\r
        }\r
        \r
-       // Called by the endpoint code when a message is received\r
-       // The manager has to:\r
-       //  a) hand message to VAMSAS app\r
-       //  b) forward it to other clients if we're the server\r
+       /**\r
+        * Handles a received message. If the manager is running in server mode,\r
+        * then it must ensure the message is also forwarded to the other clients.\r
+        * @param origin the client endpoint that received the message\r
+        * @str the message that was received\r
+        */\r
        void handleMessage(PickEndPoint origin, String str)\r
        {\r
                if (server.isServer())\r
@@ -68,10 +100,15 @@ public class PickManager
                // TODO: pass message to VAMSAS API\r
        }\r
        \r
+       /**\r
+        * Removes a client connection from the list when its connection is no\r
+        * longer valid.\r
+        * @param client the client endpoint to remove\r
+        */\r
        void removeEndPoint(PickEndPoint client)\r
        {\r
                clients.remove(client);\r
-               System.out.println("List now contains " + clients.size() + " client(s)");\r
+               logger.fine("List now contains " + clients.size() + " client(s)");\r
                \r
                // If there's no endpoints left, then we've lost all connections and\r
                // need to reinitialize\r
@@ -79,11 +116,14 @@ public class PickManager
                        new InitializeThread().start();\r
        }\r
        \r
+       /**\r
+        * Thread extension class to handle the actual initialization\r
+        */\r
        private class InitializeThread extends Thread\r
        {\r
                public void run()\r
                {\r
-                       System.out.println("Initializing connection...");\r
+                       logger.fine("Initializing connection...");\r
                        boolean connected = false;\r
                        \r
                        // Loop until we can get a connection (one way or the other)\r
@@ -99,7 +139,7 @@ public class PickManager
                                        connected = true;\r
 \r
                                // If it fails, then attempt to make a client connection...\r
-                               else if (addClientConnection(null))\r
+                               else if (addEndPoint(null))\r
                                        connected = true;\r
                        }\r
                }\r