X-Git-Url: https://mograsim.net/gitweb/?a=blobdiff_plain;f=net.mograsim.logic.model%2Fsrc%2Fnet%2Fmograsim%2Flogic%2Fmodel%2Fsnippets%2Fhighlevelstatehandlers%2Fstandard%2Fsubcomponent%2FDelegatingSubcomponentHighLevelStateHandler.java;h=12a98887c54153e0196dd5580bb34135f0896407;hb=8bed58cd47f4e53a0a83e066d38864aa6875502f;hp=3347377e432125f03098ff17ef29fa16e98dc587;hpb=3977f16649531ca3ba345c9cd5ec365e0b804783;p=Mograsim.git

diff --git a/net.mograsim.logic.model/src/net/mograsim/logic/model/snippets/highlevelstatehandlers/standard/subcomponent/DelegatingSubcomponentHighLevelStateHandler.java b/net.mograsim.logic.model/src/net/mograsim/logic/model/snippets/highlevelstatehandlers/standard/subcomponent/DelegatingSubcomponentHighLevelStateHandler.java
index 3347377e..12a98887 100644
--- a/net.mograsim.logic.model/src/net/mograsim/logic/model/snippets/highlevelstatehandlers/standard/subcomponent/DelegatingSubcomponentHighLevelStateHandler.java
+++ b/net.mograsim.logic.model/src/net/mograsim/logic/model/snippets/highlevelstatehandlers/standard/subcomponent/DelegatingSubcomponentHighLevelStateHandler.java
@@ -1,68 +1,91 @@
 package net.mograsim.logic.model.snippets.highlevelstatehandlers.standard.subcomponent;
 
-import net.mograsim.logic.model.model.components.GUIComponent;
+import net.mograsim.logic.model.model.components.ModelComponent;
 import net.mograsim.logic.model.model.components.submodels.SubmodelComponent;
-import net.mograsim.logic.model.serializing.IdentifierGetter;
+import net.mograsim.logic.model.serializing.IdentifyParams;
 import net.mograsim.logic.model.snippets.SnippetDefinintion;
-import net.mograsim.logic.model.snippets.highlevelstatehandlers.standard.HighLevelStateHandlerContext;
 import net.mograsim.logic.model.snippets.highlevelstatehandlers.standard.StandardHighLevelStateHandlerSnippetSuppliers;
 
 public class DelegatingSubcomponentHighLevelStateHandler implements SubcomponentHighLevelStateHandler
 {
 	private final SubmodelComponent parentComponent;
-	private GUIComponent delegateTarget;
+	private ModelComponent delegateTarget;
 	private String prefix;
 
-	public DelegatingSubcomponentHighLevelStateHandler(HighLevelStateHandlerContext context)
+	public DelegatingSubcomponentHighLevelStateHandler(SubmodelComponent component)
 	{
-		this(context, null);
+		this(component, null);
 	}
 
-	public DelegatingSubcomponentHighLevelStateHandler(HighLevelStateHandlerContext context,
+	public DelegatingSubcomponentHighLevelStateHandler(SubmodelComponent component,
 			DelegatingSubcomponentHighLevelStateHandlerParams params)
 	{
-		this.parentComponent = context.component;
+		this.parentComponent = component;
 		if (params != null)
 		{
 			// TODO document this
 			if (params.delegateTarget == null)
 				setDelegateTarget(parentComponent);
 			else
-				this.delegateTarget = parentComponent.submodel.getComponentsByName().get(params.delegateTarget);
-			setPrefix(prefix);
+			{
+				ModelComponent delegateTarget = parentComponent.submodel.getComponentsByName().get(params.delegateTarget);
+				if (delegateTarget == null)
+					throw new NullPointerException("No subcomponent with name " + params.delegateTarget);
+				setDelegateTarget(delegateTarget);
+			}
+			setPrefix(params.prefix);
 		}
+		parentComponent.submodel.addComponentRemovedListener(c ->
+		{
+			if (delegateTarget == c)
+				delegateTarget = null;
+		});
 	}
 
-	public void set(GUIComponent delegateTarget, String prefix)
+	public void set(ModelComponent delegateTarget, String prefix)
 	{
 		setDelegateTarget(delegateTarget);
 		setPrefix(prefix);
 	}
 
-	public void setDelegateTarget(GUIComponent delegateTarget)
+	public void setDelegateTarget(ModelComponent delegateTarget)
 	{
 		if (delegateTarget == null)
 			this.delegateTarget = parentComponent;
-		else if (parentComponent.submodel.getComponentsByName().get(delegateTarget.name) != delegateTarget)
+		else if (parentComponent.submodel.getComponentsByName().get(delegateTarget.getName()) != delegateTarget)
 			throw new IllegalArgumentException(
 					"Can only set components belonging to the submodel of the parent component of this handler as the delegate target");
 		this.delegateTarget = delegateTarget;
 	}
 
+	public ModelComponent getDelegateTarget()
+	{
+		return delegateTarget;
+	}
+
 	public void setPrefix(String prefix)
 	{
 		this.prefix = prefix;
 	}
 
+	public String getPrefix()
+	{
+		return prefix;
+	}
+
 	@Override
 	public Object getHighLevelState(String subStateID)
 	{
+		if (delegateTarget == null)
+			throw new IllegalStateException("Delegating to a component that was destroyed");
 		return delegateTarget.getHighLevelState(getDelegateTargetHighLevelStateID(subStateID));
 	}
 
 	@Override
 	public void setHighLevelState(String subStateID, Object newState)
 	{
+		if (delegateTarget == null)
+			throw new IllegalStateException("Delegating to a component that was destroyed");
 		delegateTarget.setHighLevelState(getDelegateTargetHighLevelStateID(subStateID), newState);
 	}
 
@@ -72,10 +95,16 @@ public class DelegatingSubcomponentHighLevelStateHandler implements Subcomponent
 	}
 
 	@Override
-	public DelegatingSubcomponentHighLevelStateHandlerParams getParamsForSerializing(IdentifierGetter idGetter)
+	public String getIDForSerializing(IdentifyParams idParams)
+	{
+		return "delegating";
+	}
+
+	@Override
+	public DelegatingSubcomponentHighLevelStateHandlerParams getParamsForSerializing(IdentifyParams idParams)
 	{
 		DelegatingSubcomponentHighLevelStateHandlerParams params = new DelegatingSubcomponentHighLevelStateHandlerParams();
-		params.delegateTarget = delegateTarget.name;
+		params.delegateTarget = delegateTarget.getName();
 		params.prefix = prefix;
 		return params;
 	}