カスタムViewの表示

Question

実現したいこと

タップしたら、その中の配列（プロパティ）を表示したいです

前提

FolderRowとして、カスタムViewセルをstructしています。

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

Cannot find 'folderRowArray' in scope

- 

Extraneous whitespace after '.' is not permitted

--- 

Missing arguments for parameters 'folderRowArray', 'folderRowSystemNameText', 'folderRowName' in call

該当のソースコード

struct ContentView: View {
    @Environment(\.managedObjectContext) private var viewContext
    
    @FetchRequest(
        sortDescriptors: [NSSortDescriptor(keyPath: \Item.timestamp, ascending: true)],
        animation: .default)
    private var items: FetchedResults<Item>
    
    private let list1 = [
        SampleStruct(sectionTitle: "A", listContent: ["elementA1", "elementA2", "elementA3"]),
        SampleStruct(sectionTitle: "B", listContent: ["elementB1", "elementB2", "elementB3"]),
        SampleStruct(sectionTitle: "C", listContent: ["elementC1", "elementC2", "elementC3"]),
    ]
    
    var body: some View {
        NavigationView {
            List {
                ForEach(list1) { item in
                    Section(item.sectionTitle) {
                        ForEach(item.listContent, id: \.self) { item2 in
                            Text(item2)}
                        .toolbar {
                            ToolbarItem(placement: .navigationBarTrailing) {
                                EditButton()
                            }
                        }
                    }
                }
            }
            .navigationTitle("A")
        }}}

struct FolderRow: View {
   var folderRowArray: [String]
   var systemNameText: String
   var folderRowName: String
   var body: some View {
HStack{
Image(systemName: systemNameText)
Text(folderRowName)
Text("\(folderRowArray.count)")
}
   }
}

                struct SampleStruct : Identifiable {
                    let id = UUID()
                    let sectionTitle: String
                    let listContent: [String]
                }
                
                struct ContentView_Previews: PreviewProvider {
                    static var previews: some View {
                        ContentView()
                    }
                }

試したこと

とてもややこしいのが、遷移したいのは、FolderRowの中のfolderRowArrayなので、folderRow.folderRowArrayとするのか、あるいはitemをからめて記述するのか、というところです。

（folderRowArrayもitemに保存するはずですので）

両方試しました。

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

Xcode14.1

追記

![イメージ説明]

Comments

[退会済みユーザー]

コメントありがとうございます。
修正できそうですか？

> 意としてるのは、iPhoneのメモの追加フォルダです
なるほど。
固定と言っていたと思うのですが、
フォルダ自体は増えたり減ったりしないということですかね。
(ユーザーがフォルダを追加したり、削除したりはできない？)

そうだとすると、
フォルダの表示の部分(ビュー・画面の構造体)は、
ForEachなどは使わずに、
直接Sectionを3つ("A","B","C")と続けて記述しても良いのかもしれませんね。

> あとから追加してるにもかかわらず、そのそれぞれをタップすると、そのそれぞれの配列に遷移します
ごめんなさい。
ここはまだちゃんと理解できないです・・
画面設計書とかワイヤーフレームとかイメージを共有できていれば、また違うのかもしれませんが、
ちょっと言葉だけでは伝えるのも、理解するのも、難しいところですね。
(システム開発{アプリ開発}の難しいところですね)

> プレビューコントローラーにもfolderRowへの言及の必要が書かれ、そんなこと通常であり得るの？という心情です。
構造体の場合はプロパティを定義すると
自動的にイニシャライザでそれを受け取るようになっているみたいですね。
(「Memberwise Initializers」というみたいです)

Memberwise Initializers for Structure Types
https://docs.swift.org/swift-book/documentation/the-swift-programming-language/initialization#Memberwise-Initializers-for-Structure-Types

ContentViewに`var folderRow: FolderRow`と記述してしまっていますので、
FolderRowの構造体を受け取るようなイニシャライザが自動的に実装されることになると思います。

---

ちょっとこの質問とは直接的には関係ないですが、
1つお聞きしておきたいことがあります。
フォルダごとにデータを管理すると思いますが、
このデータはどのように識別しますでしょうか？
ItemのEntityにいくつかAttributeを追加されていたようですが、
フォルダ名など追加する予定でしょうか？
＊見当違いなことを聞いてしまっていたらすみません

[sato31]

コメントありがとうございます

ちょうど更新しようと思っていたところなのですが

考えている方法がデジタルだとしたら、それができずにアナログでやろうとしてむだにおなじstructがふえていきそうだったので、

そのことについてコメントしようとおもっていました

---

リレーションでは？と思い、調べていたのですが、

そうしなくても、いける気がしています

---

ItemのattributeはひとつのEntityにまとめて と思いましたが、ひとつのセルが持つBoolなどを第二のEntityに保存して、リレーションかな…？ともおもっています
（先述とことなっています）

---

自動でイニシャライザを記述ということは、やはりプレビュープロバイダーにもfolderRowのプロパティの記述が必須なのですね

---

固定の３つのうちのひとつ（のセクション）には、あとから追加することになります。

初期段階から、３つとも名前など全て固定です

[sato31]

クラッシュするコードだったので、ひとつ前のコードにしました

[sato31]

参照リンクもありがとうございます。

[退会済みユーザー]

> むだにおなじstructがふえていきそうだったので、
正しく理解できていないかもしれませんが、
レイアウトが同じであれば、
ビュー(画面)のstructを1つ作って、
データだけを親から渡してあげれば良いのかなと思います。

---

> ItemのattributeはひとつのEntityにまとめて と思いましたが、ひとつのセルが持つBoolなどを第二のEntityに保存して、リレーションかな…？
承知しました。
引き続き検討してみてください。
CoreDataはあまり詳しくないのですが、
データベース設計としては、
同じまとまりなら一つのEntityで良いのではないかなと思います。
1:多の関係になるのでしたら、リレーションシップになる感じですかね。
(考え方は色々あると思いますので、話半分で聞いてOKです)

---

> 自動でイニシャライザを記述ということは、やはりプレビュープロバイダーにもfolderRowのプロパティの記述が必須なのですね
こちらは、解釈が逆になるかなと思います。
ContentViewに`var folderRow: FolderRow`と記述してしまうと、
自動的にfolderRowを引数に受け取るようになってしまうので、
ContentViewを呼び出す側で`ContentView(folderRow: ~)`と記述しないといけないことになってしまいます。
ですので、ContentViewには`var folderRow: FolderRow`を記述しない方針で実装する必要があると思います。

--

> 固定の３つのうちのひとつ（のセクション）には、あとから追加することになります。
セクションの一つもユーザーによって追加する感じなのですね。
(ちょっと余計イメージがわからなくなってしまいましたが・・)

---

画面イメージもありがとうございます。
Editボタンがたくさんになってしまいましたね・・

---

この質問のタイトル「カスタムViewの表示」が目的ですよね。
基本的には前の質問で使っていたのと同じようにすればOKなのですけどね。
(AnotherAnotherViewとかのことです)

現在の質問欄のコードにある`Text(item2)`の部分を、
FolderRowを呼び出すような感じになります。

[sato31]

３つの固定セルは、それぞれ用意して、if で表示するEntityのStringを３つにすれば、解決できるのでは？と思っています

親からデータだけ
これが理想なのですが、タップ判定ってできるのでしょうか？
このセルをタップしたら、このEntityのtext○を表示
のように
（text1, text2, text3でattributeを設定します）

---

イニシャライザの記述はあまり好ましくないのでfolderRowは ContentViewにはのせません

すみません、
iPhoneのフォルダ追加のようなものだとおもってください

EditButton()はバグですか？

---

できるだけ前回のものを使いたいとおもっています。
あと一点
item2がどこに宣言されているのか、わかりません

[sato31]

リレーションかな、とおもっている理由は、ひとつのセルが待つ複数のデータ（String、Bool）をデータごと に保存する必要があるのかな？とおもっているためです

[退会済みユーザー]

> これが理想なのですが、タップ判定ってできるのでしょうか？ このセルをタップしたら、このEntityのtext○を表示 のように （text1, text2, text3でattributeを設定します）
タップはできると思います。
でも、仕様がうまく把握できませんでした・・
フォルダA、フォルダB、フォルダCに1つのエンティティがそれぞれ表示されることもあるのでしょうか？
(データの関係がうまく把握できていないのかもしれません)
(フォルダAはtext1？、フォルダBはtext2？、フォルダCはtext3？のような？)

> EditButton()はバグですか？
ForEachの中にtoolbarを記述しているためですね。
3つ目の回答のコードを参考に、
どこの位置にtoolbarを記述しているか見てみてください。

> item2がどこに宣言されているのか、わかりません
item2はクロージャの中で使える引数なのですが、
ForEachに渡しているitem.listContent(Stringの配列)につきまして、
ForEachがStringの配列を1つずつに分けて
クロージャの中身({}のことです)を実行してくれます。
この時の要素1つをitem2の変数に設定して実行しれくれる形です。
(宣言がどこかというと、「in」の直前ですね)

Closure Expression Syntax
https://docs.swift.org/swift-book/documentation/the-swift-programming-language/closures/#Closure-Expression-Syntax

> リレーションかな、とおもっている理由は、ひとつのセルが待つ複数のデータ（String、Bool）をデータごと に保存する必要があるのかな？
そうなのですね。
では、1:多の関係っぽいですね。
リレーションシップで良いのかもしれませんね。

[sato31]

フォルダAはtext1？、フォルダBはtext2？、フォルダCはtext3？のような？
そうです
ほかに方法があれば、いいですが

ここでのリレーションの話は申しわけないので、やめにします

ありがとうございます

---

イニシャライザの記述が好ましくない は語弊がありますね

プレビュープロバイダーへのその他のViewのイニシャライザの記述は

です

[sato31]

カスタムViewの表示はText(text)の部分をFolderRowのようなstructとそのプロパティ でいいのですね。

[sato31]

EditButton()のForEachの記述もありがとうございます

Answer 1

入れ違いでコメントが届いたみたいなのですが、
先にコードを書いておきますね。

質問欄に画像も貼り付けてもらったことで、
少しだけ理解が進みました。

FolderRowの使い方は次のような感じになると思います。

import SwiftUI

struct ContentView: View {
    var body: some View {
        NavigationView {
            List {
                Section("A") {
                    FolderRow(
                        // --- folderRowArrayはitemsからデータを絞り込んで渡すことになると思いました。
                        folderRowArray: ["1", "2", "3", "4", "5", "6"],
                        systemNameText: "fossil.shell",
                        folderRowName: "フォルダA1")
                    FolderRow(
                        folderRowArray: ["1", "2", "3", "4", "5"],
                        systemNameText: "atom",
                        folderRowName: "フォルダA2")
                    FolderRow(
                        folderRowArray: ["1", "2", "3", "4"],
                        systemNameText: "carrot",
                        folderRowName: "フォルダA3")
                }
                Section("B") {
                    FolderRow(
                        folderRowArray: ["1", "2", "3"],
                        systemNameText: "tree.circle",
                        folderRowName: "フォルダB1")
                    FolderRow(
                        folderRowArray: ["1", "2"],
                        systemNameText: "camera.macro.circle",
                        folderRowName: "フォルダB2")
                }
                Section("C") {
                    FolderRow(
                        folderRowArray: ["1"],
                        systemNameText: "leaf.circle",
                        folderRowName: "フォルダC1")
                }
            }
            .toolbar {
                ToolbarItem(placement: .navigationBarTrailing) {
                    EditButton()
                }
            }
        }
    }
}

struct FolderRow: View {
    var folderRowArray: [String]
    var systemNameText: String
    var folderRowName: String
    var body: some View {
        HStack{
            Image(systemName: systemNameText)
            Text(folderRowName)
            Text("\(folderRowArray.count)")
        }
    }
}

struct ContentView_Previews: PreviewProvider {
    static var previews: some View {
        ContentView()
    }
}

Comments

[sato31]

ベストアンサーにさせていただきましたが、当初意としていたものは、クラッシュするといったこのコードに近いです

struct ContentView: View {
@Environment(\.managedObjectContext) private var viewContext

@FetchRequest(
sortDescriptors: [NSSortDescriptor(keyPath: \Item.timestamp, ascending: false)],
animation: .default)
private var items: FetchedResults<Item>

var body: some View {
NavigationView {
List {
ForEach(items) { item in
NavigationLink {
AnotherView(item: item)
} label: {
Text(item.text ?? "")
}
}
.onDelete(perform: deleteItems)
}
.toolbar {
#if os(iOS)
ToolbarItem(placement: .navigationBarTrailing) {
EditButton()
}
#endif
if let item1 = items.first {
ToolbarItem {
NavigationLink {
AnotherAnotherView(item: item1)
} label: {
Image(systemName: "pin")
}
}
}
}
Text("Select an item")
}
}

private func addItem() {
withAnimation {
let newItem = Item(context: viewContext)
newItem.timestamp = Date()

do {
try viewContext.save()
} catch {
// Replace this implementation with code to handle the error appropriately.
// fatalError() causes the application to generate a crash log and terminate. You should not use this function in a shipping application, although it may be useful during development.
let nsError = error as NSError
fatalError("Unresolved error \(nsError), \(nsError.userInfo)")
}
}
}

private func deleteItems(offsets: IndexSet) {
withAnimation {
offsets.map { items[$0] }.forEach(viewContext.delete)

do {
try viewContext.save()
} catch {
// Replace this implementation with code to handle the error appropriately.
// fatalError() causes the application to generate a crash log and terminate. You should not use this function in a shipping application, although it may be useful during development.
let nsError = error as NSError
fatalError("Unresolved error \(nsError), \(nsError.userInfo)")
}
}
}
}

private let itemFormatter: DateFormatter = {
let formatter = DateFormatter()
formatter.dateStyle = .short
formatter.timeStyle = .medium
return formatter
}()

struct AnotherView: View {

@Environment(\.managedObjectContext) private var viewContext
@State var textEditorText = ""
var item: Item

init(item: Item) { _textEditorText = State(initialValue: item.text ?? "")
self.item = item }

var body: some View {
TextEditor(text: $textEditorText).frame(width: .infinity, height: .infinity)

.toolbar { ToolbarItem(placement: .navigationBarLeading){ Button(action: {additem()}) {Image(systemName: "heart")}} }
}

func additem () {
withAnimation {
item.text = textEditorText

do {
try viewContext.save()
} catch {
let nsError = error as NSError
fatalError("Unresolved error \(nsError), \(nsError.userInfo)")
}
}
}}

struct AnotherAnotherView: View {

@Environment(\.managedObjectContext) private var viewContext
@State var textEditorText = ""
var item: Item

var body: some View {
TextEditor(text: $textEditorText).frame(width: .infinity, height: .infinity)

.toolbar { ToolbarItem(placement: .navigationBarLeading){ Button(action: {additem()}) {Image(systemName: "heart")}} }
}

func additem () {
withAnimation {
item.text = textEditorText

do {
try viewContext.save()
} catch {
let nsError = error as NSError
fatalError("Unresolved error \(nsError), \(nsError.userInfo)")
}
}
}}

struct ContentView_Previews: PreviewProvider {
static var previews: some View {
ContentView().environment(\.managedObjectContext, PersistenceController.preview.container.viewContext)
}
}

[退会済みユーザー]

回答者からのコメントをここに書くとメールで通知が届かないかもしれませんが、
書き込みしますね。

> ベストアンサーにさせていただきましたが、当初意としていたものは、クラッシュするといったこのコードに近いです
そうですよね。
前の質問では、フォルダを考えずitemsの一覧を表示して、編集画面を表示する感じでしたよね。
今回の質問は、主にFolderRowを呼び出す部分に関するものだと思いましたので、
AnotherView、AnotherAnotherViewなどの部分は切り離したコードにしました。

前の質問のビューの構成は次のような感じだと思います。

ContentView (itemsの一覧)
＿＿AnotherView
＿＿AnotherAnotherView

これを今回の質問も踏まえて、フォルダの一覧などのビューを組み込んだ時の構成は次のような感じになると思います。

TopView (フォルダの一覧)
＿＿FolderRow (NavigationLinkのlabelに記述する)
＿＿ContentView (itemsの一覧)(NavigationLinkのdestinationに記述する)
＿＿＿＿AnotherView
＿＿＿＿AnotherAnotherView

親のビューから子のビューへ、
データを渡しながら構成していくのが
SwiftUIっぽい感じかなと思います。

言葉だけでは不十分な気がするのですが、
伝わりますでしょうか？

---

> フォルダAはtext1？、フォルダBはtext2？、フォルダCはtext3？のような？ そうです
個人的にはちょっと違和感があります。
iPhoneのメモでは、おそらく、メモ1つがEntity1つに対応していると思います。
フォルダの概念を取り入れるとしたら、
フォルダのEntityがあって、メモのEntityがあって、
フォルダとメモの関係(リレーションシップ)が「1:多」になるかなと思いました。
(フォルダ1つに対して、メモが多数の関係)

Answer 2

ごめんなさい。
やっぱりコードからどうしたいのか理解するのが難しいです・・

とてもややこしいのが、遷移したいのは、FolderRowの中のfolderRowArrayなので、

おそらくこの部分だと思うのですが、
ご自身の中で整理しきれていないからか、
それを他者がコードから理解するのは、とても大変になっている気がします・・

1. コードのビルドエラーを解消したいのか、
2. ビルドエラーはない状態で、仕様を実現するための方法を知りたいのか

このどちらかに絞り込んだ内容にしないと、
ちょっと回答するのが難しいかもしれません。
(現在のコードはビルドエラーもあり、仕様を実現するための方法も知りたい、という状況ですよね)

根本的な方針として普通ではないかなと感じたところを1点書いておきたいと思います。
ContentViewのvar folderRow: FolderRowの部分です。
FolderRowはビュー(画面)の構造体ですが、
これをプロパティに持つケースは見たことがないです。
プロパティに持つとしたら、おそらく、
セクションタイトルと固定のリストの情報になると思います。

細かい仕様を把握しきれていないと思うので、
違うところはたくさんあると思いますが、
固定のリスト(セクション)を表示するサンプルのコードを書いてみました。

import SwiftUI

struct ContentView: View {
    private let list1 = [
        SampleStruct(sectionTitle: "A", listContent: ["elementA1", "elementA2", "elementA3"]),
        SampleStruct(sectionTitle: "B", listContent: ["elementB1", "elementB2", "elementB3"]),
        SampleStruct(sectionTitle: "C", listContent: ["elementC1", "elementC2", "elementC3"]),
    ]

    var body: some View {
        List {
            ForEach(list1) { item in
                Section(item.sectionTitle) {
                    ForEach(item.listContent, id: \.self) { item2 in
                        Text(item2)
                    }
                }
            }
        }
    }
}

struct SampleStruct : Identifiable {
    let id = UUID()
    let sectionTitle: String
    let listContent: [String]
}

struct ContentView_Previews: PreviewProvider {
    static var previews: some View {
        ContentView()
    }
}

可能でしたら、これを踏まえて、ビルドエラーがない状態まで
質問欄のコードを修正できますでしょうか？

もしそれができない場合は、この質問はビルドエラーを解消するまでで一旦終わらせた方が良いと思いました。

---

SwiftUIのチュートリアルも実践してみると良いかもしれません。

Introducing SwiftUI
https://developer.apple.com/tutorials/swiftui/

---

ArrayView()を追加修正しました。

わざわざコメントありがとうございます。
こちらも見ましたよ。

改行対応いたします

こちらもわざわざコメントありがとうございます。
いつでも大丈夫ですよ。

Comments

[sato31]

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

自分でも、ここまで複雑になるのだろうか？ という思いを持ちながらだったので、

方法がちがうのだと思います。

意としてるのは、iPhoneのメモの追加フォルダです

あとから追加してるにもかかわらず、そのそれぞれをタップすると、そのそれぞれの配列に遷移します

あの構造をつたえたつもりでしたが、

プレビューコントローラーにもfolderRowへの言及の必要が書かれ、そんなこと通常であり得るの？という心情です。

いずれにしても、いただいたコードを元に修正してみます。

[sato31]

プレビューコントローラーはプレビュープロバイダーです

Answer 3

ちょっとまだ質問の内容を理解しているところですが、
先にコードの段落について、
ちゃんと伝えられていなかったみたいですので、
先に書き込みしますね。
(コードを書き込みしたかったので、コメントではなく、回答欄に書き込みしますね)

質問欄のコードでは、
改行せずに1行で記述しているところがあると思います。
次のコードです。

List{Section(header: Text("A")){ForEach(items) { item in NavigationLink(destination: folderRow.folderRowArray, label: {FolderRow(folderLowArray: folderRowArray, folderRowSystemNameText: "gear", folderLowName: "ギア")})}} }

上のコードはちゃんと「{」の後に改行してもらえると、
回答者に優しいと思いました。
次のような感じです。

List {
    Section(header: Text("A")) {
        ForEach(items) { item in
            NavigationLink(destination: folderRow.folderRowArray, label: {
                FolderRow(folderLowArray: folderRowArray, folderRowSystemNameText: "gear", folderLowName: "ギア")
            })
        }
    }
}

コードを読むのはやっぱり大変ですので、
少しでも読みやすいようにしてあげると良いかなと思いました。
＊自分でコードを読む時にも読みやすいようにしておいた方がミスに気づけるようになると思います

Comments

[sato31]

ありがとうございます

1行で
というのは、そういうことですね

やはりコードは読むのが大変なんですね

わかりました

配慮ある助言ありがとうございます

[sato31]

コード修正でニアミスがあったようです

ArrayView()を追加修正しました。

すみません

ご確認をお願いします

[sato31]

本日は、

改行の修正ができないため

明日以降

改行対応いたします