Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
6年ほど育ったiOSプロジェクトを刷新した話 / Renewed iOS project th...
Search
ryokosuge
July 24, 2019
Technology
2
750
6年ほど育ったiOSプロジェクトを刷新した話 / Renewed iOS project that grew up for about 6 years
potatotips #63 @スマートニュース株式会社
https://potatotips.connpass.com/event/134616/
ryokosuge
July 24, 2019
Tweet
Share
More Decks by ryokosuge
See All by ryokosuge
広告実装、こうすると幸せだよ
ryokosuge
6
1.5k
アプリ開発とSDK開発の違い
ryokosuge
1
630
広告実装の苦しみ
ryokosuge
0
400
アプリを十数個作った僕が初めて書いたテストの話
ryokosuge
2
2.3k
広告SDKを駆使して 売り上げを5倍にした
ryokosuge
2
4.1k
早期発見プロジェクト
ryokosuge
0
56
Other Decks in Technology
See All in Technology
スタックチャン家庭用アシスタントへの道
kanekoh
0
110
伴走から自律へ: 形式知へと導くSREイネーブリングによる プロダクトチームの信頼性オーナーシップ向上 / SRE NEXT 2025
visional_engineering_and_design
3
230
ビジネス職が分析も担う事業部制組織でのデータ活用の仕組みづくり / Enabling Data Analytics in Business-Led Divisional Organizations
zaimy
1
310
全部AI、全員Cursor、ドキュメント駆動開発 〜DevinやGeminiも添えて〜
rinchsan
2
2.2k
united airlines ™®️ USA Contact Numbers: Complete 2025 Support Guide
flyunitedhelp
1
470
いつの間にか入れ替わってる!?新しいAWS Security Hubとは?
cmusudakeisuke
0
160
United airlines®️ USA Contact Numbers: Complete 2025 Support Guide
unitedflyhelp
0
340
CDKコード品質UP!ナイスな自作コンストラクタを作るための便利インターフェース
harukasakihara
2
200
freeeのアクセシビリティの現在地 / freee's Current Position on Accessibility
ymrl
2
280
Zero Data Loss Autonomous Recovery Service サービス概要
oracle4engineer
PRO
2
7.8k
関数型プログラミングで 「脳がバグる」を乗り越える
manabeai
2
220
Operating Operator
shhnjk
1
650
Featured
See All Featured
Understanding Cognitive Biases in Performance Measurement
bluesmoon
29
1.8k
GitHub's CSS Performance
jonrohan
1031
460k
Put a Button on it: Removing Barriers to Going Fast.
kastner
60
3.9k
Writing Fast Ruby
sferik
628
62k
CSS Pre-Processors: Stylus, Less & Sass
bermonpainter
357
30k
How STYLIGHT went responsive
nonsquared
100
5.6k
How to Ace a Technical Interview
jacobian
278
23k
Build your cross-platform service in a week with App Engine
jlugia
231
18k
Six Lessons from altMBA
skipperchong
28
3.9k
Faster Mobile Websites
deanohume
307
31k
Testing 201, or: Great Expectations
jmmastey
43
7.6k
Agile that works and the tools we love
rasmusluckow
329
21k
Transcript
6年ほど育った iOSプロジェクトを 刷新した話 potatotips #63 2019/07/24 ryokosuge
自己紹介 • 株式会社fluct ◦ SSP開発本部 SDKチーム ▪ 主にFluctSDK iOSの 開発してます
◦ 入社して9ヶ月くらい ◦ 前職はマンガアプリ作って ました Twitter: @ryo_kosuge GitHub: ryokosuge
アジェンダ • 6年かけてどう育ったか • 刷新作業(やったこと) • 刷新した結果 • まとめ
6年かけてどう育ったか
“git log --reverse” (噂ではSVN時代があるからもっと古い説あり)
広告SDKの開発について • 継続的な開発ではない ◦ がっつり作るのは新しいフォーマット対応のみ • テストはしっかり書く ◦ ほぼ100%網羅されている ◦
品質は絶対 • 作る前にしっかり設計して100%作り込んでリリースする ◦ 作ったものは容易に捨てられない世界
広告SDKのプロジェクト • 初期はstaticなSDKしか提供できなかったので`lib~.a`を生成する方式 ◦ `.framework`へ移行 ◦ スクリプトで対応(Makefile) • すごい古いプロジェクトなのでターゲットがおかしい ◦
1つの実装ファイルにたくさんのtargetが参照している ▪ 無謀なテストもかけてしまう
広告SDKのプロジェクト • ターゲットがおかしいからindexがうまくはられない ◦ テストを書く時にサジェストが出ない ◦ import error ▪ Headerファイルが見つからないらしい
• CIも時間がかかる ◦ キャッシュなど活用しても根本が解消されない ◦ 1つPRのjobに10分以上かかってしまうこともある
流石にやばい
刷新作業(やったこと) • 別ディレクトリに1からプロジェクト作成 ◦ しっかりターゲット選定をする ▪ FluctSDKとXCTestは1つのプロジェクトにして、依存するターゲット は別プロジェクトにする • 作業はworkspaceでまとめる
• ファイルはコピーして持ってきてプロジェクトに追加していく ◦ 命名規則などもこのタイミングで統一する
刷新作業(やったこと) • ターゲットを分けたことで依存しちゃいけない実装をしていた箇所を修正す る ◦ これは力技 ◦ ターゲット設計をしっかりしてちゃんと閉鎖できるようにした • テストもmockを使用することで対応可能
刷新作業(やったこと) • プロジェクトが綺麗になったタイミングでドキュメント更新 ◦ Makefileの分割 ◦ 無駄なファイルを削除 ◦ READMEもわかりやすく
結果
昔のプロジェクト すごい多いターゲット数
FluctSDKのプロジェクト
SDKに依存するプロジェクト
刷新した結果 • indexがしっかり貼られるようになったのでサジェストでる • 開発時に無駄なbuildが走らなくなった • `lib~.a`なファイルではなく、xcode標準の`.framework`をbuildで生成でき るようになった ◦ Static
+ bundleの提供方法は変えてない ▪ dynamicにすると導入手法が変わるためやらなかった
刷新した結果 • CIで落ちなくなった ◦ Travisで起きる謎のクラッシュがなくなった ▪ しっかり実行できるようになったからかな • キャッシュも効くようになったので2分ほどjobの時間を短縮 •
Warningsがほぼ0 ◦ deprecatedしているクラスのwarning以外はない
まとめ
まとめ • 6年続いたプロジェクトにまずは敬意を持つ • 何が依存していてどういう構成になっているか知るいいキッカケになった ◦ やる際はタスクをしっかり書き出してどうなると成功なのか示す必要あ る • outputは変えちゃいけない
◦ 要は大規模リファクタリング
ご静聴 ありがとうございました