suggestion on transition and condition, which should include Event about swiftstate HOT 10 CLOSED

For example:
.LoadAction(Int) Event
.CancelAction Event

.Pending State
.Loading(Int) State

.LoadAction event has an associate integer to suggest which task it wants to trigger.
.Loading state has an associate integer to suggest which task it is loading.

when the .LoadAction triggered, and the machine on .Loading state with the same associate number, then the Event should be ignored.

from swiftstate.

Hi, I added your above example as a XCTestCase in cc6bc8f.
Is this what you are looking for?

from swiftstate.

Yes.

Because transition has no information on event, we should add all possible events to the machine.

        for actionId in 1...100 {
            machine.addRouteEvent(.LoadAction(actionId), transitions: [ .AnyState => .Loading(actionId) ], condition: { $0.fromState != .Loading(actionId) })
        }

if
(1) I want the Int as greater as I want, or
(2) the associate value is a generic type from subclass's enum, or
(3) the associate value is a data

then the StateMachine cannot work on this kind of situations.
These situations have the same character: event's associate value cannot be fully enumerated by current class.

from swiftstate.

Oh, OK I got it!
You are right, and current typealias Condition = (transition: Transition) -> Bool has too few parameters to distinguish Event and other stuffs e.g. userInfo.

I think it should be something like:

typealias ConditionContext = (event: Event, transition: Transition, userInfo: Any?)
typealias Condition = ConditionContext -> Bool

And for this enhancement, we will need a major version bump.
Let me take some time to work on this 😉

from swiftstate.

Yes! Thanks!

from swiftstate.

And another suggestion is, can you use Optional.None to represent the meaning of AnyState or AnyEvent?
Because it needs code to add an nil init for an event, and it is ugly to add one more State though the fact the machine hasn't that State.

If there is a place needs nil to represent for the meaning of "AnyState" or "AnyEvent", just put "Event?" or "State?" as the parameter's type.

from swiftstate.

Thanks for above feedback as well!
This will also be another big breaking change, so I will refactor existing codes for both ideas :)

from swiftstate.

Thanks:)

from swiftstate.

Sorry for late reply.
I just finished writing a rough draft for solving your above issues in #36.

Please check and tell me your thoughts 😉

from swiftstate.

This issue has been completed in #36 and ver 4.0.0.
Please see Ver 4.0.0 Release Notes for more detail.

@frogcjn Thanks again for your contribution! 🎉

from swiftstate.