Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

how to single select cell in collection view..?

That are code properly working but when i will scroll my collection view then another cell also selected, for example 18 images are available and first show six at run time when i will select any one of the position then next six position image automatic select. why at a time two cell selecting i am confuse here. please give me solution

enter image description here

here i have take 6 cell at main show storyboard

UICollectionViewFlowLayout *flowLayout = (UICollectionViewFlowLayout*)self.collectionView.collectionViewLayout;

flowLayout.minimumLineSpacing = 15;
CGFloat availableWidthForCells = CGRectGetWidth(self.collectionView.frame) - flowLayout.sectionInset.left - flowLayout.sectionInset.right - flowLayout.minimumInteritemSpacing *2;

cellWidth = availableWidthForCells /6;
    NSLog(@"cellWidth:%f",cellWidth);
flowLayout.itemSize = CGSizeMake(cellWidth, cellWidth);

This is my Didselect And didDeselect Method

-(void)collectionView:(UICollectionView *)collectionView didSelectItemAtIndexPath:(NSIndexPath *)indexPath{

    UICollectionViewCell* cell = [collectionView cellForItemAtIndexPath:indexPath];

    cell.layer.cornerRadius = cellWidth / 2.0;
    cell.layer.backgroundColor = [UIColor blackColor].CGColor;
    NSLog(@"INDEXPATH:-%ld",(long)indexPath.row);
}
-(void)collectionView:(UICollectionView *)collectionView didDeselectItemAtIndexPath:(NSIndexPath *)indexPath{

    UICollectionViewCell* cell = [collectionView cellForItemAtIndexPath:indexPath];
    cell.layer.cornerRadius = cellWidth / 2.0;
    cell.layer.backgroundColor = [UIColor whiteColor].CGColor;

}
like image 214
Chirag Sondagar Avatar asked Apr 26 '16 09:04

Chirag Sondagar


3 Answers

This is happening because the collectionView reuse the cells;

you should store the selected cell's IndexPath in a variable:

ObjC :

@property (nonatomic, retain) NSIndexPath *selectedIndexPath;

-(void)collectionView:(UICollectionView *)collectionView didSelectItemAtIndexPath:(NSIndexPath *)indexPath {

    UICollectionViewCell* cell = [collectionView cellForItemAtIndexPath:indexPath];

    cell.layer.backgroundColor = [UIColor blackColor].CGColor;
    NSLog(@"INDEXPATH:-%ld",(long)indexPath.row);

    self.selectedIndexPath = indexPath
}
-(void)collectionView:(UICollectionView *)collectionView didDeselectItemAtIndexPath:(NSIndexPath *)indexPath{

    UICollectionViewCell* cell = [collectionView cellForItemAtIndexPath:indexPath];
    cell.layer.backgroundColor = [UIColor whiteColor].CGColor;

    self.selectedIndexPath = nil
}

Swift :

var selectedIndexPath: IndexPath?

func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) {
    let cell = collectionView.cellForItem(at: indexPath)

    cell.layer.backgroundColor = UIColor.black

    self.selectedIndexPath = indexPath
}

func collectionView(_ collectionView: UICollectionView, didDeselectItemAt indexPath: IndexPath) {
    let cell = collectionView.cellForItem(at: indexPath)

    cell.layer.backgroundColor = UIColor.white

    self.selectedIndexPath = nil
}

than in "cell for row at indexPath" check :

ObjC :

- (UICollectionViewCell *)collectionView:(UICollectionView *)collectionView cellForItemAtIndexPath:(NSIndexPath *)indexPath {

    UICollectionViewCell* cell = [collectionView cellForItemAtIndexPath:indexPath];
    cell.layer.cornerRadius = cellWidth / 2.0;
    if (self.selectedIndexPath != nil && indexPath == self.selectedIndexPath) {
        cell.layer.backgroundColor = [UIColor blackColor].CGColor;
    else {
        cell.layer.backgroundColor = [UIColor whiteColor].CGColor;
   }

    return cell
}

Swift :

func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {

    let cell = collectionView.cellForItem(at: indexPath)
    cell.layer.cornerRadius = cellWidth / 2

    if self.selectedIndexPath != nil && indexPath == self.selectedIndexPath {
        cell.layer.backgroundColor = UIColor.black
    else {
        cell.layer.backgroundColor = UIColor.white
    }
}
like image 144
Alberto Scampini Avatar answered Nov 20 '22 20:11

Alberto Scampini


Thanks for Alberto Scampini this code for swift 3.1

var selectedIndexPath: IndexPath?

 func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {

    let cell = self.collectionView.dequeueReusableCell(withReuseIdentifier: "SegmentChoiceCVCell", for: indexPath) as! SegmentChoiceCVCell
    //configure cell

    if selectedIndexPath != nil && indexPath == selectedIndexPath {
        cell.checkIcon.backgroundColor = UIColor.black

    }else{

        cell.checkIcon.backgroundColor = UIColor.white
    }

    return cell

}



func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) {

    let cell = collectionView.cellForItem(at: indexPath) as! SegmentChoiceCVCell

        cell.checkIcon.backgroundColor = UIColor.black
    self.selectedIndexPath = indexPath

}

func collectionView(_ collectionView: UICollectionView, didDeselectItemAt indexPath: IndexPath) {
    let cell: SegmentChoiceCVCell = collectionView.cellForItem(at: indexPath) as! SegmentChoiceCVCell

        cell.checkIcon.backgroundColor = white
    selectedIndexPath = nil
}
like image 26
Mostafa Sh Avatar answered Nov 20 '22 21:11

Mostafa Sh


I know that I am late to the party. I had the same problem. And after looking around in stackoverflow. Here's my solution. It may help starters understand the UICollectionViewCell a bit easier.

"cellForItemAtIndexPath" method controls how the cell will be display. And cells that are off-screen will not be updated.

Swift 4

First, you need to sub-class your cell.

class SubclassedCell: UICollectionViewCell {

      @IBOutlet var cellImage: UIImageView!
      @IBOutlet var cellCaption: UILabel!

}

extension SubclassedCell{

   func highlightEffect(){
      self.layer.borderWidth = 3.0
      self.layer.borderColor = UIColor.lightGray.cgColor
   }

   func removeHighlight(){
      
      self.layer.borderColor = UIColor.clear.cgColor
   }

} 

And now for the "cellForItemAt" method.

   func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
    
    
    let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "blahblahblah", for: indexPath) as! SubclassedCell
  
    
    cell.cellImage.image = yourImages[indexPath.item]
    cell.cellCaption.text = yourImageNames[indexPath.item]
    cell.removeHighlight()  // Call subclassed cell method.
    if indexPath.item == selectedItem {
        cell.highlightEffect() // Call subclassed cell method.
        preSelected = IndexPath(item: indexPath.item, section: indexPath.section)
        
        
    }
    
    collecttionRef = collectionView
    return cell
    
}

Then, for the "didSelectItemAt" method

  func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath){
    
    collectionView.allowsMultipleSelection = false
    if let cell = collectionView.cellForItem(at: indexPath) as! SubclassedCell?{
        let indexPathData = NSKeyedArchiver.archivedData(withRootObject: indexPath)
        UserDefaults.standard.set(indexPathData,forKey: "backgroundIndexPath")
        //I got some unwrapped crash for my App so I have to use UserDefault to fix the unwrapped problem. You could ignore these both lines.

        selectedItem = indexPath.item //selectedItem is used in the "cellForItemAt" method above.
        
        

        collectionView.reloadData() //update all cells. It could be heavy if you have many cells.
        
    }
   
}
like image 2
William Tong Avatar answered Nov 20 '22 20:11

William Tong