狠狠撸

iOSアプリエンジニアの最近
株式会社マネーフォワード
iOSアプリエンジニア
浅井勇樹

?福井県出身２８歳
?2015年07月入社
?最近はnanoblock
?Facebook：yukiasai417
自己紹介

?Merge Request
?Refactoring & Code Review
?Swift
アジェンダ

?約束
?少なくともエンジニア２人がレビュー（５人中）
?細かいことでも指摘、質問する
?精一杯優しく
?単位
?バグ単位
?タスク単位
?機能単位
惭搁体制を强化中

?約束
?少なくともエンジニア２人がレビュー（５人中）
?細かいことでも指摘、質問する
?精一杯優しく
?単位
?バグ単位
?タスク単位
?機能単位 ← これが大きくなりがち
惭搁体制を强化中

WIP（Work In Progress）でMR

iOSエンジニアの最近株式会社マネーフォワード

?約束
?作業中のコードを晒す
?WIPはマージしない
?目的
?実装方針の議論
?問題の早期発見
?他部署からの指摘も
WIPでMR

リファクタリング、コードレビュー会

?概要
?各部署のスマホエンジニアが集結
?毎週金曜（１６時～１７時）
?開始前にやることを宣言し、終了後に成果を報告

?目的
?定期的に負債を返済
?知見の共有
?最近の悩み相談

?目的
?知見の共有
結構なんでもあり

?スコープ小さくしてみたり
?再利用性を高めてみたり
?知見を共有
?いいライブラリ見つけた
?そのライブラリのコードリーディングしよう
?ここの実装もっとシンプルにできませんかね？
?勉強会で何話したらいいですかね？
例

ライブラリへ惭搁してみた

?弊社アプリはパスコード機能必須
?SwiftPasscodeLockを発見
?機能追加してMR
?パスコードの保存先を拡張
ライブラリにMRしてみた

コミュニティーに还元しよう！

Pure 厂飞颈蹿迟を目指して

?Swift 1.0 から着手
?新規コードはSwiftで記述
?既存コードも気づいたらSwiftへ移行

?Swift 1.0 から着手
?新規コードはSwiftで記述
?既存コードも気づいたらSwiftへ移行
ただいまのSwift率 33%

厂飞颈蹿迟移行の落とし穴

?Objective-Cとのインターフェース
?Optionalの扱いが雑（仕方ない）
?OptionalなIntなどを参照不可（仕方ない）
?NSArrayなどの扱いが面倒（仕方ない）
厂飞颈蹿迟移行の落とし穴

話題の
Objective-C to Swift Converter
を試してみた

?使用感
?ちょっと厳しいかな、、、
?ところどころ変換がおかしい
?.hと.mをマージしないといけない
?Webサービスなのでちょっと怖い

?使用感
?ちょっと厳しいかな、、、
?ところどころ変換がおかしい
?.hと.mをマージしないといけない
?Webサービスなのでちょっと怖い
今後に期待

Objective-CとSwift
どちらも愛しています

ありがとうございました

iOSエンジニアの最近株式会社マネーフォワード