Войти Регистрация
Код-ревью

9. Экран текущего заказа такси

Средняя

Swift · 44 строк

Экран показывает цену и время подачи машины и раз в 5 секунд обновляет их. TaxiAPI вызывает completion на фоновой очереди URLSession. order.price — Double.

Отметьте строки с ошибками.

В работе: Экран ожидания машины: пользователь держит его открытым 3–10 минут.

Как проверяем Нажмите на строку с ошибкой и коротко напишите, что не так. Баг засчитан, если отмечена хотя бы одна его строка. Зачёт — найдено не меньше 60 % багов с учётом серьёзности (критичный весит 3, обычный 2, мелкий 1), а лишних отметок не больше, чем багов в коде. Пустые строки, скобки и комментарии лишними не считаем. Ответ один на подход: после него показываем разбор.
UIKitretain cycleглавный потокTimer

Источник: Яндекс: мобильная разработка, секция «исправить ошибки в готовом коде». Условие пересказано своими словами.

Давали на собеседовании? Ответов пока мало — ваш поможет.
  1. final class OrderViewController: UIViewController {
  2. private let priceLabel = UILabel()
  3. private let etaLabel = UILabel()
  4. private let api: TaxiAPI
  5. private var timer: Timer?
  6. var orderId: String!
  8. init(api: TaxiAPI) {
  9. self.api = api
  10. super.init(nibName: nil, bundle: nil)
  11. }
  13. required init?(coder: NSCoder) { fatalError("init(coder:) не используется") }
  15. override func viewDidLoad() {
  16. super.viewDidLoad()
  17. timer = Timer.scheduledTimer(withTimeInterval: 5, repeats: true) { _ in
  18. self.refresh()
  19. }
  20. }
  22. private func refresh() {
  23. api.fetchOrder(id: orderId) { result in
  24. switch result {
  25. case .success(let order):
  26. let price = "\(order.price) ₽"
  27. self.priceLabel.text = price
  28. self.etaLabel.text = "Машина через \(order.etaMinutes) мин"
  29. case .failure:
  30. break
  31. }
  32. }
  33. }
  35. @objc private func cancelTapped() {
  36. api.cancelOrder(id: orderId) { [weak self] _ in
  37. self?.dismiss(animated: true)
  38. }
  39. }
  41. deinit {
  42. timer?.invalidate()
  43. }
  44. }