TableViewでセルを追加してもindexPathが更新されない

Question

前提・実現したいこと

下記のようなTableViewを作成しています
表示上はセルを追加されたように見えてもindexPathが増加していないため
didSelectRowAtのindexPathは常に一番目のデータが帰ってきます
どこがいけないんでしょうか？

発生している問題・エラーメッセージ

セルをタップしても、該当のIndexPathが取得できない

該当のソースコード

import UIKit
import AVFoundation
import AudioToolbox

class ScannerViewController: UIViewController, AVCaptureMetadataOutputObjectsDelegate,UITableViewDelegate,UITableViewDataSource{
    

    //var itemTable: UITableView?
    var captureDevice:AVCaptureDevice?
    var videoPreviewLayer:AVCaptureVideoPreviewLayer?
    var captureSession:AVCaptureSession?
    let detectionArea = UIView()
    var timer: Timer!
    var counter = 0
    var tableView: UITableView!
    //let items = ["Apple","Banana","Orange"]
    //var barcodeList = [String]()
    var items = [Item!]()
    var item: Item! = Item()
    
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        print("viewDidLoad...")
        // Do any additional setup after loading the view, typically from a nib.
        navigationItem.title = "Scanner"
        view.backgroundColor = .white
        
        captureDevice = AVCaptureDevice.default(for: .video)
        // カメラがあるか確認し，取得する
        if let captureDevice = captureDevice {
        
            do {
                let input = try AVCaptureDeviceInput(device: captureDevice)
                
                captureSession = AVCaptureSession()
                guard let captureSession = captureSession else { return }
                captureSession.addInput(input)
                
                // metadata取得に必要な初期設定
                let captureMetadataOutput = AVCaptureMetadataOutput()
                captureSession.addOutput(captureMetadataOutput)
                
                captureMetadataOutput.setMetadataObjectsDelegate(self, queue: .main)
                captureMetadataOutput.metadataObjectTypes = [.code128, .qr, .ean13,  .ean8, .code39] //AVMetadataObject.ObjectType
                
                captureSession.startRunning()
                
                // カメラからの取得映像を画面に表示する
                videoPreviewLayer = AVCaptureVideoPreviewLayer(session: captureSession)
                videoPreviewLayer?.videoGravity = .resizeAspectFill
                //videoPreviewLayer?.frame = view.layer.bounds
                videoPreviewLayer?.frame = CGRect(x: 0, y: 0, width: view.frame.size.width, height: view.frame.size.height / 4)
                view.layer.addSublayer(videoPreviewLayer!)
                
            } catch {
                print("Error Device Input")
            }
        } //end if let
        
        view.addSubview(codeLabel)
        codeLabel.bottomAnchor.constraint(equalTo: view.bottomAnchor).isActive = true
        codeLabel.centerXAnchor.constraint(equalTo: view.centerXAnchor).isActive = true
        codeLabel.heightAnchor.constraint(equalToConstant: 50).isActive = true
        codeLabel.widthAnchor.constraint(equalTo: view.widthAnchor).isActive = true
        
        // どの範囲を解析するか設定する
        let x: CGFloat = 0.05
        let y: CGFloat = 0.45
        let width: CGFloat = 0.9
        let height: CGFloat = 0.1
        
        //detectionArea.frame = CGRect(x: view.frame.size.width * x, y: view.frame.size.height * y, width: view.frame.size.width * width, height: view.frame.size.height * height)
        detectionArea.frame = CGRect(x: (videoPreviewLayer?.frame.width)! * x, y: (videoPreviewLayer?.frame.height)! * y, width: view.frame.size.width * width, height: view.frame.size.height * height)
        detectionArea.layer.borderColor = UIColor.red.cgColor
        detectionArea.layer.borderWidth = 3
        view.addSubview(detectionArea)
     
        timer = Timer.scheduledTimer(timeInterval: 0.5, target: self, selector: #selector(self.update), userInfo: nil, repeats: true)
        timer.fire()
        //self.tableView.register(UINib(nibName: "CustomTableViewCell", bundle: nil), forCellReuseIdentifier: "customCell")
        self.tableView = { [weak self] in
            guard let `self` = self else { return UITableView() }
            //let tableView = UITableView(frame: self.view.bounds, style: .plain)
            let tableView = UITableView(frame: CGRect(x: 0, y: (videoPreviewLayer?.frame.height)!, width: view.frame.size.width, height: view.frame.size.height - (videoPreviewLayer?.frame.height)!), style: .plain);
            tableView.autoresizingMask = [
                .flexibleWidth,
                .flexibleHeight
            ]
            tableView.delegate = self
            tableView.dataSource = self
            
            self.view.addSubview(tableView)
            
            return tableView
            }()
        
    } // end viewDidLoad

    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
        // Dispose of any resources that can be recreated.
    }
    
    override func viewWillAppear(_ animated: Bool) {
        self.tableView.reloadData()
    }
    
    func metadataOutput(_ output: AVCaptureMetadataOutput, didOutput metadataObjects: [AVMetadataObject], from connection: AVCaptureConnection) {
        if metadataObjects.count == 0 {
            //print("No Input Detected")
            codeFrame.frame = CGRect.zero
            codeLabel.text = "No Data"
            return
        }
        print("call metadataOutput")
        let metadataObject = metadataObjects[0] as! AVMetadataMachineReadableCodeObject
        
        guard let stringCodeValue = metadataObject.stringValue else { return }
        counter = 0
        
        self.item?.title = stringCodeValue
        self.item?.qty = 1
        print(self.item!.title as Any)
        print(self.item!.qty as Any)
        self.items.append(self.item)
        
        self.tableView.insertRows(at: [IndexPath(row: 0, section: 0)], with: UITableViewRowAnimation.right)
        
        guard let barcodeObject = videoPreviewLayer?.transformedMetadataObject(for: metadataObject) else { return }
        
        // Play system sound with custom mp3 file
        if let customSoundUrl = Bundle.main.url(forResource: "beep-07", withExtension: "mp3") {
            var customSoundId: SystemSoundID = 0
            AudioServicesCreateSystemSoundID(customSoundUrl as CFURL, &customSoundId)
            //let systemSoundId: SystemSoundID = 1016  // to play apple's built in sound, no need for upper 3 lines
            
            AudioServicesAddSystemSoundCompletion(customSoundId, nil, nil, { (customSoundId, _) -> Void in
                AudioServicesDisposeSystemSoundID(customSoundId)
            }, nil)
            
            AudioServicesPlaySystemSound(customSoundId)
            //captureSession?.stopRunning()
        }
        
        // Stop capturing and hence stop executing metadataOutput function over and over again
        print("stop Running")
        captureSession?.stopRunning()
        
        // Call the function which performs navigation and pass the code string value we just detected
        //displayDetailsViewController(scannedCode: stringCodeValue)
    }
    
    @objc func update(tm: Timer) {
        counter += 1
        //print(counter)
        if 1 < counter {
            captureSession?.startRunning()
            detectionArea.layer.borderColor = UIColor.red.cgColor
            detectionArea.layer.borderWidth = 3
            codeLabel.text = ""
        }
    }
    
    func numberOfSections(in tableView: UITableView) -> Int {
        print("call numberOfSections")
        return 1
    }
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        print("items.count (self.items.count)")
        return self.items.count
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        
        let cell = tableView.dequeueReusableCell(withIdentifier: "Cell")
            ?? UITableViewCell(style: .default, reuseIdentifier: "Cell")
        
        cell.textLabel?.text = self.items[indexPath.row].title
        
        print("call cellForRowAt")
        return cell
        
    }
    
    func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        
        print("Selected! (self.items[indexPath.row].title)")
        print("indexPath (self.items[indexPath.row])")
        
    }
    
    func tableView(_ tableView: UITableView, canEditRowAt indexPath: IndexPath) -> Bool {
        return true
    }
    
    func tableView(_ tableView: UITableView, commit editingStyle: UITableViewCellEditingStyle, forRowAt indexpath: IndexPath) {
        if editingStyle == UITableViewCellEditingStyle.delete {
            //リストから削除
            items.remove(at: indexpath.row)
            
            //セルを削除
            tableView.deleteRows(at: [indexpath], with: UITableViewRowAnimation.fade)
        }
    }
}

Comments

[nakasho_dev]

質問するときのヒント（https://teratail.com/help/question-tips）を読んで、ソースコードにシンタックスハイライトを付けてください。ソースが読みづらくて回答が得られにくいと思います。

Answer 1

自分でテーブルにセルを追加してはいけません。
保持している配列にデータを追加した後、テーブルをリロードして下さい。

self.items.append(self.item)
//self.tableView.insertRows(at: [IndexPath(row: 0, section: 0)], with: UITableViewRowAnimation.right)
self.tableView.reloadData()

テーブルの上に追加したいのであれば、配列の先頭にデータを追加して下さい。

self.items.insert(self.item, at: 0)

Answer 2

insertRowsの部分を下記のように変更すると何か変わりますでしょうか？

DispatchQueue.main.async { [weak self] in
  self?.tableView.beginUpdates()
                                          // ↓0から変更
  self?.tableView.insertRows(at: [IndexPath(row: self.items.count - 1, section: 0)], with: UITableViewRowAnimation.right)
  self?.tableView.endUpdates()
}

Comments

[akiraa]

回答ありがとうございます。
記述してみましたが、状況は変わりませんでした。

[newmt]

一部処理を修正しました。また、self.items.appendの後でprint(self.items)を出力すると何が出力されますでしょうか？

[akiraa]

ご回答ありがとうございます。

printさせると　[BarcodeScanner.Item] と表示されます

[newmt]

metadataOutput関数が複数呼ばれた場合にself.itemsには複数アイテムが入っていますでしょうか？また、print(self.item!.title as Any)の中身も教えて頂けましたら幸いです。

[akiraa]

ご回答ありがとうございます。
self.items には同じデーターが複数入ってました
print(self.item!.title as Any)ですが
[BarcodeScanner.Item, BarcodeScanner.Item] という内容です

Answer 3

tableView.reloadData()を呼んでいないのでテーブルの更新がされないのではないでしょうか。

【追記】
viewWillAppearでreloadDataを呼んでいるとのことですが、
metadataOutputにviewWillAppearは呼ばれているのでしょうか。
もし呼ばれていてreloadDataが実行されているとしたらself.itemsの中身が変わっていないのかもしれません。

気になったのはクラス変数としてitemを作成していることです。
itemを使いまわしているので、items[0]、items[1]と追加されても常に同じ内容になります。
metadataOutput関数内でしか使用していないので都度生成したほうが良いと考えます。

また、metadataOutput内の以下のコードで関数を抜けていないかなども確認したほうが良いのではないでしょうか。

guard let stringCodeValue = metadataObject.stringValue else { return }

Comments

[akiraa]

ご回答ありがとうございます。
viewWillAppear内にreloadData()を記述してみましたが状況は変わりませんでした。

[akiraa]

ご回答ありがとうございます。
ご指摘いただいた通りitemを局所化したところ一発で解決しました
大変助かりました、ありがとうございます