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

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

浅井勇樹

・福井県出身２８歳

・2015年07月入社

・最近はnanoblock

・Facebook：yukiasai417

自己紹介

・Merge Request

・Refactoring & Code Review

・Swift

アジェンダ

Merge Request

MR体制を強化中

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

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


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

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


WIP（Work In Progress）でMR

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

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

WIPでMR

オープンに！

Refactoring &

Code Review

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

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


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


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


結構なんでもあり

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

例

ライブラリへMRしてみた

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

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

ライブラリにMRしてみた

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

Pure Swiftを目指して

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


・Swift 1.0 から着手・新規コードはSwiftで記述・既存コードも気づいたら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 どちらも愛しています

ありがとうございました

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