Empathy: Let React now I'm a good boy and I won't mutate state

[lmno.games] / empathy / empathy.jsx

diff --git a/empathy/empathy.jsx b/empathy/empathy.jsx
index cd8062959ec67cab269d4e95db44c3167f0e49e2..959c3feaea8f685f42dd60af38fc6b086d8326ab 100644 (file)
--- a/empathy/empathy.jsx
+++ b/empathy/empathy.jsx
@@ -52,6 +52,20 @@ events.addEventListener("player-update", event => {
     window.game.set_other_player_info(info);
 });
 
+events.addEventListener("game-state", event => {
+  const state = JSON.parse(event.data);
+
+  for (let prompt of state.prompts) {
+    window.game.add_or_update_prompt(prompt);
+  }
+});
+
+events.addEventListener("prompt", event => {
+  const prompt = JSON.parse(event.data);
+
+  window.game.add_or_update_prompt(prompt);
+});
+
 /*********************************************************
  * Game and supporting classes                           *
  *********************************************************/
@@ -66,7 +80,7 @@ function copy_to_clipboard(id)
   document.body.removeChild(tmp);
 }
 
-function GameInfo(props) {
+const GameInfo = React.memo(props => {
   if (! props.id)
     return null;
 
@@ -83,9 +97,9 @@ function GameInfo(props) {
       >Copy Link</button>
     </div>
   );
-}
+});
 
-function PlayerInfo(props) {
+const PlayerInfo = React.memo(props => {
   if (! props.player.id)
     return null;
 
@@ -101,7 +115,7 @@ function PlayerInfo(props) {
       ))}
     </div>
   );
-}
+});
 
 function fetch_method_json(method, api = '', data = {}) {
   const response = fetch(api, {
@@ -122,9 +136,44 @@ async function fetch_put_json(api = '', data = {}) {
   return fetch_method_json('PUT', api, data);
 }
 
-class CategoryRequest extends React.Component {
+class CategoryRequest extends React.PureComponent {
   constructor(props) {
     super(props);
+    this.category = React.createRef();
+
+    this.handle_change = this.handle_change.bind(this);
+    this.handle_submit = this.handle_submit.bind(this);
+  }
+
+  handle_change(event) {
+    const category_input = this.category.current;
+    const category = category_input.value;
+
+    if (/[0-9]/.test(category))
+      category_input.setCustomValidity("");
+  }
+
+  handle_submit(event) {
+    const form = event.currentTarget;
+    const category_input = this.category.current;
+    const category = category_input.value;
+
+    /* Prevent the default page-changing form-submission behavior. */
+    event.preventDefault();
+
+    const match = category.match(/[0-9]+/);
+    if (match === null) {
+      category_input.setCustomValidity("Category must include a number");
+      form.reportValidity();
+      return;
+    }
+
+    fetch_post_json("prompts", {
+      items: parseInt(match[0], 10),
+      prompt: category
+    });
+
+    form.reset();
   }
 
   render() {
@@ -132,20 +181,21 @@ class CategoryRequest extends React.Component {
       <div className="category-request">
         <h2>Submit a Category</h2>
         <p>
-          Suggest a category to play with your friends. Don't forget to
-          include the number of items for each person to submit.
+          Suggest a category to play. Don't forget to include the
+          number of items for each person to submit.
         </p>
 
-        <form>
+        <form onSubmit={this.handle_submit} >
           <div className="form-field large">
             <input
               type="text"
               id="category"
               placeholder="6 things at the beach"
-              required pattern=".*[0-9]+.*"
-              title="Category must contain a number"
-              >
-            </input>
+              required
+              autoComplete="off"
+              onChange={this.handle_change}
+              ref={this.category}
+            />
           </div>
 
           <div className="form-field large">
@@ -160,13 +210,57 @@ class CategoryRequest extends React.Component {
   }
 }
 
-class Game extends React.Component {
+const PromptOptions = React.memo(props => {
+
+  function handle_click(id) {
+    fetch_post_json(`vote/${id}`);
+  }
+
+  if (props.prompts.length === 0)
+    return null;
+
+  return (
+    <div className="prompt-options">
+      <h2>Vote on Categories</h2>
+      <p>
+        Select any categories below that you'd like to play.
+        You can choose as many as you'd like.
+      </p>
+      {props.prompts.map(p => {
+        return (
+          <button
+            className="vote-button"
+            key={p.id}
+            onClick={() => handle_click(p.id)}
+          >
+            {p.prompt}
+            <div className="vote-choices">
+              {p.votes.map(v => {
+                return (
+                  <div
+                    key={v}
+                    className="vote-choice"
+                  >
+                    {v}
+                  </div>
+                );
+              })}
+            </div>
+          </button>
+        );
+      })}
+    </div>
+  );
+});
+
+class Game extends React.PureComponent {
   constructor(props) {
     super(props);
     this.state = {
       game_info: {},
       player_info: {},
       other_players: [],
+      prompts: []
     };
   }
 
@@ -195,6 +289,19 @@ class Game extends React.Component {
     });
   }
 
+  add_or_update_prompt(prompt) {
+    const prompts_copy = [...this.state.prompts];
+    const idx = prompts_copy.findIndex(p => p.id === prompt.id);
+    if (idx >= 0) {
+      prompts_copy[idx] = prompt;
+    } else {
+      prompts_copy.push(prompt);
+    }
+    this.setState({
+      prompts: prompts_copy
+    });
+  }
+
   render() {
     const state = this.state;
 
@@ -213,6 +320,10 @@ class Game extends React.Component {
       <p key="spacer"></p>,
       <CategoryRequest
         key="category-request"
+      />,
+      <PromptOptions
+        key="prompts"
+        prompts={state.prompts}
       />
     ];
   }