[Swift] CollectionView Cell _ Lingo Feedback

 

 

 

 

 

func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
        guard let cell = collectionView.dequeueReusableCell(withReuseIdentifier: SelectPlanetCollectionViewCell.identifier, for: indexPath) as? SelectPlanetCollectionViewCell else { return UICollectionViewCell() }
        
        let selectedPlanetName = planetList[indexPath.row].planetName
        let selectedPlanetImage = planetList[indexPath.row].planetImage
        
        cell.planetNameLabel.text = selectedPlanetName
        cell.planetImageView.image = selectedPlanetImage
        
        return cell
    }
    
    func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) {
        let cell = collectionView.dequeueReusableCell(withReuseIdentifier: SelectPlanetCollectionViewCell.identifier, for: indexPath) as? SelectPlanetCollectionViewCell
        
        let selectedPlanet = planetList[indexPath.row]
        
        if (cell?.isSelected) != nil {
            contentsViewController.planet = selectedPlanet
            navigationController?.pushViewController(contentsViewController, animated: true)
        }
    }

 

 

 

1. Cell의 identifier 

: 여러 개의 Cell 파일이 있을 수 있다. (꼭 하나의 Cell만 사용하는 게 아니니까)

-> identifier를 통해 불러올 수 있도록, 함수 내에서 cell 선언 시 활용하는 것

-> cell의 재사용 : dequeueReusableCell

    : 재사용할 일이 없다면, 굳이 dequeueReusableCell 안써도 된다! ex) 간단한 설정 뷰 등등

 

 

2. " 쌍따옴표 " 내부에 들어가는 문자열 주의

: 오타가 발생해도, 컴파일러가 인식할 수 없다. (인간 읽으라고 만든 것)

-> 따라서 해당 내용들은 여러번 사용될 경우, 따로 변수로 선언하고 쓰는 것이 좋다.

-> 오타가 나더라도 OK -> 해당 변수로 엮이면 코드는 오류 없이 돌아가니까.

 

 

3. Planet을 넘기면 좋지 않을까?

-> 기존에는 setPlanet 함수를 통해, planet 이름과 이미지 값을 넘김

-> 이후 행성 모델에 다른 데이터가 추가될 수 있으니까, Planet 자체를 넘기는 것이 좋지 않을까?

-> REFACTOR Commit으로 수정해서 머지했다 :)