React SetState Twice in Same Function










-1














Assume you have a Chess class with a make move function like the one below. Right after the user makes a move we let the chess engine reply with its own move. All is well, except for the fact that the browser does not repaint until the second setState is called which means the user move and the engine move show up at the same time with a 3 second delay. This is obviously not good for the user since the piece they move disappears and then two moves appear immediately. My question is how would you structure these functions or class so that the browser can render the users move before it computes the engine move. Thanks.



class Chess extends Component {
constructor(props)
super(props);
this.state =
chess_board: Array(64).set(null);


//User makes a move
makemove_on_dragend(move)
const chess_board = this.state.chess_board;
if (move is legal)
let new_board = chess_board.make_move(move);
this.setState(chess_board: new_board)

this.chess_engine_move();

// Chess engine makes a move
chess_engine_move()
const chess_board = this.state.chess_board;
let search_time = 3 seconds;
let move = alphabeta(search_time, chess_board);
let new_board = chess_board.make_move(move);
this.setState(chess_board: new_board);










share|improve this question


























    -1














    Assume you have a Chess class with a make move function like the one below. Right after the user makes a move we let the chess engine reply with its own move. All is well, except for the fact that the browser does not repaint until the second setState is called which means the user move and the engine move show up at the same time with a 3 second delay. This is obviously not good for the user since the piece they move disappears and then two moves appear immediately. My question is how would you structure these functions or class so that the browser can render the users move before it computes the engine move. Thanks.



    class Chess extends Component {
    constructor(props)
    super(props);
    this.state =
    chess_board: Array(64).set(null);


    //User makes a move
    makemove_on_dragend(move)
    const chess_board = this.state.chess_board;
    if (move is legal)
    let new_board = chess_board.make_move(move);
    this.setState(chess_board: new_board)

    this.chess_engine_move();

    // Chess engine makes a move
    chess_engine_move()
    const chess_board = this.state.chess_board;
    let search_time = 3 seconds;
    let move = alphabeta(search_time, chess_board);
    let new_board = chess_board.make_move(move);
    this.setState(chess_board: new_board);










    share|improve this question
























      -1












      -1








      -1







      Assume you have a Chess class with a make move function like the one below. Right after the user makes a move we let the chess engine reply with its own move. All is well, except for the fact that the browser does not repaint until the second setState is called which means the user move and the engine move show up at the same time with a 3 second delay. This is obviously not good for the user since the piece they move disappears and then two moves appear immediately. My question is how would you structure these functions or class so that the browser can render the users move before it computes the engine move. Thanks.



      class Chess extends Component {
      constructor(props)
      super(props);
      this.state =
      chess_board: Array(64).set(null);


      //User makes a move
      makemove_on_dragend(move)
      const chess_board = this.state.chess_board;
      if (move is legal)
      let new_board = chess_board.make_move(move);
      this.setState(chess_board: new_board)

      this.chess_engine_move();

      // Chess engine makes a move
      chess_engine_move()
      const chess_board = this.state.chess_board;
      let search_time = 3 seconds;
      let move = alphabeta(search_time, chess_board);
      let new_board = chess_board.make_move(move);
      this.setState(chess_board: new_board);










      share|improve this question













      Assume you have a Chess class with a make move function like the one below. Right after the user makes a move we let the chess engine reply with its own move. All is well, except for the fact that the browser does not repaint until the second setState is called which means the user move and the engine move show up at the same time with a 3 second delay. This is obviously not good for the user since the piece they move disappears and then two moves appear immediately. My question is how would you structure these functions or class so that the browser can render the users move before it computes the engine move. Thanks.



      class Chess extends Component {
      constructor(props)
      super(props);
      this.state =
      chess_board: Array(64).set(null);


      //User makes a move
      makemove_on_dragend(move)
      const chess_board = this.state.chess_board;
      if (move is legal)
      let new_board = chess_board.make_move(move);
      this.setState(chess_board: new_board)

      this.chess_engine_move();

      // Chess engine makes a move
      chess_engine_move()
      const chess_board = this.state.chess_board;
      let search_time = 3 seconds;
      let move = alphabeta(search_time, chess_board);
      let new_board = chess_board.make_move(move);
      this.setState(chess_board: new_board);







      javascript reactjs






      share|improve this question













      share|improve this question











      share|improve this question




      share|improve this question










      asked Nov 13 '18 at 1:02









      Chris AtkesonChris Atkeson

      143




      143






















          1 Answer
          1






          active

          oldest

          votes


















          0














          First,setState is an asynchronous function,so you can't decide when the first setState runs.So I recommend to force it run synchronous.



          this.setState(chess_board: new_board,()=> console.log(this.state.chess_board))


          Second,use a setTimeout to analogy the delay



          setTimeout(this.chess_engine_move(),3000);// delay 3s





          share|improve this answer




















          • Thank you. setTimeout is exactly the kind of thing I was looking for. Sorry new to javascript.
            – Chris Atkeson
            Nov 13 '18 at 14:13










          Your Answer






          StackExchange.ifUsing("editor", function ()
          StackExchange.using("externalEditor", function ()
          StackExchange.using("snippets", function ()
          StackExchange.snippets.init();
          );
          );
          , "code-snippets");

          StackExchange.ready(function()
          var channelOptions =
          tags: "".split(" "),
          id: "1"
          ;
          initTagRenderer("".split(" "), "".split(" "), channelOptions);

          StackExchange.using("externalEditor", function()
          // Have to fire editor after snippets, if snippets enabled
          if (StackExchange.settings.snippets.snippetsEnabled)
          StackExchange.using("snippets", function()
          createEditor();
          );

          else
          createEditor();

          );

          function createEditor()
          StackExchange.prepareEditor(
          heartbeatType: 'answer',
          autoActivateHeartbeat: false,
          convertImagesToLinks: true,
          noModals: true,
          showLowRepImageUploadWarning: true,
          reputationToPostImages: 10,
          bindNavPrevention: true,
          postfix: "",
          imageUploader:
          brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
          contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
          allowUrls: true
          ,
          onDemand: true,
          discardSelector: ".discard-answer"
          ,immediatelyShowMarkdownHelp:true
          );



          );













          draft saved

          draft discarded


















          StackExchange.ready(
          function ()
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53272282%2freact-setstate-twice-in-same-function%23new-answer', 'question_page');

          );

          Post as a guest















          Required, but never shown

























          1 Answer
          1






          active

          oldest

          votes








          1 Answer
          1






          active

          oldest

          votes









          active

          oldest

          votes






          active

          oldest

          votes









          0














          First,setState is an asynchronous function,so you can't decide when the first setState runs.So I recommend to force it run synchronous.



          this.setState(chess_board: new_board,()=> console.log(this.state.chess_board))


          Second,use a setTimeout to analogy the delay



          setTimeout(this.chess_engine_move(),3000);// delay 3s





          share|improve this answer




















          • Thank you. setTimeout is exactly the kind of thing I was looking for. Sorry new to javascript.
            – Chris Atkeson
            Nov 13 '18 at 14:13















          0














          First,setState is an asynchronous function,so you can't decide when the first setState runs.So I recommend to force it run synchronous.



          this.setState(chess_board: new_board,()=> console.log(this.state.chess_board))


          Second,use a setTimeout to analogy the delay



          setTimeout(this.chess_engine_move(),3000);// delay 3s





          share|improve this answer




















          • Thank you. setTimeout is exactly the kind of thing I was looking for. Sorry new to javascript.
            – Chris Atkeson
            Nov 13 '18 at 14:13













          0












          0








          0






          First,setState is an asynchronous function,so you can't decide when the first setState runs.So I recommend to force it run synchronous.



          this.setState(chess_board: new_board,()=> console.log(this.state.chess_board))


          Second,use a setTimeout to analogy the delay



          setTimeout(this.chess_engine_move(),3000);// delay 3s





          share|improve this answer












          First,setState is an asynchronous function,so you can't decide when the first setState runs.So I recommend to force it run synchronous.



          this.setState(chess_board: new_board,()=> console.log(this.state.chess_board))


          Second,use a setTimeout to analogy the delay



          setTimeout(this.chess_engine_move(),3000);// delay 3s






          share|improve this answer












          share|improve this answer



          share|improve this answer










          answered Nov 13 '18 at 8:25









          RootRoot

          1,513128




          1,513128











          • Thank you. setTimeout is exactly the kind of thing I was looking for. Sorry new to javascript.
            – Chris Atkeson
            Nov 13 '18 at 14:13
















          • Thank you. setTimeout is exactly the kind of thing I was looking for. Sorry new to javascript.
            – Chris Atkeson
            Nov 13 '18 at 14:13















          Thank you. setTimeout is exactly the kind of thing I was looking for. Sorry new to javascript.
          – Chris Atkeson
          Nov 13 '18 at 14:13




          Thank you. setTimeout is exactly the kind of thing I was looking for. Sorry new to javascript.
          – Chris Atkeson
          Nov 13 '18 at 14:13

















          draft saved

          draft discarded
















































          Thanks for contributing an answer to Stack Overflow!


          • Please be sure to answer the question. Provide details and share your research!

          But avoid


          • Asking for help, clarification, or responding to other answers.

          • Making statements based on opinion; back them up with references or personal experience.

          To learn more, see our tips on writing great answers.





          Some of your past answers have not been well-received, and you're in danger of being blocked from answering.


          Please pay close attention to the following guidance:


          • Please be sure to answer the question. Provide details and share your research!

          But avoid


          • Asking for help, clarification, or responding to other answers.

          • Making statements based on opinion; back them up with references or personal experience.

          To learn more, see our tips on writing great answers.




          draft saved


          draft discarded














          StackExchange.ready(
          function ()
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53272282%2freact-setstate-twice-in-same-function%23new-answer', 'question_page');

          );

          Post as a guest















          Required, but never shown





















































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown

































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown







          這個網誌中的熱門文章

          Barbados

          How to read a connectionString WITH PROVIDER in .NET Core?

          Node.js Script on GitHub Pages or Amazon S3