Sprint 09 #1
Sprint 09 #1
Conversation
| override func prepare(for segue: UIStoryboardSegue, sender: Any?) { | ||
| if segue.identifier == ShowSingleImageSegueIdentifier { | ||
| let viewController = segue.destination as! SingleImageViewController | ||
| let indexPath = sender as! IndexPath | ||
| let image = UIImage(named: photosName[indexPath.row]) | ||
| viewController.image = image | ||
| } else { | ||
| super.prepare(for: segue, sender: sender) | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
здесь поехало форматирование, давай поправим
| if segue.identifier == ShowSingleImageSegueIdentifier { | ||
| let viewController = segue.destination as! SingleImageViewController | ||
| let indexPath = sender as! IndexPath |
There was a problem hiding this comment.
Никогда не используй force unwrap в тестовом проекте или проектах для портфолио.
Проверяющий обращает на них пристальное внимание и может сделать поспешные выводы. Гораздо выигрышнее будет смотреться безопасное извлечение опционалов через if let a = b, и мягкий каст if let c = d as? D
| @IBOutlet private var tableViewImage: UITableView! | ||
|
|
||
| private var photosName = [String]() | ||
| private let ShowSingleImageSegueIdentifier = "ShowSingleImage" |
There was a problem hiding this comment.
предлагаю придерживаться распространненого правила по именованию свойств – нижний camelCase. Эта константа не является отдельным типом
| // @IBOutlet weak var exitButton: UIButton! | ||
| // @IBOutlet weak var imageProfile: UIImageView! | ||
| // @IBOutlet weak var name: UILabel! | ||
| // @IBOutlet weak var nickName: UILabel! | ||
| // @IBOutlet weak var status: UILabel! | ||
|
|
There was a problem hiding this comment.
душу наверное греет, но лучше все-таки мертвый код удалять-вырезать-выкорчевывать :)
| let image = UIImageView() | ||
| let labelName = UILabel() | ||
| let labelNickname = UILabel() | ||
| let labelStatus = UILabel() | ||
| let button = UIButton.systemButton(with: UIImage(systemName: "ipad.and.arrow.forward")!, target: ProfileViewController.self, action: nil) | ||
|
|
There was a problem hiding this comment.
пока что не хватает модификаторов доступа
|
|
||
|
|
||
| labelName.text = "Екатерина Новикова" |
There was a problem hiding this comment.
максимум одна пустая строка между логическими блоками
|
|
||
| @IBOutlet weak private var buttonBack: UIButton! | ||
| @IBOutlet weak var imageView: UIImageView! | ||
| @IBOutlet weak var scrollView: UIScrollView! |
| scrollView.setContentOffset(CGPoint(x: x, y: y), animated: false) | ||
| } | ||
|
|
||
| @IBAction func clickSharingButton(_ sender: Any) { |
|
Поправил все замечания в коммите "fix after review" |
| @IBOutlet weak private var imageView: UIImageView! | ||
| @IBOutlet weak private var scrollView: UIScrollView! | ||
|
|
||
| @IBOutlet weak var sharingButton: UIButton! |
|
Можно смерджить |
3 participants
No description provided.