web-dev-qa-db-ja.com

Swift：セルのindexPathを選択するUICollectionViewの問題

誰かがcellを選択する_Collection View_を実行しようとしていますが、選択するたびに、選択内容を保持する別の_View Controller_に移動します。ただし、このコード行をdidSelectItemAtIndexPathに適用すると、問題が発生します。

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

シミュレータで実行すると、cellの選択はindexPathに従って動作しますが、新しいcellを選択するたびに選択が記憶されます。たとえば、各cellには写真とラベルがあり、cellで最初のindexPathを選択すると、segueは最初に空白のビューになり、次に選択したcellになります。 cellで別のindexPathの番号3を選択すると、空のビューは前の選択からの最初のcellになり、その後、選択した3番目のcellになります。そのたびにそれを行う。 performSegueWithIdentiferコードを削除すると（Xcode 6.2（6.1.1ではランダム））選択は以前の選択であり、「selectedCell」ではありませんが、viewを取得するために少なくとも2回ではなく1回のみ選択します。 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コードを削除すると、正しいindexPathで選択を取得するためにprepareForSegueで何ができますか？

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の送信者をindexPathに変更しようとしましたが、問題はまだ残っています。

EDIT 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に連れて行く前に2つのビューを渡しています。 didSelectItemAtIndexPathのコードも少し修正しましたが、それでも問題は解決しませんでした。

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

performSegueWithIdentifier("showDetail", sender: cell) 
}
_

ただし、他の提案に従って、StoryBoardで、_Collection View_ segueからcellを、_show_IDという識別子を持つDetailViewControllerに追加しました。 segueを削除すると、cellsから何も選択できません。

performSegueWithIdentiferコードはダブルビューのトリガーであるようですが、削除するとcellが1回しか選択されないため、問題はindexPath選択のcellが空のビューで最初に選択されるため正しくないことでした（それはshowDetail segue？）を処理することです。これにより、indexPathが同期しなくなります。

編集-解決済み

これにより、二重選択が停止しました（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
   }
}
_

助けてくれて本当にありがとうございます ！！！！

swiftuicollectionviewuicollectionviewcelldidselectrowatindexpath
10
j03T

（注：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:)は必要ないのではないかと疑っていますが、いずれにしても、セグエの準備は機能するはずです。

11
Jeffery Thomas

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)")
}
1
Samiul Islam Sami