Swift开发Philips Hue应用：ViewModel属性订阅失效问题排查

你的核心问题是Combine订阅的生命周期管理错误：当调用bulb.$isOn.sink时，返回的AnyCancellable没有被存储，会在闭包执行完毕后立即被销毁，导致订阅仅在初始赋值时触发一次，后续isOn变更时不会收到通知。另外你怀疑的“数组存储对象副本”不成立——BulbViewModel是类（引用类型），数组中存储的是对象引用，而非副本。

方案一：让BulbViewModel自身管理订阅与业务逻辑（推荐） #

这种方式符合单一职责原则，每个灯泡ViewModel自己负责状态变更后的业务逻辑（比如调用Hue桥接器接口）。

修改BulbViewModel.swift #

class BulbViewModel: ObservableObject, Identifiable {
    @Published var color = 250.0
    @Published var amplitude = 250.0
    @Published var isOn = false
    @Published var name: String = ""
    @Published var id: String
    
    private let hueController: HueController
    private var cancellables = Set<AnyCancellable>()
    
    var isOnText: String {
        isOn ? "ON" : "OFF"
    }
    
    init(id: String, hueController: HueController){
        self.id = id
        self.hueController = hueController
        
        // 订阅isOn状态变更
        $isOn
            .sink { [weak self] isOn in
                guard let self = self else { return }
                print("\(isOn) set on \(self.name)")
                // 调用Hue控制器发送状态更新请求
                self.hueController.updateLightState(bulbId: self.id, isOn: isOn)
            }
            .store(in: &self.cancellables)
    }
}

修改ApplicationData.swift #

class ApplicationData: ObservableObject {
    @Published var hueResources: HueSession
    @Published var bulbs: [BulbViewModel]
    
    var hue = HueController()
    
    init(){
        bulbs = []
        hueResources = HueSession()
        
        hue.GetLightsList() { [weak self] resources in
            guard let self = self, let resources = resources else { return }
            self.hueResources = resources
            
            self.hueResources.data.forEach { value in
                // 初始化ViewModel时传入Hue控制器
                let bulb = BulbViewModel(id: value.id, hueController: self.hue)
                bulb.name = value.metadata.name
                bulb.isOn = value.on.on
                
                self.bulbs.append(bulb)
            }
        }
    }
}

方案二：在ApplicationData中统一管理所有订阅 #

如果希望集中管理所有灯泡的状态变更逻辑，可以在ApplicationData中存储所有订阅引用：

修改ApplicationData.swift #

class ApplicationData: ObservableObject {
    @Published var hueResources: HueSession
    @Published var bulbs: [BulbViewModel]
    
    var hue = HueController()
    private var cancellables = Set<AnyCancellable>()
    
    init(){
        bulbs = []
        hueResources = HueSession()
        
        hue.GetLightsList() { [weak self] resources in
            guard let self = self, let resources = resources else { return }
            self.hueResources = resources
            
            self.hueResources.data.forEach { value in
                let bulb = BulbViewModel(id: value.id)
                bulb.name = value.metadata.name
                bulb.isOn = value.on.on
                
                // 订阅isOn变更并存储引用
                bulb.$isOn
                    .sink { isOn in
                        print("\(isOn) set on \(bulb.name)")
                        // 调用Hue控制器发送更新请求
                        self.hue.updateLightState(bulbId: bulb.id, isOn: isOn)
                    }
                    .store(in: &self.cancellables)
                
                self.bulbs.append(bulb)
            }
        }
    }
}

关键注意事项 #

1. 必须存储AnyCancellable：所有Combine订阅返回的AnyCancellable都需要存储到集合中，否则订阅会立即失效。
2. 避免循环引用：在闭包中使用[weak self]或[weak bulb]，防止对象因强引用无法被释放。
3. 业务逻辑与UI分离：两种方案都避免了在BulbView中处理业务逻辑，符合MVVM的最佳实践。

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