What's wrong with this circuit?

[Aero]

I'm making the controller for letting people choose what floor they want from inside the car, but I can't seem to get it to work right. This is how it looks:



For some reason, it only goes up and down in increments of 4, letting me only chose floors 1, 5, 9, 13 and 15. Does anyone know what I did wrong? I just started with wire a few days ago

[Razoul]

What happens if you hold the + or - button, does it keep going up, or change only once?

My only thought would be the legnth of time your holding the butting is allowing the add to run through more than once, it would pass 4 (in this case) differant values, setting those to the memory just to be added again.

Run my test, if it continues to change while holding a button then the problem is what I stated above, if it changes only once while holding the button then I am at a loss for what is going on.

You might also try using the Debugger tool on your chips to see where the problem is occuring.

[Aut]

Don't use D-latch memory. Use Latch(edgetriggered) and see, D-latch will change the value while the button is pressed, Latch(edge) will only change the value when the clock first flips from 0 to 1.

[Aero]

When I hold it, it doesn't keep going up, it only goes up by 4 once, until I press it again.

I switched to Latch (edge triggered) but with this, the value doesn't change at all, it stays at 1 when I press the button

Edit: I think this is because the clock get switched on before the value is updated, so it always stays at what it is. I changed it so that the clock was based on a delta, and this allowed me to go upwards properly, but not down. It half works...

[Razoul]

Might be a bit overkill, but I would suggest using an expression for this (fewer chips, easier to change/add later)

N@Elevator
I@Up Down
O@Floor
~Up & Up -> Floor = clamp(Floor + 1, 1, 15);
~Down & Down -> Floor = clamp(Floor - 1, 1, 15);

Edit: Made it so Floor will never be greater than 15, or less than 1.

[Aero]

Thanks, it works now with the expression gate.