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/

おかやまん

April 28, 2022
Tweet

More Decks by おかやまん

Other Decks in Programming

Transcript

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

    View Slide

  2. 軽く自己紹介タイム

    View Slide

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

    View Slide

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

    View Slide

  5. 絶賛、公開中!

    View Slide

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

    View Slide

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

    View Slide

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

    View Slide

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

    View Slide

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

    View Slide

  11. ~ 2021-08-31
    ● コントリビューターを表示するアプ
    リを作成
    ● 一覧・詳細画面のみ
    ● 選考ポイントを記載
    ● コードの提出方法を記載
    ● 期限は1週間以内
    ● 非公開

    View Slide

  12. View Slide

  13. View Slide

  14. 、、と思いきや

    View Slide

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

    View Slide

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

    View Slide

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

    View Slide

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

    View Slide

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

    View Slide

  20. フィードバックDB

    View Slide

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

    View Slide

  22. View Slide

  23. 感動

    View Slide

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

    View Slide

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

    View Slide

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

    View Slide

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

    View Slide

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

    View Slide

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

    View Slide

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

    View Slide

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

    View Slide

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

    View Slide

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

    View Slide

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

    View Slide

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

    View Slide

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

    View Slide

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

    View Slide