Changeset 906 for trunk/src/org/expeditee/io

Timestamp:

04/15/14 12:56:30 (10 years ago)

Author:

bln4

Message:

Added Magnetic Constraints

Magnetic Constraints are a new type of constraint between items; particularly text. Items can have their neighbours set (for example: who is to my right) and then have custom actions run when a item and its neighbour interact in a defined way. For example: one text item grows enough for hits its right neighbour; so its right neighbour moves over to compensate.
Files changed: org.expeditee.{ gui.DisplayIO, io.DefaultFrameReader, io.DefaultFrameWriter, items.Item, items.Text }

Added ability to have 'delayed' properties on items. With the invention of Magnetic Constraints it was found that the saving and loading of properties now required (greatly benifited from) a way to set a property after all items have been loaded in; rather than as the perticular item was being loaded in. This is that ability.

Files changed: org.expeditee.io.{ DefaultFrameReader, ExpReader }

Let through some non letter characters as names of properties. The tokens ',', '^{', and '_' have been used as property names to describe neighbors. In order to do this the isValidLine(String) method was altered to let these through. This is not the ideal solution, and should be looked at later.}

Files changed: org.expeditee.io.ExpReader

Bug fix: setting a custom title support was added previously. If a custom title had been set the title was: [Your Custom Title] ~ [Expeditee Title]. However the tildas where showing up even without a custom title set. This is now fixed.

Location:

trunk/src/org/expeditee/io

Files:

• DefaultFrameReader.java (modified) (4 diffs)
• DefaultFrameWriter.java (modified) (1 diff)
• ExpReader.java (modified) (7 diffs)
• flowlayout/XGroupItem.java (modified) (1 diff)

trunk/src/org/expeditee/io/DefaultFrameReader.java

@@ -12 +12 @@
 import java.util.LinkedHashMap;
 import java.util.List;
+import java.util.LinkedList;
 
 import org.expeditee.gui.Frame;
@@ -24 +25 @@
 public abstract class DefaultFrameReader implements FrameReader {
         protected static LinkedHashMap<Character, Method> _ItemTags = null;
+        
+        protected static List<Character> _DelayedItemTags = null;
 
         protected static LinkedHashMap<Character, Method> _FrameTags = null;
@@ -49 +52 @@
                 _ItemTags = new LinkedHashMap<Character, Method>();
                 _FrameTags = new LinkedHashMap<Character, Method>();
+                _DelayedItemTags = new LinkedList<Character>();
 
                 try {
@@ -139 +143 @@
                         _ItemTags.put('L', Line.class.getMethod("setStartItem", pItem));
                         _ItemTags.put('C', Constraint.class.getMethod("getID", (Class[]) null));
+                        
+                        _ItemTags.put('[', Item.class.getMethod("setMagnetizedItemLeft", pInt));
+                        _DelayedItemTags.add('[');
+                        _ItemTags.put(']', Item.class.getMethod("setMagnetizedItemRight", pInt));
+                        _DelayedItemTags.add(']');
+                        _ItemTags.put('^', Item.class.getMethod("setMagnetizedItemTop", pInt));
+                        _DelayedItemTags.add('^');
+                        _ItemTags.put('_', Item.class.getMethod("setMagnetizedItemBottom", pInt));
+                        _DelayedItemTags.add('_');
                 } catch (Exception e) {
                         e.printStackTrace();

trunk/src/org/expeditee/io/DefaultFrameWriter.java

@@ -123 +123 @@
                         _ItemTags.put('p', Item.class.getMethod("getPermission"));
 
+                        _ItemTags.put('[', Item.class.getMethod("getMagnetizedItemLeft"));
+                        _ItemTags.put(']', Item.class.getMethod("getMagnetizedItemRight"));
+                        _ItemTags.put('^', Item.class.getMethod("getMagnetizedItemTop"));
+                        _ItemTags.put('_', Item.class.getMethod("getMagnetizedItemBottom"));
                 } catch (Exception e) {
                         e.printStackTrace();

trunk/src/org/expeditee/io/ExpReader.java

@@ -6 +6 @@
 import java.lang.reflect.Method;
 import java.util.HashMap;
+import java.util.LinkedList;
+import java.util.List;
 
 import org.expeditee.gui.Frame;
@@ -48 +50 @@
         protected static boolean isValidLine(String s) {
                 return s.length() >= 2 && s.charAt(1) == ' '
-                                && Character.isLetter(s.charAt(0));
+                                && (Character.isLetter(s.charAt(0)) || s.charAt(0) == '[' || s.charAt(0) == ']' || s.charAt(0) == '^' || s.charAt(0) == '_');
         }
 
@@ -69 +71 @@
                         newFrame.setName(_frameName);
 
+                        final List<DelayedAction> delayedActions = new LinkedList<DelayedAction>();
+                        
                         // First read all the header lines
                         while (_reader.ready() && !(next = _reader.readLine()).equals("Z")) {
@@ -95 +99 @@
                                                 _linePoints.put(currentItem.getID(), currentItem);
                                                 newFrame.addItem(currentItem);
+                                        } else if (currentItem != null && actionShouldBeDelayed(getTag(next))) { 
+                                                delayedActions.add(new DelayedAction(currentItem, next));
                                         } else if (currentItem != null) {
                                                 processBodyLine(currentItem, next);
+                                        } else {
+                                                System.err.println("Error while reading in frame (ExpReader): Found body line but no current item to apply it to.");
                                         }
                                 }
@@ -140 +148 @@
                                 }
                         }
+                        
+                        for(DelayedAction action: delayedActions) 
+                                action.exec();
 
                         // Read the stats
@@ -173 +184 @@
         }
 
+        private class DelayedAction {
+                private Item theItem;
+                private String theLine;
+
+                DelayedAction(final Item theItem, final String theLine) {
+                        this.theItem = theItem;
+                        this.theLine = theLine; 
+                }
+                
+                void exec() {
+                        processBodyLine(theItem, theLine);
+                }
+        }
+        
         // Stores points used when constructing lines
         private HashMap<Integer, Item> _linePoints = new HashMap<Integer, Item>();
@@ -213 +238 @@
                 else
                         return "";
+        }
+        
+        protected static boolean actionShouldBeDelayed(Character c) {
+                return _DelayedItemTags.contains(c);
         }
 

trunk/src/org/expeditee/io/flowlayout/XGroupItem.java

@@ -213 +213 @@
         }
         
+        public int getItemSpanLength() {
+                return yitems_span_array.length;
+        }
+        
         public YOverlappingItemsSpan getYOverlappingItemsSpan(int y) 
         {