【redux】reduxのベストプラクティス6選〜Action編〜

# tl;dr
ReduxのActionに関するベストプラクティスを
本家のdocumentやdiscussion含め日英の文献にあたりながら模索してみました。

1. Action名は「システムが行うこと」ではなく「実際に起こったこと」を書く
1. Actionのフォーマットは、Fluxスタンダードにあったものを使う
* APIのデータはnormalizrで正規化する
* 「基本的に」データの加工はActionで完結させる
* ActionがFatになりすぎたら①〜副作用がないデータ加工はReducerでやる〜
* ActionがFatになりすぎたら②〜redux-sagaに非同期処理を書き、分ける〜

**※※※ 特に後者3つに関して、コメントでご指摘を頂いておりますのでそちらも必ずご参照ください。**

## まえがき
* 筆者はReactを使っていますが、Vue等他のライブラリを仕様した際も転用可能な議論かなと思います。
* 冒頭にも書きましたが、「ベスト・プラクティス6選」などとでしゃばってしまいましたが、まだ使いたてですので、それは違うでしょ、というところがあればじゃんじゃん批判してくださいまし。。m(_ _)m
* [markerikson/react-redux-links](https://github.com/markerikson/react-redux-links/blob/master/redux-architecture.md)に、かなりの数のReduxのアーキテクチャに関する良記事が載っているので、もしよければご参考くださいませ。
* もし時間と需要があれば、Reducer/Containerなどに関してもポストしたいなと思っております

# Action名は「システムが行うこと」ではなく「実際に起こったこと」を書く。

## よくある課題
Actionの命名に、説得力があるルールが見つけられず、カオスる。

## 解決策
Action名は「システムが行うこと」ではなく「外の世界で実際に起こったこと」を書く。

## 詳細
ReduxのAction名は、「システムが行うこと」ではなく「外の世界で実際に起こったこと」をベースに命名します。

> The only way to change the state tree is to emit an action, an object describing what happened.  [(Redux公式より)](https://github.com/reactjs/redux#the-gist)

>  Every action only describes what happened in the outside world (user wants to do this, server said that) [(reduxコミッターのdtinthさんより)](https://github.com/reactjs/redux/issues/1171#issuecomment-167615246)

この議論に乗っ取ると、例えばSNSなどのサービスで、あるユーザーがコメントを投稿する場合のAction名は、以下のようになります。

```
(x) CREATE_COMMENT
(o) POST_COMMENT
```
またサーバーからAjax通信をしたときレスポンスも、
「外の世界にあるサーバーが、特定のレスポンスを返した（という事実が起きた）」、
と解釈することができます。

したがって、Ajaxを使ったコメントを取得に成功/失敗した場合ですと、それぞれ

```
(When on Success) FETCH_COMMENT_SUCCESS
(When on Failure) FETCH_COMMENT_FAILURE
```
というアクション名を当てはめることができます。

この考え方に従えば、Actionの命名ばブレにくくなり、
保守性が向上します。

なお、余談ですがActionTypeはで以下のように定義するのが慣例となっております。

```js
export const EXAMPLE_ACTION = 'EXAMPLE_ACTION';
```

こちらについては、[ReducingBoilerplate](http://redux.js.org/docs/recipes/ReducingBoilerplate.html)をご参考ください。

# Actionのフォーマットは、Fluxスタンダードにあったものを使う

## よくある課題
ActionのフォーマットがActionごとに変わり、Reducer側で混乱をきたす。

## 解決策
Actionのフォーマットは、Fluxスタンダードを使う。

## 詳細
Actionのフォーマットは、Reducer側でActionの形のランダムさに混乱しないためにも、
下記のようなフォーマットで統一します。

```js
{
  type: FOO_TYPE,      // must アクションタイプ
  payload: {object},   // optional 主なデータ
  meta: {object},      // optional　追加情報
  error: false, true, undefined, null, ... // optional エラーかどうか
}
```

決まったフォーマットを用いることで、Reducer側を書く際に
Actionの形をわざわざ毎回確認する必要がなくなります。

また、FluxスタンダードのActionをサポートするための、
[redux-actions](https://github.com/acdlite/redux-actions)というライブラリもあります。

* (もちろん,payload/metaの中身は各アクションごとに変わりますので、
TypeScriptやFlowで細かく型指定してあげるのが良いのかもしれません。)
* (もしこちらのうまいやり方をご存知でしたらコメントください、、><)

詳しくは、[こちらの記事](http://qiita.com/yasuhiro-okada-aktsk/items/a14f7f37262fb6cf0bf8#action-%E3%81%AE%E6%A7%8B%E9%80%A0)にわかりやすく書かれておりますので、参照してください。

# APIのデータはnormalizrで正規化する

## よくある課題
WebAPIから受け取ったデータをStoreに保存する際に、保存の仕方が属人化してしまう。

## 解決策
[normalizr](https://github.com/paularmstrong/normalizr)を使って、レスポンスデータを標準化する。

## 詳細
SPAを開発していると、WebAPIから受け取ったデータが下の様に
しばしば便宜のためにentityやVOがネストされた状態で送られてくる場合があるかと思います。

```json
{
  "id": "123",
  "author": {
    "id": "1",
    "name": "Paul"
  },
  "title": "My awesome blog post",
  "comments": [
    {
      "id": "324",
      "commenter": {
        "id": "2",
        "name": "Nicole"
      }
    }
  ]
} 

(公式より)
```

このようにネストされたオブジェクトを以下のように標準化してくれるのがnormalizrになります。

```js
{
  result: "123",
  entities: {
    "articles": { 
      "123": { 
        id: "123",
        author: "1",
        title: "My awesome blog post",
        comments: [ "324" ]
      }
    },
    "users": {
      "1": { "id": "1", "name": "Paul" },
      "2": { "id": "2", "name": "Nicole" }
    },
    "comments": {
      "324": { id: "324", "commenter": "2" }
    }
  }
}

(公式より)
```

normalizrを使った標準化をActionでAPIからデータを受け取った時点でかけておくと、
Reducerでのデータの格納、及び[reselect](https://github.com/reactjs/reselect)などでデータを取り出すときに非常に楽になりますし、なによりも、どういう形で保存をしようかと悩む必要がなくなるので、スピーディーにコーディングができます。

ただ、normalizrは公式に乗ってこそはいるものの、
今後Entityの扱い方のProtocolがFluxスタンダートとして決まった場合、
大幅にコードの変更が必要になりそうなので、注意が必要かなと思いました。

なお、normalizrを使った後に、reducerでデータをStoreに格納する際に
[こちらのTODOアプリの例](https://github.com/hokuma/redux-normalizr-todo)が非常に参考になりました。immutablejsも使っておりますが、こちらを見ればnormalizrで実装する際のインスピレーションがかなり得れるかと思います。


# 「基本的に」データ加工は、Actionで完結させる(要検討)

## よくある課題
Action側でデータをどこまで加工してReducerに渡せばいいのかわからない。

## 解決策
データ加工は基本的にAction側で行ってしまい、
Reducerは単純に受け取ったデータをstoreに格納するだけの"Dummy(馬鹿な) reducer"にとどまらせる。

## 備考
Viewから受け取ったデータをStoreに格納する上で、
高い確率でどこかのタイミングでそのデータを加工する必要が生じます。

WEBAPI経由で非同期で加工される場合、
Formに記入してもらったデータの消し込みなどbrowser内でできる場合
など多種多様なデータ加工があるなか、
基本的にはその全てをActionで加工するのがきれいかなと考えました。

加工する場所の選択肢として、

* View(Container/Component)
* Action
* Reducer

が挙げられますが、Viewはそもそもユーザーへのデータの表示と
ユーザーからのデータの受取が責任のためお門違い、
Reducerが純粋である必要があるため、副作用も扱える
Actionですべてデータを加工するのがいいのかな、と思いました。

ただ、全てActionに書くと、
MVCのFat Model病のようにFat Action化してしまいます。
そのために、大きくなってき場合は、下のように
一部の処理をReducerとredux-sagaに切り分けるのが現実的かなと思っております。

# ActionがFatになりすぎたら①〜副作用がないデータ加工はReducerでやる〜

## よくある課題
上に従って、Actionにできるだけ全部ぶっこんでいったら、肥大化してわけわかめになる。

## 解決策
副作用がないデータ加工であれば、Reducerでやってしまう。

## 詳細

> The reducer is a pure function that takes the previous state and an action, and returns the next state.[(redux公式より](http://redux.js.org/docs/basics/Reducers.html)

ReducerはReduxのドキュメントにもある通り、純粋である必要が有ります。
が一方、純粋であるという条件さえ満たせれば、
Actionで書いていたデータ加工ロジックの一部を肩代わりすることも可能になります。

繰り返しになりますが上述したとおり、純粋か副作用ありかでデータ加工ロジックを書く場所を変えてしまうのは、関心の分離的に正しくないかなあともおもっておりますので、
臨機応変にしていく必要がありそうです。

詳しくは[こちら](https://github.com/reactjs/redux/issues/1171#issuecomment-205888533)を参照してください。


# ActionがFatになりすぎたら②〜redux-sagaに非同期処理をかき分ける〜

## よくある課題
上に従って、Actionにできるだけ全部ぶっこんでいったら、肥大化してわけわかめになる

## 解決策
redux-sagaにデータ加工ロジックを書き、分ける

## 詳細
reducerには純粋な処理を切り分けることができましたが、
redux-sagaを使う場合は、sagaにそのまま純粋・副作用がある処理の両方を任せることができます。

この場合ですと、stupid actions/ stupid reducers / wise saga
と言うかたちで、かなりきれいに住み分けができる様になります。

欠点としては当たり前ですがデータ加工の記載箇所が、
sagaにもろ依存してしまうところがあるのと、
sagaがfatになる可能性が拭えないことかなと思いました。

sagaに関しては、[redux-sagaで非同期処理と戦う](http://qiita.com/kuy/items/716affc808ebb3e1e8ac)に詳しく書かれておりますので、
ご覧になってください。


## 参考

* [Recommendations for best practices regarding action-creators, reducers, and selectors](https://github.com/reactjs/redux/issues/1171)
* [Redux without profanity](https://www.google.co.jp/search?q=redux+profaint&oq=redux+profaint&aqs=chrome..69i57.4995j0j4&sourceid=chrome&ie=UTF-8)
* [Redux Documentation](http://redux.js.org/)
* [Redux best practices](https://medium.com/lexical-labs-engineering/redux-best-practices-64d59775802e)
* [React使い必見！ Immutable.jsでReactはもっと良くなる](https://en-jp.wantedly.com/companies/wantedly/post_articles/28935
http://inside.pixiv.net/entry/2015/12/19/113746)
* [FluxのActionとStoreをちゃんと分ける話](http://inside.pixiv.net/entry/2015/12/19/113746)
* [Redux: Actionのコーディング規約 と redux-actions](http://qiita.com/yasuhiro-okada-aktsk/items/a14f7f37262fb6cf0bf8#action-%E3%81%AE%E6%A7%8B%E9%80%A0)
* [redux-sagaで非同期処理と戦う](http://qiita.com/kuy/items/716affc808ebb3e1e8ac)
* [markerikson/react-redux-links](https://github.com/markerikson/react-redux-links/blob/master/redux-architecture.md)
* [immutablejs/reselect/noramlizrを使ったTODO例](https://github.com/hokuma/redux-normalizr-todo)