レビュー経験ほぼ0だった自分が レビューで心掛けていること

Transcript

1. レビュー経験ほぼ0だった⾃分が レビューで⼼掛けていること 【学⽣向け勉強会】teamLab×DMM tech meetup ~frontend~ 柏原 太⼀郎（カシハラ タイチロウ） 1
2. © teamLab Inc ⾃⼰紹介 柏原 太⼀郎 Kashihara Taichiro • 所属 : パッケージチーム フロントエンド班 • 年次 : 新卒 1年⽬  ◦ プロジェクトに配属されてから約8ヶ⽉経過 2
3. © teamLab Inc ⼊社前 現在 レビュー経験ほぼゼロ レビューの⽅法がわからない 指摘もできない レビューの⽅法を掴み始める （多少）バグなどを⾒つける ？ レビュー経験 3
4. © teamLab Inc レビューで⼼掛けていること1 -まず、実装ミス‧バグをなくす - 仕様書‧デザイン UI‧動作‧コード 既存システムとの 整合性 ユーザーが 利⽤できるものか 4
5. © teamLab Inc 仕様‧デザイン - 既存システムとの整合性 - • 追加する機能と既存の機能に⽭盾がないか 個⼈ 考えている点 ⼩規模‧シンプル ⼤規模‧複雑 仕事 記事A 〜〜〜 記事B 〜〜〜 記事C 〜〜〜 記事B 優先度1 記事C 優先度2 記事A 優先度5 • 現在のデザインからずれていないのか 5
6. © teamLab Inc UI‧動作‧コード - ユーザーが利⽤できるものか - • ロジックが正しいかどうか 考えている点 個⼈ シンプルな仕様‧⾃分の環境 複雑な仕様‧決められた環境 仕事 height : autoの アニメーションを interpolate-sizeで実装! [引用] Can I use interpolate-size [参考] MDN interpolate-size • ユーザーが利⽤する環境で操作できるかどうか 6
7. © teamLab Inc レビューで⼼掛けていること2 -まず、レビューをしてみる- レビュー初⼼者‧⼊社したて ハードルが⾼い 時間‧難易度 レビューへの不安 ハードルを低くする 部分的にレビュー 学ぶつもりでレビュー 7
8. EOF まとめ まず、実装ミス‧バグをなくす まず、レビューをしてみる © teamLab Inc 8
9. © teamLab Inc 9 仕様書のイラスト https://www.irasutoya.com/2019/11/blog-post_860.html Figma https://www.figma.com/ja-jp/using-the-figma-brand/ プログラムのキャラクター https://www.irasutoya.com/2017/10/blog-post_931.html スマートフォンで絵を描いているイラスト https://www.irasutoya.com/2017/04/blog-post_83.html イラストや https://www.irasutoya.com/ その他 プログラミングをする人のイラスト（男性） https://www.irasutoya.com/2018/12/blog-post_627.html 高いハードルのイラスト（男性） https://www.irasutoya.com/2019/01/blog-post_22.html 低いハードルのイラスト（男性） https://www.irasutoya.com/2019/01/blog-post_24.html JAF [Q]初心者マーク（若葉マーク）などは付けないと違反になるのでしょうか？ https://jaf.or.jp/common/kuruma-qa/category-accident/subcategory-traffic-violation/faq306 イラストリンク (引用)