Tidy up code and naming

This commit is contained in:
Dean Herbert
2022-09-09 15:11:26 +09:00
parent 2709a4d398
commit 715e9018da

View File

@ -19,16 +19,16 @@ namespace osu.Game.Rulesets.Catch.UI
{ {
public class CatchTouchInputMapper : VisibilityContainer public class CatchTouchInputMapper : VisibilityContainer
{ {
private Dictionary<object, TouchCatchAction> trackedActions = new Dictionary<object, TouchCatchAction>(); private Dictionary<object, TouchCatchAction> trackedActionSources = new Dictionary<object, TouchCatchAction>();
private KeyBindingContainer<CatchAction> keyBindingContainer = null!; private KeyBindingContainer<CatchAction> keyBindingContainer = null!;
private Container mainContent = null!; private Container mainContent = null!;
private ArrowHitbox leftBox = null!; private InputArea leftBox = null!;
private ArrowHitbox rightBox = null!; private InputArea rightBox = null!;
private ArrowHitbox leftDashBox = null!; private InputArea leftDashBox = null!;
private ArrowHitbox rightDashBox = null!; private InputArea rightDashBox = null!;
public override bool PropagatePositionalInputSubTree => true; public override bool PropagatePositionalInputSubTree => true;
public override bool PropagateNonPositionalInputSubTree => true; public override bool PropagateNonPositionalInputSubTree => true;
@ -61,7 +61,7 @@ namespace osu.Game.Rulesets.Catch.UI
Origin = Anchor.CentreLeft, Origin = Anchor.CentreLeft,
Children = new Drawable[] Children = new Drawable[]
{ {
leftBox = new ArrowHitbox(TouchCatchAction.MoveLeft, ref trackedActions, colours.Gray3) leftBox = new InputArea(TouchCatchAction.MoveLeft, ref trackedActionSources, colours.Gray3)
{ {
Anchor = Anchor.CentreRight, Anchor = Anchor.CentreRight,
Origin = Anchor.CentreRight, Origin = Anchor.CentreRight,
@ -69,7 +69,7 @@ namespace osu.Game.Rulesets.Catch.UI
RelativePositionAxes = Axes.Both, RelativePositionAxes = Axes.Both,
Width = 0.5f, Width = 0.5f,
}, },
leftDashBox = new ArrowHitbox(TouchCatchAction.DashLeft, ref trackedActions, colours.Gray2) leftDashBox = new InputArea(TouchCatchAction.DashLeft, ref trackedActionSources, colours.Gray2)
{ {
Anchor = Anchor.CentreLeft, Anchor = Anchor.CentreLeft,
Origin = Anchor.CentreLeft, Origin = Anchor.CentreLeft,
@ -88,7 +88,7 @@ namespace osu.Game.Rulesets.Catch.UI
Origin = Anchor.CentreRight, Origin = Anchor.CentreRight,
Children = new Drawable[] Children = new Drawable[]
{ {
rightBox = new ArrowHitbox(TouchCatchAction.MoveRight, ref trackedActions, colours.Gray3) rightBox = new InputArea(TouchCatchAction.MoveRight, ref trackedActionSources, colours.Gray3)
{ {
Anchor = Anchor.CentreLeft, Anchor = Anchor.CentreLeft,
Origin = Anchor.CentreLeft, Origin = Anchor.CentreLeft,
@ -96,7 +96,7 @@ namespace osu.Game.Rulesets.Catch.UI
RelativePositionAxes = Axes.Both, RelativePositionAxes = Axes.Both,
Width = 0.5f, Width = 0.5f,
}, },
rightDashBox = new ArrowHitbox(TouchCatchAction.DashRight, ref trackedActions, colours.Gray2) rightDashBox = new InputArea(TouchCatchAction.DashRight, ref trackedActionSources, colours.Gray2)
{ {
Anchor = Anchor.CentreRight, Anchor = Anchor.CentreRight,
Origin = Anchor.CentreRight, Origin = Anchor.CentreRight,
@ -137,7 +137,7 @@ namespace osu.Game.Rulesets.Catch.UI
// Loop through the buttons to avoid keeping a button pressed if both mouse buttons are pressed. // Loop through the buttons to avoid keeping a button pressed if both mouse buttons are pressed.
foreach (MouseButton i in e.PressedButtons) foreach (MouseButton i in e.PressedButtons)
trackedActions[i] = touchCatchAction; trackedActionSources[i] = touchCatchAction;
calculateActiveKeys(); calculateActiveKeys();
return true; return true;
@ -147,7 +147,7 @@ namespace osu.Game.Rulesets.Catch.UI
{ {
Show(); Show();
trackedActions[e.Touch.Source] = getTouchCatchActionFromInput(e.ScreenSpaceTouch.Position); trackedActionSources[e.Touch.Source] = getTouchCatchActionFromInput(e.ScreenSpaceTouch.Position);
calculateActiveKeys(); calculateActiveKeys();
base.OnTouchMove(e); base.OnTouchMove(e);
@ -165,24 +165,6 @@ namespace osu.Game.Rulesets.Catch.UI
base.OnTouchUp(e); base.OnTouchUp(e);
} }
private void calculateActiveKeys()
{
if (trackedActions.ContainsValue(TouchCatchAction.DashLeft) || trackedActions.ContainsValue(TouchCatchAction.MoveLeft))
keyBindingContainer.TriggerPressed(CatchAction.MoveLeft);
else
keyBindingContainer.TriggerReleased(CatchAction.MoveLeft);
if (trackedActions.ContainsValue(TouchCatchAction.DashRight) || trackedActions.ContainsValue(TouchCatchAction.MoveRight))
keyBindingContainer.TriggerPressed(CatchAction.MoveRight);
else
keyBindingContainer.TriggerReleased(CatchAction.MoveRight);
if (trackedActions.ContainsValue(TouchCatchAction.DashRight) || trackedActions.ContainsValue(TouchCatchAction.DashLeft))
keyBindingContainer.TriggerPressed(CatchAction.Dash);
else
keyBindingContainer.TriggerReleased(CatchAction.Dash);
}
private bool handleDown(object source, Vector2 position) private bool handleDown(object source, Vector2 position)
{ {
TouchCatchAction catchAction = getTouchCatchActionFromInput(position); TouchCatchAction catchAction = getTouchCatchActionFromInput(position);
@ -190,7 +172,7 @@ namespace osu.Game.Rulesets.Catch.UI
if (catchAction == TouchCatchAction.None) if (catchAction == TouchCatchAction.None)
return false; return false;
trackedActions[source] = catchAction; trackedActionSources[source] = catchAction;
calculateActiveKeys(); calculateActiveKeys();
@ -199,11 +181,29 @@ namespace osu.Game.Rulesets.Catch.UI
private void handleUp(object source) private void handleUp(object source)
{ {
trackedActions.Remove(source); trackedActionSources.Remove(source);
calculateActiveKeys(); calculateActiveKeys();
} }
private void calculateActiveKeys()
{
if (trackedActionSources.ContainsValue(TouchCatchAction.DashLeft) || trackedActionSources.ContainsValue(TouchCatchAction.MoveLeft))
keyBindingContainer.TriggerPressed(CatchAction.MoveLeft);
else
keyBindingContainer.TriggerReleased(CatchAction.MoveLeft);
if (trackedActionSources.ContainsValue(TouchCatchAction.DashRight) || trackedActionSources.ContainsValue(TouchCatchAction.MoveRight))
keyBindingContainer.TriggerPressed(CatchAction.MoveRight);
else
keyBindingContainer.TriggerReleased(CatchAction.MoveRight);
if (trackedActionSources.ContainsValue(TouchCatchAction.DashRight) || trackedActionSources.ContainsValue(TouchCatchAction.DashLeft))
keyBindingContainer.TriggerPressed(CatchAction.Dash);
else
keyBindingContainer.TriggerReleased(CatchAction.Dash);
}
private TouchCatchAction getTouchCatchActionFromInput(Vector2 inputPosition) private TouchCatchAction getTouchCatchActionFromInput(Vector2 inputPosition)
{ {
if (leftDashBox.Contains(inputPosition)) if (leftDashBox.Contains(inputPosition))
@ -228,7 +228,7 @@ namespace osu.Game.Rulesets.Catch.UI
mainContent.FadeOut(300); mainContent.FadeOut(300);
} }
private class ArrowHitbox : CompositeDrawable, IKeyBindingHandler<CatchAction> private class InputArea : CompositeDrawable, IKeyBindingHandler<CatchAction>
{ {
private readonly TouchCatchAction handledAction; private readonly TouchCatchAction handledAction;
@ -238,7 +238,7 @@ namespace osu.Game.Rulesets.Catch.UI
private bool isHiglighted; private bool isHiglighted;
public ArrowHitbox(TouchCatchAction handledAction, ref Dictionary<object, TouchCatchAction> trackedActions, Color4 colour) public InputArea(TouchCatchAction handledAction, ref Dictionary<object, TouchCatchAction> trackedActions, Color4 colour)
{ {
this.handledAction = handledAction; this.handledAction = handledAction;
this.trackedActions = trackedActions; this.trackedActions = trackedActions;