异步操作未被队列启动却标记完成：是否需处理及方案有效性？

首先咱们先搞明白这个警告的来龙去脉：SomeOperation went isFinished=YES without being started by the queue it is in，这个警告是OperationQueue在告诉你，它管理的某个操作违反了状态流转的规则——一个操作必须先被队列调用start()方法启动，才能进入isFinished=true的状态。而你的代码里，在队列还没来得及调用start()的时候，就直接把操作标记为完成了，这就触发了队列的校验警告。

1. 这个警告能忽略吗？问题严重吗？ #

绝对不建议忽略！虽然短期内可能不会直接崩溃，但这违反了Operation的状态契约，会给队列的状态管理埋下隐患：

• 队列内部会维护操作的执行计数和依赖关系，这种异常的状态流转可能导致依赖该操作的后续任务出现不可预期的行为；
• 长期来看，这种不规范的状态修改可能引发难以复现和排查的奇怪bug，比如队列卡住、操作状态混乱等。所以这个问题必须重视，不能视而不见。

2. 原代码的问题出在哪？ #

看你的cancel()实现：

public override func cancel() {
    super.cancel()
    markAsCompleted() //isExecuting = false, isFinished = true
}

当你把操作加入队列后，立刻调用cancel()，这时候队列还没来得及调用操作的start()方法。但你的cancel()方法直接调用了markAsCompleted()，把isFinished设为true——相当于操作跳过了“被队列启动”的步骤直接完成，这完全不符合OperationQueue的状态管理逻辑，所以警告必然会出现。

3. 用户提出的解决方案是否有效？ #

先看你给出的修改后的cancel()：

public override func cancel() {
    isExecuting = true //防止操作在启动前被取消
    super.cancel()
    markAsCompleted()
}

这个方案能暂时消除警告，但并不是正确的做法。手动设置isExecuting=true相当于模拟了操作已经被启动的状态，让队列误以为操作是被它启动后才完成的，但这其实是绕过了队列的状态管理流程，可能会带来其他潜在问题（比如KVO通知的时序混乱，或者队列对操作状态的跟踪出错）。

4. 正确的解决方式是什么？ #

咱们应该严格遵循Operation的状态流转规则：cancel()只负责标记操作的取消状态，不要在cancel()里直接修改isExecuting和isFinished，把状态流转的逻辑交给start()方法处理。

修改后的代码应该是这样：

public class SomeOperation : AsyncOperation {
    //MARK: Start
    public override func start() {
        // 首先必须检查是否已被取消，这是规范操作
        guard !isCancelled else {
            markAsCompleted()
            return
        }
        isExecuting = true
        doSomethingAsynchronously { [weak self] in
            self?.markAsCompleted()
        }
    }
    //MARK: Cancel
    public override func cancel() {
        super.cancel()
        // 这里只调用super.cancel()，不要调用markAsCompleted()
        // 如果异步任务已经在执行，可以在这里取消异步任务（比如取消网络请求、定时器等）
    }
}

这样修改后，当你在队列启动操作前调用cancel()，队列后续调用start()时，会先检查isCancelled，然后调用markAsCompleted()——这时候start()是被队列调用的，完全符合状态流转规则，警告自然就消失了。

另外补充：如果你的异步任务已经开始执行（比如doSomethingAsynchronously已经启动），那在cancel()里可以添加逻辑去取消这个异步任务（比如取消URLSessionTask、停止定时器等），但依然不要直接调用markAsCompleted()，而是让异步任务完成后（或者被取消后）再调用markAsCompleted()，或者在start()的逻辑里处理。

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