精灵套件。从可选更改为强制展开会使应用程序崩溃

Question

我正在关注一个关于 IF 语句有问题的 SpriteKit 教程。该行的逻辑如下：如果子弹和小行星相撞，则将它们移除。

if body1.categoryBitMask == PhysicsCategories.bullet && body2.categoryBitMask == PhysicsCategories.asteroid {
   // remove bullet and asteroid
}

在尝试确保小行星 (body2.node) 在关闭之前位于可播放区域内时会出现问题。为此，作者添加了以下内容：

body2.node?.position.y < self.size.height

如下制作完整的 IF 语句：

if body1.categoryBitMask == PhysicsCategories.bullet && body2.categoryBitMask == PhysicsCategories.asteroid && body2.node?.position.y < self.size.height {
   // remove bullet and asteroid
}

显然，该行适用于 Swift 2，但是 Swift 3 进行了更正，将位置从可选位置更改并强制展开位置。

    if body1.categoryBitMask == PhysicsCategories.bullet && body2.categoryBitMask == PhysicsCategories.asteroid && body2.node!.position.y < self.size.height {
        // remove bullet and asteroid        
    }

通过强制展开位置，当三个物体发生碰撞时，应用程序会以“I THINK”的形式崩溃。看着屏幕真的很难分辨。

我正在测试下面的代码，目前还没有遇到任何问题。你们认为下面的修复会起作用吗？我在想的是，如果我确保 body2.node 不是 nil，那么应用程序没有理由崩溃，因为在尝试强制解包时不会遇到 nil。

if body1.categoryBitMask == PhysicsCategories.bullet && body2.categoryBitMask == PhysicsCategories.asteroid {
    // If the bullet has hit the asteroid
  if body2.node != nil {
    if ( body2.node!.position.y < self.size.height ) {
       // remove bullet and asteroid
    }           
  }           
}

或者，如果你们可以提出另一种方法来编写原始 IF 语句？

谢谢

Answer 1

是的，if != nil 语句（如目前所写的那样）将防止强制解包引起的崩溃。

另一种方法是在 Swift 中使用 if let 语法：

if body1.categoryBitMask == PhysicsCategories.bullet && body2.categoryBitMask == PhysicsCategories.asteroid {
    // If the bullet has hit the asteroid
    if let body2Node = body2.node {
        if body2Node.position.y < self.size.height {
           // remove bullet and asteroid
        }           
    }           
}

好处是它从您的代码中删除了!，并且更清楚地将 nil 检查与您稍后使用的变量联系起来。

Answer 2

nathan 有正确的答案，但更好的选择是使用 guard 来保护您的函数：

...

guard let  body1Node = body1.node, let  body2Node = body2.node else {return}
//Beyond this point we need to guarentee both nodes exist

if body1.categoryBitMask == PhysicsCategories.bullet && body2.categoryBitMask == PhysicsCategories.asteroid {
    // If the bullet has hit the asteroid
    if body2Node.position.y < self.size.height {
       // remove bullet and asteroid
    }           

}

通过使用守卫，我们减少了嵌套，并且我们知道，超出这一点，物理体必须包含一个节点，除非我们在函数结束之前将其删除，这将消除进一步检查节点是否存在的需要。

作为旁注，我始终建议在更新阶段结束时删除节点以避免此类问题，而只是以某种方式标记节点，您将不再使用它。