良いコードとは何か - エンジニア新卒研修 スライド公開｜CyberZ Developer

CyberZ CTO室のメンバーの森 (@at_sushi_at) です。 先日、株式会社サイバーエージェントの2021年度 エンジニア新卒研修でコードの品質に関する講義を行いました。 そこで話した内容とスライドを完全公開します。 45分の内容のため、かなり長いですが、個人的にぜひ一読して欲しい内容になっています。 はじめに こんにちは、森 篤史と言います。2019年度入社で今年で3年目になります。株式会社CyberZのOPENREC.tvというプロダクトでAndroidアプリチームのリーダをやっています。 最近はプログラムを書く仕事以外に、次世代マネジメント室という全社横断組織でDevelopers Blogの改善プロジェクトを実行したり、CyberZ CTO室で組織活性化に取り組んでいます。 あと、2019年度の未踏スーパークリエータにも認定されました。 メインの仕事としては、入社して

[white_trevally36]

「いざリファクタリングをするとき」なんて来るんですかねえ

[suginoy]

結合度と凝集度は基本情報で出題されるんだけどプログラミングと関連付けて理解してる人あまりいないんだよなぁ。

[joker1007]

これは良い資料だな。基本的には単一責任原則と依存関係が単一方向であること、オブジェクト指向の基本に則ってメッセージ結合とデータ結合を意識してコードを書くと、見通しが良くなる。

[t-wada]

私の「質とスピード」と技術的負債の翻訳、 @nuits_jp さんの「世界一わかりやすいClean Architecture」、 @sonatard さんの「オブジェクト指向のその前に-凝集度と結合度」をベースにわかりやすくまとめている研修資料

[NOV1975]

一般論としては正しいけど、金科玉条としてしまうとできないエンジニアになりかねない。

[yamadar]

技術的負債、普段はレビューで防いでるけど気付くと溜まっていて、時々一気に手を入れてる。一気に手を入れなくて済むよう計画的に返済したい...

[ledsun]

凝集度と結合度を分類するのはじめて知りました。参考文献も読んでみます。

[ko-ya-ma]

主に凝縮度と結合度についてわかりやすくまとまってる

[calpo]

凝集度、結合度について。単一責任の原則にも。

[masuda220]

技術的負債の発生理由のまとめが参考になった。ドメイン知識の問題の意図的な負債・変化による負債・学びによる負債。変化による負債は「ドメイン知識の変化」より「事業環境や事業活動の変化」という感じかな。

[shingo-sasaki-0529]

凝集度と結合度ちょっと誤解してた節あったから読んでよかった。 こういうのは新卒よりある程度経験してからの方が刺さると思うけど、サイバーの新卒は大抵の企業のミドルエンジニアより優秀か。

[hi-hats]

たしかに最後に参考資料として出典をまとめてあるんだが、読んでいてずっとこれはアレだよな、、、次はアレか、、と確信が強くなっていくのがモヤる内容だった。

[sha51]

すごい 漠然と思ってたのここに全部書いてあった

[edo_m18]

めちゃめちゃいい資料だった・・。同意しかない・・。

[heguro]

FEのいい復習

[razokulover]

凝集度と結合度のレベル別で名前付いてたの知らなかった

[ebo-c]

結合度(e.g.スタンプ結合)の分類のような凝集度の分類を初めて知った

[temtan]

こっちに詳しい解説付であった。

[daaaaaai]

いい

[ginpei]

良い資料。書き直すだけのリファクタリングは無意味、良いコードについて考えよ。綺麗にする「後で」は来ない、高速開発のために品質を保つ、凝集度と良いモジュール、結合度と良い関係、境界と依存方向。

[ys0000]

元となった参考文献も読んでみようと思う。

[pmint]

いまだに答えの出せない無駄議論。品質とスピードの対立はちょっとおかしい。考えるなら要求対スピード。品質に選択幅なんて無いし、ユーザーに影響しない品質も無い。凝集度の評価は「適切な名前か」でいい。

[efcl]

凝集性と良くないコードの言語化

[engisuke]

ボブおじさんの本のエッセンスが抽出されていて、入門にいい。凝縮度や結合度は各レベルの呼び名をすぐ忘れるので、サンプル助かる。

[nilab]

「先日、株式会社サイバーエージェントの2021年度 エンジニア新卒研修でコードの品質に関する講義を行いました。そこで話した内容とスライドを完全公開します」

[monster-energy-zx14]

良さげ

[katzchang]

“良いコードが何かを知らなければ、たとえ書き直したとしても良いコードにならない” それも含めてリスクテイクなのだ（複雑！）

[diveintounlimit]

一部手段の目的化に見えるが新卒向けには悪くない資料かもしれない？

[getcha]

相当大規模なリファクタリング回やったのは1回かな。もう一回は要件把握して全て作り直した。整理された汎化された例が現実に当てはまる事はまれだよ。盗めそうなアイディアを盗む程度で。良いコードは要件で違う。

[nonsugertan]

こういうことをしっかりと言語化できて、チームに落とし込めるスペシャルワンがいるのは素直に凄い。

[gyochan]

良い悪いはわからないけど、mM7とか#11thみたいな、オーギュメントが絡んだコードが好きですね

[phoope]

いいこと言う

[Nyoho]

いい発表だなー。凝集度、結合度、実際になおす例もある。

[m_yamashita]

これはすごい…。よくまとまってる。

[dekokun]

凝集度、結合度のレベルわけ知らんかったな〜

[advancive61]

"その「後で」はこない"

[taruhachi]

品質より速度を優先して後で問題になるケース、単一のシステムではなく、複数のシステムの結合の時によく発生する。べき論ではAのシステムを改修すべきだが、調整に時間がかかるのでBを改修して対応するとか。

[t1mvverr]

新しい技術を使うことでより良いコードになるどころか、ドメイン知識不足で書かれたコードや過剰に抽象化したコードとかが散りばめられ、ゴミが完成したのであった。

[khtokage]

これは素晴らしい。というか私もなんとなくで理解してた部分がちょくちょくあって反省した。

[Balmaufula]

担当してるOPENREC.tv自体はアプリは悪くもないが運営とかのせいか配信アプリとしてはパっとせんよなと関係ないこと思いました