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

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

ただいまの
回答率

89.55%

Reactで作るtodoアプリです

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 1,643

yuki_nagahama

score 15

現在ReactにてTodoアプリケーションを作っています。

<TodoApp>
  <TodoCreator />
  <TodoList />
</TodoApp>

全体はTodoを作るアプリケーションで、その中にTodoCreatorにてinputフィールドとbuttonを設置していて、
inputに言葉を入力するとbuttonをクリックした時に全体の持つstateが増えて、TodoListにてリスト表示をしております。

完成画面です

この場合上のinputフィールドとbuttonがTodoCreatorで、
その下に表示されているリストはTodoListです。

※ここでどうしてもできない問題が発生しました。
Todolistに<Checkbox onClick={this._onChange.bind(this))} />
のように左側のチェックボックスをクリックした時にこのコンポーネントが持つ_onChangeが発火するはずなのですが、bindしても{::this._onChange}をしてもなぜか
_onChange of undifined
のエラーが出てしまうのです。。
(ちなみに、ES6構文のエラーは全て試しましたので、ロジック的にできないのかと考えております。)

理由はリストにて表示しているので、
構築した後のListにonclickをしてもデータの流れは一方通行だと聞いたことがあるので、
もしかするとそのような問題なのかなあと思っております。

完成目標では_onChange関数で、選択したリストは斜線を引くとい風にしたいのです。
どうかよろしくお願いいたします。

↓問題になっているコードです。

class TodoValue extends React.Component{

  _onDelete(i){
    console.log(i);
  }

  render(){
    return(
        <List>
          {
            this.props.todos.map(function(todo, i){
              return <ListItem key={i} primaryText={todo.item} rightIcon={<ActionInfo />} leftCheckbox={<Checkbox onClick={this._onDelete.bind(this, i)} />} />
            })
          }
        </List>
    );
  }
}

↓全文です。

import React from 'react';
import ReactDOM from 'react-dom';
import MuiThemeProvider from 'material-ui/styles/MuiThemeProvider';
import injectTapEventPlugin from 'react-tap-event-plugin';
injectTapEventPlugin();

import TextField from 'material-ui/TextField';
import RaisedButton from 'material-ui/RaisedButton';

import {List, ListItem} from 'material-ui/List';
import Subheader from 'material-ui/Subheader';
import Divider from 'material-ui/Divider';
import Checkbox from 'material-ui/Checkbox';
import Toggle from 'material-ui/Toggle';
import ActionInfo from 'material-ui/svg-icons/action/info';

import { withGoogleMap } from "react-google-maps";

const style = {
  margin: 12,
};


class TodoInput extends React.Component{

  constructor(){
    super();
    this.state = {
      value: ""
    }
  }

  _onAdd(){
    const newTodo = this.refs.inputText.getValue();
    this.props.onAdd(newTodo)
    this.setState({
      value: ""
    });
  }

  _onChange(){
    this.setState({
      value: this.refs.inputText.getValue()
    });
  }

  render(){
    return(
      <div>
        <TextField
          hintText="ここにTodoを入力"
          ref="inputText"
          value={this.state.value}
          onChange={this._onChange.bind(this)}
        />
        <RaisedButton label="追加" secondary={true} style={style} onClick={this._onAdd.bind(this)} />
      </div>
    );
  }
}

class TodoValue extends React.Component{

  _onDelete(i){
    console.log(i);
  }

  render(){
    return(
        <List>
          {
            this.props.todos.map(function(todo, i){
              return <ListItem key={i} primaryText={todo.item} rightIcon={<ActionInfo />} leftCheckbox={<Checkbox onClick={this._onDelete.bind(this, i)} />} />
            })
          }
        </List>
    );
  }
}


class TodoApps extends React.Component{

  constructor(){
    super();
    this.state = {
      todos: []
    }
    this.onAdd = this.onAdd.bind(this);
  }

  onAdd(newTodo){
    this.setState({
      todos: this.state.todos.concat({item:newTodo, status:0})
    });
  }

  render(){
    return(
      <MuiThemeProvider>
        <div>
          <TodoInput onAdd={this.onAdd} />
          <TodoValue todos={this.state.todos} />
        </div>
      </MuiThemeProvider>
    );
  }
}

ReactDOM.render(<TodoApps />, document.getElementById("content"));
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+1

this.props.todos.map((todo, i) => {
  return <ListItem key={i} primaryText={todo.item} rightIcon={<ActionInfo />} leftCheckbox={<Checkbox onClick={this._onDelete.bind(this, i)} />} />
})

と言う風に、mapに渡す関数についてアロー関数=>を使ってみてください。


【解説】

Array.property.mapに渡すコールバック関数について、その関数の実行時にthisはグローバルオブジェクト(windowやglobal)になります。そのため、mapでのコールバック関数内で相当通りのthisを使用する場合は、

  1. アロー関数=>を使う。
  2. 無名関数functionに対してbindを使って束縛する。
  3. mapに第二引数を与える。(第二引数でthisが束縛される)

のいずれかを行う必要があります。ES6以降またはTypeScriptやCoffeeScriptを使っている場合は、1.のアロー関数を使うのが短くて一番わかりやすいと思います。他のコールバック関数についても基本的にアロー関数を使うとした方が問題が起きにくいです。

参考: Array.prototype.map() - JavaScript | MDN

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/12/20 09:42

    アロー関数について詳しく教えていただきありがとうございました。
    調べてもなかなかこの解決方法には至りませんでしたので、
    1日で解決できたことすごく嬉しいです。
    本当にありがとうございました。

    キャンセル

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

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