Upgrade to Pro — share decks privately, control downloads, hide ads and more …

Yumemi.apk #6 ~ゆめみのAndroidエンジニア 日頃の成果大発表会!~ Session 2

Yumemi.apk #6 ~ゆめみのAndroidエンジニア 日頃の成果大発表会!~ Session 2

コーディング試験のレビューを2年間続けて気づいたこと

https://yumemi.connpass.com/event/243080/

7e5089310d68eca7e2ad404a1c8f2292?s=128

おかやまん

April 28, 2022
Tweet

More Decks by おかやまん

Other Decks in Programming

Transcript

  1. コーディング試験のレビューを 2年間続けて気づいたこと Android 採用WG

  2. 軽く自己紹介タイム

  3. 内容 1. コーディング試験(1分) 2. コーディング試験の歴史(4分) 3. レビューで気づいたこと(5分)

  4. 1. コーディング試験

  5. 絶賛、公開中!

  6. コーディング試験の内容 メチャクチャなコードがあるので それを改善してください という内容です ざっくりいうと

  7. ゆめみ側で GitHub のリポジトリを検 索するアプリを既に作成しています。 しかし、このアプリとコードには、改善 可能な部分がたくさんあります。 アプリ仕様を守りながら、課題に従っ て改善していってください。

  8. 2. コーディング試験の歴史

  9. ~ 2020-03-24 • http://todomvc.com のような TODOアプリを作成 • 期限は1週間以内 • 非公開

  10. 課題点 • 似たような実装になる • 機能が多く実装コストが高い • ネットワーク通信がない • 課題の提示方法が不親切

  11. ~ 2021-08-31 • コントリビューターを表示するアプ リを作成 • 一覧・詳細画面のみ • 選考ポイントを記載 •

    コードの提出方法を記載 • 期限は1週間以内 • 非公開
  12. None
  13. None
  14. 、、と思いきや

  15. • 新卒の方にとっては難易度高い • 応募へのハードルは高いまま • レビューコストが高い😇 課題点

  16. 現在 • 課題を難易度別に作成 • 作成されたアプリが存在 • 機能追加は不要 • 公開

  17. 3. レビューで気づいたこと

  18. ゆめみでは合否に関係なく 「良かった点」「機会点」 のフィードバックを送っています

  19. 実際のフィードバック

  20. フィードバックDB

  21. Git履歴がきれいだと コードもきれいな場合が多い レビューで気づいたこと:1つ目

  22. None
  23. 感動

  24. どんな場合がきれいじゃない?

  25. 例1「コミットメッセージはただの飾り」

  26. 例2「ブランチ行ったり来たり」

  27. 例3「全てを1つにまとめてきた」

  28. 可読性・保守性に関する フィードバックが多い レビューで気づいたこと:2つ目

  29. どんなフィードバック?

  30. 例1「コードぐちゃぐちゃ」

  31. 例1「コードぐちゃぐちゃ」

  32. 例2「例外の握りつぶし」

  33. 例2「例外の握りつぶし」

  34. 例3「いろんなものが詰め込まれてる」

  35. 例3「いろんなものが詰め込まれてる」

  36. • Git履歴がきれいだとコードもきれ いな場合が多い • 可読性・保守性に関するフィード バックが多い

  37. ご静聴ありがとうございました