大規模プロジェクトに学ぶ、新卒エンジニアの教訓

⼤規模プロジェクトに学ぶ、新卒エンジニアの教訓⼭路直⼈ Fusic Tech Live Vol.13 Rubyistの集いVol.2

⾃⼰紹介 • Fusic CO.LTD • 技術開発第⼆部⾨エンジニア • 学部卒新卒2年⽬ •
Ruby ⼭路直⼈ ( やまじなおと ) @naoyama_ruby @NaotoCoding

3 構成 1. プロジェクトの状況 2. 困ったこと 3. 僕なりの解決⽅法 4. 今後やりたいこと・まとめ

1研修後、プロジェクトに参加

研修後にプロジェクトに参加 • 稼働開始から10年弱経つ⼤規模アプリケーション • 8ヶ⽉間に及ぶ⼤規模改修 • メイン開発者︓開始から2ヶ⽉間は1名(僕)、以後1名ジョイン • 技術相談役兼レビュワーとしてシニアエンジニア2名状況

2 困ったこと

困ったこと⼀覧 • 仕様が分からない • コーディング能⼒不⾜ • チーム開発が経験したい • (いわゆる)レガシーコード、技術スタック

困ったこと１ • 現⾏の仕様 • 追加機能の仕様仕様が分からない

困ったこと１ - 仕様が分からない • 参加時の説明で細部まで網羅することは難しい • ましてや今回は稼働10年弱のアプリケーション ◦ 僕、中学⽣だなぁ、⾃分の事も全部覚えてない… 現⾏の仕様

困ったこと１ - 仕様が分からない • フワッとした仕様決定の状態でプロジェクトが開始する場合もある • 知らないアプリケーション上に追加機能の仕様よしなに実装は難しい追加機能の仕様え、、、そんな仕様だったの、、、︖

困ったこと２ • 仕様は分かるけど、、、 ◦ 現⾏コードを読んでも理解できない ◦ 追加機能を仕様通りに実装できないコーディング能⼒不⾜

困ったこと３ • 研修からチーム開発の経験がない ◦ 別のプロジェクトでも今後確実に経験するもの ◦ チーム開発に早めに慣れたかったチーム開発がしたい

困ったこと４ • あれ、このコード、良くないよね、、、︖ ◦ 研修ではタブーとされた記法の存在 ◦ 現⾏コードの記法に従うべきなのか、、、レガシーコード、技術スタック

困ったことの解決が僕の目標！

このプロジェクトにおける僕の⽬標 • 仕様を深く理解 • コーディング能⼒向上 • チーム開発の経験 • レガシーコード改善

3 僕なりの解決策

解決策⼀覧 • ⾊々聞きまくってコードを読め、書け︕ • ペアプロをやろう︕ • 研修で学んだことを活かせ︕

解決策１色々聞きまくってコードを読め、書け！

解決策１⾊々聞きまくってコードを読め、書け︕ 効⼒仕様の理解コーディング能⼒向上チーム開発の経験レガシーコード改善

解決策１ • きっかけ ◦ プロジェクト開始時、コードを読めば仕様が分かるのではと粘った結果、何も分からずに時間浪費 ◦ このままじゃ駄⽬だ︕先輩を頼ろう︕ ⾊々聞きまくってコードを読め、書け︕

解決策１ • やったこと ◦ コード、仕様が15分かけて理解できなかったら聞いた︕ ◦ ⼤規模だからこそ⾊んなコードをどんどん読んだ︕ ▪ そしてその後どんどん聞いた︕ ⾊々聞きまくってコードを読め、書け︕

解決策１ • 結果 ◦ 時間の浪費が激減︕ ◦ 様々な実装⽅法を学習 ▪ メタプログラミング ▪
ビット演算⾊々聞きまくってコードを読め、書け︕

解決策２ペアプロをやろう！

解決策２ペアプロをやろう︕ 効⼒仕様の理解チーム開発の経験レガシーコード改善コーディング能⼒向上

解決策２ • きっかけ ◦ チーム開発をしたいとPMに相談 ▪ 開発初期はメイン実装者が僕⼀⼈ペアプロをやろう︕

解決策２ • やったこと(やってもらったこと) ◦ 週⼀⽇⼆時間ほどプロジェクトマネージャー(PM)とペアプロ ◦ ドライバー︓新卒 ◦ ナビゲーター︓PM ペアプロをやろう︕

解決策２ • 結果１ ◦ これまでにないレベルでコードに集中する経験 ▪ ⾃分ってこんなスピードでコード書けるんだ、、、ペアプロをやろう︕

解決策２ • 結果２ ◦ リアルタイムのアドバイスがとにかく嬉しい ▪ 実装の記憶が残る︕ ▪ いつでも相談できる環境
▪ 難しい実装に⽴ち向かいやすいペアプロをやろう︕ 転けてもすぐに⽴ち上がれる

解決策２ • 結果３ ◦ 上司との距離感が近くなる ▪ 今後質問がしやすくなる︕ ペアプロをやろう︕ 流⽯にこの距離感ではないよ︕

解決策２ • 注意点 ◦ ナビゲータが意⾒するのはドライバーの実装が⼀段落ついた時 • 理由 ◦ 新卒にとっては全てのコーディングが学び ◦
不合格のコードでもそれを書くために⾊々調べて学ぶこともあるペアプロをやろう︕

解決策３研修で学んだことを活かせ！

解決策３研修で学んだことを活かせ︕ 効⼒仕様の理解チーム開発の経験レガシーコード改善コーディング能⼒向上

解決策３ • きっかけ ◦ 既存コードの修正、、、怖い、、、 ◦ 研修で学んだことが活かせるのでは︖︖ ▪ テスト ▪
コードのリンター ▪ リーダブルコード研修で学んだことを活かせ︕

解決策３ • やったこと ◦ 積極的なテスト(RSpec)の導⼊ ▪ 既存コードの修正時にはテストコード記述 ◦ 既存コードの転⽤時にはリーダブルコードに修正 ◦
新規ファイルに対してコードのリンター(RuboCop)を適⽤研修で学んだことを活かせ︕

解決策３ • 結果１ ◦ 積極的なテスト導⼊の結果 ▪ 変更しやすいアプリケーション ▪ 仕様の理解が深まる ▪
テストコード作成能⼒の向上研修で学んだことを活かせ︕

解決策３ • 結果２ ◦ RuboCop、リーダブルコードを適⽤した結果 ▪ 理解しやすいコードが増えた ▪ 今後も理解しやすいコードが増えていく可能性⼤研修で学んだことを活かせ︕

4 今後やりたいこと・まとめ

今後やりたいこと • RuboCop ◦ 新規ファイルにのみ⾃動で適⽤できるように設定 ◦ 可能な限りコード不正検出を減らしたい保守しやすいアプリにしたい

今後やりたいこと • ドキュメント ◦ 本来なら開発中に作るべきであった ◦ 同アプリを触る⼈のためにもドキュメントを整備したい保守しやすいアプリにしたい

まとめ • ⾊々聞きまくってコードを読め、書け︕ • ペアプロをやろう︕ • 研修で学んだことを活かせ︕

ご清聴ありがとうございました。

More Decks by なおと