Hs096/Questionable
1
0
Fork 0
forked from liza/Questionable
Code Pull Requests Activity

Only count successful jumps for for repeatedJumps

Browse Source
This commit is contained in:
Liza 2024-09-05 14:46:05 +02:00
parent b92531f2ec
commit e1ff326708
Signed by: liza
GPG Key ID: 7199F8D727D55F67
1 changed files with 7 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
Questionable/Controller/Steps/Interactions/Jump.cs
View File

@ -1,4 +1,5 @@
using System; using System;
using Dalamud.Game.ClientState.Conditions;
using Dalamud.Plugin.Services; using Dalamud.Plugin.Services;
using FFXIVClientStructs.FFXIV.Client.Game; using FFXIVClientStructs.FFXIV.Client.Game;
using Microsoft.Extensions.DependencyInjection; using Microsoft.Extensions.DependencyInjection;
@ -15,6 +16,7 @@ internal static class Jump
MovementController movementController, MovementController movementController,
IClientState clientState, IClientState clientState,
IFramework framework, IFramework framework,
ICondition condition,
ILoggerFactory loggerFactory) : SimpleTaskFactory ILoggerFactory loggerFactory) : SimpleTaskFactory
{ {
public override ITask? CreateTask(Quest quest, QuestSequence sequence, QuestStep step) public override ITask? CreateTask(Quest quest, QuestSequence sequence, QuestStep step)
@ -38,7 +40,7 @@ internal static class Jump
public ITask RepeatedJumps(uint? dataId, JumpDestination jumpDestination, string? comment) public ITask RepeatedJumps(uint? dataId, JumpDestination jumpDestination, string? comment)
{ {
return new DoRepeatedJumps(dataId, jumpDestination, comment, movementController, clientState, framework, return new DoRepeatedJumps(dataId, jumpDestination, comment, movementController, clientState, framework,
loggerFactory.CreateLogger<DoRepeatedJumps>()); condition, loggerFactory.CreateLogger<DoRepeatedJumps>());
} }
} }
@ -91,6 +93,7 @@ internal static class Jump
MovementController movementController, MovementController movementController,
IClientState clientState, IClientState clientState,
IFramework framework, IFramework framework,
ICondition condition,
ILogger<DoRepeatedJumps> logger) ILogger<DoRepeatedJumps> logger)
: DoSingleJump(dataId, jumpDestination, comment, movementController, clientState, framework) : DoSingleJump(dataId, jumpDestination, comment, movementController, clientState, framework)
{ {
@ -108,7 +111,7 @@ internal static class Jump
public override ETaskResult Update() public override ETaskResult Update()
{ {
if (DateTime.Now < _continueAt) if (DateTime.Now < _continueAt || condition[ConditionFlag.Jumping])
return ETaskResult.StillRunning; return ETaskResult.StillRunning;
float stopDistance = _jumpDestination.CalculateStopDistance(); float stopDistance = _jumpDestination.CalculateStopDistance();
@ -120,10 +123,10 @@ internal static class Jump
_jumpDestination.Position.Y - 0.5f); _jumpDestination.Position.Y - 0.5f);
unsafe unsafe
{ {
ActionManager.Instance()->UseAction(ActionType.GeneralAction, 2); if (ActionManager.Instance()->UseAction(ActionType.GeneralAction, 2))
++_attempts;
} }
++_attempts;
if (_attempts >= 50) if (_attempts >= 50)
throw new TaskException("Tried to jump too many times, didn't reach the target"); throw new TaskException("Tried to jump too many times, didn't reach the target");