X-Git-Url: https://mograsim.net/gitweb/?a=blobdiff_plain;f=net.mograsim.logic.model.editor%2Fsrc%2Fnet%2Fmograsim%2Flogic%2Fmodel%2Feditor%2FEditor.java;h=c0a504aff2499fa88ad2ea61ce791790d0a08ff6;hb=95a225aed8eae0fc080440dba779abfdf867ff3d;hp=e4c92928d012eb785e4bdf1c4312acbd0042beb8;hpb=f87464acdb28197031f7235e60ed7de1ac206723;p=Mograsim.git

diff --git a/net.mograsim.logic.model.editor/src/net/mograsim/logic/model/editor/Editor.java b/net.mograsim.logic.model.editor/src/net/mograsim/logic/model/editor/Editor.java
index e4c92928..c0a504af 100644
--- a/net.mograsim.logic.model.editor/src/net/mograsim/logic/model/editor/Editor.java
+++ b/net.mograsim.logic.model.editor/src/net/mograsim/logic/model/editor/Editor.java
@@ -13,6 +13,7 @@ import com.google.gson.JsonParser;
 import com.google.gson.JsonSyntaxException;
 
 import net.haspamelodica.swt.helper.swtobjectwrappers.Point;
+import net.mograsim.logic.model.am2900.Am2900Loader;
 import net.mograsim.logic.model.editor.handles.ComponentHandle;
 import net.mograsim.logic.model.editor.handles.Handle;
 import net.mograsim.logic.model.editor.handles.HandleManager;
@@ -24,6 +25,7 @@ import net.mograsim.logic.model.model.ViewModelModifiable;
 import net.mograsim.logic.model.model.components.GUIComponent;
 import net.mograsim.logic.model.model.wires.GUIWire;
 import net.mograsim.logic.model.serializing.DeserializedSubmodelComponent;
+import net.mograsim.logic.model.serializing.IdentifierGetter;
 import net.mograsim.logic.model.serializing.IndirectGUIComponentCreator;
 
 public final class Editor
@@ -53,10 +55,10 @@ public final class Editor
 		dialogManager = new DialogManager(gui.shell);
 
 		toBeEdited.submodel.addComponentRemovedListener(c -> identifierPerComponent.remove(c));
-		
+
 		gui.open();
 	}
-	
+
 	public ViewModelModifiable getSubmodel()
 	{
 		return toBeEdited.getSubmodelModifiable();
@@ -67,8 +69,8 @@ public final class Editor
 		return selection;
 	}
 
-	//TODO: Remove this error prone method: Relative offset may change between multiple moves,
-	//because Handles have different ways of responding to reqMove(...), causing strange behaviour
+	// TODO: Remove this error prone method: Relative offset may change between multiple moves,
+	// because Handles have different ways of responding to reqMove(...), causing strange behaviour
 	@Deprecated
 	public void moveSelection(double x, double y)
 	{
@@ -84,7 +86,7 @@ public final class Editor
 			c.reqMove(newX, newY);
 		}
 	}
-	
+
 	public void moveHandles(double x, double y, Map<Handle, Point> handleOffsetMap)
 	{
 		Point snapped = new Point(x, y);
@@ -113,7 +115,7 @@ public final class Editor
 		for (Handle h : selection)
 		{
 			Optional<ComponentInfo> cInfo = h.reqCopy(refPoint);
-			if(cInfo.isPresent())
+			if (cInfo.isPresent())
 				copyBuffer.add(cInfo.get());
 		}
 	}
@@ -130,18 +132,22 @@ public final class Editor
 		}
 		moveSelection(x, y);
 	}
-	
+
 	public void save()
 	{
 		saveManager.save();
 	}
 
+	public void saveAs()
+	{
+		saveManager.openSaveAsDialog();
+	}
+
 	public void addComponent(double x, double y)
 	{
 		boolean successful = false;
 		JsonElement params = JsonNull.INSTANCE;
-		outer:
-		while(!successful)
+		outer: while (!successful)
 		{
 			String selected = gui.getAddListSelected();
 			try
@@ -152,27 +158,28 @@ public final class Editor
 				moveSelection(x, y);
 				successful = true;
 			}
-			catch(UnsupportedOperationException | JsonSyntaxException | NumberFormatException e)
+			catch (@SuppressWarnings("unused") UnsupportedOperationException | JsonSyntaxException | NumberFormatException e)
 			{
 				String result = DialogManager.openMultiLineTextDialog("Add component", "Create", "Cancel", "Parameters:");
-				if(result == null)
+				if (result == null)
 					break outer;
 				params = new JsonParser().parse(result);
 			}
 		}
 	}
-	
+
 	private GUIComponent addComponent(String identifier, JsonElement params)
 	{
-		GUIComponent comp = IndirectGUIComponentCreator.createComponent(toBeEdited.getSubmodelModifiable(), identifier,
-				params);
+		GUIComponent comp = IndirectGUIComponentCreator.createComponent(toBeEdited.getSubmodelModifiable(), identifier, params);
 		identifierPerComponent.put(comp, identifier);
 		return comp;
 	}
-	
+
 	public static String getIdentifier(GUIComponent c)
 	{
-		return identifierPerComponent.get(c);
+		if (identifierPerComponent.containsKey(c))
+			return identifierPerComponent.get(c);
+		return new IdentifierGetter().componentIDs.apply(c);
 	}
 
 	public void duplicate()
@@ -189,8 +196,10 @@ public final class Editor
 		case OFF:
 			break;
 		case ABSOLUTE:
-			newP.x -= newP.x % snapX;
-			newP.y -= newP.y % snapY;
+			newP.x = (int) (newP.x / snapX + .5) * snapX;
+			newP.y = (int) (newP.y / snapY + .5) * snapY;
+			break;
+		default:
 			break;
 		}
 	}
@@ -200,7 +209,7 @@ public final class Editor
 		public final double relX, relY;
 		public final String identifier;
 		public final JsonElement params;
-		
+
 		public ComponentInfo(double relX, double relY, String identifier, JsonElement params)
 		{
 			this.relX = relX;
@@ -210,6 +219,7 @@ public final class Editor
 		}
 	}
 
+	@SuppressWarnings("unused")
 	public void addWire(PinHandle a, PinHandle b)
 	{
 		new GUIWire(toBeEdited.getSubmodelModifiable(), a.getPin(), b.getPin(), new Point[0]);
@@ -218,16 +228,17 @@ public final class Editor
 	public static enum Snapping
 	{
 		OFF, ABSOLUTE;
-		
+
 		@Override
 		public String toString()
 		{
 			return super.toString().toLowerCase();
 		}
 	}
-	
+
 	public static void main(String[] args) throws IOException
 	{
+		Am2900Loader.setup();
 		SaveLoadManager.openLoadDialog();
 	}