Effects class instantiated twice? about fluxor HOT 6 CLOSED

Hi

I looked at the source code to refresh my memory. Classes with [EffectMethod] methods in them are registered as Scoped, so you should get the same instance for every method in the same class. If you are seeing something different then I'll need to see a project that reproduces the problem.

You are correct about sharing state. If you inject IState<MyState> into more than one component then you should get the same state object. Again, if this is not what you are seeing then I'll need a project that reproduces the problem.

from fluxor.

Thank you @mrpmorris for the kind reply. Before I get too deep into creating an example project (which I'm more than happy to do), there's one part of my code that doesn't smell right, which might explain the issue.

In the aforementioned effects class, some of my EffectMethod handlers require access to the state, outside of what I supply in the action object. To make this possible, I inject IState into the effects class using a parameter on the effect class constructor. I then store a private reference to the state, which my methods access. In my mind this should work, but maybe it is the wrong approach?

I have traced the issue to this state reference in my effects class reflecting stale state data.

Example:

    private readonly IState<MyState> _myState;

    public Effects(IState<MyState> myState)
    {
        _myState = myState;
    }

    [EffectMethod]
    public async Task HandleMyAction(MyAction action, IDispatcher dispatcher)
    {
        var foo = _myState.Value.Blerg;
        ...
        dispatcher.Dispatch(new MyResultAction(...));
    }

The reason I don't simply pass the necessary parts of the state in the action object is that the effect requires access to tons of state properties, so instantiating the action object would be very cumbersome.

from fluxor.

Sorry to have wasted your time. This behavior was caused by an error in my state class design, not related to Fluxor.

from fluxor.

Can you share any information so others can benefit?

from fluxor.

Normally I would have, but in this case the error was pure stupidity on my part. My state class constructor basically had an error like this:

 class MyState {
      public int Foo { get; }
      public MyState(int foo)
      {
           Foo = foo;
           Foo = 0;
      }
 }

So the passed value of foo was always ignored.

from fluxor.

Thanks :)

from fluxor.