質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.76%

  • Swift

    6700questions

    Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

  • Xcode

    3850questions

    Xcodeはソフトウェア開発のための、Appleの統合開発環境です。Mac OSXに付随するかたちで配布されています。

  • iOS

    3798questions

    iOSとは、Apple製のスマートフォンであるiPhoneやタブレット端末のiPadに搭載しているオペレーションシステム(OS)です。その他にもiPod touch・Apple TVにも搭載されています。

[Swift] CollectionViewの複数のセル内容を表示させる方法

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 147

s1209

score 5

CollectionViewのセルに編集画面で入力した値を表示させ続けたいのですが、ある1つのセルに表示させたあと違うセルに表示させると、前に表示させたセルの内容が消えてしまいます。

流れとしては、ViewControllerのCollectionViewのセルをタップ→編集画面に遷移→TextFieldに表示させたい値を入力→完了ボタンを押すと画面がViewControllerに戻り、タップしたセルのみに値が表示される。
タップしたセルの番号(cellNumber)とindexPath.rowが一致した時に値を表示させています。

値を表示させることはできるのですが、常に1つのセルしか表示させられません。

cellForRowAtの中身が間違ってるのでしょうか?

<該当コード(一部省略)>

import UIKit

class ViewController: UIViewController, UICollectionViewDelegate, UICollectionViewDataSource, UICollectionViewDelegateFlowLayout  {

    static var cellNumber: Int = 0
    @IBOutlet weak var collectionView: UICollectionView!

    static var testArray: [Class] = []

    override func viewDidLoad() {
        super.viewDidLoad()

        collectionView.delegate = self
        collectionView.dataSource = self

        collectionView.register(UINib(nibName: "CollectionViewCell", bundle: nil), forCellWithReuseIdentifier: "CollectionViewCell")

    }


    override func viewWillAppear(_ animated: Bool) {

//        取得
        if let data = UserDefaults.standard.object(forKey: "TimeTable") as? Data, let array = NSKeyedUnarchiver.unarchiveObject(with: data) as? [Class] {
            print("取得:\(array)")
        } else {
            print("Error")
        }

        collectionView.reloadData()
    }


//    セルが選択された時
    func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) {

        ViewController.cellNumber = indexPath.row

        if ViewController.cellNumber == indexPath.row {
            print("セル番号:\(indexPath.row)")
        }

        performSegue(withIdentifier: "Modal1", sender: self)
    }


//    セルの内容
    func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
        let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "CollectionViewCell", for: indexPath) as! CollectionViewCell
        cell.backgroundColor = UIColor.groupTableViewBackground

        for data in ViewController.testArray {
        if indexPath.row == data.cellNumber {
            cell.classLabel.text = data.className
            cell.roomLabel.text = data.roomName
        } else {
            cell.classLabel.text = ""
            cell.roomLabel.text = ""
        }
        }
        return cell
    }
import UIKit

class Class: NSObject, NSCoding {


    let className: String
    let roomName: String
    let cellNumber: Int

    init(className: String, roomName: String, cellNumber: Int) {

        self.className = className
        self.roomName = roomName
        self.cellNumber = cellNumber
    }

    func encode(with aCoder: NSCoder) {
        aCoder.encode(self.className, forKey: "name")
        aCoder.encode(self.roomName, forKey: "room")
        aCoder.encode(self.cellNumber, forKey: "number")
    }

    required init?(coder aDecoder: NSCoder) {
        self.className = aDecoder.decodeObject(forKey: "name") as! String
        self.roomName = aDecoder.decodeObject(forKey: "room") as! String
        self.cellNumber = aDecoder.decodeInteger(forKey: "number")
    }
}

class Edit: UIViewController {

    @IBOutlet weak var classTextField: UITextField!
    @IBOutlet weak var roomTextField: UITextField!

    override func viewDidLoad() {
        super.viewDidLoad()

    }


//    完了ボタンを押した時
    @IBAction func tappedButton(_ sender: UIButton) {

        let classData = Class(className:classTextField.text!, roomName: roomTextField.text!, cellNumber: ViewController.cellNumber)
        ViewController.testArray.append(classData)

//        保存
        UserDefaults.standard.set(NSKeyedArchiver.archivedData(withRootObject: ViewController.testArray), forKey: "TimeTable")
        UserDefaults.standard.synchronize()

        print("配列の中身:\(ViewController.testArray)")
        print("保存:\(UserDefaults.standard.set(NSKeyedArchiver.archivedData(withRootObject: ViewController.testArray), forKey: "TimeTable"))")

        dismiss(animated: true) {}
    }
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

下記の箇所でcellに空文字を設定しているからではないでしょうか?

if indexPath.row == data.cellNumber {
     cell.classLabel.text = data.className
     cell.roomLabel.text = data.roomName
} else {

     // ↓空文字を設定
     cell.classLabel.text = ""
     cell.roomLabel.text = ""
}
for data in ViewController.testArray {
        if indexPath.row == data.cellNumber {
            cell.classLabel.text = data.className
            cell.roomLabel.text = data.roomName
        }
}

[追記]

if let data = ViewController.testArray.first(where: { $0.cellNumber == indexPath.row }) {
    cell.roomLabel.text = data.roomName
    cell.classLabel.text = data.className
} else {
    cell.classLabel.text = ""
    cell.roomLabel.text = ""
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/05/09 16:27

    ありがとうございます。
    空文字部分を消したらセルに値を表示させ続けることができました!

    しかし、表示させるセルを増やすと、以前に表示させていたセルの値が指定していないセルに複製されて表示されてしまいます。
    このような場合はreloadDataの書き方が悪いのでしょうか?

    キャンセル

  • 2018/05/09 18:37

    全体像がわからないので可能性でしか言えないのですが、カスタムセル側に設定した値を保持するとどうなりますでしょうか?

    let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "CollectionViewCell", for: indexPath) as! CollectionViewCell
    はセルを再利用するという意味で、以前表示されていたセルを使用します。

    キャンセル

  • 2018/05/09 18:58

    カスタムセルには、ドラッグ&ドロップでラベルを配置しているだけでコードを書いたり値を設定したりなどはしていません…。

    セルの再利用が原因の可能性もあるのでしょうか…

    キャンセル

  • 2018/05/09 19:18

    セルを再利用しているので、以前使用していた値が表示されているのかと思いました。セルに値を取得する部分を追記したコードに変えるとどうなりますでしょうか?

    キャンセル

  • 2018/05/09 20:23

    ありがとうございます!
    コードを変えてみたら複製されなくなりましたが、1つのセル内に表示させたい値が2種類あり、追記コードだと1つの値は複製されずもう1つの値は複製される、という表示になってしまいました。。。

    このような場合はどこを書き換えればいいでしょうか?

    キャンセル

  • 2018/05/09 20:52

    間違いがあったので修正しました。同じ変数に空文字設定していました。。。

    キャンセル

  • 2018/05/09 21:30

    私も間違いに気づきませんでした…

    修正後、希望通りに表示させることができました。
    しばらく悩んでいたところなので解決できて本当に嬉しいです。

    ちなみに、first(where:)のコードは初見なので調べてみたのですが、配列の最初の要素を取り出す?という解釈で合ってますでしょうか?

    キャンセル

  • 2018/05/10 06:22

    はい、その認識で合っています。
    参考記事
    https://qiita.com/shtnkgm/items/928630d692cf1e5b0846

    キャンセル

  • 2018/05/10 08:18

    ありがとうございました。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.76%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • Swift

    6700questions

    Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

  • Xcode

    3850questions

    Xcodeはソフトウェア開発のための、Appleの統合開発環境です。Mac OSXに付随するかたちで配布されています。

  • iOS

    3798questions

    iOSとは、Apple製のスマートフォンであるiPhoneやタブレット端末のiPadに搭載しているオペレーションシステム(OS)です。その他にもiPod touch・Apple TVにも搭載されています。