【Swift】コードレビューを受けたみた2
今週に作ったパスワード生成アプリのコードレビューをSlackのメンバーにお願いしてもらい、数々の指摘をもらったので、備忘録として書きます。
↓
「パスワード生成アプリ」
・辞書のキーを数字にすると、いちいち中身を確認しないと、値が何なのかがわからない
・だから、キーはなるべく単語やキーワードにすべし
let dic = [0:false, 1:false, 2:false]
let dic = ["button1":false, "button2":false, "button3":false]
・ viewDidLoadの中身がsuper.viewDidLoad()だけなら、削除してもOK
・ if文でBool値の真偽によって変数に代入する値が変わる処理を施すなら、
変数 = 条件式(Bool値かどうか) ? trueの場合 : falseの場合
num = sender.isOn ? 5 : 0
・ 条件式の値とif文の中身で変数に代入する値が同じなら、
変数 = 条件式 ? true : false
num = sender.isON ? true : false
・ ループ処理の条件式をわかりやすくするために、while&条件式を変数で構成する方がベスト
for _ in 0…count {}
while num < count {}
・ viewWIlAppearにはsuper.viewWillAppearを入れろ
・ アンラップ時、as?で型を推測しているので、変数に型を指定する必要はない
× if let 変数 : String = … as? String {}
○ if let 変数 = ... as? String {}
・セルに格納したデータを参照する場合は、セルから直接参照しなくても、データから参照すればいい
//配列の要素をセルに代入するとする
//配列がその.swift内にあれば、
print(array[indexPath])
//と、配列から直接値を参照できる
