SwiftUI MVVM架构下产品单选逻辑的正确实现方式咨询

你的实现方向是符合MVVM规范的——将状态（产品列表、选中的产品）统一放在ViewModel中管理，视图仅负责渲染数据和转发用户交互，没有在视图层持有状态，这完全契合MVVM“视图做展示、ViewModel管状态和逻辑”的核心原则。

不过当前实现还有可以优化的地方，让代码的职责划分更清晰、健壮性更强：

一、当前实现的正确性确认 #

你通过@Published在ViewModel中维护list和selectedProduct，视图通过ForEach遍历渲染，点击时修改选中状态，这种模式是合规的：

• 视图没有业务逻辑，只做“展示+触发交互”
• 状态完全由ViewModel掌控，便于后续扩展逻辑（比如选中时触发网络请求、统计事件等）

唯一需要注意的点：如果Product是值类型（struct），当列表中的Product实例更新时，selectedProduct可能和列表中的实例不是同一个对象（值类型赋值是拷贝），可能导致选中状态判断失效。

二、更优的实现方式 #

1. 封装选中逻辑为ViewModel方法，避免视图直接修改状态 #

不要在视图中直接赋值viewModel.selectedProduct = product，而是在ViewModel中定义专门的方法处理选中逻辑。这样ViewModel可以完全掌控状态变化的过程，方便后续添加校验、日志、联动操作等。

示例代码：

class ProductsViewModel: ObservableObject {
    @Published var list: [Product] = []
    @Published var selectedProduct: Product? // 建议设为可选，支持取消选中

    // 封装选中方法
    func selectProduct(_ product: Product) {
        // 可选逻辑：点击已选中的产品时取消选中
        selectedProduct = selectedProduct?.id == product.id ? nil : product
        // 这里可以添加其他业务逻辑，比如上报选中事件、更新其他关联状态
    }
}

视图中调用方法：

ForEach(viewModel.list) { product in
    ProductRow(product: product, isSelected: viewModel.selectedProduct?.id == product.id)
        .onTapGesture {
            viewModel.selectProduct(product)
        }
}

2. 用产品唯一ID管理选中状态，替代直接存储Product实例 #

如果Product是值类型，直接存储实例容易出现“实例不匹配”的问题。改用产品的唯一标识（比如id: UUID）存储选中状态，能避免这个问题，同时让状态更轻量化。

示例代码：

class ProductsViewModel: ObservableObject {
    @Published var list: [Product] = []
    @Published var selectedProductId: UUID? // 存储选中ID

    func selectProduct(withId id: UUID) {
        selectedProductId = selectedProductId == id ? nil : id
    }
}

视图中判断选中状态：

ForEach(viewModel.list) { product in
    ProductRow(product: product, isSelected: product.id == viewModel.selectedProductId)
        .onTapGesture {
            viewModel.selectProduct(withId: product.id)
        }
}

3. 让ProductRow成为纯展示组件 #

不要让ProductRow依赖ViewModel或自行处理选中逻辑，而是通过传入isSelected参数让它只负责展示样式。这样ProductRow的复用性更强，也符合单一职责原则。

示例代码：

struct ProductRow: View {
    let product: Product
    let isSelected: Bool

    var body: some View {
        HStack {
            Text(product.name)
                .foregroundColor(isSelected ? .blue : .primary)
            Spacer()
            if isSelected {
                Image(systemName: "checkmark.circle.fill")
                    .foregroundColor(.blue)
            }
        }
        .padding(.vertical, 8)
    }
}

总结 #

你的初始实现是符合MVVM规范的，上述优化方案主要是让状态管理更健壮、职责划分更清晰，同时提升代码的可维护性和复用性。

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