Processing碰撞检测逻辑失效：Ship与Asteroid未接触时持续触发碰撞判定

核心错误点 #

• draw()函数中的碰撞判断语句存在语法错误：if (collision == true); 句尾的分号会让if分支只执行空语句，后续的print("There is a hit")不受判断条件控制，每帧都会无条件执行，这就是控制台持续输出碰撞提示的最直接原因。
• 原有碰撞检测逻辑完全不具备空间判断意义：代码中将小行星的x、y坐标相加，与飞船的x、y坐标相加的结果做相等比较。这种判断完全不代表两个对象在二维空间上的位置接近，例如坐标为(20, 80)的小行星和坐标为(80, 20)的飞船，x+y结果都是100，哪怕两个对象相距极远也会被判定为位置重合。同时对浮点坐标直接用==做相等比较本身就存在精度偏差，几乎不可能精准匹配真实的碰撞位置。
• 硬编码数组索引的设计缺陷：hitShip()方法写死遍历下标0、1、2，一旦小行星数组长度调整就会触发数组越界错误，完全不具备通用性。
• 完全遗漏对象尺寸的判断：创建小行星时传入的random(60, 99)参数是小行星的半径属性，原有代码从未用到这个参数做距离判断，根本无法识别两个对象的实际碰撞范围。

具体修复方案 #

1. 移除if语句后的多余分号，将碰撞提示逻辑纳入判断分支内：

   if (collision) {
     println("There is a hit");
     // 替换原来的print为println，避免输出内容逐行粘连
   }
   

2. 基于圆形碰撞的通用规则重写hitShip()方法：当两个圆形对象的圆心直线距离小于两个对象的半径之和时，判定为碰撞。为了减少开平方运算的性能损耗，直接比较距离平方与半径和的平方：

   boolean hitShip(Ship ship, Asteroid[] asteroid) {
     for (int i = 0; i < asteroid.length; i++) {
       Asteroid rock = asteroid[i];
       // 计算两圆心的坐标差
       float dx = rock.getXPos() - ship.getXPos();
       float dy = rock.getYPos() - ship.getYPos();
       // 计算距离平方
       float distSq = dx * dx + dy * dy;
       // 读取两个对象的半径，计算半径和的平方
       float radiusSum = rock.getRadius() + ship.getRadius();
       if (distSq < radiusSum * radiusSum) {
         return true;
       }
     }
     return false;
   }
   

   补充说明：如果你的Asteroid类没有对外暴露获取半径的getRadius()方法，需要在类中补充该方法，将构造函数传入的半径值返回即可；同理Ship类也需要补充返回自身半径的getRadius()方法。

3. 修正初始化阶段的冗余逻辑：原代码在小行星的for循环内反复创建Satellite对象，循环结束后只会保留最后一次创建的实例，完全没有意义。应将Satellite的初始化代码移出for循环，仅执行一次创建。

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