Как реализовать Выбор ячейки в UICollectionView didSelectItemAt?

Question

[Dima Grib]

Приветствую
Первый раз работаю с UICollectionView и делаю программно, не могу добиться срабатывания метода func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath)
Буду очень признателен за Ваши советы

Код следующий

class UserViewController: UIViewController, UICollectionViewDelegate, UICollectionViewDelegateFlowLayout, UICollectionViewDataSource {

let collectionView: UICollectionView = {
    let layout = UICollectionViewFlowLayout()
    layout.minimumLineSpacing = 16
    layout.scrollDirection = .vertical
    let cv = UICollectionView(frame: .zero, collectionViewLayout: layout)
    cv.backgroundColor = .clear
    return cv
}()

let layout = UICollectionViewFlowLayout()

override func viewDidLoad() {
    super.viewDidLoad()
    setupViews()
}

 func setupViews() {
    collectionView.register(videoCell.self, forCellWithReuseIdentifier: imagesCellId)
    collectionView.register(previewCell.self, forCellWithReuseIdentifier: albumCellId)
    collectionView.delegate = self
    collectionView.dataSource = self
    collectionView.allowsSelection = true
    collectionView.isUserInteractionEnabled = true
    view.addSubview(collectionView)
    collectionView.translatesAutoresizingMaskIntoConstraints = false
    collectionView.topAnchor.constraint(equalTo: view.topAnchor, constant: 20).isActive = true
    collectionView.bottomAnchor.constraint(equalTo:  view.bottomAnchor, constant: 5).isActive = true
    collectionView.rightAnchor.constraint(equalTo: view.rightAnchor, constant: 0).isActive = true
    collectionView.leftAnchor.constraint(equalTo: view.leftAnchor, constant: 0).isActive = true
}


func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
if indexPath.section == 1 {
    let cell = collectionView.dequeueReusableCell(withReuseIdentifier: albumCellId, for: indexPath) as! previewCell
return cell
}
    let cell = collectionView.dequeueReusableCell(withReuseIdentifier: imagesCellId, for: indexPath) as! videoCell
return cell
}
}


func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) {
self.collectionView.selectItem(at: IndexPath(item: 0, section: 0), animated: true, scrollPosition: .left)
self.collectionView(self.collectionView, didSelectItemAt: IndexPath(item: 0, section: 0))
}



////////////////////////////////////////////////
////////////////////////////////////////////////




class videoCell: UICollectionViewCell, UICollectionViewDelegate, UICollectionViewDelegateFlowLayout, UICollectionViewDataSource, AVPlayerViewControllerDelegate, UIGestureRecognizerDelegate {

let collectionView: UICollectionView = {
    let layout = UICollectionViewFlowLayout()
    layout.minimumLineSpacing = 30
    layout.scrollDirection = .horizontal
    let cv = UICollectionView(frame: .zero, collectionViewLayout: layout)
    cv.backgroundColor = UIColor.clear
    //cv.allowsSelection = true
    return cv
}()

override init(frame: CGRect) {
    super.init(frame: frame)
    setup()
}

func setup() {
    addSubview(collectionView)
    collectionView.translatesAutoresizingMaskIntoConstraints = false
    collectionView.topAnchor.constraint(equalTo: self.topAnchor, constant: 15).isActive = true
    collectionView.bottomAnchor.constraint(equalTo:  self.bottomAnchor, constant: 15).isActive = true
    collectionView.rightAnchor.constraint(equalTo: self.rightAnchor, constant: 5).isActive = true
    collectionView.leftAnchor.constraint(equalTo: self.leftAnchor, constant: 5).isActive = true
    collectionView.delegate = self
    collectionView.dataSource = self
    collectionView.allowsSelection = true
    collectionView.isUserInteractionEnabled = true
    collectionView.register(InsideVideoCell.self, forCellWithReuseIdentifier: videCellId)
    tap.delegate = self
    tap.cancelsTouchesInView = false
    tap.isEnabled = true
}

func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
    let cell = collectionView.dequeueReusableCell(withReuseIdentifier: videCellId, for: indexPath) as! InsideVideoCell
return cell
}

required init?(coder aDecoder: NSCoder) {
    fatalError("Error VideoCell")
}


 private class InsideVideoCell: UICollectionViewCell {
public var videoPlayerController = AVPlayerViewController()
.....
}

Comments

[doublench21]

После просмотра данного кода, я кажется "ослеп". Приведите код в человекочитаемый вид. Покажите скриншот экрана.

[Dima Grib]

Простите, поправил

---

[doublench21]

Dima Grib, Что-то вы крайне странное с этим методом вытворяете. Конечно родительский метод didSelect ничего не получает, потому-что его получают дочерние. А вы наоборот берете и пытаетесь обработать касания в родительском. Обрабатывайте didSelect в дочерних коллекциях.

[Dima Grib]

doublench21, Я пробывал это делать в дочерних. Сейчас еще раз повторно попробывал но не срабатывает

К родительском классе ячейку у меня

tap.delegate = self
tap.cancelsTouchesInView = false
tap.isEnabled = true

К ячейке добавил
cell.addGestureRecognizer(tap)

В методе didSelectItemAt

func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) {
        self.collectionView.selectItem(at: IndexPath(item: 0, section: 0), animated: true, scrollPosition: .bottom)
        self.collectionView(self.collectionView, didSelectItemAt: IndexPath(item: 0, section: 0))
}

Все методы в дочерних классах срабатывают
CollectionView willDisplay VideoCell
CollectionView didEndDisplaying VideoCell
Prepare For Reuse

Что я мог упустить?

[doublench21]

Dima Grib, Вы меня не поняли. Зачем дабавлять GestureRecognizer??

ParentCV:

• Cell 1: ChildCV1
  
• Cell 2: ChildCV2
  

ParentCVCell1:

• ChildCV1.delegate = self
  
• ChildCV1.dataSource = self
  
• Тут и надо обрабатывать в didSelectItemAt нажатие на ячейки ChildCV1
  

ParentCVCell2:

• ChildCV2.delegate = self
  
• ChildCV2.dataSource = self
  
• Тут и надо обрабатывать в didSelectItemAt нажатие на ячейки ChildCV2
  

ШОК

Answer 1

[Сергей Дунаев]

Из того кода, что написан, максимально приближенный рабочий вариант будет выглядеть следующим образом. Чтобы хорошо разобраться в теме рекомендую посмотреть этот курс по Collection View.

import UIKit
import AVKit

class ViewController: UIViewController, UICollectionViewDelegate, UICollectionViewDataSource {
    
    let imagesCellId = "ImageCellId"
    let albumCellId = "AlbumCellId"
    
    let collectionView: UICollectionView = {
        let layout = UICollectionViewFlowLayout()
        layout.minimumLineSpacing = 16
        layout.scrollDirection = .vertical
        layout.itemSize = CGSize(width: 50, height: 50)
        
        let cv = UICollectionView(frame: .zero, collectionViewLayout: layout)
        cv.backgroundColor = .clear
        
        return cv
    }()
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        setupViews()
    }
    
    func setupViews() {
        collectionView.register(VideoCell.self, forCellWithReuseIdentifier: imagesCellId)
        collectionView.register(PreviewCell.self, forCellWithReuseIdentifier: albumCellId)
        collectionView.delegate = self
        collectionView.dataSource = self
        collectionView.allowsSelection = true
        collectionView.isUserInteractionEnabled = true
        
        view.addSubview(collectionView)
        
        collectionView.translatesAutoresizingMaskIntoConstraints = false
        collectionView.topAnchor.constraint(equalTo: view.topAnchor, constant: 20).isActive = true
        collectionView.bottomAnchor.constraint(equalTo:  view.bottomAnchor, constant: 5).isActive = true
        collectionView.rightAnchor.constraint(equalTo: view.rightAnchor, constant: 0).isActive = true
        collectionView.leftAnchor.constraint(equalTo: view.leftAnchor, constant: 0).isActive = true
    }
    
    func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
        if indexPath.section == 1 {
            let cell = collectionView.dequeueReusableCell(withReuseIdentifier: albumCellId, for: indexPath) as! PreviewCell
            cell.backgroundColor = UIColor.red
            return cell
        }
        
        let cell = collectionView.dequeueReusableCell(withReuseIdentifier: imagesCellId, for: indexPath) as! VideoCell
        
        cell.backgroundColor = UIColor.green
        return cell
    }
    
    func numberOfSections(in collectionView: UICollectionView) -> Int {
        return 2
    }
    
    func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
        return 1
    }
    
    func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) {

        // Вот тут совсем не понятно, что хотел сказать автор
        
//        collectionView.selectItem(at: IndexPath(item: 0, section: 0), animated: true, scrollPosition: .left)
//        collectionView(collectionView, didSelectItemAt: IndexPath(item: 0, section: 0))
        
        print("Выбрана ячейка: (\(indexPath.section), \(indexPath.item))")
    }
    
}

class VideoCell: UICollectionViewCell, AVPlayerViewControllerDelegate {
    var videoPlayerController = AVPlayerViewController()
}
class PreviewCell: UICollectionViewCell {

}

Comments

[Dima Grib]

Шикарно. Благодарю Вас. Курс обязательно куплю и пройду
Последний вопрос

- В чем может быть проблема, если в секции где imageView все отлично работает

- В секции где к cell cell.addSubview(videoPlayerController.view) не работает. Если убрать suview c видео то все работает.

- В коде вроде все отмечено для корректной работы

cell.contentView.isUserInteractionEnabled = true
            cell.videoPlayerController.view.isUserInteractionEnabled = true

[Сергей Дунаев]

А что именно не работает? Дайте весь кусок кода.

[Dima Grib]

в дочернем классе, который отвечает за секцию с видео, не работает метод didselect если к cell.addsubview добавлено view плеера, если вью плеера убрать, то didselect работает

class VideoCollectionViewCell: UICollectionViewCell, UICollectionViewDelegate, UICollectionViewDelegateFlowLayout, UICollectionViewDataSource, AVPlayerViewControllerDelegate {

    
    let videCellId = "CellId"
    var player = AVPlayer()
    let indexPath = IndexPath(item: 0, section: 0)
    
    
    var stories: [String]? {
        didSet {
            collectionView.reloadData()
        }
    }
    
    
    let collectionView: UICollectionView = {
        let layout = UICollectionViewFlowLayout()
        layout.minimumLineSpacing = 30
        layout.scrollDirection = .horizontal
        let cv = UICollectionView(frame: .zero, collectionViewLayout: layout)
        cv.backgroundColor = UIColor.clear
        return cv
    }()
    
    
    override init(frame: CGRect) {
        super.init(frame: frame)
        setup()
    }
    
    
    func setup() {
        //backgroundColor = .cyan
        addSubview(collectionView)
        collectionView.translatesAutoresizingMaskIntoConstraints = false
        collectionView.topAnchor.constraint(equalTo: self.topAnchor, constant: 15).isActive = true
        collectionView.bottomAnchor.constraint(equalTo:  self.bottomAnchor, constant: 15).isActive = true
        collectionView.rightAnchor.constraint(equalTo: self.rightAnchor, constant: 5).isActive = true
        collectionView.leftAnchor.constraint(equalTo: self.leftAnchor, constant: 5).isActive = true
        collectionView.delegate = self
        collectionView.dataSource = self
        collectionView.allowsSelection = true
        collectionView.register(InsideVideoCell.self, forCellWithReuseIdentifier: videCellId)
    }
    
    
    func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
        return stories?.count ?? 5
    }
    
    
    func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
        let cell = collectionView.dequeueReusableCell(withReuseIdentifier: videCellId, for: indexPath) as! InsideVideoCell
        
        if let storiesArray = stories?[indexPath.item] {
            let url = URL(string: storiesArray)
            let playerItem = AVPlayerItem(url: url!)
            self.player = AVPlayer(playerItem: playerItem)
            player.actionAtItemEnd = .none
            let videoPlayerLayer = AVPlayerLayer(player: self.player)
            videoPlayerLayer.videoGravity = AVLayerVideoGravity.resizeAspectFill
            videoPlayerLayer.frame = cell.bounds
            cell.videoPlayerController.player = self.player
            cell.videoPlayerController.view.frame = videoPlayerLayer.frame
            cell.videoPlayerController.view.clipsToBounds = true
            cell.videoPlayerController.view.layer.cornerRadius = 15
            cell.contentView.isUserInteractionEnabled = true
            cell.videoPlayerController.view.isUserInteractionEnabled = true
            cell.addSubview(cell.videoPlayerController.view)
        }
        return cell
    }
    
    
    func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) {
        print("Выбрана ячейка: (\(indexPath.section), \(indexPath.item))")
    }
    
    
    func collectionView(_ collectionView: UICollectionView, layout collectionViewLayout: UICollectionViewLayout, sizeForItemAt indexPath: IndexPath) -> CGSize {
        return CGSize(width: frame.width - 174, height: frame.height - 18)
    }
  
    
    required init?(coder aDecoder: NSCoder) {
        fatalError("Error VideoCell")
    }
    
    

    private class InsideVideoCell: UICollectionViewCell, AVPlayerViewControllerDelegate {
        
        public var videoPlayerController = AVPlayerViewController()

        func addControls() {
            videoPlayerController.showsPlaybackControls = false
        }
        
        override init(frame: CGRect) {
            super.init(frame: frame)
            setup()
            addControls()
        }

        func setup() {
            videoPlayerController.delegate = self
        }

        required init?(coder aDecoder: NSCoder) {
            fatalError("Error ImagesCell")
        }
    }
}

[Сергей Дунаев]

Насколько я понял вы хотите сделать в ячейке еще одну вложенную collection view. Верно?
Что касается работоспособности didSelect, то проблема заключается в том, что на всей ячейке располагается представление плеера, он-то и перехватывает все жесты. Соответственно, плеер надо подложить под основное представление ячейки. Воспользуйтесь методом insertSubview(_:at:), вместо addSubview, и укажите 0 индекс, чтобы плеер добавился под основное представление ячейки. И сделайте прозрачный фон для ячейки.

[Dima Grib]

Сергей Дунаев, очень круто. Все работает. Очень Вам благодарен. Возьму курс и разберусь. Вы очень помогли!!!!!!!!