post Image
iOSエンジニアがAndroidプロジェクトのレビュー効率的にする方法(Swift → Kotlin)

はじめに

この記事ではiOSエンジニアがAndroidプロジェクトのレビューをしてみて思ったこと(こうしたらいいんじゃね?)をだらだらと書いております。

背景

iOSとAndroidのエンジニアが1人ずつしかいないプロジェクトにおいて、レビュー誰もしてくれない問題を回避したいと思いお互いにレビューすればいんじゃね?となったので、実際にやってみた感想とかポイントとかまとめておきます。

もともと他のプロジェクト人にレビューしてもらっていたのですが、仕様がわからないのでそもそもレビューできない等の意見がでたので試してみました。

レビューしてみての全体的な感想

やっぱり言語がわからないのは結構辛い。ただ、if文だけを見れば大体のロジックはわかるのでそこだけ重点的に見るならありかなと。結局の所レビューで何を担保したいのかによるなーと思いました。
仕様的な確認を行いたいのであれば同プロジェクトの人がチェックするといいと思いますし、コードの読みやすさとかをチェックしたいなら同言語を使用している人に見てもらうといいと思います。

以下効率的にレビューするポイント

ディレクトリ構成とアーキテクチャーの説明だけは最初に教えてもらっておく

ディレクトリ構成とアーキテクチャーの説明を受けないでレビューしようと思うと、全体がつかめないしどこに何が書いてあるかもわからないのでレビューする気が起きません。
まぁ、同言語のレビューする場合でも大事なことなんですけど、忘れがちなので。

xmlファイルは見ない

iOSでいうところのxibとかstoryboardに当たるファイルです。xibとかstoryboardに比べれば読みやすいですが、そんなに見なくてもいいかなと。チェックする時間の割にチェックしてもあまりいいことがない気がします。
ただ注意しないといけないのはxmlにもロジックがかけるらしいのでコードだけ見ていると罠にはめられる可能性があります。

Resource系も見ない

ImageとかStringとか(Stringもxmlだけどね)AnimationとかDrawableとかColorとかとか…
見ない!
ココらへんは実際にアプリをポチポチテストしているときに確認してもらったらいいかなーと

gradleも見ない

iOSのprojectファイルみたいなもの(だと思ってる)ので、見ない

import文も見ない

確か勝手にAndroidStudioが管理してくれるので読み飛ばします

View周りは見ない

LayoutファイルとかViewとかActivityとかAndroidの癖が強めなところは見ない
MVPとかだったらVに当たる部分ですね
View周りはデザイナーさんが実機で確認してくれると思うのできっと大丈夫!

基本if文が書いてある部分だけ見る

仕様どおりにコードが動くかってことを考えるとif文のところが一番怪しいのでそこを重点的に確認します。
if文だったら言語関係なくほぼ同じ書き方になると思うので言語&OSが異なってもすらすら読めます。

困ったらAndroidエンジニアと一緒にレビューする

調べながらレビューしてもいいのですが、基礎がなさすぎると1日中勉強しないといけないことになってしまいます。
そんなときはできるAndroidエンジニアを捕まえて一緒にレビューするとめちゃめちゃ捗ります。
Android側の知識だけでなくiOSの知識を再度見直すいい機会になったりします。

nininsankyaku_men.png


『 Swift 』Article List
Category List

Eye Catch Image
Read More

Androidに関する現役のエンジニアのノウハウ・トレンドのトピックなど技術的な情報を提供しています。コード・プログラムの丁寧な解説をはじめ、初心者にもわかりやすいように写真や動画を多く使用しています。

Eye Catch Image
Read More

AWSに関する現役のエンジニアのノウハウ・トレンドのトピックなど技術的な情報を提供しています。コード・プログラムの丁寧な解説をはじめ、初心者にもわかりやすいように写真や動画を多く使用しています。

Eye Catch Image
Read More

Bitcoinに関する現役のエンジニアのノウハウ・トレンドのトピックなど技術的な情報を提供しています。コード・プログラムの丁寧な解説をはじめ、初心者にもわかりやすいように写真や動画を多く使用しています。

Eye Catch Image
Read More

CentOSに関する現役のエンジニアのノウハウ・トレンドのトピックなど技術的な情報を提供しています。コード・プログラムの丁寧な解説をはじめ、初心者にもわかりやすいように写真や動画を多く使用しています。

Eye Catch Image
Read More

dockerに関する現役のエンジニアのノウハウ・トレンドのトピックなど技術的な情報を提供しています。コード・プログラムの丁寧な解説をはじめ、初心者にもわかりやすいように写真や動画を多く使用しています。

Eye Catch Image
Read More

GitHubに関する現役のエンジニアのノウハウ・トレンドのトピックなど技術的な情報を提供しています。コード・プログラムの丁寧な解説をはじめ、初心者にもわかりやすいように写真や動画を多く使用しています。

Eye Catch Image
Read More

Goに関する現役のエンジニアのノウハウ・トレンドのトピックなど技術的な情報を提供しています。コード・プログラムの丁寧な解説をはじめ、初心者にもわかりやすいように写真や動画を多く使用しています。

Eye Catch Image
Read More

Javaに関する現役のエンジニアのノウハウ・トレンドのトピックなど技術的な情報を提供しています。コード・プログラムの丁寧な解説をはじめ、初心者にもわかりやすいように写真や動画を多く使用しています。

Eye Catch Image
Read More

JavaScriptに関する現役のエンジニアのノウハウ・トレンドのトピックなど技術的な情報を提供しています。コード・プログラムの丁寧な解説をはじめ、初心者にもわかりやすいように写真や動画を多く使用しています。

Eye Catch Image
Read More

Laravelに関する現役のエンジニアのノウハウ・トレンドのトピックなど技術的な情報を提供しています。コード・プログラムの丁寧な解説をはじめ、初心者にもわかりやすいように写真や動画を多く使用しています。

Eye Catch Image
Read More

Pythonに関する現役のエンジニアのノウハウ・トレンドのトピックなど技術的な情報を提供しています。コード・プログラムの丁寧な解説をはじめ、初心者にもわかりやすいように写真や動画を多く使用しています。

Eye Catch Image
Read More

Rubyに関する現役のエンジニアのノウハウ・トレンドのトピックなど技術的な情報を提供しています。コード・プログラムの丁寧な解説をはじめ、初心者にもわかりやすいように写真や動画を多く使用しています。

Eye Catch Image
Read More

Scalaに関する現役のエンジニアのノウハウ・トレンドのトピックなど技術的な情報を提供しています。コード・プログラムの丁寧な解説をはじめ、初心者にもわかりやすいように写真や動画を多く使用しています。

Eye Catch Image
Read More

Swiftに関する現役のエンジニアのノウハウ・トレンドのトピックなど技術的な情報を提供しています。コード・プログラムの丁寧な解説をはじめ、初心者にもわかりやすいように写真や動画を多く使用しています。

Eye Catch Image
Read More

Unityに関する現役のエンジニアのノウハウ・トレンドのトピックなど技術的な情報を提供しています。コード・プログラムの丁寧な解説をはじめ、初心者にもわかりやすいように写真や動画を多く使用しています。

Eye Catch Image
Read More

Vue.jsに関する現役のエンジニアのノウハウ・トレンドのトピックなど技術的な情報を提供しています。コード・プログラムの丁寧な解説をはじめ、初心者にもわかりやすいように写真や動画を多く使用しています。

Eye Catch Image
Read More

Wordpressに関する現役のエンジニアのノウハウ・トレンドのトピックなど技術的な情報を提供しています。コード・プログラムの丁寧な解説をはじめ、初心者にもわかりやすいように写真や動画を多く使用しています。

Eye Catch Image
Read More

機械学習に関する現役のエンジニアのノウハウ・トレンドのトピックなど技術的な情報を提供しています。コード・プログラムの丁寧な解説をはじめ、初心者にもわかりやすいように写真や動画を多く使用しています。