Download to read offline
大切な名前[Intro]公開版
- 1.
- 2. もくじ 1. 自己紹介 2. テーマ 3.ダメコードとは 4. コードを読む 5. 命名 6. 最後に 7. 予定 8. 会社より
- 3. 1. 自己紹介 名前 :中野 明 現在 所属 : インフィニットループ仙台支社 (1 年 ) 業務 : スマホゲーのサーバ開発 (PHP) 過去 業務 : ゲーム開発 (10 年 ) コンシューマ・スマホ・ガラケー 趣味 お菓子づくり、レザークラフト 最近ボビンレース始めました
- 4.
- 5. 2. テーマ テーマ「大切な名前」 ● レビューで名前を指定することが多い (若干過剰 ) ● なぜ、自分がそこにこだわるのか整理したい
- 6. 2. テーマ テーマ「大切な名前」 ● レビューで名前を指定することが多い (若干過剰 ) ● なぜ、自分がそこにこだわるのか整理したい 「俺は間違ってない！」と叫びたい
- 7. 2. テーマ テーマ「大切な名前」 ● レビューで名前を指定することが多い (若干過剰 ) ● なぜ、自分がそこにこだわるのか整理したい 「俺は間違ってない！」と叫びたい 今回はテーマ説明のみです
- 8.
- 9.
- 10.
- 11. 3. ダメコードとは コードの使われ方 ● 新規 (write) -----以下の作業が多数 ----- ● 修正 (read, write) ● 追加 (read, write) ● 削除 (read, delete) ● レビュー (read)
- 12.
- 13.
- 14.
- 15. 4. コードを読む コードを読むとは ? ● 処理単位での意味の理解 ● 問題になりそうな箇所を把握する ➔ (I/O,DB, GPU, マルチスレッド ) ● 自分の処理をどこに置くか
- 16. 4. コードを読む コードを読むとは ? ● 処理単位での意味の理解 ● 問題になりそうな箇所を把握する ➔ (I/O,DB, GPU, マルチスレッド ) ● 自分の処理をどこに置くか 行動のためのヒントを探す
- 17.
- 18.
- 19.
- 20.
- 21. 5. 命名 要は見やすければいいんでしょ ● 単純な名前つけてるよ ● bool を返すメソッドはIs でいいんでしょ ● やっていることは全部書いているよ
- 22. 5. 命名 要は見やすければいいんでしょ ● 単純な名前つけてるよ ➔ 意味を持たない Get/Service/Managerが氾濫してません？ ● bool を返すメソッドは Is でいいんでしょ ➔ そのメソッドはもっと大きなことしてません ? ● やっていることは全部書いているよ ➔ 説明過多になっていません？
- 23. 5. 命名 要は見やすければいいんでしょ ● 単純な名前だけで構成されている Get だけで意図を表現できる？ functionversion() { $database = DataBase::get(); $handler = $database->getHandler('environment'); $entity = $handler->getEntity(); return $entity->getVersion(); }
- 24. 5. 命名 要は見やすければいいんでしょ ● 単純な名前だけで構成されている Get だけで意図を表現できる？ functionversion() { $database = DataBase::instance(); $handler = $database->createHandler('environment'); $entity = $handler->fetchEntity(); return $entity->getVersion(); }
- 25. 5. 命名 要は見やすければいいんでしょ ● 名前以上のことをやっている bool うんぬんより検証作業の意味合いが強い classEntity { private $id; public function isEnable() { return STATUS_OK === DataBase::instance() ->createHandler('entity') ->fetch($this->id) ->status(); } }
- 26. 5. 命名 要は見やすければいいんでしょ ● 名前以上のことをやっている bool うんぬんより検証作業の意味合いが強い classEntity { private $id; public function validate() { return STATUS_OK === DataBase::instance() ->createHandler('entity') ->fetch($this->id) ->status(); } }
- 27. 5. 命名 要は見やすければいいんでしょ ● やたらと説明的 (引数被り / 冗長 ) 引数で説明できません？ function createBurgerFromBreadAndCheese( Bread $bread, Cheese $cheese ){ /* 処理いろいろ */ }
- 28. 5. 命名 要は見やすければいいんでしょ ● やたらと説明的 (引数被り / 冗長 ) 引数で説明できません？ function createBurger( Bread $bread, Cheese $cheese ){ /* 処理いろいろ */ }
- 29. 5. 命名 要は見やすければいいんでしょ ● 内部ロジックの露出 モンスターの持つ処理としてメモリ解放はどうでしょう ? classMonster { private $buffer; public function freeMemory() { Memory::free($this->buffer); } }
- 30. 5. 命名 要は見やすければいいんでしょ ● 内部ロジックの露出 モンスターの持つ処理としてメモリ解放はどうでしょう ? classMonster { private $buffer; public function destroy() { Memory::free($this->buffer); } }
- 31.
- 32.
- 33.
- 34.
- 35.
- 36.
- 37.
- 38.
- 39.
- 40.
- 41.