await或者异步回调用多了，得加isValid判断，搞得整个人都提心掉胆的

Creator 3.x
#50

难道你用回调就不需要判断isValid了吗？

#51

自己封装下promise呗，然后建个基类管理下

#52

难道你用回调就不需要判断isValid了吗？

#53

【包教包会】Promise?async?await?终于有人把异步解释清楚了 稍微改造下就能满足了 注意自己工程的使用场景

#54

你都异步了为啥不加个事件通知处理呢，只要销毁的时候移除事件监听不就好了

#55

但是调试比较蛋疼

#56

const perfab=await load(…)//等待加载资源完成
const node=cc.clone(prefab)
node.runAction(xxxx)
node…
this.xxxxx
…

楼主说的类似这种情况吧 await之后的所有执行

#57
1赞
#58

哦，那我理解错了，我以为回调的情况呢

#59

习惯加就好了

#60

用装饰器呀，比如你定制一个函数的装饰器，这个装饰器可以看作是对函数功能拓展，比如你说的安全性校验，就可以放到装饰器内
function promise_function(_: any, __: string, descriptor: PropertyDescriptor) {
const original = descriptor.value;

descriptor.value = async function (...args: any[]) {
    // 执行前节点校验
    if (!cc.isValid(this.node)) {
        cc.warn(`[promise_function] node invalid before executing ${original.name}`);
        return;
    }

    try {
        const result = await original.apply(this, args);

        // 执行后再校验一次
        if (!cc.isValid(this.node)) {
            cc.warn(`[promise_function] node invalid after executing ${original.name}`);
            return;
        }

        return result;
    } catch (err) {
        cc.error(`[promise_function] error in ${original.name}:`, err);
    }
};

return descriptor;

}

假设function1 是一个await，
@promise_function
async function1 () {
const res = await this.fakeAsyncLoad();
this.node.setPosition(cc.v3(0, 0, 0)); // 如果 node 已经被销毁，就不会走到这里
} 在项目中，我是这么使用的

2赞
#61

以下是AI给的建议,未实际验证.

// config flags（按你的项目常量来，用 CC_DEBUG / CC_DEV 也行）
const __DEV__ = typeof CC_DEBUG !== 'undefined' ? CC_DEBUG : true;

type Options = {
  log?: boolean;       // 是否打印 warn/error
  rethrow?: boolean;   // 发生异常后是否继续抛出给上层
  afterCheck?: boolean;// 是否在 await 后再二次校验（默认 true）
};
export function promise_function(options: Options = {}) {
  const { log = __DEV__, rethrow = true, afterCheck = true } = options;

  return function (_: any, __: string, descriptor: PropertyDescriptor) {
    const original = descriptor.value;

    descriptor.value = async function (...args: any[]) {
      // 1) 执行前校验
      const node: cc.Node | undefined = (this as any)?.node;
      if (!node || !cc.isValid(node, true)) { // 第二个参数true: 包括正在销毁的情况
        if (log) cc.warn('[promise_function] node invalid before:', original.name);
        return;
      }

      try {
        const ret = await original.apply(this, args);

        // 2)（可选）执行后再校验一次，防“中途被销毁”
        if (afterCheck) {
          if (!cc.isValid(node, true)) {
            if (log) cc.warn('[promise_function] node invalid after:', original.name);
            return;
          }
        }
        return ret;
      } catch (err) {
        if (log) cc.error('[promise_function] error in', original.name, err);
        if (rethrow) throw err; // 默认：记录后继续抛，让上层能感知
        return;
      }
    };

    return descriptor;
  };
}

使用:

@promise_function({ log: true, rethrow: true }) // 开发期
async function1 () {
  const res = await this.fakeAsyncLoad();
  this.node.setPosition(cc.v3(0, 0, 0));
}

// 需要极致性能的构建（发布态）可以这样统一关闭日志：
@promise_function({ log: false, rethrow: true })
1赞
#62

isValid越少，说明封装越深，说明适用范围越少。
所以不要在乎这些，适用自己就行了。

#63

和 界面结合起来就得判断。有时候还得 try catch. 烦

#64

额 fakeAsyncLoad完成后 this没了 不一样吗

#65

isvaild不丢人，主要是担心项目组又来了应届毕业生，review代码的时候提心吊胆的 :rofl: :rofl: :rofl:

#66

确实，老老实实判断下节点或者资源状态挺好的，方便查看和调试。就怕新人来搞些不一样的花活 :sweat_smile:

#67

对于界面上的虚拟列表上的节点，有时候加了 isVaild 也不行，还得加别的判断 哈哈哈哈

#68

解决方案就是把isValid自己封装在框架里，或者用语法糖编译后生成。。。 你真的需要这样的方案吗

#69

新人一来就给底层封装好的逻辑加几个if判断，看明白代码是很难的，最后觉得，直接来个if搞一下能跑就行 :crazy_face: