Generalize FlyCircle activity Aircraft caching naming

To make a possible future merger (or inheritance or other code-sharing) of these activities easier.
This commit is contained in:
reaperrr
2018-08-04 23:09:21 +02:00
committed by reaperrr
parent ae92255ded
commit c3a0d129a3
2 changed files with 14 additions and 14 deletions

View File

@@ -16,21 +16,21 @@ namespace OpenRA.Mods.Common.Activities
{ {
public class FlyCircle : Activity public class FlyCircle : Activity
{ {
readonly Aircraft plane; readonly Aircraft aircraft;
readonly WDist cruiseAltitude; readonly WDist cruiseAltitude;
int remainingTicks; int remainingTicks;
public FlyCircle(Actor self, int ticks = -1) public FlyCircle(Actor self, int ticks = -1)
{ {
plane = self.Trait<Aircraft>(); aircraft = self.Trait<Aircraft>();
cruiseAltitude = plane.Info.CruiseAltitude; cruiseAltitude = aircraft.Info.CruiseAltitude;
remainingTicks = ticks; remainingTicks = ticks;
} }
public override Activity Tick(Actor self) public override Activity Tick(Actor self)
{ {
// Refuse to take off if it would land immediately again. // Refuse to take off if it would land immediately again.
if (plane.ForceLanding) if (aircraft.ForceLanding)
{ {
Cancel(self); Cancel(self);
return NextActivity; return NextActivity;
@@ -45,8 +45,8 @@ namespace OpenRA.Mods.Common.Activities
return NextActivity; return NextActivity;
// We can't possibly turn this fast // We can't possibly turn this fast
var desiredFacing = plane.Facing + 64; var desiredFacing = aircraft.Facing + 64;
Fly.FlyToward(self, plane, desiredFacing, cruiseAltitude); Fly.FlyToward(self, aircraft, desiredFacing, cruiseAltitude);
return this; return this;
} }

View File

@@ -17,17 +17,17 @@ namespace OpenRA.Mods.Common.Activities
{ {
public class HeliFlyCircle : Activity public class HeliFlyCircle : Activity
{ {
readonly Aircraft helicopter; readonly Aircraft aircraft;
public HeliFlyCircle(Actor self) public HeliFlyCircle(Actor self)
{ {
helicopter = self.Trait<Aircraft>(); aircraft = self.Trait<Aircraft>();
} }
public override Activity Tick(Actor self) public override Activity Tick(Actor self)
{ {
// Refuse to take off if it would land immediately again. // Refuse to take off if it would land immediately again.
if (helicopter.ForceLanding) if (aircraft.ForceLanding)
{ {
Cancel(self); Cancel(self);
return NextActivity; return NextActivity;
@@ -36,14 +36,14 @@ namespace OpenRA.Mods.Common.Activities
if (IsCanceled) if (IsCanceled)
return NextActivity; return NextActivity;
if (HeliFly.AdjustAltitude(self, helicopter, helicopter.Info.CruiseAltitude)) if (HeliFly.AdjustAltitude(self, aircraft, aircraft.Info.CruiseAltitude))
return this; return this;
var move = helicopter.FlyStep(helicopter.Facing); var move = aircraft.FlyStep(aircraft.Facing);
helicopter.SetPosition(self, helicopter.CenterPosition + move); aircraft.SetPosition(self, aircraft.CenterPosition + move);
var desiredFacing = helicopter.Facing + 64; var desiredFacing = aircraft.Facing + 64;
helicopter.Facing = Util.TickFacing(helicopter.Facing, desiredFacing, helicopter.TurnSpeed / 3); aircraft.Facing = Util.TickFacing(aircraft.Facing, desiredFacing, aircraft.TurnSpeed / 3);
return this; return this;
} }