Changeset 96 for java/main/src/main/java/com/framsticks/hosting/InstanceClient.java

Timestamp:

07/04/13 20:29:50 (11 years ago)

Author:

psniegowski

Message:

HIGHLIGHTS:

• cleanup Instance management
  • extract Instance interface
  • extract Instance common algorithms to InstanceUtils?
• fix closing issues: Ctrl+C or window close button

properly shutdown whole program

• add tests regarding connecting to ObjectInstance? hosted

by Java Framsticks framework

• fix parsing and printing of all request types
• hide exception passing in special handle method of closures
  • substantially improve readability of closures
  • basically enable use of exception in asynchronous closures

(thrown exception is transported back to the caller)

• implement call request on both sides

CHANGELOG:
Further improve calling.

Improve instance calling.

Calling is working on both sides.

Improve exception handling in testing.

Waiters do not supercede other apllication exception being thrown.

Finished parsing and printing of all request types (with tests).

Move implementation and tests of request parsing to Request.

Add tests for Requests.

Improve waits in asynchronours tests.

Extract more algorithms to InstanceUtils?.

Extract Instance.resolve to InstanceUtils?.

Improve naming.

Improve passing exception in InstanceClient?.

Hide calling of passed functor in StateCallback?.

Hide Exception passing in asynchronous closures.

Hide exception passing in Future.

Make ResponseCallback? an abstract class.

Make Future an abstract class.

Minor change.

Move getPath to Path.to()

Move bindAccess to InstanceUtils?.

Extract common things to InstanceUtils?.

Fix synchronization bug in Connection.

Move resolve to InstanceUtils?.

Allow names of Joinable to be dynamic.

Add support for set request server side.

More fixes in communication.

Fix issues with parsing in connection.

Cut new line characters when reading.

More improvements.

Migrate closures to FramsticksException?.

Several changes.

Extract resolveAndFetch to InstanceUtils? algorithms.

Test resolving and fetching.

More fixes with function signature deduction.

Do not print default values in SimpleAbstractAccess?.

Add test of FramsClass? printing.

Improve FramsticksException? messages.

Add explicit dispatcher synchronization feature.

Rework assertions in tests.

Previous solution was not generic enough.

Allow addition of joinables to collection after start.

Extract SimulatorInstance? from RemoteInstance?.

Remove PrivateJoinableCollection?.

Improve connections.

Move shutdown hook to inside the Monitor.

It should work in TestNG tests, but it seems that
hooks are not called.

In ServerTest? client connects to testing server.

Move socket initialization to receiver thread.

Add proper closing on Ctrl+C (don't use signals).

Fix bugs with server accepting connections.

Merge Entity into Joinable.

Reworking ServerInstance?.

Extract more algorithm to InstanceUtils?.

Extract some common functionality from AbstractInstance?.

Functions were placed in InstanceUtils?.

Hide registry of Instance.

Use ValueParam? in Instance interface.

Minor change.

Extract Instance interface.

Old Instance is now AbstractInstance?.

File:

• java/main/src/main/java/com/framsticks/hosting/InstanceClient.java (modified) (5 diffs)

java/main/src/main/java/com/framsticks/hosting/InstanceClient.java

@@ -1 +1 @@
 package com.framsticks.hosting;
+
+import static com.framsticks.util.lang.Strings.assureNotEmpty;
 
 import com.framsticks.communication.*;
 import com.framsticks.communication.queries.ApplicationRequest;
+import com.framsticks.communication.queries.CallRequest;
 import com.framsticks.communication.queries.GetRequest;
 import com.framsticks.communication.queries.InfoRequest;
+import com.framsticks.communication.queries.SetRequest;
 import com.framsticks.core.Instance;
+import com.framsticks.core.InstanceUtils;
 import com.framsticks.core.Path;
 import com.framsticks.params.*;
+import com.framsticks.params.types.ProcedureParam;
 import com.framsticks.parsers.Savers;
-import com.framsticks.core.LocalInstance;
+import com.framsticks.util.FramsticksException;
 import com.framsticks.util.dispatching.AbstractJoinable;
 import com.framsticks.util.dispatching.Dispatching;
@@ -16 +22 @@
 import com.framsticks.util.dispatching.JoinableParent;
 import com.framsticks.util.dispatching.JoinableState;
+import static com.framsticks.core.InstanceUtils.*;
 
 import java.net.Socket;
-import java.util.ArrayList;
 import java.util.LinkedList;
 import java.util.List;
-import com.framsticks.util.dispatching.RunAt;
 
 /**
@@ -28 +33 @@
 public class InstanceClient extends AbstractJoinable implements RequestHandler, JoinableParent {
 
-        protected final LocalInstance instance;
+        protected final Server server;
+        protected final Instance instance;
         protected final ServerConnection connection;
 
-        public InstanceClient(LocalInstance instance, Socket socket) {
-                this.instance = instance;
-                connection = new ServerConnection(socket, this);
+        public InstanceClient(Server server, Socket socket) {
+                this.server = server;
+                this.instance = server.hosted;
+                this.connection = new ServerConnection(socket, this);
         }
 
         @Override
-        public String toString() {
-                return instance + "|" + connection.toString();
+        public String getName() {
+                return connection + " to " + server;
         }
 
         @Override
         public void handle(final ApplicationRequest request, final ResponseCallback<?> responseCallback) {
-                instance.dispatch(new RunAt<Instance>() {
+                assureNotEmpty(request.getPath());
+
+                resolve(instance, request.getPath(), new Future<Path>(responseCallback) {
                         @Override
-                        public void run() {
-                                final Path path = instance.getPath(request.getPath());
-                                if (!path.isResolved(request.getPath())) {
-                                        responseCallback.process(new Response(false, "\"invalid path\"", null));
+                        protected void result(final Path path) {
+
+                                // final AccessInterface access = instance.prepareAccess(path);
+                                final AccessInterface access = instance.prepareAccess(path.getTop().getParam());
+
+                                if (request instanceof SetRequest) {
+                                        SetRequest set = (SetRequest) request;
+                                        //TODO Proxy - here is break of chain, instance should have hosted
+                                        //hosted set
+                                        AccessInterface access2 = InstanceUtils.bindAccess(path);
+                                        int flag = access2.set(set.getField(), set.getValue());
+
+                                        responseCallback.process(new Response(true, Flags.write(flag, null), null));
                                         return;
                                 }
+
                                 if (request instanceof GetRequest) {
-                                        instance.findInfo(path, new Future<FramsClass>() {
+                                        InstanceUtils.findInfo(path, new Future<FramsClass>(responseCallback) {
                                                 @Override
-                                                public void result(FramsClass result, Exception e) {
+                                                protected void result(FramsClass result) {
                                                         if (result == null) {
-                                                                responseCallback.process(new Response(false, "\"failed to find info for access bind\"", null));
-                                                                return;
+                                                                throw new FramsticksException().msg("failed to find info for access bind");
                                                         }
                                                         List<File> files = new LinkedList<File>();
-                                                        AccessInterface access = instance.bindAccess(path);
+                                                        AccessInterface access = bindAccess(path);
 
                                                         if (access == null) {
+                                                                throw new FramsticksException().msg("failed to bind access");
+                                                        }
 
-                                                                responseCallback.process(new Response(false, "\"failed to bind access\"", null));
-                                                                return;
-                                                        }
                                                         ListSink sink = new ListSink();
                                                         access.save(sink);
@@ -75 +92 @@
                                         return;
                                 }
-                                if (request instanceof InfoRequest) {
-                                        instance.findInfo(path, new Future<FramsClass>() {
+                                if (request instanceof CallRequest) {
+                                        final CallRequest callRequest = (CallRequest) request;
+                                        instance.call(path, access.getFramsClass().getParamEntry(callRequest.getProcedure(), ProcedureParam.class), callRequest.getArguments().toArray(), new Future<Object>(responseCallback) {
                                                 @Override
-                                                public void result(FramsClass result, Exception e) {
-                                                        if (result == null) {
-                                                                responseCallback.process(new Response(false, "\"info not found\"", null));
-                                                                return;
+                                                protected void result(Object result) {
+                                                        ListSink sink = new ListSink();
+                                                        sink.print("Result:").breakLine();
+                                                        sink.print("value:").print("[");
+                                                        if (result != null) {
+                                                                sink.print(result);
                                                         }
-                                                        ListSink sink = new ListSink();
-                                                        Savers.saveFramsClass(sink, result);
-                                                        List<File> files = new ArrayList<File>();
-                                                        files.add(new File(path.getTextual(), new ListSource(sink.getOut())));
-                                                        responseCallback.process(new Response(true, null, files));
+                                                        sink.print("]");
+
+                                                        responseCallback.process(new Response(true, "", File.single(new File("", new ListSource(sink.getOut())))));
                                                 }
                                         });
                                         return;
                                 }
-                                responseCallback.process(new Response(false, "invalid", null));
+                                if (request instanceof InfoRequest) {
+                                        findInfo(path, new Future<FramsClass>(responseCallback) {
+                                                @Override
+                                                protected void result(FramsClass result) {
+                                                        if (result == null) {
+                                                                throw new FramsticksException().msg("info not found");
+                                                        }
+                                                        responseCallback.process(new Response(true, null, File.single(new File(path.getTextual(), new ListSource(Savers.saveFramsClass(new ListSink(), result).getOut())))));
+                                                }
+                                        });
+                                        return;
+                                }
+
+                                throw new FramsticksException().msg("invalid request type: " + request.getCommand());
                         }
                 });
@@ -122 +153 @@
         }
 
+
 }