tableViewについてあれこれ

Question

//ViewController.swift
import UIKit

class ViewController: UIViewController,UITableViewDataSource,UITableViewDelegate{
  
  @IBOutlet weak var mytableView: UITableView!
  
  //構造体
  let initialItems: [String] = []
  var kei: [Cell] = Array()

  var item: [String] = []
  var cnt  = 0
  var itemTotal: [[String]] = [[]]
  
  func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
    if (itemTotal.count == 0
      || itemTotal.count > 0 && itemTotal[0].count == 0) {
      return 0
    } else {
      return itemTotal.count
    }
  }
  
  func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
    let cell = UITableViewCell()
    let index = itemTotal.count - 1 - indexPath.row
    let item = itemTotal[index]
    
    guard item.count > 0 else {
      return cell
    }
    
    var labels: [UILabel] = []
    //labelについて
    for i in item {
      let label = UILabel()
      label.backgroundColor = UIColor.darkGray
      label.text = i
      
      
      // textLabel という名前の変数に格納された UILabel にフォントサイズの自動調整を設定します。
      label.adjustsFontSizeToFitWidth = true
      label.minimumScaleFactor = 10.0
      //丸みに対して
      label.layer.cornerRadius = 5
      label.layer.masksToBounds = true
      cell.contentView.addSubview(label)
      labels.append(label)
    }
    
    var preLabel: UILabel? = nil
    for l in labels {
      
      l.translatesAutoresizingMaskIntoConstraints = false
      if (preLabel == nil) {
        l.leftAnchor.constraint(equalTo: cell.contentView.leftAnchor, constant: 12).isActive = true
      } else {
        l.leftAnchor.constraint(equalTo: preLabel!.rightAnchor, constant: 20).isActive = true
      }
      cell.contentView.heightAnchor.constraint(equalTo: l.heightAnchor, multiplier: 1).isActive = true
      preLabel = l
    }
    
    
    return cell
  }
  
  
  @IBAction func addcellbtr(_ sender: Any) {
    
    alertnormal()
  }
  
  func alertnormal(){
    // テキストフィールド付きアラート表示
    
    let alert = UIAlertController(title: "タイトル", message: "メッセージ", preferredStyle: .alert)
    
    // OKボタンの設定
    let okAction = UIAlertAction(title: "OK", style: .default, handler: {
      (action:UIAlertAction!) -> Void in
      
      // OKを押した時入力されていたテキストを表示
      if let textFields = alert.textFields {
        
        // アラートに含まれるすべてのテキストフィールドを調べる
        for textField in textFields {
          self.item.insert(textField.text!, at: 0)
          self.mytableView.insertRows(at: [IndexPath(row: 0, section: 0)],with: UITableViewRowAnimation.automatic)
          print(textField.text!)
        }
        self.mytableView.reloadData()
      }
    })
    alert.addAction(okAction)
    
    // キャンセルボタンの設定
    let cancelAction = UIAlertAction(title: "Cancel", style: .cancel, handler: nil)
    alert.addAction(cancelAction)
    
    // テキストフィールドを追加
    alert.addTextField(configurationHandler: {(textField: UITextField!) -> Void in
      textField.placeholder = "テキスト"
    })
    
    alert.view.setNeedsLayout() // シミュレータの種類によっては、これがないと警告が発生
    
    // アラートを画面に表示
    self.present(alert, animated: true, completion: nil)

  }
  
  
  func tableView(_ tableView: UITableView, trailingSwipeActionsConfigurationForRowAt indexPath: IndexPath) -> UISwipeActionsConfiguration? {
    
    let edit = UIContextualAction(style: .normal,title: "追加", handler: { (action: UIContextualAction, view: UIView, success :(Bool) -> Void) in
      
      let alert = UIAlertController(title: "タイトル", message: "メッセージ", preferredStyle: .alert)
      
      // OKボタンの設定
      let okAction = UIAlertAction(title: "OK", style: .default, handler: {
        (action:UIAlertAction!) -> Void in
        
        // OKを押した時入力されていたテキストを表示
        if let textFields = alert.textFields {
          
          // アラートに含まれるすべてのテキストフィールドを調べる
          for textField in textFields {
            self.item.insert(textField.text!, at: 0)
            self.itemTotal[self.cnt] = self.item
            
            if self.item.count % 5 == 0, self.itemTotal[0].count > 1 {
              self.cnt += 1
              self.itemTotal.append([])
              self.item = []
              print(textField.text!)
              //self.Gesture()
              //self.doubleclic()
            }
          self.mytableView.reloadData()
          }
        }
      })
      alert.addAction(okAction)
      
      // キャンセルボタンの設定
      let cancelAction = UIAlertAction(title: "Cancel", style: .cancel, handler: nil)
      alert.addAction(cancelAction)
      
      // テキストフィールドを追加
      alert.addTextField(configurationHandler: {(textField: UITextField!) -> Void in
        textField.placeholder = "テキスト"
      })
      
      
      alert.view.setNeedsLayout() // シミュレータの種類によっては、これがないと警告が発生
      
      // アラートを画面に表示
      self.present(alert, animated: true, completion: nil)
      print("edit")
      
      success(true)
    })
    
    edit.backgroundColor = .blue
    
    return UISwipeActionsConfiguration(actions: [edit])
  }
  
  
  
  // trueを返すことでCellのアクションを許可しています
  func tableView(_ tableView: UITableView, canEditRowAt indexPath: IndexPath) -> Bool {
    
    if item.count > (indexPath.row + 1) {
      return true
    }else{
      return false
    }
  }

  
  //注意
  
  func alert(){
    // テキストフィールド付きアラート表示
    
    let alert = UIAlertController(title: "タイトル", message: "メッセージ", preferredStyle: .alert)
    
    // OKボタンの設定
    let okAction = UIAlertAction(title: "OK", style: .default, handler: {
      (action:UIAlertAction!) -> Void in
      
      // OKを押した時入力されていたテキストを表示
      if let textFields = alert.textFields {
        
        // アラートに含まれるすべてのテキストフィールドを調べる
        for textField in textFields {
          self.item.insert(textField.text!, at: 0)
          self.itemTotal[self.cnt] = self.item
          
          if self.item.count % 5 == 0, self.itemTotal[0].count > 1 {
            self.cnt += 1
            self.itemTotal.append([])
            self.item = []
            print(textField.text!)
          //self.Gesture()
          //self.doubleclic()
          }
           self.mytableView.reloadData()
        }
      }
    })
    alert.addAction(okAction)
    
    // キャンセルボタンの設定
    let cancelAction = UIAlertAction(title: "Cancel", style: .cancel, handler: nil)
    alert.addAction(cancelAction)
    
    // テキストフィールドを追加
    alert.addTextField(configurationHandler: {(textField: UITextField!) -> Void in
      textField.placeholder = "テキスト"
    })
    
    alert.view.setNeedsLayout() // シミュレータの種類によっては、これがないと警告が発生
    
    // アラートを画面に表示
    self.present(alert, animated: true, completion: nil)
    
  }
  
  func tableView(_ tableView: UITableView, heightForRowAt indexPath: IndexPath) -> CGFloat {
    return 40
  }
  
  override func viewDidLoad() {
    super.viewDidLoad()
    
    mytableView.delegate = self
    mytableView.dataSource = self
    mytableView.estimatedRowHeight = 60
    //mytableView.backgroundColor = UIColor(red: 0.0, green: 0.8, blue: 1.0, alpha: 0.1)
    
    for item in initialItems {
      kei.append(Cell(item: item, cellcolor: false))
    }
  }
  
  

  override func didReceiveMemoryWarning() {
    super.didReceiveMemoryWarning()
    // Dispose of any resources that can be recreated.
  }


}

今やっている事

まずボタンをタップした時に題名となるcellとその中身となるcellを挿入しようとしています。
中身となるcellはスワイプによって編集できるようにしています。

困っている事
最初に題名となるセルを追加しようとコードを加えたのですが実行時にはエラーは起きないのですが実際にアラートを呼び出しOKとタップすると落ちてしまいます。
エラーはsignal SIGABRTによるものです。

//cell.swift
struct Cell {
  //var category: String
  //var name: String
  var title: String
  var item: String
  var cellcolor: Bool
  var cnt  = 0
  
  init(item: String,cellcolor: Bool) {
    self.init(title: item,item: item,cellcolor: cellcolor)
    self.item = item;
    self.cellcolor = cellcolor;
    //self.itemTotal = item;
  }
  
  init(title: String,item: String,cellcolor: Bool) {
    self.title = title;
    self.item = item;
    self.cellcolor = cellcolor;
    //self.itemTotal = item;
}

}

最新のコードです。
構造体でtitleセルとdetailセルをグループ化しようとしています。

Answer 1

原因となっているのは、

func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
    if (itemTotal.count == 0
        || itemTotal.count > 0 && itemTotal[0].count == 0) {
        return 0
    } else {
        return itemTotal.count
    }
}

でitemTotalのサイズを数えていますが、

func alertnormal(){

            ...(省略)
            
            // アラートに含まれるすべてのテキストフィールドを調べる
            for textField in textFields {
                self.item.insert(textField.text!, at: 0)
                self.mytableView.insertRows(at: [IndexPath(row: 0, section: 0)],with: UITableViewRowAnimation.automatic)
                print(textField.text!)
            }
            self.mytableView.reloadData()

ここではitemTotalにデータを追加していないため、
self.mytableView.insertRowsで行を追加しようとしていますが、
itemTotalの数と合わなくなっていることが原因と考えられます。

解決方法としては、alertnormalでもitemTotalにデータを追加する必要があります。

とりあえずの回避方法としては

func alertnormal(){

            ...(省略)
            // アラートに含まれるすべてのテキストフィールドを調べる
            for textField in textFields {
                self.item.insert(textField.text!, at: 0)
                
                if (self.cnt != 0) {
                    self.itemTotal.append([])
                }
                
                self.itemTotal[self.cnt].append(textField.text!)
                self.tableView.insertRows(at: [IndexPath(row: 0, section: 0)],with: UITableViewRowAnimation.automatic)
                
                self.cnt += 1
                
                print(textField.text!)
            }
            self.tableView.reloadData()

のような形にするとエラーは出なくなりました。

Comments

[退会済みユーザー]

alertnormalにitemTotalのデータを追加してしまうとラベルとして追加されてしまうのでそれを避ける為にitemTotalを使わないようにしていたのですが、ご教示して頂いたコートですとラベルとして追加されてしまうのでただのテキストとしてセルに追加することは出来ないのででしょうか？
私としてはtitleとしてのセルとdetailとなるセルを区別にする為にstructで使ってグルーピングしたいと思っているのですが配列の入ってる今回のようなコードだとどう構造体を使って書けばいいか分からないでいるのでご教示いただけないでしょうか？

[退会済みユーザー]

構造体を使ってtitleとなるセルとdetailとなるセルをグループにして表示させようとしているのですが下記のコードをどうstructの中に下記入れればいいか分からなくて困っています。
var item: [String] = []
var cnt = 0
var itemTotal: [[String]] = [[]]
イメージとしては
apple商品　　　　　　　　　←titleセル
[iPhone] [iPad] [iMac] [Macbook] [Apple Watch] ←detailセル
このような感じになります。以前、構造体でセルのグループ分けの仕方は知っているのですがラベルのあるセルでは作ったことがなかったので教えて頂けると幸いです。

[newmt]

やられたいことから考えると、セルで分けるのではなくセクションを使った方が良いのではないかと思います。以下の内容が参考になるかと思います。
http://docs.fabo.io/swift/uikit/019_sectionuitableview.html

構造体を使うとなると
struct Product {
let title: String
let detail: [String]
}
として
var products: [Product]という配列で管理するかなと思います。

[退会済みユーザー]

懇切に教えていただき有難うございます。
配列の部分でのエラーはなくなりました。
それでなんですがitemの部分をproductsに置き換えているのですがitemTotalの部分は何に置き換えて行けばいいのでしょうか？それともitemTotalはそのまま残す感じでしょうか？
var cnt = 0はカウントする為に残しましたがitemTotalはどうすればいいか分からないのですご教示願いたいです。

[newmt]

私のイメージですと、let detail: [String]がitemになり、var products: [Product]がitemTotalのイメージです。cntは配列の順番と行の順番が連動している(indexPathから取得できる)ので特に必要ないのかなと思いました。

[退会済みユーザー]

https://teratail.com/questions/126260
私なりに置き換えてみたのですが上手く動かなかったので見てもらえませんか？

[退会済みユーザー]

numberOfRowsInSectionのメソッドで多次元配列のitemTotalの部分いprodauctsを入れるとエラーが起こるのですがどう対処すれば良いのでしょうか？

Answer 2

まずは Xcode のデバッガを利用して、 OK をタップした後の処理をステップ実行し、どのコードが原因か絞り込むのが良いと思います。

Comments

[退会済みユーザー]

デバッグを見てみたのですがあまり理解できなかったです。
でも多分、cellForRowAtの中にラベルに関するコードのせいでエラーが起きてるんじゃないんかと思っています。

[退会済みユーザー]

題名となるセルに中身となるラベルの処理を呼ばれないようにするにはどうしたらいいですか？
多分、cellForRowAtの中の処理が呼ばれてエラーが起きたのではないかと思います。

[newmt]

signal SIGABRT以外に何かメッセージは出ていませんか？

「多分、cellForRowAtの中の処理が呼ばれてエラーが起きたのではないかと思います。」
これはブレークポイントを設定した場合に呼ばれていたということでしょうか？その場合、どこの行でエラーが発生しているか特定できていますでしょうか？

[退会済みユーザー]

メッセージはsignal SIGABRTしか出ていないと思われます。

私が思うには今回のアラートはただ入力したものをcellに反映させたいだけなのでcellForRowAt内でのlabelの処理で落ちたのかなと思います。
実際に、旧アラート(挿入時にlabelとして挿入されるアラート)では問題はありませんでした。