Использовать локальное закрытие?

Question

Мне было отмечено, что есть лучший способ написать этот код без использования локального замыкания. Как я могу это сделать? Переключайте изображения по клику.

@IBAction func btnMuteMicPressed(_ sender: Any) {
    publisher.publishAudio.toggle()

    let buttonImage: UIImage  = {
        if !publisher.publishAudio {
            return #imageLiteral(resourceName: "group3-1")
        } else {
            return #imageLiteral(resourceName: "group3")
        }
    }()

    btnMuteMic.setImage(buttonImage, for: .normal)
}

Answer 1

Это можно сократить до,

@IBAction func btnMuteMicPressed(_ sender: Any) {
    publisher.publishAudio.toggle()
    let buttonImage = !publisher.publishAudio ? #imageLiteral(resourceName: "group3-1") : #imageLiteral(resourceName: "group3")
    btnMuteMic.setImage(buttonImage, for: .normal)
}

Я не думаю, что closure для получения buttonImage требуется здесь. Это определенно не означает, что ваш код ошибочен.

Answer 2

Закрытие определенно можно сбросить здесь:

@IBAction func btnMuteMicPressed(_ sender: Any) {
    publisher.publishAudio.toggle()

    let buttonImage: UIImage
    if !publisher.publishAudio {
        buttonImage = #imageLiteral(resourceName: "group3-1")
    } else {
        buttonImage = #imageLiteral(resourceName: "group3")
    }

    btnMuteMic.setImage(buttonImage, for: .normal)
}

Но в старом способе инициализации нет ничего неправильного.