SwiftUI中CoreData Location实体取数失败及ForEach警告问题求助

核心问题1：用错了ForEach #

你在getLocationNames()里用的是SwiftUI的ForEach视图组件，它是用来生成UI元素的，不是普通的循环工具。你创建了它但没把它加到视图树里，所以它根本不会执行遍历逻辑，自然读不到数据，还会弹出“未使用初始化结果”的警告。

核心问题2：冗余的数组维护 #

手动维护locationNames完全没必要，@FetchRequest返回的FetchedResults<Location>本身就是可遍历的集合，而且会自动响应CoreData数据的变化，直接用它生成Picker选项更可靠。

具体修复步骤 #

1. 替换错误的遍历逻辑：删掉getLocationNames()里的ForEach，改用普通的for-in循环遍历locations集合。
2. 清理冗余代码：去掉onReceive的多余监听，Picker的selection绑定会自动更新selectedLocation。
3. 修复强制解包风险：把locationType!、locationName!这类强制解包改成可选绑定或默认值，避免空值崩溃。
4. 优化Picker数据源：直接用locations作为Picker的数据源，不用手动维护locationNames数组（更简洁可靠）。

修正后的完整代码 #

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

    @FetchRequest(
        sortDescriptors: [NSSortDescriptor(keyPath: \Location.locationName, ascending: true)],
        animation: .default)
    private var locations: FetchedResults<Location>

    @State private var selectedLocation: String? = nil

    var body: some View {
        NavigationView {
            ScrollView(showsIndicators: false) {
                HStack {
                    Text("Select Location")

                    Picker("Location", selection: $selectedLocation) {
                        ForEach(locations, id: \.self) { location in
                            Text(location.displayString)
                                .tag(location.displayString)
                        }
                    }
                    .pickerStyle(MenuPickerStyle())
                    .onTapGesture {
                        print("当前已加载\(locations.count)个地点")
                    }
                }
                .header {
                    Text("**Current**")
                        .padding(.top, 10)
                }
            }
            .navigationTitle("Locations")
            .toolbar {
                ToolbarItem(placement: .confirmationAction) {
                    Button("Detail", action: onAdd)
                        .foregroundColor(.blue)
                }
            }
            .onAppear {
                print("页面加载，共\(locations.count)个地点")
            }
        }
    }

    private func onAdd() {
        // 补充你的onAdd逻辑
    }
}

// 给Location实体加扩展，生成显示用的字符串
extension Location {
    var displayString: String {
        let type = locationType ?? "未知类型"
        let name = locationName ?? "未知名称"
        let zip = zip ?? "无邮编"
        return "\(type); \(name), \(zip)"
    }
}

额外优化说明 #

• 把生成显示字符串的逻辑放到Location的扩展里，代码更模块化，避免视图里混杂数据处理逻辑。
• 直接用locations作为Picker数据源，FetchedResults会自动监听CoreData变化，数据更新时Picker自动刷新，无需手动维护数组。
• 用默认值替代强制解包，避免空值导致程序崩溃。
• 删掉冗余的状态变量，简化视图状态管理。

内容的提问来源于stack exchange，提问作者vrao