React & Redux: State of component is not updating even with mapStateToProps

Question

I posted an answer below, but if someone can explain why this is necessary you'll get the bounty, I went through a redux tutorial and feel like I didn't learn about mapDispatchToProps, only mapStateToProps. If you can explain at a deeper level what exactly mapStateToProps and mapDispatchToProps are doing and how they are different I'll give you the bounty.

Minimum Reproducible Example

I have a mapStateToProps function that looks like

const mapStateToProps = state => {
  return {
    firstName: state.firstName,
    middleName: state.middleName,
    lastName: state.lastName,
  }
}

const ReduxTabForm = connect(mapStateToProps)(MyTab)

In my MyTab component I have a button that is supposed to be inactive if these 2 field do not have anything entered in, but the state of whether or not the button is disabled does not change

function App() {
  const {firstName, lastName} = store.getState().formData

  const isDisabled = () => {
    const {firstName, lastName} = store.getState().form
    const requiredFields = [firstName, lastName]
    alert(requiredFields)
    for(let i = 0; i < requiredFields.length; i=i+1){
      if (!requiredFields[i]){
        return true
      }
    }
    return false
  }

  return (

    <div className="App">
        <div className='bg-light rounded'>
          <div className='px-sm-5 pt-0 px-4 flexCenterCol mx-5'>

            <div>
                <input 
                    type='text'
                    className="form-control"
                    value={store.getState().formData['firstName']}
                    placeholder="First Name"
                    onChange={(e) => {
                        store.dispatch(setFormData({'firstName': e.target.value}))
                    }}
               ></input>
               <input 
                    type='text'
                    className="form-control"
                    value={store.getState().formData['lastName']}
                    placeholder="Last Name"
                    onChange={(e) => {
                        store.dispatch(setFormData({'lastName': e.target.value}))
                    }}
               ></input>

            </div>
            <button
                type="submit"
                disabled={isDisabled()}
            >
                Button
            </button>
        </div>
    </div>
  </div>
 )
}

That alert statement executes on page refresh, but does not execute any time after that when I enter data in. I have checked that the redux state updating and it is. The button will not update though, and the isDisabled function will not run more than once

const mapStateToProps = state => ({ firstName: state.firstName, middleName: state.middleName, lastName: state.lastName, }) Can you write like i have written above. — Aftab22, Sep 25 '20 at 18:47

score 1 · Answer 1 · answered Sep 25 '20 at 18:36

I don`t know if this will solve your problems, but maybe it is one of the things below: 1 - As Mohammad Faisal said the correct form of calling props should be

const { firstName, lastName } = props;

2 - Instead of reduxTabForm, maybe you could use this instead:

export default connect(mapStateToProps)(MyTab);

3 - And finally, maybe it is an error in the "isDisabled":

for(let i = 0; i < requiredFields.length; i=i+1){
  if (!requiredFields){
    return false
  }
}

If you look carefully, you can see that you are not checking if there is an error inside requeiredFields, your are looking if that doesnt exist if (!requiredFields), maybe changing the condition to if(!requiredFields[i]) so it check each variable and not if the array doesn`t exists.

Edit: the return condition is correct? Returning False when something doesn`t exists?

I posted an [MRE](https://github.com/samgermain/redux-problem) and yes, it's supposed to re-render, but I don't think it is — Sam, Sep 25 '20 at 19:43

score 1 · Answer 2 · answered Sep 29 '20 at 10:04

const ReduxTabForm = connect(mapStateToProps,null)(MyTab)

Try this code snippet, as you are not passing dispatch function to your component. It is better to pass null value. mapdispatchtoProps is the same basic theory of mapStateToProps.You are storing the function inside the store(which usually in actions) and when rendering the component you attach those function in store to your props. After rendering the component you will be able to run the functions which are in store.

score 1 · Accepted Answer · answered Oct 03 '20 at 04:03

I looked at your reducer code and it looks like this:

...
const reducer = combineReducers({
  formData: formReducer,
})

export default reducer

Which means your redux state structure is like this:

state = {
  formData: {
    firstName: <value>,
    middleName: <value>,
    lastName: <value>,
  }
}

Solution

So, to make your component re-render when the redux state is changed, you need to subscribe to the correct state variables in your mapStateToProps function. Change it to this and will work:

const mapStateToProps = state => {
  return {
    firstName: state.formData.firstName, // Note that I added "formData"
    middleName: state.formData.middleName,
    lastName: state.formData.lastName
  }
}

Couple of side notes:

It's a better to use the props instead of directly accessing the redux store.
For debugging, console.log is preferred over alert. React DevTools and Redux DevTools are even better.

score 0 · Answer 4 · answered Sep 25 '20 at 18:00

0

your state values are passed to your component as props. so if you want to access them inside component you should do something like this

 const {firstName, lastName} = props

answered Sep 25 '20 at 18:00

Mohammad Faisal

2,265
1
10
16

So this also doesn't work, I get that props is undefined – Sam Sep 25 '20 at 18:08
I can’t sorry, I can try to make an mre, it might be up in an hour and a bit – Sam Sep 25 '20 at 18:15
just one thing .... your component should start like the following. export default function MyTab(props) { ..your component code here.. } – Mohammad Faisal Sep 25 '20 at 18:17
Yeah it has that, my code posted here is just a snippet – Sam Sep 25 '20 at 18:39
I posted an [MRE](https://github.com/samgermain/redux-problem) – Sam Sep 25 '20 at 19:42

A Zarqam · Answer 5 · 2020-09-25T20:01:56.883

0

What you could do is something like:

<button 
      type="submit"
      disabled={!fistname && !lastname}
    />

this way if any of your fields be falsy, button is disabled. (empty string is falsy)

edited Sep 25 '20 at 20:01

answered Sep 25 '20 at 18:47

A Zarqam

338
2
3
12

But then it will never be enabled – Sam Sep 25 '20 at 19:41
@Sam remove the true and keep it like !fistname && !lastname – A Zarqam Sep 25 '20 at 20:01
It doesn't fix anything. You can check with the MRE – Sam Sep 25 '20 at 20:28

score 0 · Answer 6 · answered Sep 28 '20 at 15:53

0

The React redux documentation also explains mapDispatchToProps

If you call your action called setFormData from your redux/actions.js then you will be mapping the action setFormData (which is an object) to your component. The action setFromData replace mapDispatchToProps. This is what the documentation says about mapping actions to your components

If it’s an object full of action creators, each action creator will be turned into a prop function that automatically dispatches its action when called.

To fix your problem, change your connect function call to this.

const ReduxApp = connect(
  setFormData,
  mapStateToProps
)(App)

answered Sep 28 '20 at 15:53

Sam

1,765
11
82
176

Actually, this example didn't work when I transferred it to my actual program. I don't know what I can do – Sam Sep 28 '20 at 16:26
I added an answer which will fix your issue. I can share more details if you need any explanation. – Dipen Shah Sep 30 '20 at 23:30

score 0 · Answer 7 · answered Sep 29 '20 at 18:27

Your issues is with this code isDisabled()

     const isDisabled = () => {
    const {firstName, lastName} = store.getState().form
    const requiredFields = [firstName, lastName]
    alert(requiredFields)
    for(let i = 0; i < requiredFields.length; i=i+1){
      if (!requiredFields){
        return false
      }
    }
    return true
  }

You are trying to test in loop !requiredFields, an array you created which always return false, even if it doesn't have values. It's a reference type. What you can do now is

const isDisabled = () => {
    const {firstName, lastName} = store.getState().form
    const requiredFields = [firstName, lastName]
    alert(requiredFields)
    for(let i = 0; i < requiredFields.length; i=i+1){
      if (!requiredFields[i]){
        return false
      }
    }
    return true
  }

Your loop will check firstNAme and LastName values if they are undefined or not and test should respond with this.

That's not the issue, it's a side thing, the state isn't getting mapped to props — Sam, Sep 29 '20 at 18:31
Have you attempted it and it didn't work? Your code is bad and will the condition you set will never change. One is correcting is you, you're saying that's not the issue. — Niyi Aromokeye, Sep 29 '20 at 18:36

score 0 · Answer 8 · answered Sep 30 '20 at 23:21

The problem is in the mapStateToProps method. Your code is setting firstNAme and lastName as formData within state object and you are trying to access it from state object directly.

This codesandbox has your project updated with the fix. I didn't fix any other thing in your code so everything is as it is only mapStateToProps should be:

const mapStateToProps = (state) => {
  return {
    firstName: state.formData["firstName"],
    middleName: state.middleName,
    lastName: state.formData["lastName"]
  };
};

and this will fix your issue. mapStateToPropsis kind of a projection function which will project entire store to some object which will only have properties based on requirements of your component.

score 0 · Answer 9 · answered Oct 04 '20 at 17:01

Redux re-render components only when store state changes. Check if you are only updating store's state property not whole state because redux compare state by comparing their references, so if you are doing something like this inside your reducer:

State.firstName = action.payload.firstName;

return State;

Then change this to this:

return {...State, firstName: action.payload.firstName}

Note: If you unable to grasp that then kindly provide your reducer code too so that I can see how you are updating your store state.

Tom · Answer 10 · 2020-10-04T17:31:11.910

Looking at your MRE on GitHub, the first thing I want to say is that your button will only have one state and it is the one that the isDisabled() method returns when you refresh the page. This is because the App component it's not getting refresh every time you write on the input fields, so you will never be able to make it change.
you need to subscribe to the correct state variables in your mapStateToProps function. Like this:

const mapStateToProps = state => {
    return {
        firstName: state.formData.firstName,
        middleName: state.formData.middleName,
        lastName: state.formData.lastName
    }
}

So now that you have this right, you have to introduce this props into your component, like this:

     function  App({firstName, lastName}) { ...

Another thing to add, is that your are not initializing the states when you create your reducer in reducer.js. If you don't do this, your initial states for firstName and lastName will be null. Here is how you should do it:

import {combineReducers} from 'redux'
import {SET_FORM_DATA} from './actions'

const initialState = {
  firstName: "",
  lastName: ""
}

const formReducer = (state = initialState, action) => {
  if (action.type === SET_FORM_DATA){
    return {
      ...state,
      ...action.payload
    }
  }else{
    return state
  }
}

const reducer = combineReducers({
  formData: formReducer,
})

export default reducer

Finally you have to update App:

function  App({firstName, lastName}) {
  return (
      <div className="App">
        <div className='bg-light rounded'>
          <div className='px-sm-5 pt-0 px-4 flexCenterCol mx-5'>

            <div>
            <input 
              type='text'
              className="form-control"
              placeholder="First Name"
              onChange={(e) => {
                store.dispatch(setFormData({'firstName': e.target.value}));
                console.log(firstName === "h");
              }}
            ></input>
            <input 
              type='text'
              className="form-control"
              placeholder="Last Name"
              onChange={(e) => {
                store.dispatch(setFormData({'lastName': e.target.value}))
                alert(JSON.stringify(store.getState()))

              }}
            ></input>

            </div>
            <button
              type="submit"
              disabled={firstName == "" || lastName == ""}
            >
              Button
            </button>
          </div>
        </div>
      </div>
  );
}

So in this way you will be able to update the states from your store and have the dynamic behavior that you were looking for.

score 0 · Answer 11 · answered Oct 04 '20 at 20:47

In isDisabled function you read data from form: const {firstName, lastName} = store.getState().form but I guess they are saved to formData.

Changing const {firstName, lastName} = store.getState().form to const {firstName, lastName} = store.getState().formData should help.

React & Redux: State of component is not updating even with mapStateToProps

11 Answers11

Solution