Reactにて、selectを使って画像を変更したいです。

前提・実現したいこと

reactの練習で、selectを使って人の画像を変更するプログラムを書きたいのですが、エラーが出てしまい、思うようにいきません。
プルダウンで変更すると、以下のようなエラーが出てしまいます。
恐らく、setStateが行われる関数（imgChange）まで、optionのvalueの値が渡っていないのだと思うのですが、、
どなたかアドバイスをご教授頂けないでしょうか。

ちなみにエラーメッセージは同じものが二つ出ています。

発生している問題・エラーメッセージ

Uncaught TypeError: Cannot read property 'value' of undefined
    at onChange (<anonymous>:49:38)
    at HTMLUnknownElement.callCallback (react-dom.development.js:182)
    at Object.invokeGuardedCallbackDev (react-dom.development.js:231)
    at invokeGuardedCallback (react-dom.development.js:286)
    at invokeGuardedCallbackAndCatchFirstError (react-dom.development.js:300)
    at executeDispatch (react-dom.development.js:383)
    at executeDispatchesInOrder (react-dom.development.js:408)
    at executeDispatchesAndRelease (react-dom.development.js:3401)
    at executeDispatchesAndReleaseTopLevel (react-dom.development.js:3410)
    at forEachAccumulated (react-dom.development.js:3382)

該当のソースコード

<body>
  <div id="root"></div>
  <script type="text/babel">
    (() => {

      const pcts = [
        "img0のサンプル",
        "img1のサンプル",
        "img2のサンプル",
        "img3のサンプル"
      ];
     

      function AppHeader(props) {
        return (
          <div>
            <img src={props.pct} />
              <select name="slcGiv" onChange={() => props.imgChange(this.value)}>
                <option value="0">たけし</option>
                <option value="1">ももこ</option>
                <option value="2">ゆうた</option>
                <option value="3">えみこ</option>
              </select>
          </div>
        );
      }

    

      class App extends React.Component {
        constructor() {
          super();
          this.state = {
            pcts: pcts,
            pct: "img0のサンプル"
          };
          this.imgChange = this.imgChange.bind(this);
        }

        imgChange(num) {
          const pcts = this.state.pcts.slice();
          const pos = num;
          const pct = pcts[pos]
          this.setState({
            pct: pct
          });
        }

        render() {
          return (
            <div className="container">
              <AppHeader
                pct={this.state.pct}
                imgChange={this.imgChange}
              />
            </div>
          );
        }
      }


      ReactDOM.render(
        <App />,
        document.getElementById('root')
      );
    })();
  </script>
</body>

試したこと

なにが間違っているのか分からず、手が出ません。

補足情報（FW/ツールのバージョンなど）

react javascript html/css VScode

行動規範の内容に同意します

回答1件

ベストアンサー

こんにちは

最小限の修正で済ませるとすれば、select の changeリスナーとして与えている関数を以下のように修正すればよいかと思います。

diff
1- onChange={() => props.imgChange(this.value)}
2+ onChange={evt => props.imgChange(+evt.target.value)}

補足

参考までに、ご質問に挙げられているコードを諸点でリファクタリングした一例も挙げておきます。（画像はdummyimageを使用しています。）

jsx
1const members = [
2  { name: "たけし", pct: "https://dummyimage.com/100x100/1e00ff/fff.png&text=img0" },
3  { name: "ももこ", pct: "https://dummyimage.com/100x100/ff0000/fff.png&text=img1" },
4  { name: "ゆうた", pct: "https://dummyimage.com/100x100/059100/fff.png&text=img2" },
5  { name: "えみこ", pct: "https://dummyimage.com/100x100/e000e0/fff.png&text=img3" }
6];
7
8const AppHeader = ({ selectedIndex, onChange }) => (
9  <div>
10    <img src={members[selectedIndex].pct} />
11    <select name="slcGiv" onChange={onChange}>
12      {members.map((member, i) => 
13         <option key={`member-${i}`} value={i} selected={i === selectedIndex}>
14          {member.name}
15        </option>)
16      }      
17    </select>
18  </div>
19);
20
21class App extends React.Component {
22  constructor() {
23    super();
24    this.state = { selectedIndex: 0 };
25  }
26
27  handleChange = evt  => {
28    this.setState({ selectedIndex: +evt.target.value });
29  }
30
31  render() {
32    return (
33      <div className="container">
34        <AppHeader
35          selectedIndex={this.state.selectedIndex}
36          onChange={this.handleChange}
37        />
38      </div>
39    );
40  }
41}
42
43ReactDOM.render(
44  <App />,
45  document.getElementById('root')
46);