Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
Yumemi.apk #6 ~ゆめみのAndroidエンジニア 日頃の成果大発表会!~ Ses...
Search
おかやまん
April 28, 2022
Programming
1
610
Yumemi.apk #6 ~ゆめみのAndroidエンジニア 日頃の成果大発表会!~ Session 2
コーディング試験のレビューを2年間続けて気づいたこと
https://yumemi.connpass.com/event/243080/
おかやまん
April 28, 2022
Tweet
Share
More Decks by おかやまん
See All by おかやまん
具体と抽象の行き来を意識する
blendthink
0
400
FlutterKaigi Day1「Dart のこれから」
blendthink
1
6k
公開している Notion サイトから人名を抽出して、誤って個人情報を記載していないか検出できるようにしてみた
blendthink
0
440
株式会社ゆめみの Android の採用コーディング試験を公開しました
blendthink
0
84
Yumemi.apk #5 ~Kotlin夏祭り~ Session 2
blendthink
0
380
Developers Boost Session 6
blendthink
1
870
Other Decks in Programming
See All in Programming
Package Traits
ikesyo
1
210
Lookerは可視化だけじゃない。UIコンポーネントもあるんだ!
ymd65536
1
130
盆栽転じて家具となる / Bonsai and Furnitures
aereal
0
1.8k
快速入門可觀測性
blueswen
0
500
混沌とした例外処理とエラー監視に秩序をもたらす
morihirok
13
2.3k
PSR-15 はあなたのための ものではない? - phpcon2024
myamagishi
0
400
ASP.NET Core の OpenAPIサポート
h455h1
0
120
Alba: Why, How and What's So Interesting
okuramasafumi
0
210
ErdMap: Thinking about a map for Rails applications
makicamel
1
630
Асинхронность неизбежна: как мы проектировали сервис уведомлений
lamodatech
0
1.3k
アクターシステムに頼らずEvent Sourcingする方法について
j5ik2o
6
700
知られざるDMMデータエンジニアの生態 〜かつてツチノコと呼ばれし者〜
takaha4k
1
420
Featured
See All Featured
Refactoring Trust on Your Teams (GOTO; Chicago 2020)
rmw
33
2.7k
The Pragmatic Product Professional
lauravandoore
32
6.4k
Rails Girls Zürich Keynote
gr2m
94
13k
Imperfection Machines: The Place of Print at Facebook
scottboms
267
13k
Stop Working from a Prison Cell
hatefulcrawdad
267
20k
Raft: Consensus for Rubyists
vanstee
137
6.7k
Save Time (by Creating Custom Rails Generators)
garrettdimon
PRO
29
960
Building Flexible Design Systems
yeseniaperezcruz
328
38k
Building an army of robots
kneath
302
45k
RailsConf 2023
tenderlove
29
970
Documentation Writing (for coders)
carmenintech
67
4.5k
StorybookのUI Testing Handbookを読んだ
zakiyama
28
5.4k
Transcript
コーディング試験のレビューを 2年間続けて気づいたこと Android 採用WG
軽く自己紹介タイム
内容 1. コーディング試験(1分) 2. コーディング試験の歴史(4分) 3. レビューで気づいたこと(5分)
1. コーディング試験
絶賛、公開中!
コーディング試験の内容 メチャクチャなコードがあるので それを改善してください という内容です ざっくりいうと
ゆめみ側で GitHub のリポジトリを検 索するアプリを既に作成しています。 しかし、このアプリとコードには、改善 可能な部分がたくさんあります。 アプリ仕様を守りながら、課題に従っ て改善していってください。
2. コーディング試験の歴史
~ 2020-03-24 • http://todomvc.com のような TODOアプリを作成 • 期限は1週間以内 • 非公開
課題点 • 似たような実装になる • 機能が多く実装コストが高い • ネットワーク通信がない • 課題の提示方法が不親切
~ 2021-08-31 • コントリビューターを表示するアプ リを作成 • 一覧・詳細画面のみ • 選考ポイントを記載 •
コードの提出方法を記載 • 期限は1週間以内 • 非公開
None
None
、、と思いきや
• 新卒の方にとっては難易度高い • 応募へのハードルは高いまま • レビューコストが高い😇 課題点
現在 • 課題を難易度別に作成 • 作成されたアプリが存在 • 機能追加は不要 • 公開
3. レビューで気づいたこと
ゆめみでは合否に関係なく 「良かった点」「機会点」 のフィードバックを送っています
実際のフィードバック
フィードバックDB
Git履歴がきれいだと コードもきれいな場合が多い レビューで気づいたこと:1つ目
None
感動
どんな場合がきれいじゃない?
例1「コミットメッセージはただの飾り」
例2「ブランチ行ったり来たり」
例3「全てを1つにまとめてきた」
可読性・保守性に関する フィードバックが多い レビューで気づいたこと:2つ目
どんなフィードバック?
例1「コードぐちゃぐちゃ」
例1「コードぐちゃぐちゃ」
例2「例外の握りつぶし」
例2「例外の握りつぶし」
例3「いろんなものが詰め込まれてる」
例3「いろんなものが詰め込まれてる」
• Git履歴がきれいだとコードもきれ いな場合が多い • 可読性・保守性に関するフィード バックが多い
ご静聴ありがとうございました