Swiftのコードレビュー勘所 - 文字っぽいの。

はじめに Swift with iOSなコードをレビューする時に僕が気をつけて見ているところをざっくりまとめました 今年からSwift始めます！って人が読めばクラッシュしやすいコードを書きにくくなるはず ロジック面についてもちゃんと確認しましょう 勘所 AnyObject使ってる 不適切に使われてる（型を消すとか）場合には絶対に直させる。型を消したい場合は protocol + typealias とかgenerics でなんとかできる場合が多い。というか AnyObject を使ってるコードでは、型を消す必要がない場合が多い。 配列にindex指定してアクセスしている let item = items[index] とか。基本的にはindexなど使わずにアクセスするのが一番なので、書き直せないか確認する。 書き直せない場合はロジック自体の設計を変えられないか、もしくははみ出ない処理がちゃ

[FromAtom]

'??' はコンパイル遅くなるって教えてもらった。

[sawat]

“非同期処理の中で self 呼んでるのに weak 〜” はブロックがselfをownするので「先に消える」ことはないのでは？逆に循環参照の心配をする必要がある（だからweakにすることが多い。dispatch_asyncなら必要ないが）

[nsnil]

早期リターンされているコードはネストが浅くて読みやすい。

[koogawa]

ブコメで訂正してくれる文化とても良い

[masayoshinym]

“エンバグすると人が死ぬ。”マジか。

[tondol]

既にブコメで指摘されているけど、ブロック内でselfを参照した場合は基本強参照としてキャプチャされるはず。メモリリークの原因になる。むしろクラッシュする可能性があるのはunownedでキャプチャされているとき。

[yu-robeat]

の使い所とunwrapが最近の悩みなので、循環参照についてきちんと学ばなきゃいけないんだけど、どれが正解かわからない...dispatch_asyncの中では、わざわざweak参照する必要ない？

[hiragram]

ブロック内でselfをweakやunownedにしなかったときって強参照だからそもそも解放されないのでは

[gomi_ningen]

"クラッシュ原因ほとんどはこれ。安易にForce Unwrapすべきではない。" 分かり手だ。iOSのクラッシュ理由ほとんどこれだよなー

[asonas]

良さある