zdogma/AikatsuLyric

SwiftLint に則って Caution 対応したい

Closed this issue · 0 comments

Caution の数が多そうであれば種類別に PR を分けて対応する。
#1 (comment)

特に
force_cast
force_try
force_unwrapping
をなるべく避けるようにするとよりSwiftらしくなると思います。

例えばこの辺のforce unwrappingを治すとしたら、
var dataを暗黙的アンラップ型で定義して、dataに値が渡ってなかったら非正常系というのを型で表現し、try ~ catchで例外を補足して適当にエラー表示する
Optional Chaining、Optional Bindingを使って、dataにデータが渡されてるとき(正常系)と、渡されてないとき(非正常系)を表示分けする
if let thumbnailUlr = data?.thumbnailUrl, !thumbNailUrl.isEmpty {
// データがあるとき
} else {
// データがないとき
}
のどちらかといった感じです。