大規模リファクタリングで痛感したSwiftのOptionalとの正しい付き合い方 - ZOZO TECH BLOG

iOSアプリチームの@hiragramです。 最近、ファーストリリース時からあった画面の大規模なリファクタリングを担当しました。 コードは遅かれ早かれ賞味期限が切れて少しずつ腐っていくものですが、その賞味期限を少しでも伸ばすために、普段コードを書く時にSwiftのOptionalについて意識していることを記事にします。 「とりあえずOptional」をやめる SwiftのOptionalは便利ですが、「Optionalを使えば、nilを安全に扱えて良い」と捉えてしまうと、気づくとモデルのプロパティがOptionalだらけになっていて使う側で毎回アンラップをしなくてはいけないような状況に必ずなります。 そうではなく、「Optionalの存在のおかげで、非Optionalなところにnilが絶対入ってこないことが保証されて良い」と捉えるべきだと思っています。 nilに口なしといいますが、Opti

[holyshared]

カジュアルにOptionalにするのが悪いのでは... Optionalにしたくなるケースそんなにないでしょ...

[gfx]

ViewController#viewDidLoaded() でプロパティを初期化するときの問題はAndroid x Kotlinでも同じ問題があるんだよなー。React Nativeはさすがに設計が新しいおかげでこの問題がなくて よい。

[nyamadori]

nilに口なし

[FromAtom]

ユーザーにクラッシュを強いてデバッガーとしてクラッシュログを集めさせてる感じがあるなぁ。Crashlyticsにnon-fatalで送らない理由はなんかあるんだろうか？

[mather314]

「データが存在しない」ことが通常の使用時に起こりうる場合にOptionalを使うのであって、想定外の事態にOptionalやnilも使うべきではなく動作を止めるべき。

[luccafort]

Swiftに限らないんだけどエラーや例外を握り潰す方向に舵取りをしたら危険信号だなとは思う。

[gomi_ningen]

開発者目線では大筋同意できるんだけど、ユーザーにとっての“正しく動き続ける"とはクラッシュしないことを指す気がするのだよな。落ちる=正しくない挙動だと思う。

[koogawa]

nilに口なし！

[yu_dotnet2004]

Optionalが安全というのであれば、Objective-Cのnilも同じくらい安全と言えるだろう。つまり大切なのは正しいモデル設計だと思う。

[kakajika]

プロパティを利用する側が、プロパティが初期化されていない可能性に気付きにくいという点で設計的にはよろしくない気もする。難しいな～

[zetta1985]

Optionalにextension 生やして、そのメソッド内でforce unwrapするほうが好み

[katzchang]

よくわかんないけど、swiftのoptionってこんな使いにくいの？？

[hiragram]

ちょっとした文を書いた