エンタープライズRuby on Rails ~エンプラでぶち当たった2つの壁と突破法~
Download as pptx, pdf0 likes2,036 views
関西Ruby会議2017での「エンタープライズRuby on Rails ~エンプラでぶち当たった2つの壁と突破法~」の発表で用いた講演資料です。
More Related Content
What's hot (20)
Similar to エンタープライズRuby on Rails ~エンプラでぶち当たった2つの壁と突破法~ (20)
エンタープライズRuby on Rails ~エンプラでぶち当たった2つの壁と突破法~
- 1. エンタープライズRubyOnRails Future Architect, Inc. Hiroki Tanaka エンプラでぶち当たった2つの壁と突破法 関西Ruby会議2017
- 2. 自己紹介 田中 宏基(たなか ひろき) ・ フューチャーアーキテクト株式会社 Technology Innovation Group所属 ・ 社内でのRuby on Railsのプレゼンス向上活動に 従事 ・ 茶道の師範ライセンス所持
- 3. 会社紹介
- 6. フューチャーをざっくり言うと 通 常 の I T ベ ン ダ ー と 違 い 、 コ ン サ ル か ら S I ま で 一 気 通 貫 で 行 う 。 中 立 ( ニ ュ ー ト ラ ル ) の ポ ジ シ ョ ン 。 「 な い も の は つ く る 」 の 精 神 。 ・ 自 社 サ ー ビ ス も 行 う 。
- 7. 巷ではこう言われている…
- 11. 本日お伝えしたいこと 巷 で は こ ん な こ と が 言 わ れ て い ま す が 、 実 際 に エ ン タ ー プ ラ イ ズ シ ス テ ム 開 発 が 得 意 な 会 社 で R u b y o n Ra i l s を 使 っ た 際 、 よ り 具 体 的 な 壁 に ぶ つ か り ま し た 。 そ れ が ど ん な 壁 な の か 、 ま た 、 そ の 突 破 法 を 本 日 お 伝 え し ま す 。 Enterprise
- 12. プロジェクト概要 ●中規模開発 ・ 開発期間：3.5ヶ月 ・ 機能数：777 ・ Model数：160 ・ 開発人数：最大10人 ・ ソース管理： GitLab ・ ドキュメント管理：Subversion 開発チーム ●経験者少数 ・ 経験者数：1人
- 13. 2つの壁にぶち当たった
- 15. コードのメンテナンス性低下
- 16. 経験者が不足するとどうなるか 一 貫 性 の な い コ ー ド が 量 産 さ れ る 。 コンフリクトメンテナンス性
- 17. 経験者が不足するとどうなるか 一 貫 性 の な い コ ー ド が 量 産 さ れ る 。 each文内 の深いネスト コンフリクトメンテナンス性
- 18. 経験者が不足するとどうなるか 一 貫 性 の な い コ ー ド が 量 産 さ れ る 。 each文内 の深いネスト 無駄なself コンフリクトメンテナンス性
- 19. 経験者が不足するとどうなるか 一 貫 性 の な い コ ー ド が 量 産 さ れ る 。 each文内 の深いネスト If文内に unless 無駄なself コンフリクトメンテナンス性
- 20. 経験者が不足するとどうなるか 一 貫 性 の な い コ ー ド が 量 産 さ れ る 。 each文内 の深いネスト If文内に unless If文内に 後置if 無駄なself コンフリクトメンテナンス性
- 21. 経験者が不足するとどうなるか 一 貫 性 の な い コ ー ド が 量 産 さ れ る 。 each文内 の深いネスト If文内に unless If文内に 後置if 無駄なself コンフリクトメンテナンス性
- 22. メンテナンス性が低下するとどうなるか レ ビ ュ ア ー の 負 担 が 増 加 す る 。 イケてないコード 読みづらい… わかりづらい… コンフリクトメンテナンス性 レビュアー
- 23. メンテナンス性が低下するとどうなるか レ ビ ュ ア ー の 負 担 が 増 加 す る 。 イケてないコード 読みづらい… わかりづらい… コンフリクトメンテナンス性 レビュアー
- 24. コンフリクトの多発
- 25. 中規模開発を行うとどうなるか Ra i l s の 特 性 上 、 同 一 フ ァ イ ル を 複 数 人 で 編 集 す る こ と が 多 い 。 Model 例 ： M o d e l コンフリクトメンテナンス性
- 26. 中規模開発を行うとどうなるか 同 じ フ ァ イ ル を 編 集 し た 様 々 な fe a t u r e ブ ラ ン チ が 存 在 す る 。 master Model MergeRequest featureブランチ コンフリクトメンテナンス性
- 27. 中規模開発を行うとどうなるか 各 ブ ラ ン チ か ら MergeRequestが発行される。 master Model コンフリクトメンテナンス性 MergeRequest featureブランチ
- 28. 中規模開発を行うとどうなるか M e r g e Re q u e s t の 1 つ を マ ー ジ す る 。 master Model Merge! コンフリクトメンテナンス性 MergeRequest featureブランチ
- 29. 中規模開発を行うとどうなるか 他 の M e r g e Re q u e s t で コ ン フ リ ク ト が 発 生 す る 。 master Model Merge! コンフリクトメンテナンス性 MergeRequest featureブランチ
- 30. コンフリクトが多発するとどうなるか レ ビ ュ ア ー は マ ー ジ の 順 番 や コ ー ド の 整 合 性 が わ か ら ず 、 マ ー ジ コ ス ト が 増 加 す る 。 コンフリクト多発中のMR 何が正しいのか わからない… コンフリクトメンテナンス性 レビュアー
- 31. コンフリクトが多発するとどうなるか レ ビ ュ ア ー は マ ー ジ の 順 番 や コ ー ド の 整 合 性 が わ か ら ず 、 マ ー ジ コ ス ト が 増 加 す る 。 コンフリクト多発中のMR 何が正しいのか わからない… コンフリクトメンテナンス性 レビュアー
- 32. レビュアーの負担が激増！！
- 33. レビュアーのつらみ 日 を 追 う 毎 に 重 く な る レ ビ ュ ア ー の 負 担 メ ン テ ナ ン ス 性 の 低 い コ ー ド コ ン フ リ ク ト の 多 発 レ ビ ュ ー に 時 間 が 掛 か る チ ー ム の 遅 延 の 一 因 に や っ と の 思 い で 機 能 を 実 装 何 と か 1 つ マ ー ジ す る 増 加 し て い る コ ン フ リ ク ト 更 に レ ビ ュ ー に 時 間 が 取 ら れ る 自 担 当 機 能 の 遅 延 溜 ま っ て い る レ ビ ュ ー 依 頼 コンフリクトメンテナンス性
- 34. レビュアーのつらみ 日 を 追 う 毎 に 重 く な る レ ビ ュ ア ー の 負 担 メ ン テ ナ ン ス 性 の 低 い コ ー ド コ ン フ リ ク ト の 多 発 レ ビ ュ ー に 時 間 が 掛 か る チ ー ム の 遅 延 の 一 因 に や っ と の 思 い で 機 能 を 実 装 何 と か 1 つ マ ー ジ す る 増 加 し て い る コ ン フ リ ク ト 更 に レ ビ ュ ー に 時 間 が 取 ら れ る 自 担 当 機 能 の 遅 延 溜 ま っ て い る レ ビ ュ ー 依 頼 コンフリクトメンテナンス性
- 35. 壁は想像以上に高かった
- 37. どうするか
- 40. 実行してみた
- 41. 開発統制
- 43. 各開発者の実装時 RuboCop×SublimeText3×独自チェッカーによって、 強制フォーマットと規約違反をチェック rbファイル ERBファイル SublimeText3RuboCop 独自チェッカー 開発統制 自動生成 コンフリクトメンテナンス性
- 44. 各開発者の実装時 RuboCop×SublimeText3×独自チェッカーによって、 強制フォーマットと規約違反をチェック rbファイル ERBファイル SublimeText3RuboCop 独自チェッカー 開発統制 自動生成 コンフリクトメンテナンス性
- 45. コードの強制フォーマット 使 用 す る コ ー ド イケてないコード 開発統制 自動生成 コンフリクトメンテナンス性
- 46. コードの強制フォーマット フ ォ ー マ ッ ト の 様 子 開発統制 自動生成 コンフリクトメンテナンス性
- 47. コードの強制フォーマット フ ォ ー マ ッ ト 後 開発統制 自動生成 コンフリクトメンテナンス性
- 48. コードの強制フォーマット フ ォ ー マ ッ ト 後 開発統制 自動生成 コンフリクトメンテナンス性 コード複雑度 への警告
- 49. コードの強制フォーマット フ ォ ー マ ッ ト 後 開発統制 自動生成 コンフリクトメンテナンス性 コード複雑度 への警告 無駄なself を削除
- 50. コードの強制フォーマット フ ォ ー マ ッ ト 後 開発統制 自動生成 コンフリクトメンテナンス性 コード複雑度 への警告 無駄なself を削除 unless→if への変換
- 51. コードの強制フォーマット フ ォ ー マ ッ ト 後 開発統制 自動生成 コンフリクトメンテナンス性 コード複雑度 への警告 無駄なself を削除 unless→if への変換 後置ifへの 警告
- 52. コードの強制フォーマット フ ォ ー マ ッ ト 後 開発統制 自動生成 コンフリクトメンテナンス性 コード複雑度 への警告 無駄なself を削除 unless→if への変換 後置ifへの 警告
- 53. ERBの規約チェック 使 用 す る コ ー ド 開発統制 自動生成 コンフリクトメンテナンス性
- 54. ERBの規約チェック 使 用 す る コ ー ド 開発統制 自動生成 コンフリクトメンテナンス性 ERBコメントではなく、 HTMLコメントを使用
- 55. ERBの規約チェック 規 約 チ ェ ッ ク 実 行 警告 ポップアップ 開発統制 自動生成 コンフリクトメンテナンス性
- 56. ERBの規約チェック 規 約 チ ェ ッ ク 実 行 警告 ポップアップ 開発統制 自動生成 コンフリクトメンテナンス性
- 59. パスしなかったMergeRequest マージ 不可能！ 開発統制 自動生成 コンフリクトメンテナンス性 チ ェ ッ ク を 通 ら な か っ た M e r g e Re q u e s t は 、 マ ス タ ー マ ー ジ で き な い 。
- 60. パスしなかったMergeRequest マージ 不可能！ 開発統制 自動生成 コンフリクトメンテナンス性 チ ェ ッ ク を 通 ら な か っ た M e r g e Re q u e s t は 、 マ ス タ ー マ ー ジ で き な い 。
- 62. 自動生成
- 64. こんな社内ツールが出来始めた ・リポジトリへのコミットでJenkinsジョブ起動 ・設計書を解析してDBへロード ・エクセル設計書は勿論、ER図にも対応 設計書 SVN DB ①コミット Jenkins②ジョブ起動 ③設計書DBロード 開発統制 自動生成 コンフリクトメンテナンス性
- 70. Routingについてちょっと補足 Routing分割の効果 ・可読性向上 ・一部は手メンテ可能 (gemで勝手に追加されるRoutingなど) 開発統制 自動生成 コンフリクトメンテナンス性
- 77. 様々なファイルを自動生成 (一部の例) URL・権限設計書 ER図 画面設計書 Routing permission.yml (権限定義ファイル) menu.yml (メニュー定義ファイル) Model RSpec Migration attributes.yml View item.ja.yml (画面ラベル定義ファイル) Controller ・ロジックが不要な部分は全て自動生成 凡例： 半自動生成 開発統制 自動生成 コンフリクトメンテナンス性
- 78. 自動生成の仕組み まとめ 設計書 Jenkins SVN DB Git featureブランチ Git masterブランチ 自動生成ソース 凡例： データの流れ ①コミット ③設計書情報 をDBにロード ②コミット を検知 Jenkinsジョブ ④-①DB情報を基にソースを自動生成 ④-②Gitにブランチを作成し、Commit&Push ④-③GitLabにMRを投げる ・設計書を一度DBに抽出、そこからコードを生成させる ・設計書をコミットしたら自動でMRが上がってくる仕組みを実現 GitLab管理 開発統制 自動生成 コンフリクトメンテナンス性
- 79. 開発フローをまとめると.. 設計書 Jenkins SVN DB Git featureブランチ ①コミット ③設計書情報 をDBにロード ②コミット を検知 ④-①DB情報を基にソースを自動生成 ④-②Gitにブランチを作成し、Commit&Push 独自チェッカー Git featureブランチ ⑥常にフォーマット、 チェックされながら開発 ⑦Commit&Push ⑤自動生成された ブランチをチェックアウト Git masterブランチ OK? YES ⑩Master Merge ⑨MR accept 独自チェッカー No ⑧リモートへのPushをフック、 リモート側でもRuboCopと独自 チェッカーでコードスタイルをチェック Start 開発統制 自動生成 コンフリクトメンテナンス性
- 83. 壁を突破！
- 84. まとめ 自 動 化 を 推 進 す る こ と で Ru b y o n Ra i l s は エ ン タ ー プ ラ イ ズ の 開 発 手 法 に お い て も 取 り 得 る 選 択 肢 と な る 。 経験者が少数 コードの メンテナンス性低下 開発統制 10人以上の同時開発 コンフリクト多発 自動生成
- 85. Enterprise