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

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

ただいまの
回答率

90.87%

  • JavaScript

    14759questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • PostgreSQL

    951questions

    PostgreSQLはオープンソースのオブジェクトリレーショナルデータベース管理システムです。 Oracle Databaseで使われるPL/SQLを参考に実装されたビルトイン言語で、Windows、 Mac、Linux、UNIX、MSなどいくつものプラットフォームに対応しています。

  • React.js

    654questions

    Reactは、アプリケーションのインターフェースを構築するためのオープンソースJavaScriptライブラリです。

  • Spring Boot

    414questions

    Spring Bootは、Javaのフレームワークの一つ。Springプロジェクトが提供する様々なフレームワークを統合した、アプリケーションを高速で開発するために設計されたフレームワークです。

  • Redux

    89questions

propsの'innId'がreducerに渡されない件です。

受付中

回答 0

投稿 編集

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

Saggitarie

score 2

 前提・実現したいこと

ここに質問の内容を詳しく書いてください。
(例)JavaScript(React/Redux)とSpring MyBatisで民泊サイト的なシステムを作っています。
クライアントサイド側で入力された情報をDBに挿入する機能を実装中に以下のエラーメッセージが発生しました。

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

Uncaught TypeError: Cannot read property 'innId' of undefined

 該当のソースコード(1部抜粋)

民泊の絞り込み検索をする機能(components/InnList)

const InnList = (props) => {
    const {classes} = props;
    return (
        <List>
            <p>施設レビュー数…施設利用者による施設レビューの掲載数です。</p>
            <p>第三者評価数…行政や企業などの認可された団体がおすすめしているコメントの掲載数です。</p>
            {props.inns.filter((inn) => inn.prefecture.id === props.prefecture).map((inn) => {
                return(
                    <ListItem key={inn.id} divider={true}>
                        <Avatar alt='Remy Sharp' src={inn.url}/>
                        <ListItemText primary={inn.name} secondary={inn.address}/>
                        <ListItemText primary={'最大宿泊人数'} secondary={inn.numOfPeople + '人'}/>
                        <ListItemText primary={'値段'} secondary={inn.price + '円'}/>
                        <ListItemText primary={'施設レビュー数'} secondary={inn.countReview + '件'}/>
                        <ListItemText primary={'第三者評価数'} secondary={inn.countEvaluation + '件'}/>
                        <Button
                            variant='raised'
                            color='primary'
                            className={classes.button}
                            onClick={(event) => {
                                props.openMessage(inn.id)
                            }}>
                            この施設を予約
                        </Button>
                    </ListItem>
                );
            })
            }
        </List>
    );
};
export default withStyles(styles)(InnList);


reservation/actionCreators

export const openMessage = (innId) => {
    return {
        type: ACTION_MESSAGE_OPEN,
        innId: innId
    };
};

reservation/actionTypes

/*
 * メッセージダイアログに関するアクションタイプ
 */

export const ACTION_MESSAGE_OPEN = 'ACTION_MESSAGE_OPEN';

reservation/reducers

    //ユーザ情報の一時格納
    userInfo: {
        username: {
            value: '',
        },
        name: {
            value: '',
            isError: false,
        },
        innId:{
            value:'',
        },
        numOfPeople: {
            value: '',
            isError: false,
        },
        tel: {
            value: '',
            isError: false,
        },
        checkIn: {
            value: '',
            isError: false,
        },
        checkOut: {
            value: '',
            isError: false,
        },
        comment: {
            value: '',
            isError: false,
        }
    },
};


export const openMessage = (state, action) => {
    return Object.assign({}, state, {
        messageStatus: true,
        userInfo: Object.assign({}, state.userInfo, {
            innId: Object.assign({}, state.userInfo.innId, {
                innId: action.innId}
                )
        })
    })
}

(合計3つを統合する為の)./reducers

/* Redux関連 */
import {combineReducers} from 'redux';
/* リデューサー */
import accountReducer from './account/reducers';
import innsReducer from './inns/reducers';
import reservationReducer from './reservation/reducers';

/* リデューサーを統合して公開 */
export default combineReducers({
    account: accountReducer,
    inns: innsReducer,
    messagedialog: reservationReducer
});

containers

/**
 * MessageDialog
 */
export const MessageDialog = connect(
    (state) => ({
        account: state.app.account,
        inns: state.app.inns.inns,
        message: state.app.messagedialog
    }),
    (dispatch) => {
        const messageActionCreators = bindActionCreators(_messageActionCreators, dispatch);
        const jumpTopPage = bindActionCreators(push,dispatch);
        return {
            closeMessage: messageActionCreators.closeMessage,
            closeConfirmation: messageActionCreators.closeConfirmation,
            openConfirmation: messageActionCreators.openConfirmation,
            openComp: messageActionCreators.openComp,
            closeComp: messageActionCreators.closeComp,
            jumpTopPage: jumpTopPage,
            userUpdate: messageActionCreators.userUpdate,
            postReservation: messageActionCreators.postReservation,
            openMessage: messageActionCreators.openMessage
        }
    }

)(_MessageDialog);
class MessageDialog extends React.Component {

    constructor(props) {
        super(props);

        this.state = {

            open: false,
            userInfo: {
                username: {
                    value: '',
                },
                name: {
                    value: '',
                    isError: false,
                },
                innId:{
                    value:'',
                },
                numOfPeople: {
                    value: '',
                    isError: false,
                },
                tel: {
                    value: '',
                    isError: false,
                },
                checkIn: {
                    value: '',
                    isError: false,
                },
                checkOut: {
                    value: '',
                    isError: false,
                },
                comment: {
                    value: '',
                    isError: false,
                }
            },
        };
    }


    handleNameChange (event) {
        const name = event.target.value;
        const userInfo = Object.assign({}, this.state.userInfo, {
            name: Object.assign({}, this.state.userInfo.name, {
                value: name,
                isError: !event.target.validity.valid
            })
        });
        this.setState({userInfo: userInfo});
    }
    handleNumChange (event) {
        const numOfPeople = event.target.value;
        let error = false;
        if (numOfPeople === '')
            error = true;

        const userInfo = Object.assign({}, this.state.userInfo, {
            numOfPeople: Object.assign({}, this.state.userInfo.numOfPeople, {
                value: numOfPeople,
                isError: error
            })
        });
        this.setState({userInfo: userInfo});
    }
    handleTelChange (event) {
        const tel = event.target.value;
        const userInfo = Object.assign({}, this.state.userInfo, {
            tel: Object.assign({}, this.state.userInfo.tel, {
                value: tel,
                isError: !event.target.validity.valid
            })
        });
        this.setState({userInfo: userInfo});
    }
    handleCheckInChange (event) {
        const checkIn = event.target.value;
        const userInfo = Object.assign({}, this.state.userInfo, {
            checkIn: Object.assign({}, this.state.userInfo.checkIn, {
                value: checkIn,
                isError: !event.target.validity.valid
            })
        });
        this.setState({userInfo: userInfo});
    }
    handleCheckOutChange (event) {
        const checkOut = event.target.value;
        const userInfo = Object.assign({}, this.state.userInfo, {
            checkOut: Object.assign({}, this.state.userInfo.checkOut, {
                value: checkOut,
                isError: !event.target.validity.valid
            })
        });
        this.setState({userInfo: userInfo});
    }
    handleCommentChange (event) {
        const comment = event.target.value;
        const userInfo = Object.assign({}, this.state.userInfo, {
            comment: Object.assign({}, this.state.userInfo.comment, {
                value: comment,
            })
        });

        this.setState({userInfo: userInfo});
    }

    static getDerivedStateFromProps(props, state) {
        if (props.message.userInfo.innId.value !== '') {
            return Object.assign({}, state, {
                userInfo: Object.assign({}, state.userInfo, {
                    innId: Object.assign({}, state.userInfo.innId, {
                        value: props.message.userInfo.innId.value
                    })
                })
            })
        }
        if (props.account.claims !== null && props.account.claims.sub !== '') {
            return Object.assign({}, state, {
                userInfo: Object.assign({}, state.userInfo, {
                    username: Object.assign({}, state.userInfo.username, {
                        value: props.account.claims.sub
                    })
                })
            })
        }
        return state;

    };

 試したこと

・Google ChromeのDevToolsを使ってエラーがどのタイミングで発生するのか確認しました。
1)InnListコンポーネントで「この施設を予約する」ボタンを押すと、openMessage関数が引数「inn.id(例えば3)」を持った状態で呼び出されます。
*innsはサーバーサイドからfetch関数で受け取った配列です。

2)/reservation/actionCreatorsのopenMessage関数に引数としてinnIdに「3」という数字が渡されている。

3)/reservation/reducersのデバック時にactionとして渡されているはずの「innId」がなぜかundefinedになっている。

* 「redux.js」というクラスの  

try {
      isDispatching = true;
      currentState = currentReducer(currentState, action);
    } finally {
      isDispatching = false;
    }


の時点ではまだcurrentStateとして「type:ACTION_MESSAGE_OPEN, innId:3」は存在します。この後reducersに移る時、なぜかactionがundefinedになってしまいます。

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

propsのinnIdが認識されないのがなぜか解明したいです。
足りないコードがあれば言ってください。よろしくお願いします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正の依頼

  • jun68ykt

    2018/06/12 20:32

    こちらのご質問、すでに解決されましたでしょうか?「このあたりが怪しいのでは?」と思うところが無くはないです。

    キャンセル

まだ回答がついていません

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

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

関連した質問

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

  • JavaScript

    14759questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • PostgreSQL

    951questions

    PostgreSQLはオープンソースのオブジェクトリレーショナルデータベース管理システムです。 Oracle Databaseで使われるPL/SQLを参考に実装されたビルトイン言語で、Windows、 Mac、Linux、UNIX、MSなどいくつものプラットフォームに対応しています。

  • React.js

    654questions

    Reactは、アプリケーションのインターフェースを構築するためのオープンソースJavaScriptライブラリです。

  • Spring Boot

    414questions

    Spring Bootは、Javaのフレームワークの一つ。Springプロジェクトが提供する様々なフレームワークを統合した、アプリケーションを高速で開発するために設計されたフレームワークです。

  • Redux

    89questions