React.js + Flux ソースレビューのポイント
1499 ワード
オレオレFluxではなくFacebook公式に出来るだけ寄せる方針。
React.js
propとstate
- stateは最低限の利用にできているか?stateが必要以上に使われていないか?propで置き換えはできないか?
- stateをsetStateを使わず直接代入していないか(バグ)(listとか弄る時注意)
- componentのなかでpropの書き換えを行っていないか?(バグ)
- 適切にpropTypesを利用して型付けできているか
- ref,getDOMNodeを使っているか?使うべき理由があるか確認する。
- getInitialStateの中でpropsを使っていないか?使われている場合同期されないことを正しく認識して実装しているか? Props in getInitialState Is an Anti-Pattern
component
- JSXで記載されているか?
- renderは複数回呼ばれても問題を起こさないか(冪等性)
- Componentは正しく階層化されているか
- formはLinkedStateMixin使って短く書く
Flux
* Componentにビジネスロジックを書かない
* StoreにViewのロジックを書かない
* Storeのテストコードを書きましょう
* Dispatcher <-> Component はお互いを知らないはず
* Store <-> Action はお互いを知らないはず
* 通信処理はActionに寄せて通信終了の通知をDispatcherに伝えるべき
Author And Source
この問題について(React.js + Flux ソースレビューのポイント), 我々は、より多くの情報をここで見つけました https://qiita.com/iida-hayato/items/85b663741bc2c7bb28ae著者帰属:元の著者の情報は、元のURLに含まれています。著作権は原作者に属する。
Content is automatically searched and collected through network algorithms . If there is a violation . Please contact us . We will adjust (correct author information ,or delete content ) as soon as possible .