Maybe needs review: fixed bug in ManualSwitch concerning the input

[Mograsim.git] / net.mograsim.logic.core / src / net / mograsim / logic / core / components / ManualSwitch.java

diff --git a/net.mograsim.logic.core/src/net/mograsim/logic/core/components/ManualSwitch.java b/net.mograsim.logic.core/src/net/mograsim/logic/core/components/ManualSwitch.java
index fc6e943..fa31f0c 100644 (file)
--- a/net.mograsim.logic.core/src/net/mograsim/logic/core/components/ManualSwitch.java
+++ b/net.mograsim.logic.core/src/net/mograsim/logic/core/components/ManualSwitch.java
@@ -1,9 +1,14 @@
 package net.mograsim.logic.core.components;
 
+import java.util.ArrayList;
+import java.util.Collection;
 import java.util.List;
 
+import net.mograsim.logic.core.LogicObservable;
+import net.mograsim.logic.core.LogicObserver;
 import net.mograsim.logic.core.timeline.Timeline;
 import net.mograsim.logic.core.types.Bit;
+import net.mograsim.logic.core.types.BitVector;
 import net.mograsim.logic.core.wires.Wire.ReadEnd;
 import net.mograsim.logic.core.wires.Wire.ReadWriteEnd;
 
@@ -13,50 +18,62 @@ import net.mograsim.logic.core.wires.Wire.ReadWriteEnd;
  * @author Christian Femers
  *
  */
-public class ManualSwitch extends Component
+public class ManualSwitch extends Component implements LogicObservable
 {
+       private Collection<LogicObserver> observers;
        private ReadWriteEnd output;
-       private boolean isOn;
+       private BitVector inputValues;
 
        public ManualSwitch(Timeline timeline, ReadWriteEnd output)
        {
                super(timeline);
-               if (output.length() != 1)
-                       throw new IllegalArgumentException("Switch output can be only a single wire");
+               observers = new ArrayList<>();
                this.output = output;
+               this.inputValues = output.getInputValues();
        }
 
-       public void switchOn()
+       public void switchFullOn()
        {
-               setState(true);
+               setState(BitVector.of(Bit.ONE, output.width()));
        }
 
-       public void switchOff()
+       public void switchFullOff()
        {
-               setState(false);
+               setState(BitVector.of(Bit.ZERO, output.width()));
        }
 
        public void toggle()
        {
-               setState(!isOn);
+               if (isFullOn())
+                       switchFullOff();
+               else
+                       switchFullOn();
        }
 
-       public void setState(boolean isOn)
+       public void setState(Bit bit)
        {
-               if (this.isOn == isOn)
+               setState(BitVector.of(bit));
+       }
+
+       public void setState(BitVector bits)
+       {
+               if (bits.length() != output.width())
+                       throw new IllegalArgumentException("Incorrect bit vector length");
+               if (bits.equals(inputValues))
                        return;
-               this.isOn = isOn;
-               output.feedSignals(getValue());
+               inputValues = bits;
+               output.feedSignals(bits);
+               notifyObservers();
        }
 
-       public boolean isOn()
+       public boolean isFullOn()
        {
-               return isOn;
+               return BitVector.of(Bit.ONE, output.width()).equals(output.getInputValues());
        }
 
-       public Bit getValue()
+       public BitVector getValues()
        {
-               return isOn ? Bit.ONE : Bit.ZERO;
+               return inputValues;
        }
 
        @Override
@@ -71,4 +88,22 @@ public class ManualSwitch extends Component
                return List.of(output);
        }
 
+       @Override
+       public void registerObserver(LogicObserver ob)
+       {
+               observers.add(ob);
+       }
+
+       @Override
+       public void deregisterObserver(LogicObserver ob)
+       {
+               observers.remove(ob);
+       }
+
+       @Override
+       public void notifyObservers()
+       {
+               observers.forEach(ob -> ob.update(this));
+       }
+
 }