Changeset 101 for java/main/src/main/java/com/framsticks/core/AbstractTree.java

Timestamp:

07/14/13 23:20:04 (11 years ago)

Author:

psniegowski

Message:

HIGHLIGHTS:

• improve tree side notes
• improve GUI layout
• add foldable list of occured events to EventControl?
• improve automatic type conversion in proxy listeners
• implement several Access functionalities as algorithms independent of Access type
• introduce draft base classes for distributed experiments
• automatically register dependant Java classes to FramsClass? registry
• add testing prime experiment and configuration
• simplify and improve task dispatching

CHANGELOG:
Improve task dispatching in RemoteTree?.

GUI no longer hangs on connection problems.

Make all dispatchers joinables.

Refactorize Thread dispatcher.

Remove Task and PeriodicTask?.

Use Java utilities in those situations.

Reworking tasks dispatching.

Fix bug in EventControl? listener dispatching.

Minor improvements.

Add testing configuration for ExternalProcess? in GUI.

More improvement to prime.

Support for USERREADONLY in GUI.

Add that flag to various params in Java classes.

Remove redundant register clauses from several FramsClassAnnotations?.

Automatically gather and register dependant classes.

Add configuration for prime.

Improve Simulator class.

Add prime.xml configuration.

Introduce draft Experiment and Simulator classes.

Add prime experiment tests.

Enclose typical map with listeners into SimpleUniqueList?.

Needfile works in GUI.

Improve needfile handling in Browser.

More improvement with NeedFile?.

Implementing needfile.

Update test.

Rename ChangeEvent? to TestChangeEvent?.

Automatic argument type search in RemoteTree? listeners.

MultiParamLoader? uses AccessProvider?. By default old implementation
enclosed in AccessStash? or Registry.

Minor changes.

Rename SourceInterface? to Source.

Also improve toString of File and ListSource?.

Remove unused SimpleSource? class.

Add clearing in HistoryControl?.

Show entries in table at EventControl?.

Improve EventControl?.

Add listeners registration to EventControl?.

Add foldable table to HistoryControl?.

Add control row to Procedure and Event controls.

Improve layout of controls.

Another minor change to gui layout.

Minor improvement in the SliderControl?.

Minor changes.

Move ReflectionAccess?.Backend to separate file.

It was to cluttered.

Cleanup in ReflectionAccess?.

Move setMin, setMax, setDef to AccessOperations?.

Extract loading operation into AccessOperations?.

Append Framsticks to name of UnsupportedOperationException?.

The java.lang.UnsupportedOperationException? was shadowing this class.

Rename params.Util to params.ParamsUtil?.

Several improvements.

• Merge ValueControlListener? into ControlOwner?.
• Remove several deprecated utility classes.
• Add EventListeners? utility container.

Minor changes.

Implement revert functionality.

Improve local changes management.

Minor improvement.

Remove methods rendered superfluous after SideNoteKey? improvement.

Improve SideNoteKey?.

It is now generic type, so explicit type specification at
call site is no more needed.

Introduce SideNoteKey? interface.

Only Objects implementing that key may be used as side note keys.

Minor improvements.

Use strings instead of ValueControls? in several gui mappings.

File:

• java/main/src/main/java/com/framsticks/core/AbstractTree.java (modified) (17 diffs)

java/main/src/main/java/com/framsticks/core/AbstractTree.java

@@ -1 +1 @@
 package com.framsticks.core;
 
+import java.util.Comparator;
+import java.util.Iterator;
 import java.util.Map;
+import java.util.PriorityQueue;
 
 import javax.annotation.Nonnull;
@@ -9 +12 @@
 import org.apache.logging.log4j.LogManager;
 
+import com.framsticks.communication.queries.NeedFile;
+import com.framsticks.communication.queries.NeedFileAcceptor;
 import com.framsticks.params.Access;
 import com.framsticks.params.CompositeParam;
 import com.framsticks.params.FramsClass;
+import com.framsticks.params.ParamFlags;
 import com.framsticks.params.ParamsPackage;
 import com.framsticks.params.Registry;
@@ -20 +26 @@
 import com.framsticks.util.Misc;
 import com.framsticks.util.dispatching.AbstractJoinable;
+import com.framsticks.util.dispatching.BufferedDispatcher;
 import com.framsticks.util.dispatching.Dispatcher;
 import com.framsticks.util.dispatching.Dispatching;
 import com.framsticks.util.dispatching.ExceptionResultHandler;
 import com.framsticks.util.dispatching.Joinable;
-import com.framsticks.util.dispatching.JoinableDispatcher;
 import com.framsticks.util.dispatching.JoinableParent;
 import com.framsticks.util.dispatching.JoinableState;
@@ -30 +36 @@
 import com.framsticks.util.dispatching.Thread;
 import com.framsticks.util.dispatching.ThrowExceptionHandler;
-import com.framsticks.util.lang.Casting;
+import com.framsticks.util.lang.Pair;
 
 /**
@@ -36 +42 @@
  */
 @FramsClassAnnotation
-public abstract class AbstractTree extends AbstractJoinable implements Dispatcher<Tree>, Tree, JoinableParent {
+public abstract class AbstractTree extends AbstractJoinable implements Dispatcher<Tree>, Tree, JoinableParent, NeedFileAcceptor {
 
         private static final Logger log = LogManager.getLogger(AbstractTree.class);
@@ -43 +49 @@
         private ExceptionResultHandler handler = ThrowExceptionHandler.getInstance();
 
-        private JoinableDispatcher<Tree> dispatcher;
+        protected final BufferedDispatcher<Tree> bufferedDispatcher = new BufferedDispatcher<>();
+
+        protected final PriorityQueue<Pair<Integer, NeedFileAcceptor>> needFileAcceptors = new PriorityQueue<>(32, new Comparator<Pair<Integer, NeedFileAcceptor>>() {
+
+                @Override
+                public int compare(Pair<Integer, NeedFileAcceptor> arg0, Pair<Integer, NeedFileAcceptor> arg1) {
+                        if (arg0.first < arg1.first) {
+                                return -1;
+                        }
+                        if (arg0.first > arg1.first) {
+                                return 1;
+                        }
+                        return 0;
+                }
+        });
 
         @Override
@@ -152 +172 @@
          */
         @Override
-        public JoinableDispatcher<Tree> getDispatcher() {
-                return dispatcher;
+        public Dispatcher<Tree> getDispatcher() {
+                return bufferedDispatcher.getTargetDispatcher();
         }
 
@@ -160 +180 @@
          */
         @Override
-        public void setDispatcher(JoinableDispatcher<Tree> dispatcher) {
-                if (this.dispatcher != null) {
-                        throw new FramsticksException().msg("dispatcher is already set").arg("tree", this).arg("dispatcher", dispatcher);
-                }
-                this.dispatcher = dispatcher;
+        public void setDispatcher(Dispatcher<Tree> dispatcher) {
+                if (bufferedDispatcher.getTargetDispatcher() != null) {
+                        throw new FramsticksException().msg("dispatcher is already set").arg("tree", this).arg("dispatcher", bufferedDispatcher.getTargetDispatcher());
+                }
+                bufferedDispatcher.setTargetDispatcher(dispatcher);
         }
 
@@ -170 +190 @@
          * @return the name
          */
-        @ParamAnnotation
+        @ParamAnnotation(flags = ParamFlags.USERREADONLY)
         public String getName() {
                 return name;
@@ -193 +213 @@
         @Override
         protected void joinableStart() {
-                if (dispatcher == null) {
-                        dispatcher = new Thread<Tree>();
-                }
-                Dispatching.use(dispatcher, this);
+                if (bufferedDispatcher.getTargetDispatcher() == null) {
+                        bufferedDispatcher.setTargetDispatcher(new Thread<Tree>());
+                }
+                Dispatching.use(bufferedDispatcher.getTargetDispatcher(), this);
         }
 
         @Override
         protected void joinableInterrupt() {
-                Dispatching.drop(dispatcher, this);
+                Dispatching.drop(bufferedDispatcher.getTargetDispatcher(), this);
         }
 
@@ -211 +231 @@
         @Override
         protected void joinableJoin() throws InterruptedException {
-                Dispatching.join(dispatcher);
+                Dispatching.join(bufferedDispatcher.getTargetDispatcher());
         }
 
         @Override
         public void childChangedState(Joinable joinable, JoinableState state) {
-                if (joinable == dispatcher) {
+                if (joinable == bufferedDispatcher.getTargetDispatcher()) {
                         proceedToState(state);
                 }
@@ -223 +243 @@
         @Override
         public boolean isActive() {
-                if (dispatcher == null) {
-                        throw new FramsticksException().msg("no dispatcher is set for tree yet").arg("tree", this);
-                }
-                return dispatcher.isActive();
+                return bufferedDispatcher.isActive();
         }
 
         @Override
         public void dispatch(RunAt<? extends Tree> runnable) {
-                if (dispatcher == null) {
-                        throw new FramsticksException().msg("no dispatcher is set for tree yet").arg("tree", this);
-                }
-                dispatcher.dispatch(runnable);
+                bufferedDispatcher.dispatch(runnable);
         }
 
@@ -242 +256 @@
 
         @Override
-        public void putSideNote(Object object, Object key, Object value) {
+        public <T> void putSideNote(Object object, SideNoteKey<T> key, T value) {
                 assert isActive();
                 Misc.throwIfNull(object);
@@ -253 +267 @@
                 }
                 @SuppressWarnings("unchecked")
-                Map<Object, Object> sideNotesMap = (Map<Object, Object>) sideNote;
+                Map<SideNoteKey<?>, Object> sideNotesMap = (Map<SideNoteKey<?>, Object>) sideNote;
                 sideNotesMap.put(key, value);
         }
 
-        @Override
-        public <T> T getSideNote(Object object, Object key, Class<T> valueType) {
+        @SuppressWarnings("unchecked")
+        @Override
+        public <T> T getSideNote(Object object, SideNoteKey<T> key) {
                 assert isActive();
                 Misc.throwIfNull(object);
@@ -266 +281 @@
                         return null;
                 }
-                return Casting.nullOrThrowCast(valueType, ((Map<?, ?>) sideNote).get(key));
-        }
-
-        @Override
-        public boolean removeSideNote(Object object, Object key) {
+                Object value = ((Map<SideNoteKey<?>, Object>) sideNote).get(key);
+                if (value == null) {
+                        return null;
+                }
+                return (T) value;
+        }
+
+        @Override
+        public boolean removeSideNote(Object object, SideNoteKey<?> key) {
+                assert isActive();
                 Object sideNote = sideNotes.get(object);
                 if (sideNote == null) {
@@ -276 +296 @@
                 }
                 @SuppressWarnings("unchecked")
-                Map<Object, Object> sideNotesMap = (Map<Object, Object>) sideNote;
+                Map<SideNoteKey<?>, Object> sideNotesMap = (Map<SideNoteKey<?>, Object>) sideNote;
                 boolean result = (sideNotesMap.remove(key) != null);
                 if (sideNotesMap.isEmpty()) {
@@ -284 +304 @@
         }
 
+        @Override
+        public void addNeedFileAcceptor(int priority, NeedFileAcceptor acceptor) {
+                assert isActive();
+                needFileAcceptors.add(Pair.make(priority, acceptor));
+        }
+
+        @Override
+        public void removeNeedFileAcceptor(NeedFileAcceptor acceptor) {
+                assert isActive();
+                Iterator<Pair<Integer, NeedFileAcceptor>> i = needFileAcceptors.iterator();
+                while (i.hasNext()) {
+                        if (i.next().second == acceptor) {
+                                i.remove();
+                                break;
+                        }
+                }
+        }
+
+        @Override
+        public boolean acceptNeed(final NeedFile needFile) {
+                Dispatching.dispatchIfNotActive(this, new RunAt<AbstractTree>(needFile.getFuture()) {
+
+                        @Override
+                        protected void runAt() {
+                                for (Pair<Integer, NeedFileAcceptor> acceptor : needFileAcceptors) {
+                                        if (acceptor.second.acceptNeed(needFile)) {
+                                                return;
+                                        }
+                                }
+                                throw new FramsticksException().msg("failed to find need file acceptor in tree").arg("tree", AbstractTree.this);
+                        }
+                });
+                return true;
+        }
+
 }