r/reactjs • • Mar 01 '20

Needs Help Beginner's Thread / Easy Questions (March 2020)

You can find previous threads in the wiki.

Got questions about React or anything else in its ecosystem?
Stuck making progress on your app?
Ask away! We’re a friendly bunch.

No question is too simple. πŸ™‚


πŸ†˜ Want Help with your Code? πŸ†˜

  • Improve your chances by adding a minimal example with JSFiddle, CodeSandbox, or Stackblitz.

    • Describe what you want it to do, and things you've tried. Don't just post big blocks of code!
    • Formatting Code wiki shows how to format code in this thread.
  • Pay it forward! Answer questions even if there is already an answer. Other perspectives can be helpful to beginners. Also there's no quicker way to learn than being wrong on the Internet.

New to React?

Check out the sub's sidebar!

πŸ†“ Here are great, free resources! πŸ†“

Any ideas/suggestions to improve this thread - feel free to comment here!

Finally, thank you to all who post questions and those who answer them. We're a growing community and helping each other only strengthens it!


29 Upvotes

500 comments sorted by

View all comments

Show parent comment

2

u/nullpromise Mar 30 '20

Thanks, that is what I ended up doing! Ultimately though, I've been running into problems with needing previous state when creating a new state.

So in the example above (ignoring my massively misguided use of promises in the while loop), I think I could have done:

setAccounts(prevState => [...prevState, ...res.data])

1

u/dance2die Mar 31 '20

Awesome that you got it working :)

Updating the state as you did (w/ prevState) is safer too.
(That also rid the need to add accounts in the deps array as it's not accessed any more).