0

Swiftにてカスタムセル 二つ それぞれちがうVC から追加したいのですが

大元を Tab - Navi - List ( UIVC ) ほかの二つを Post Fol とします どちらも UIVC です

そのさい カスタムセル を表示する List にはどのように書けばよろしいでしょうか ?

func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return mainArray.count
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let postcell = tableView.dequeueReusableCell(withIdentifier: "postcell")!
        postcell.textLabel?.text = mainArray[indexPath.row]
        return postcell
    } 

だと ひとつしか 登録できない とおもうのですが

追記

大元のVC

class List: UIViewController, UITableViewDataSource, UITableViewDelegate {
    
    var mainArray: [String] = []
    let initArray: [String] = []
    
    override func viewDidLoad() {
        super.viewDidLoad()
        tableView.dataSource = self
        tableView.delegate = self
        
        // Do any additional setup after  the view.
    }
    
    @IBOutlet weak var tableView: UITableView!
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return mainArray.count
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let postcell = tableView.dequeueReusableCell(withIdentifier: "postcell")!
        postcell.textLabel?.text = mainArray[indexPath.row]
        return postcell
    }
    
    func tableView(_ tableView: UITableView, didSelectRowAt className: UITableViewCell) {
        
        switch className {
        case is PostCell:
            
            guard let viewControlelr = storyboard?.instantiateViewController(withIdentifier: "Post") as? Post else {
                return
            }; navigationController?.pushViewController(viewControlelr, animated: true)
        
            
        case is FolderCell:
            
            guard let list = storyboard?.instantiateViewController(withIdentifier: "List") as? List else {
                return
            }; navigationController?.pushViewController(list, animated: true)
            
        default:
            return
        }
        
    }

}

投稿するVC Post

class Post: UIViewController {

    var postString: String = ""
    
    @IBOutlet weak var postTextField: UITextField!
    
    override func viewDidLoad() {
        super.viewDidLoad()

        // Do any additional setup after loading the view.
    }
    
    @IBAction func postBack(_ sender: Any) {
    self.navigationController?.popToRootViewController(animated: true)
        
        postTextField.text = postString
        
        guard let list = tabBarController?.viewControllers?[0] as? List else {
            return
        }
        
        let postString = postTextField.text ?? ""
        
        list.tableView?.beginUpdates()
        list.tableView?.insertRows(at: [IndexPath(row: 0, section: 0)], with: .automatic)
        list.tableView?.endUpdates()
    } 

} 

Folder用のセルの追加のVC Fol

まだコードはないのですが Post とほぼ同じで 追加するカスタムセル が FolderCell なだけです

追記

List に protcol を追加すると

Protocol 'ListEntryDelegate' cannot be nested inside another declaration

と、extention では

Declaration is only valid at file scope 

Use of undeclared type 'ListEntryDelegate' 

というエラーが出ます

なにかちがうのでしょうか ?

追記 1

struct ListEntry {
    enum CellType: String {
        case post   = "postcell"
        case folder = "foldercell"
    }
    /// 種類
    var cellType: CellType
    
    /// テキスト
    var text: String
    
    var array: [String]
    
    // 一つのセルに表示する情報が増えればプロパティを増やす
    //...
}

に Post また Fol から

listView.mainArray.insert(folString, at: 0) 

しようとすると

Cannot convert value of type 'String' to expected argument type 'ListEntry' 

が出ます

追記 2

PostCell をタップしたときはPostにpostStringを入れて FolderCellをタップしたときはListのmainArrayにFolCellの配列folArrayを入れてNavigation Controllerで遷移したいのです

追記

Post でButtonを押した場合は PostCell を追加 Fol でButtonを押した場合は FolCell というのは Post のInsartになにを付け足せばいいのでしょうか ?

3
  • 「それぞれちがうVC から追加したい」と言いつつ、「どちらも UIVC です」とあるので、ご質問内容が私の頭の中でうまく整理できません。あまり抽象化しようとせず、関連するview controllerと、セルクラスのクラス名を全部具体的に挙げて説明し直してもらえないでしょうか?
    – OOPer
    Commented 2020年6月30日 9:19
  • ご質問内容の更新ありがとうございます。今は手が離せないのですぐに回答の形にまとめることは出来ませんが、やりたいことは概ね伝わってきたように思います。先に他の方が回答をあげてしまうかもしれませんが、のんびりお待ちいただければと思います。
    – OOPer
    Commented 2020年6月30日 9:55
  • わかりました、ありがとうございます
    – tony
    Commented 2020年6月30日 10:15

1 件の回答 1

0

更新後のご質問内容を読んで、以下のように理解しました。

  • 2種類のカスタムセルを表示したいUITableViewList(ViewController)に一つだけ
  • そのUITableViewでは、セクション0だけを使用する
  • そのセクション0内では、post用のセル、folder用のセルが混在して表示される

何か認識間違いがあれば、早めにお知らせください。

(動作に支障があるわけではないんですが、viewControllerにListなんて名前をつけるのは、あまりiOSアプリにはないのでコードが読みづらいですね。ListViewControllerか、せめてListVCくらいにしておいた方が良いと思います。作りかけのアプリのクラス名を変えるといろいろ本質的でないところで苦労する場合があるため、以下ではList(やPost)のままでいきます。)


さて、一つのUITableViewに「2種類のカスタムセルを表示したい」なんてことをするとなれば、「どこかにセルの種類を区別するための情報が必要」と言うことになります。

mainArrayとは別の配列を用意してそちらに情報を入れると言う方法は、アプリが複雑になるうちにどんどん処理が煩雑になっていずれ破綻するのでやめておきます。

「一つのセル」に対応する情報は、「一つの型」で表すようにしましょう。

ListEntry.swift

import Foundation

struct ListEntry {
    enum CellType: String {
        case post   = "postcell"
        case folder = "foldercell"
    }
    /// 種類
    var cellType: CellType
    
    /// テキスト
    var text: String
    
    // 一つのセルに表示する情報が増えればプロパティを増やす
    //...
}

ListmainArrayは、Stringの配列ではなく、このListEntryの配列にします。

class List: UIViewController, UITableViewDataSource, UITableViewDelegate {
    
    var mainArray: [ListEntry] = []
    //let initArray: [String] = []
    
    override func viewDidLoad() {
        super.viewDidLoad()
        tableView.dataSource = self
        tableView.delegate = self
        
        // Do any additional setup after  the view.
        //以下はstoryboardで設定しているなら不要
        tableView.register(PostCell.self, forCellReuseIdentifier: ListEntry.CellType.post.rawValue)
        tableView.register(FolderCell.self, forCellReuseIdentifier: ListEntry.CellType.folder.rawValue)
    }

    //...
}    

register(_:forCellReuseIdentifier:)の呼び出しは、storyboardでやっているのなら不要ですが、やっていないなら、セルの種類全てについて必ず呼び出しておいて下さい。


さて、実際にセルの種類によって処理を分けないといけないところは、ListEntrycellTypeプロパティの値をもとに場合分けしてやります。

例えば、tableView(_:cellForRowAt:)はこんな感じにしないといけないでしょう。

    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let listEntry = mainArray[indexPath.row]
        switch listEntry.cellType {
        case .post:
            let postcell = tableView.dequeueReusableCell(withIdentifier: listEntry.cellType.rawValue, for: indexPath) as! PostCell
            //実際には`textLabel`でなく、`PostCell`の専用プロパティを使用する必要があると思われる
            postcell.textLabel?.text = listEntry.text
            return postcell
        case .folder:
            let foldercell = tableView.dequeueReusableCell(withIdentifier: listEntry.cellType.rawValue, for: indexPath) as! FolderCell
            //実際には`textLabel`でなく、`FolderCell`の専用プロパティを使用する必要があると思われる
            foldercell.textLabel?.text = listEntry.text
            return foldercell
        }
    }

(ちなみに、危険とされているas!を使っているのは、「設定間違えてたらクラッシュしてくれ」と言う意図でわざとです。)


さて、tableView(_:didSelectRowAt:)の方ですが、「種類による場合わけ」が間違った形で書かれているので、大幅に書き直してもらわないと予測不能の障害が起こります。

どうせ大幅に書き換えるならと言うことで、Post(ViewController)内でListにアクセスしている部分を書き換えておきます。現在のコードですと、どのタブにどのVCを表示するのか、と言うデザイン上の変更があると、コードまで書き換えないといけなくなります。ある程度は仕方ないのですが、できるだけコードとデザインの分離をする、と言うのが良い設計と言うことになっています。

今のところPostでやっているのは要素の追加だけのようなので、次のようなプロトコルを用意して、Listクラスにそのプロトコルを実装させておきます。

(実際には「要素の編集」も必要な気がしますが、その辺り「2種類のセルを使い分ける」と言う主題から離れすぎるので、この回答ではこれ以上深入りしないことにします。)

(Listと同じファイルに追加すること想定。)

protocol ListEntryDelegate: class {
    func addEntry(_ entry: ListEntry, at index: Int)
}

extension List: ListEntryDelegate {
    func addEntry(_ entry: ListEntry, at index: Int) {
        mainArray.insert(entry, at: index)
        tableView.insertRows(at: [IndexPath(row: index, section: 0)], with: .automatic)
    }
}

上記のデリゲートを使用する前提のPostは次のような感じになるでしょう。

class Post: UIViewController {
    
    weak var delegate: ListEntryDelegate? //<- 追加
    
    var postString: String = ""
    
    @IBOutlet weak var postTextField: UITextField!
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        // Do any additional setup after loading the view.
    }
    
    @IBAction func postBack(_ sender: Any) {
        self.navigationController?.popToRootViewController(animated: true)
        
        let postString = postTextField.text ?? ""
        
        //↓自前で`List`のメソッドを呼び出したりせず、デリゲートのメソッドを呼び出す
        let newEntry = ListEntry(cellType: .post, text: postString)
        delegate?.addEntry(newEntry, at: 0) //<- いつでも先頭に追加で良い?
    }
}

さて、Fol(ViewController)側(VCのクラス名なんて、そうなんども書く必要があるわけでもないので、もう少しわかりやすい名前にした方が良いと思います)にも同様の変更を施すと、先ほどのListクラスのtableView(_:didSelectRowAt:)は次のような感じになるでしょう。

    func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        let listEntry = mainArray[indexPath.row]
        switch listEntry.cellType {
        case .post:
            guard let postVC = storyboard?.instantiateViewController(withIdentifier: "Post") as? Post else {
                return
            }
            postVC.delegate = self
            postVC.postString = listEntry.text
            //その他`postVC`に伝えるべき情報があればここで設定
            //postVC.〜 = 〜
            navigationController?.pushViewController(postVC, animated: true)
        case .folder:
            guard let folderVC = storyboard?.instantiateViewController(withIdentifier: "Folder") as? Fol else {
                return
            }
            folderVC.delegate = self
            //その他`folderVC`に伝えるべき情報があればここで設定
            //folderVC.〜 = 〜
            navigationController?.pushViewController(folderVC, animated: true)
        }
    }

tableView(_:didSelectRowAt:)didSelectRowAt:に渡されるのはIndexPath型の値です。それをUITableViewCell型で宣言すると、実行時に予測不可能な障害が発生する可能性があります。

(実際にはSwiftコンパイラが「これはUITableViewDelegatetableView(_:didSelectRowAt:)じゃない!」と判断して、呼び出しさえされないはずですが。)


変更が多岐にわたって少しみにくいかと思いますが、どのクラスのどの部分を修正するのかをよく読み取ってお試しいただければと思います。

なにぶん読みにくくなってしまったので、わからない点やうまくいかない点など出てくると思いますが、

  • 「2種類のカスタムセルを表示したい」→この質問へのコメント
  • それ以外の事柄 →新規の質問を立てる

と言った使い分けをしてもらえると、後からこのスレを読まれる方がこれ以上混乱せずに済むかと思います。


ご質問の「追記」について → Listクラスのまとめ。

説明の都合を優先して、コードをバラバラに小出しにしてしまったので、混乱させてしまったようです。Listクラスを記載するファイル(List.swift ?)に上記のすべての修正を適用すると、以下のような感じになります。

import UIKit

class List: UIViewController, UITableViewDataSource, UITableViewDelegate {
    
    var mainArray: [ListEntry] = []
    //let initArray: [String] = []
    
    override func viewDidLoad() {
        super.viewDidLoad()
        tableView.dataSource = self
        tableView.delegate = self
        
        // Do any additional setup after  the view.
        //以下はstoryboardで設定しているなら不要
        tableView.register(PostCell.self, forCellReuseIdentifier: ListEntry.CellType.post.rawValue)
        tableView.register(FolderCell.self, forCellReuseIdentifier: ListEntry.CellType.folder.rawValue)
    }
    
    @IBOutlet weak var tableView: UITableView!
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return mainArray.count
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let listEntry = mainArray[indexPath.row]
        switch listEntry.cellType {
        case .post:
            let postcell = tableView.dequeueReusableCell(withIdentifier: listEntry.cellType.rawValue, for: indexPath) as! PostCell
            //実際には`textLabel`でなく、`PostCell`の専用プロパティを使用する必要があると思われる
            postcell.textLabel?.text = listEntry.text
            return postcell
        case .folder:
            let foldercell = tableView.dequeueReusableCell(withIdentifier: listEntry.cellType.rawValue, for: indexPath) as! FolderCell
            //実際には`textLabel`でなく、`FolderCell`の専用プロパティを使用する必要があると思われる
            foldercell.textLabel?.text = listEntry.text
            return foldercell
        }
    }
    
    func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        let listEntry = mainArray[indexPath.row]
        switch listEntry.cellType {
        case .post:
            guard let postVC = storyboard?.instantiateViewController(withIdentifier: "Post") as? Post else {
                return
            }
            postVC.delegate = self
            postVC.postString = listEntry.text
            //その他`postVC`に伝えるべき情報があればここで設定
            //postVC.〜 = 〜
            navigationController?.pushViewController(postVC, animated: true)
        case .folder:
            guard let folderVC = storyboard?.instantiateViewController(withIdentifier: "Folder") as? Fol else {
                return
            }
            folderVC.delegate = self
            //その他`folderVC`に伝えるべき情報があればここで設定
            //folderVC.〜 = 〜
            navigationController?.pushViewController(folderVC, animated: true)
        }
    }
} //<- ここで`List`のクラス定義は一旦終わっている

protocol ListEntryDelegate: class {
    func addEntry(_ entry: ListEntry, at index: Int)
}

extension List: ListEntryDelegate {
    func addEntry(_ entry: ListEntry, at index: Int) {
        mainArray.insert(entry, at: index)
        tableView.insertRows(at: [IndexPath(row: index, section: 0)], with: .automatic)
    }
}

ListEntryDelegateListEntryDelegateを記載する場所が、やってみたのと違っているのではないでしょうか? (わかりにくくてすいませんでした。)この内容で再度お試しください。

25
  • ありがとうございます 内容はそちらで合ってます 今日はできないので明日朝一でやりたいとおもいます
    – tony
    Commented 2020年6月30日 14:59
  • ちなみに class と struct , protcol のちがいが何度読んでもいまいちわからないので 新しく質問を立てるのでそちらのほうもよろしくお願いします
    – tony
    Commented 2020年6月30日 19:17
  • 朝一といったのにすみません やる事ができてしまいそちらに付きっきりでした こちらをすべておわらせてからご連絡しようとおもいましたが エラーがでるのでさきにご連絡します List に protcol を追加すると … 追記します
    – tony
    Commented 2020年7月1日 7:54
  • ちなみに CellType の case post と case folder はidentifier でしょうか ?
    – tony
    Commented 2020年7月1日 13:51
  • @poury さん、返信が遅れて申し訳ありません。お急ぎの場合もあるかもしれませんが、こちらも手すきの時にしか書き込み等できませんので、ご容赦ください。で、Swiftでidentifierと言うのは単に「識別子」、緩く言うと「名前」程度の意味しかありませんので、その通りですと言うことになるんですが、例えば「post」「folder」を区別する項目をInt型の01なんかで表すとプログラムが読みにくくなってしまうので、enumを使ってListEntry.CellType.postListEntry.CellType.folderなんて意味のない数字ではなく分かりやすい名前を使えるようにしたわけです。(SwiftではListEntry.CellTypeの部分は省略できて、.post, .folderとだけ書けば済む場合も多い。)さらにSwiftのenumには、rawValueなんてものを持たせることができるので、rawValueがセルのreuseIdentifierになるようにしています。エラー問題については回答の方を編集してお示しします。
    – OOPer
    Commented 2020年7月1日 14:11

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。