Optimize world border check

This commit is contained in:
AJ Ferguson 2024-07-03 07:13:37 -04:00
parent 264b925990
commit 3a18ca9d16
2 changed files with 31 additions and 35 deletions

View file

@ -555,12 +555,11 @@ public class VehicleComponent<T extends LivingEntity & ClientVehicle> {
motion = motion.mul(movementMultiplier);
}
Vector3d correctedMovement = vehicle.getSession().getCollisionManager().correctMovement(
motion.toDouble(), boundingBox, vehicle.isOnGround(), this.stepHeight, true, vehicle.canWalkOnLava()
Vector3d correctedMovement = vehicle.getSession().getWorldBorder().correctMovement(boundingBox, motion.toDouble());
correctedMovement = vehicle.getSession().getCollisionManager().correctMovement(
correctedMovement, boundingBox, vehicle.isOnGround(), this.stepHeight, true, vehicle.canWalkOnLava()
);
correctedMovement = vehicle.getSession().getWorldBorder().correctMovement(boundingBox, correctedMovement);
boundingBox.translate(correctedMovement);
ctx.loadSurroundingBlocks();

View file

@ -205,48 +205,45 @@ public class WorldBorder {
Vector3d bbMin = boundingBox.getMin();
Vector3d bbMax = boundingBox.getMax();
// If outside border, don't adjust movement
if (bbMin.getX() + COLLISION_TOLERANCE < GenericMath.floor(minX)
|| bbMin.getZ() + COLLISION_TOLERANCE < GenericMath.floor(minZ)
|| bbMax.getX() - COLLISION_TOLERANCE > GenericMath.ceil(maxX)
|| bbMax.getZ() - COLLISION_TOLERANCE > GenericMath.ceil(maxZ)) {
double correctedX;
if (movement.getX() < 0) {
correctedX = -limitMovement(-movement.getX(), bbMin.getX() - GenericMath.floor(minX));
} else {
correctedX = limitMovement(movement.getX(), GenericMath.ceil(maxX) - bbMax.getX());
}
// Outside of border, don't adjust movement
if (Double.isNaN(correctedX)) {
return movement;
}
double diff; // Difference between edge of bounding box and world border
double correctedX = movement.getX();
diff = GenericMath.floor(minX) - (bbMin.getX() + movement.getX());
if (diff > 0) {
correctedX += diff;
double correctedZ;
if (movement.getZ() < 0) {
correctedZ = -limitMovement(-movement.getZ(), bbMin.getZ() - GenericMath.floor(minZ));
} else {
diff = (bbMax.getX() + movement.getX()) - GenericMath.ceil(maxX);
if (diff > 0) {
correctedX -= diff;
}
correctedZ = limitMovement(movement.getZ(), GenericMath.ceil(maxZ) - bbMax.getZ());
}
if (Math.abs(correctedX) < COLLISION_TOLERANCE) {
correctedX = 0;
}
double correctedZ = movement.getZ();
diff = GenericMath.floor(minZ) - (bbMin.getZ() + movement.getZ());
if (diff > 0) {
correctedZ += diff;
} else {
diff = (bbMax.getZ() + movement.getZ()) - GenericMath.ceil(maxZ);
if (diff > 0) {
correctedZ -= diff;
}
}
if (Math.abs(correctedZ) < COLLISION_TOLERANCE) {
correctedZ = 0;
if (Double.isNaN(correctedZ)) {
return movement;
}
return Vector3d.from(correctedX, movement.getY(), correctedZ);
}
private double limitMovement(double movement, double limit) {
if (limit < 0) {
// Return NaN to indicate outside of border
return Double.NaN;
}
if (limit < COLLISION_TOLERANCE) {
return 0;
}
return Math.min(movement, limit);
}
/**
* Updates the world border's minimum and maximum properties
*/