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

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

浅井 勇樹

・福井県出身２８歳

・2015年07月入社

・最近はnanoblock

・Facebook：yukiasai417

自己紹介

・Merge Request

・Refactoring & Code Review

・Swift

アジェンダ

Merge Request

MR体制を強化中

・約束 ・少なくともエンジニア２人がレビュー（５人中） ・細かいことでも指摘、質問する ・精一杯優しく

・単位 ・バグ単位 ・タスク単位 ・機能単位

MR体制を強化中

・約束 ・少なくともエンジニア２人がレビュー（５人中） ・細かいことでも指摘、質問する ・精一杯優しく

・単位 ・バグ単位 ・タスク単位 ・機能単位 ← これが大きくなりがち

MR体制を強化中

WIP（Work In Progress）でMR

・約束 ・作業中のコードを晒す ・WIPはマージしない

・目的 ・実装方針の議論 ・問題の早期発見 ・他部署からの指摘も

WIPでMR

オープンに！

Refactoring &

Code Review

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

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

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

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

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

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

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

結構なんでもあり

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

例

ライブラリへMRしてみた

・弊社アプリはパスコード機能必須 ・SwiftPasscodeLockを発見

・機能追加してMR ・パスコードの保存先を拡張

ライブラリにMRしてみた

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

Swift

Pure Swiftを目指して

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

Pure Swiftを目指して

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

Pure Swiftを目指して

ただいまのSwift率 33%

Swift移行の落とし穴

・Objective-Cとのインターフェース ・Optionalの扱いが雑（仕方ない） ・OptionalなIntなどを参照不可（仕方ない） ・NSArrayなどの扱いが面倒（仕方ない）

Swift移行の落とし穴

話題の Objective-C to Swift Converter

を試してみた

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

Objective-C to Swift Converter

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

Objective-C to Swift Converter

今後に期待

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

ありがとうございました