(Swift/Firebase/Firestore)Int型のデータをラベルに表示できない

Question

Firestoreに記録しているInt型のフィールドを取得・CollectionViewのセル内ラベルに表示したい

お世話になります。
現在、食材検索・食事記録サービスを開発しながらSwiftとFirebaseを学んでいる初学者です。
以前下記リンクのような質問をさせていただきました。
https://teratail.com/questions/199897

この質問にいただいた回答を基に、Firestore内に入れているInt型のデータをラベルに表示するコードを以下のように書いたのですが、実行するとアプリが落ちてしまいます。

なお、Firestoreには

• 食品名(String)
• メーカー名(String)
• 価格(Number)

というデータが2つ入っております。

String型のデータは正しく表示されていることもあり、どうしてInt型だと配列に追加・ラベルに上手く表示できないのか分かりません。
どなたか解決策をご存知の方がいらっしゃいましたら、教えていただきたいです。

該当のソースコード

// ViewController
var foodNameArray: [String] = []
var priceArray: [Int] = []
class ViewController: UIViewController, UICollectionViewDataSource, UISearchBarDelegate {
    //中略

    // セルを表示する内容
    func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
        
        let cell = collectionView.dequeueReusableCell(withReuseIdentifier: reuseIdentifier, for: indexPath) as! CollectionViewCell2
        cell.label1.text = foodNameArray[indexPath.row]
      // 表示されないラベルは以下
        cell.label2.text = String(priceArray[indexPath.row])
        cell.btn1.tag = indexPath.row
        cell.setCell()
        return cell
    }
    
  // Firestoreからデータをもってくる
  private func loadData() {
        
        let db = Firestore.firestore()
        db.collection("items").getDocuments() { (querySnapshot, err) in
            if let err = err {
                print("Error getting documents: (err)")
            }
            for document in querySnapshot!.documents {
                if let foodName = document.data()["name"] as? String{
                    foodNameArray.append(foodName)
                }
                if let priceDetail = document.data()["price"] as? Int{
                  priceArray.append(priceDetail)
                }
    }
            self.collectionViewObj.reloadData()
        }
    }

質問内容に不備・不明瞭な点がありましたら、ご指摘いただけますと助かります。
よろしくお願いいたします。

補足情報（FW/ツールのバージョンなど）

Swift version 5.0.1
Xcode version 10.2.1

Comments

[fuzzball]

saltDetailArrayってなんでしょうか？定義されてないですが。

[fuzzball]

>>ビルドするとアプリが落ちてしまいます
「アプリが落ちて」ではなく、エラーが出るということでしょうか？
それとも、「実行すると」アプリが落ちるのでしょうか？

[haruhi.]

fuzzballさん

ご指摘いただきありがとうございます。
配列名は以前使っていた名前のまま記載してしまいました。
正しくは『priceArray』です。

また、『アプリが落ちる』とは『実行し、シミュレーターで該当ページを表示したときにアプリが落ちる』です。
『cell.label2.text = String(priceArray[indexPath.row])』という箇所に『Thread 1: Fatal error: Index out of range』と表示されます。
以上、2点修正いたしました。引き続きよろしくお願いいたします。

[fuzzball]

Firebaseのことはよく分からないのですが、"price"が設定されていない、もしくは数値ではないデータがありませんか？

[haruhi.]

お返事いただきありがとうございます。
FirestoreのitemsコレクションにNumberという型でpriceを設定しています。

[fuzzball]

それは質問に書いてあるので知ってます。

具体的なコードで言うと、
document.data()["price"] as? Int
ここでIntに変換できていないデータがあるのではないか？と聞いています。

Answer 1

コードを見た感じの問題点は

• 適切な型を作る
• グローバル変数は使わない
• エラーを処理する

です
もちろん例外はありますが、このあたりは基本です。

適切な型を作る

name、priceを別々の配列で保持していますが本当にそれで良いでしょうか？
これらはともに[food]の属性のはずです。
こういう時はそれにあった型を作ります。

struct Food {
    let name: String
    let price: Int
//    let manufacturer: String
}

そして、このFoodの配列を持つようにします。

グローバル変数は使わない

そのままです。
この場合はプロパティとして持つようにします。

class ViewController: UIViewController, UICollectionViewDataSource, UISearchBarDelegate {
  
    private var foods: [Food] = []
    
    ...
}

エラーを処理する

fuzzballさんがおっしゃるようにdocument.data()["price"] as? Intがnilであるためエラーが発生している可能性があります。
ただし、今のままではそれが当たっているかどうかが分かりません。
こういった場合のためエラー処理をします。

// Firestoreからデータをもってくる
private func loadData() {
    
    // reset foods
    foods = []
    
    let db = Firestore.firestore()
    db.collection("items").getDocuments() { (querySnapshot, err) in
        
        if let err = err {
            print("Error getting documents: (err)")
            return
        }
        
        for document in querySnapshot!.documents {
            gaurd let name = document.data()["name"] as? String else {
                print("food name is nil.")
                return
            }
            gaurd let price = document.data()["price"] as? Int else {
                print(document.data(), "\n", "food price is nil.")
                return
            }
            
            self.foods.append(Food(name: name, price: price))
        }
        DispatchQueue.main.sync {
            self.collectionViewObj.reloadData()
        }
    }
}

これはいわゆるprintデバッグというもので開発中に行うものです。

---

これで問題解決には至らずとも問題は絞られます。