質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.92%

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

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 180

we_puee

score 1

前提・実現したいこと

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

こんにちは

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

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

補足

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

const members = [
  { name: "たけし", pct: "https://dummyimage.com/100x100/1e00ff/fff.png&text=img0" },
  { name: "ももこ", pct: "https://dummyimage.com/100x100/ff0000/fff.png&text=img1" },
  { name: "ゆうた", pct: "https://dummyimage.com/100x100/059100/fff.png&text=img2" },
  { name: "えみこ", pct: "https://dummyimage.com/100x100/e000e0/fff.png&text=img3" }
];

const AppHeader = ({ selectedIndex, onChange }) => (
  <div>
    <img src={members[selectedIndex].pct} />
    <select name="slcGiv" onChange={onChange}>
      {members.map((member, i) => 
         <option key={`member-${i}`} value={i} selected={i === selectedIndex}>
          {member.name}
        </option>)
      }      
    </select>
  </div>
);

class App extends React.Component {
  constructor() {
    super();
    this.state = { selectedIndex: 0 };
  }

  handleChange = evt  => {
    this.setState({ selectedIndex: +evt.target.value });
  }

  render() {
    return (
      <div className="container">
        <AppHeader
          selectedIndex={this.state.selectedIndex}
          onChange={this.handleChange}
        />
      </div>
    );
  }
}

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

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/07/14 09:58

    jun68yktさん、非常に丁寧にご回答いただき、ありがとうございます。無事解決致しました。
    私はまだ初学者で、補足で頂いた一例には理解出来ない点も多いのですが、いつかこのような綺麗なコードが書けるように日々頑張りたいと思います。
    ありがとうございました。

    キャンセル

  • 2020/07/14 12:19

    どういたしまして😀
    > 無事解決致しました。
    とのことでよかったです👏

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.92%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る