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
620
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
410
FlutterKaigi Day1「Dart のこれから」
blendthink
1
6.1k
公開している Notion サイトから人名を抽出して、誤って個人情報を記載していないか検出できるようにしてみた
blendthink
0
450
株式会社ゆめみの Android の採用コーディング試験を公開しました
blendthink
0
88
Yumemi.apk #5 ~Kotlin夏祭り~ Session 2
blendthink
0
390
Developers Boost Session 6
blendthink
1
880
Other Decks in Programming
See All in Programming
Domain-Driven Transformation
hschwentner
2
1.9k
Honoのおもしろいミドルウェアをみてみよう
yusukebe
1
210
Honoとフロントエンドの 型安全性について
yodaka
7
1.4k
昭和の職場からアジャイルの世界へ
kumagoro95
1
410
なぜイベント駆動が必要なのか - CQRS/ESで解く複雑系システムの課題 -
j5ik2o
12
4.4k
ML.NETで始める機械学習
ymd65536
0
210
sappoRo.R #12 初心者セッション
kosugitti
0
270
Formの複雑さに立ち向かう
bmthd
1
900
第3回 Snowflake 中部ユーザ会- dbt × Snowflake ハンズオン
hoto17296
4
380
PRレビューのお供にDanger
stoticdev
1
210
DROBEの生成AI活用事例 with AWS
ippey
0
140
データの整合性を保つ非同期処理アーキテクチャパターン / Async Architecture Patterns
mokuo
53
18k
Featured
See All Featured
Bash Introduction
62gerente
611
210k
Helping Users Find Their Own Way: Creating Modern Search Experiences
danielanewman
29
2.4k
YesSQL, Process and Tooling at Scale
rocio
172
14k
Making Projects Easy
brettharned
116
6k
Rails Girls Zürich Keynote
gr2m
94
13k
Code Review Best Practice
trishagee
67
18k
Become a Pro
speakerdeck
PRO
26
5.1k
GraphQLとの向き合い方2022年版
quramy
44
13k
Statistics for Hackers
jakevdp
797
220k
Being A Developer After 40
akosma
89
590k
The Language of Interfaces
destraynor
156
24k
The Cost Of JavaScript in 2023
addyosmani
47
7.3k
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履歴がきれいだとコードもきれ いな場合が多い • 可読性・保守性に関するフィード バックが多い
ご静聴ありがとうございました