【これからiOS頑張りたい方向け】2年半iOSアプリ開発をしてハッとした瞬間まとめ

はじめに

iOS Advent Calendar 2020 17日目です。

2年半くらいiOSアプリ開発してきてハッとした瞬間をまとめました。(iOSとかswiftに限った話じゃない学びもあるけど。)
がんばらなくても読めるけど、なんとなく勉強にもなる記事を目指しました。
タイトルに近い方が初歩的なやつです。
もし時間あればみていただけると嬉しいです。

お品書き

• 返り値でBoolを返す時はそのBool自身を返せばいい
• 三項演算子を使うとif else がワンライナーで書ける
• var +=は計算型プロパティにできる。
• ネストは早期returnで減らせる
• 2重否定はifでいい。
• 型が明確な時のinitializerは.initに省略できる
• trailing closureは引数から省略できる
• enumとswitchを組み合わせて網羅性をチェックする
• Bool値が複数ある場合の場合分けはswitchとパターンマッチを使うと見やすくなる。
• var+forは大抵高階関数で書き換えられる
• 配列のmappingにはmapが使える
• for文にifを挟む処理はfor+whereに書き換えできる。
• for+whereはfilter+forEachに書き換えできる。
• 引数の変更を参照元に反映させたい時はinoutが使える
• filter + first or lastはfirst(where: ) or last(where: )で書ける。
• StackViewを使うと制約をグッと減らせる。
• StackView + ScrollViewでお手軽に拡張しやすいスクロール可能なレイアウトを構築できる。
• 見えているViewの裏のViewのタッチイベントを取得したい時はhittestが使える
• 同じ処理をアノテーション化して省略する時はPropertyWrapperが使える

返り値でBoolを返す時はそのBool自身を返せばいい

Before

var hoge: Bool {
    if fugaBool {
        return true
    } else {
        return false
    }
}

After

var hoge: Bool {
    return fugaBool
}

今思えば当たり前なんですけどね。
〇〇がtrueの時trueを返さなきゃってなって〇〇自体が真偽値なことに気づいてなかったんでしょうね。

三項演算子を使うと if else がワンライナーで書ける

Before

if fugaBool {
    hogeLabel.text = "fugaは真です"
} else {
    hogeLabel.text = "fugaは偽です"
}

After

hogeLabel.text = fugaBool ? "fugaは真です" : "fugaは偽です"

三項演算子すこ。

var +=は計算型プロパティにできる。

Before

func presentHogeAlert() {
    var message = ""
    if validationA {
        message.append("Aがあかんかった")
    } else if validationB {
        message.append("Bがあかんかった")
    } else {
        message.append("あかんかった")
    }
    let alert = UIAlertController(title: "エラー", message: message, preferredStyle: .alert)
    alert.addAction(.init(title: "OK", style: .default))
    present(alert, animated: true)
}

After

func presentHogeAlert() {
    var message: String {
        if validationA {
            return "Aがあかんかった"
        } else if validationB {
            return "Bがあかんかった"
        } else {
            return "あかんかった"
        }
    }

    let alert = UIAlertController(title: "エラー", message: message, preferredStyle: .alert)
    alert.addAction(.init(title: "OK", style: .default))
    present(alert, animated: true)
}

計算型の方が返り値がわかりやすくてスッキリしますよね。

ネストは早期returnで減らせる

Before

func hoge() {
    if yagi {
        // 処理A
        if let mogu = mogu {
            // 処理B
        } else {
            if fuga {
                // 処理C
            } else {
                // 処理D
            }
        }
    } else {
        // 処理E
    }
}

After

func hoge() {
    guard yagi else {
        // 処理E
        return
    }

    // 処理A

    guard let mogu = mogu else {
        if fuga {
            // 処理C
        } else {
            // 処理D
        }
        return 
    }

    // 処理B
}

もうちょっといい例を用意したかったです。
ネストが多くて長いだけで読むのが辛くなっちゃいますよね。

2重否定はifでいい。

Before

guard !fugaBool else {
    // do something
    return
}

After

if fugaBool {
    // do something
    return
}

プロジェクトによってはguardで明示的にreturnを表現するためにあえて2重否定を好むところもあります。
真偽値一つならまだ分かるけどこれが!fugaBool || mogu > 1 && yagi.isEmpty みたいな条件になってきたら発狂しますよね。

型が明確な時のinitializerは.initに省略できる

Before

view.frame = CGRect(origin: CGPoint(x: 0, y: 0), size: CGSize(width: 0, height: 0))

After

view.frame = .init(
    origin: .init(
        x: 0,
        y: 0
    ),
    size: .init(
        width: 0,
        height: 0
    )
)

長い型名を省略できるので好き。
多用すると読みにくくなるので注意。

trailing closureは引数から省略できる

Before

func hogeChanged(fugaBool: Bool, hogeCompletionHandler: @escaping ((String) -> Void)) {
    guard fugaBool else { return }

    let hoge = getChangedHogeValue
    hogeCompletionHandler(hoge)
}

hogeChanged(fugaBool: true, hogeCompletionHandler: @escaping { [weak self] hoge in

    guard let self = self else { return }

    self.hogeLabel.text = hoge
})

After

func hogeChanged(fugaBool: Bool, hogeCompletionHandler: @escaping ((String) -> Void)) {
    guard fugaBool else { return }

    let hoge = getChangedHogeValue
    hogeCompletionHandler(hoge)
}

hogeChanged(fugaBool: true) { [weak self] hoge in

    guard let self = self else { return }

    self.hogeLabel.text = hoge
})

クロージャの引数名大抵長くなるから助かります。
読みづらくなる場合もあります。

enumとswitchを組み合わせて網羅性をチェックする

例えばTabBarControllerのtabの種類をenumにするとか。

Before

    override func viewDidLoad() {
        super.viewDidLoad()

        var viewControllers: [UIViewController] = []

        let hogeVC = HogeViewController())
        hogeVC.tabBarItem = UITabBarItem(title: "hoge", image: nil, tag: 1)
        viewControllers.append(hogeVC)

        let fugaVC = FugaViewController()
        fugaVC.tabBarItem = UITabBarItem(title: "fuga", image: nil, tag: 2)
        viewControllers.append(fugaVC)

        setViewControllers(viewControllers, animated: false)
    }

After

enum TabType: Int, CaseIterable {
    case hoge = 0
    case fuga = 1

    private var baseViewController: UIViewController {
        switch self {
        case .hoge:
            return HogeViewController()

        case .fuge:
            return FugaViewController()

        }
    }

    private var title: String {
        switch self {
        case .hoge:
            return "hoge"

        case .fuga:
            return "fuga"

        }
    }

    var tabItem: UITabBarItem {
        .init(title: title, image: nil, tag: self.rawValue)
    }

    var viewController: UIViewController {
        let viewController = baseViewController
        viewController.tabBarItem = tabItem
        return viewController
    }
}

// 使う時
override func viewDidLoad() {
    super.viewDidLoad()
    setViewControllers(TabType.allCases.map(\.viewController), animated: false)
}

caseを追加すれば他の設定値も自ずと追加が必要になるので漏れを防げます。

Bool値が複数ある場合の場合分けはswitchとパターンマッチを使うと見やすくなる。

Before

フラグが何個かあってtrueとfalseの組み合わせを表現したい時。

let hogeFlag: Bool
let fugaFlag: Bool

if hogeFlag && fugaFlag {
    // true true
} else if !hogeFlag && fugaFlag {
    // false true
} else if hogeFlag && !fugaFlag {
    // true flase
} else {
    // false false
}

After

switch (hogeFlag, fugaFlag) {
case (true, true):   break
case (false, true):  break
case (true, false):  break
case (false, false): break
}

switchの見やすさは異常。

var+forは大抵高階関数で書き換えられる

Before

var titles: [String] = []
for i in 0...10 {
    let title = "りんごが\(i)個あります。"
    titles.append(title)
}
titles.joined(separator: "\n")
return titles

After

(0...10).map {
    "りんごが\($0)個あります。"
}
.joined(separator: "\n")

https://qiita.com/shtnkgm/items/600009917d8e572e6780
こちらの記事が詳しいです。

配列のmappingにはmapが使える

Before

struct Hoge {
    let aaa: String
    let iii: String
    let uuu: String
}

// Hogeの配列があります。
let hogeList = [Hoge]()

After

// 特定のプロパティのみ取り出し
let aaaList = hogeList.map(\.aaa)
// 別の型にmapping
struct Fuga {
    let aaa: String
    let eee: String
    let ooo: String
}
let fugaList = hogeList.map { Fuga(aaa: $0.aaa, eee: "", ooo: "") }

API叩いてデータ加工してViewに渡す過程で絶対必要になりますよね。

for文にifを挟む処理はfor+whereに書き換えできる。

こういう動物達がいるとします。

protocol Animal {
    var name: String { get }
}

protocol Runable: Animal {
    func run()
}
extension Runable {
    func run() {
        print("run!!!")
    }
}

class Cat: Animal, Runable {
    var name: String = "猫"
}

class Dog: Animal, Runable {
    var name: String = "犬"
}

class Penguin: Animal {
    var name: String = "ペンギン"
}

let animals: [Animal] = [Cat(), Dog(), Penguin()]

Before

for animal in animals {
    if animal is Runable {
        print("この動物は走れます！")
    }
    if let runableAnimal = animal as? Runable {
        runableAnimal.run()
    }
}

After

for animal in animals where animal is Runable {
    print("この動物は走れます！")

    if let runableAnimal = animal as? Runable {
        runableAnimal.run()
    }
}

whereを使いこなしてるとなんかできる感出てきますよね。

for+whereはfilter+forEachに書き換えできる。

先ほどの例から

Before

for animal in animals where animal is Runable {
    print("\(animal.name)は走れます！")

    if let runableAnimal = animal as? Runable {
        runableAnimal.run()
    }
}

After

animals.filter { $0 is Runable }.forEach {
    print("\($0.name)は走れます！")

    if let runableAnimal = $0 as? Runable {
        runableAnimal.run()
    }
}

高階関数すこ。
この例ならキャストしてcompactMapにかけた方が早いかも。

animals.compactMap { $0 as? Runable }.forEach {
    print("\($0.name)は走れます！")
    $0.run()
}

引数の変更を参照元に反映させたい時はinoutが使える

いわゆる参照渡しです。

Before

var fuga = "fugaです。"

print(fuga) // fugaです。

func addHogeString() {
    fuga.append("\n")
    fuga.append("hogeを追加します")
}

addHogeString() 

print(fuga) // fugaです。\n hogeを追加します。

After

func addHogeString(strings: inout String) {
    strings.append("\n")
    strings.append("hogeを追加します")
}

var fuga = "fugaです。"

print(fuga) // fugaです。

addHogeString(strings: &fuga)

print(fuga) // fugaです。\n hogeを追加します。

この&を使った参照渡しの表現はCombineのassignでも使いますね。

filter + first or lastはfirst(where: ) or last(where: )で書ける。

Before

animals.filter { !($0 is Runable) }.first // Penguin

After

animals.first { !($0 is Runable) } // Punguin

firstを使う方がパフォーマンスにも優れるとかなんとか。

StackViewを使うと制約をグッと減らせる。

こんなViewを用意したいとします。(枠が見えやすいように色を変えています。)

Before

class ViewController: UIViewController {

    private lazy var hogeLabel: UILabel = {
        let label = UILabel()
        label.text = "hogehoge"
        return label
    }()

    private lazy var fugaLabel: UILabel = {
        let label = UILabel()
        label.text = "fugafuga"
        return label
    }()

    private lazy var moguLabel: UILabel = {
        let label = UILabel()
        label.text = "mogumogu"
        return label
    }()

    override func viewDidLoad() {
        super.viewDidLoad()

        setupLayout()
    }

    private func setupLayout() {
        [hogeLabel, fugaLabel, moguLabel].forEach {
            $0.translatesAutoresizingMaskIntoConstraints = false
            $0.backgroundColor = .gray
            view.addSubview($0)
        }

        NSLayoutConstraint.activate([
            hogeLabel.topAnchor.constraint(equalTo: view.topAnchor, constant: 80),
            hogeLabel.rightAnchor.constraint(equalTo: view.rightAnchor, constant: -16),
            hogeLabel.leftAnchor.constraint(equalTo: view.leftAnchor, constant: 16),

            fugaLabel.topAnchor.constraint(equalTo: hogeLabel.bottomAnchor, constant: 16),
            fugaLabel.leftAnchor.constraint(equalTo: view.leftAnchor, constant: 16),
            fugaLabel.rightAnchor.constraint(equalTo: view.rightAnchor, constant: -16),

            moguLabel.topAnchor.constraint(equalTo: fugaLabel.bottomAnchor, constant: 16),
            moguLabel.leftAnchor.constraint(equalTo: view.leftAnchor, constant: 16),
            moguLabel.rightAnchor.constraint(equalTo: view.rightAnchor, constant: -16),
        ])
    }
}

After

class ViewController: UIViewController {

...

    private lazy var stackView: UIStackView = {
        let stackView = UIStackView()
        stackView.spacing = 16
        stackView.axis = .vertical
        return stackView
    }()

    override func viewDidLoad() {
        super.viewDidLoad()

        setupLayout()
    }

    private func setupLayout() {
        view.backgroundColor = .lightGray

        stackView.translatesAutoresizingMaskIntoConstraints = false
        view.addSubview(stackView)

        [hogeLabel, fugaLabel, moguLabel].forEach {
            $0.translatesAutoresizingMaskIntoConstraints = false
            $0.backgroundColor = .gray
            stackView.addArrangedSubview($0)
        }

        NSLayoutConstraint.activate([
            hogeLabel.topAnchor.constraint(equalTo: view.topAnchor, constant: 80),
            hogeLabel.rightAnchor.constraint(equalTo: view.rightAnchor, constant: -16),
            hogeLabel.leftAnchor.constraint(equalTo: view.leftAnchor, constant: 16),
        ])
    }
}

StackViewすこ。使いすぎるとパフォーマンス落ちるとの声もあるので注意。

StackView + ScrollViewでお手軽に拡張しやすいスクロール可能なレイアウトを構築できる。

こんな感じにスクロールしたい画面を作る時はUIScrollViewを使いますが、ScrollViewはその中身のViewのサイズによってスクロールが発生するかどうかが決まるので中身をViewを用意してあげないといけません。そのViewをStackViewにしてやれば、stackViewにaddArreangedSubViewするだけでサイズが変わってくれるので、Viewの高さ計算とかしなくても楽にスクロールが発生します。

• ScrollViewをSuperViewの全方向に0pxで制約を貼る
• ScrollViewにStackViewをaddSubView
• StackViewとFrameLayoutGuideを同じ横幅にする
• StackViewをContentLayoutGuideの上下左右に0pxで制約を貼る

後はstackViewにViewを追加していくだけでStackViewのサイズが画面サイズより大きくなった時にスクロールが発生します。

スクロールなし	スクロールあり

見えているViewの裏のViewのタッチイベントを取得したい時はhittestが使える

こういうScrollViewが重なった画面があるとします。

画像は今開発中の岩の位置を記録するアプリの画面です。

Before

そのままだと前面の縦ScrollViewがタッチイベントを吸収し、裏の横ScrollViewが反応しません。

After

そこでhittestです。
こいつでnilを返すようにoverrideすればタッチイベントを無視できます。

final class HeaderIgnorableScrollView: UIScrollView {
    override func hitTest(_ point: CGPoint, with event: UIEvent?) -> UIView? {
        let view = super.hitTest(point, with: event)

        if view == self,
           point.x < UIScreen.main.bounds.width && point.y < UIScreen.main.bounds.height * (9/16) {
            return nil
        }
        return view
    }
}

裏面の横ScrollViewもタッチできるようになりました。

感動しました。

同じ処理をアノテーション化して省略する時はPropertyWrapperが使える

例えばKeychainへのsetとgetの処理。

Before

こんな感じでgetとsetをしてたとします。

final class KeychainManager {

    /// キー
    struct Key {
        static let accessToken = "accessToken"
    }

    /// キーチェーンインスタンス
    static let keychain = Keychain(service: Bundle.main.bundleIdentifier ?? "")

    static var accessToken: String {
        get { get(key: Key.accessToken) ?? "" }
        set { set(key: Key.accessToken, value: newValue) }
    }

    private static func get<T>(key: String) -> T? {
        do {
            return try KeychainManager.keychain.get(key) as? T

        } catch {
            print(error.localizedDescription)
            assertionFailure("Keychainからのデータの取得に失敗しました。")
            return nil
        }
    }

    private static func set(key: String, value: String) {
        do {
            return try KeychainManager.keychain.set(value, key: key)

        } catch {
            print(error.localizedDescription)
            assertionFailure("Keychainへのデータの保存に失敗しました。。")
        }
    }
}

// 利用時
KeychainManager.accessToken = accessToken // set
let accessToken = KeychainManager.accessToken

After

propertyWrapperを使えばsetとかgetを書かなくてよくなります。

import KeychainAccess

@propertyWrapper
class KeychainStorage<T: LosslessStringConvertible> {

    private let key: String

    var keychain: Keychain {
        guard let identifier = Bundle.main.object(forInfoDictionaryKey: UUID().uuidString) as? String else {
            return Keychain(service: "")
        }
        return Keychain(service: identifier)
    }

    init(key: String) {
        self.key = key
    }

    var wrappedValue: T? {
        get {
            do {
                guard let result = try keychain.get(key) else { return nil }
                return T(result)
            } catch {
                print(error.localizedDescription)
                return nil
            }
        }
        set {
            do {
                guard let new = newValue else {
                    try keychain.remove(key)
                    return
                }
                try keychain.set(String(new), key: key)
            } catch {
                print(error.localizedDescription)
            }
        }
    }
}

final class KeychainDataHolder {

    private enum Key: String {
        case uid = "_accessToken"
    }

    static let shared: KeychainDataHolder = KeychainDataHolder()

    private init() {}

    @KeychainStorage(key: Key.accessToken.rawValue)
    var accessToken: String?

}

// 利用時
KeychainDataHolder.shared.accessToken = accessToken // set
let accessToken = KeychainDataHolder.shared.accessToken //get

keyとプロパティを増やせばsetとgetは書かなくてよくなりますね。

終わりに

読んでいただいてありがとうございました！
もっといっぱいあると思うんですけどいざ出そうとすると出てこないんですよね。
明日は@takashicoさんです！