Swift: UICollectionView выбор проблем indexPath ячейки

я пытаюсь сделать Collection View когда кто-то выбирает cell и для каждого выбора он принимает их к другому View Controller который содержит содержимое выборки. Однако я сталкиваюсь с трудностями, как только я применяю эту строку кода к didSelectItemAtIndexPath;

self.performSegueWithIdentifer("showDetail", sender: self)

а затем запустите его в симуляторе cell отбор работающих по indexPath но его запоминание Выбора каждый раз, когда я выбираю new cell. Так, например,cell имеет фото и ярлык и если я выберу первый cell на indexPath the segue берет меня сначала на пустой вид, а затем на мой выбранный cell. Если я выберу другой cell, номер 3 на indexPath пустой вид теперь первый cell из моего предыдущего выбора, после которого он принимает мой выбранный третий cell . Это происходит каждый раз. Если я удалю performSegueWithIdentifer код (из Xcode 6.2 (в 6.1.1 он был случайным)) выбор-мой предыдущий выбор и никогда не мой "selectedCell", но тогда, по крайней мере, его единственный выбор один раз вместо двух, чтобы добраться до view. Там что-то происходит не так на indexPath. Это код для моего prepareForSegue

override func prepareForSegue(segue: UIStoryBoardSegue, sender: AnyObject?) {
      if segue.identifer == "showDetail" {
          let detailVC:DetailViewController = segue.destinationViewController as DetailViewController
             detailVC.selectedImageName = selectedImage
             detailVC.selectedLabel = selectedLabels
             }
      }

я застрял на том, что делать и какое решение применять. Я держу performSegueWithIdentifer код и создать Equatable для реализации find(array, selection) на indexPath? Или я могу написать цикл (что кажется намного проще), который будет проходить через indexPath на основе выбора, и это удалит ячейку, которая больше не выбрана. Однако я не конечно, какое условие писать в цикле, потому что я не знаю значения свойства 'selectedCell', потому что его необязательно.

for (index, value) in enumerate(cellItems) {
//something here to remove 'selectedItem' in the indexPath
}

если я удалить performSegueWithIdentifer код didSelectItemAtIndexPath что я могу сделать в моем prepareForSegue чтобы получить выбор на правильном indexPath?

отредактируйте полный код в didSelectItemAtIndexPath

override func collectionView(collectionView: UICollectionView, didSelectItemAtIndexPath indexPath: NSIndexPath) {
     selectedImage = cellImages[indexPath.row] as String
     selectedLabels = cellLabels[indexPath.row] as String
     self.performSegueWithIdentifer("showDetail", sender: self)
   }

я попытался изменить отправителя в performSegueWithIdentifer to indexPath но проблема все равно остается.

Изменить 2 полный код мой CollectionViewController

class CollectionViewController: UIViewController, UICollectionViewDataSource, UICollectionViewDelegate {

@IBOutlet weak var collectionView: UICollectionView!

var selectedImage = String()
var selectedLabels = String()

var cellImages:[String] = ["1.jpg", "2.jpg", "3.jpg", "4.jpg", "5.jpg", "6.jpg", "7.jpg", "8.jpg", "9.jpg", "10.jpg", "11.jpg", "13.jpg", "14jpg"]

var cellLabels:[String] = ["Photo 1", "Photo 2", "Photo 3", "Photo 4", "Photo 5", "Photo 6", "Photo 7", "Photo 8", "Photo 9", "Photo 10", "Photo 11", "Photo 12", "Photo 13", "Photo 14"]

func collectionView(collectionView: UICollectionView, numberOfNumberItemsInSection: Int) -> Int {
    return cellImages.count
}

func collectionView(collectionView: UICollectionView, cellForItemAtIndexPath indexPath: NSIndexPath) -> UICollectionViewCell {
    let cell: PhotoViewCell = collectionView.dequeueReuseableCellWithReuseIdentifier("Cell", forIndexPath: indexPath) as PhotoViewCell 
    cell.labelCell.text = cellLabels[indexPath.row]
    cell.ImageCell.image = UIImage(named: cellImages[indexPath.row])
    return cell
}

 override func collectionView(collectionView: UICollectionView, didSelectItemAtIndexPath indexPath: NSIndexPath) {
     selectedImage = cellImages[indexPath.row] as String
     selectedLabels = cellLabels[indexPath.row] as String
     self.performSegueWithIdentifer("showDetail", sender: self)
   }

override func prepareForSegue(segue: UIStoryBoardSegue, sender: AnyObject?) {
      if segue.identifer == "showDetail" {
          let detailVC:DetailViewController = segue.destinationViewController as DetailViewController
          detailVC.selectedImageName = selectedImage
          detailVC.selectedLabel = selectedLabels
          }
      }
}

PhotoViewCell

class PhotoViewCell: UICollectionViewCell {

@IBOutlet var labelCell: UILabel!
@IBOutlet var ImageCell: UIImage!

}

редактировать 3-исправлено

я попробовал ваше предложение, и, к сожалению, проблема все еще сохраняется на двойных представлениях-он все еще проходит два представления, прежде чем он приведет меня к фактическому выбранному cell. Я также немного изменил код в didSelectItemAtIndexPath но это все равно не решило проблему.

if let cell = collectionView.cellForItemAtIndexPath(indexPath) as? PhotoViewCell {

performSegueWithIdentifier("showDetail", sender: cell) 
}
другие предложения в моей StoryBoard я добавил а segue от меня Collection View cell мой DetailViewController, который имеет идентификатор "showDetail". Если я удалю segue ничто не может быть выбран из моего cells.

хотя кажется performSegueWithIdentifer код является триггером для двойных представлений, потому что когда я удаляю его,cell выбирается только один раз, проблема в том, что indexPath на cell выбор был неправильным, потому что он сначала выбирается на пустом представлении (это связано с showDetail segue?), который тогда ставит мой indexPath рассинхронизация.

EDIT-Solved

это остановило двойной выбор (тег performSegueWithIdentifier строка была удалена): -

override func collectionView(collectionView: UICollectionView, didSelectItemAtIndexPath indexPath: NSIndexPath) {
if let cell = collectionView.cellForItemAtIndexPath(indexPath) {
cellLabels[indexPath.row] as String
cellImages[indexPath.row] as String
   }
}

большое спасибо за вашу помощь !!!!

2 ответов


(Примечание: я обновил это для Swift 4 и более современных методов.)

Я придерживаюсь UIView как можно больше объектов.

override func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) {
    guard let cell = collectionView.cellForItem(at: indexPath) else { return }

    performSegue(withIdentifier: "showDetail", sender: cell)
}

затем в prepare(for:sender:)

override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
    switch segue.identifier {
    case "showDetail":
        guard let indexPath = (sender as? UIView)?.findCollectionViewIndexPath() else { return }
        guard let detailViewController = segue.destination as? DetailViewController else { return }

        detailViewController.selectedImageName = cellImages[indexPath.row]
        detailViewController.selectedLabel = cellLabels[indexPath.row]

    default: return
    }
}

я использовал расширение, которое я создал некоторое время назад findCollectionViewIndexPath()

extension UIView {

    func findCollectionView() -> UICollectionView? {
        if let collectionView = self as? UICollectionView {
            return collectionView
        } else {
            return superview?.findCollectionView()
        }
    }

    func findCollectionViewCell() -> UICollectionViewCell? {
        if let cell = self as? UICollectionViewCell {
            return cell
        } else {
            return superview?.findCollectionViewCell()
        }
    }

    func findCollectionViewIndexPath() -> IndexPath? {
        guard let cell = findCollectionViewCell(), let collectionView = cell.findCollectionView() else { return nil }

        return collectionView.indexPath(for: cell)
    }

}

у меня есть подозрение, что у вас уже есть сегмент в раскадровке и вам не нужно func collectionView(, didSelectItemAtIndexPath:), но в любом случае, подготовьте сегмент должен работать.


Swift 3.0

override func prepare(for segue: UIStoryboardSegue, sender: Any?) {

    let iPath = self.collectionView.indexPathsForSelectedItems
    let indexPath : NSIndexPath = iPath![0] as NSIndexPath
    let rowIndex = indexPath.row
    print("row index = \(rowIndex)")
}