Add Enemy Control

[xAstroBoy]

Closes #250

[winstxnhdw]

Oh dear, there's a lot more problems than I thought.

    public EnemyIdentity IdentifyEnemy(EnemyAI enemy) {
        return enemy is null
            ? EnemyIdentity.None
            : enemy switch {
                CentipedeAI => EnemyIdentity.Centipede,
                BaboonBirdAI => EnemyIdentity.Baboon,
                FlowermanAI => EnemyIdentity.Flowerman,
                ForestGiantAI => EnemyIdentity.ForestGiant,
                HoarderBugAI => EnemyIdentity.HoarderBug,
                JesterAI => EnemyIdentity.Jester,
                MaskedPlayerEnemy => EnemyIdentity.Masked,
                MouthDogAI => EnemyIdentity.MouthDog,
                NutcrackerEnemyAI => EnemyIdentity.Nutcracker,
                PufferAI => EnemyIdentity.Puffer,
                SandWormAI => EnemyIdentity.Sandworm,
                SpringManAI => EnemyIdentity.Springman,
                _ => EnemyIdentity.Default,
            };
    }

Why did you not just return the enemy instead? You are making a lot of unnecessary indirections, which you later have to indirect back again.

Please be patient while I fix this.

[xAstroBoy]

Oh dear, there's a lot more problems than I thought.

    public EnemyIdentity IdentifyEnemy(EnemyAI enemy) {
        return enemy is null
            ? EnemyIdentity.None
            : enemy switch {
                CentipedeAI => EnemyIdentity.Centipede,
                BaboonBirdAI => EnemyIdentity.Baboon,
                FlowermanAI => EnemyIdentity.Flowerman,
                ForestGiantAI => EnemyIdentity.ForestGiant,
                HoarderBugAI => EnemyIdentity.HoarderBug,
                JesterAI => EnemyIdentity.Jester,
                MaskedPlayerEnemy => EnemyIdentity.Masked,
                MouthDogAI => EnemyIdentity.MouthDog,
                NutcrackerEnemyAI => EnemyIdentity.Nutcracker,
                PufferAI => EnemyIdentity.Puffer,
                SandWormAI => EnemyIdentity.Sandworm,
                SpringManAI => EnemyIdentity.Springman,
                _ => EnemyIdentity.Default,
            };
    }

Why did you not just return the enemy instead? You are making a lot of unnecessary indirections, which you later have to indirect back again.

Please be patient while I fix this.

that is how to identify which enemy class is to specifically control without casting the enemy over and over?

[winstxnhdw]

You could probably just do this.

T IdentifyEnemy(EnemyAI enemy) where T : EnemyAI {
   ...
}

without casting the enemy over and over?

Also, you are doing a lot of casting anyways.

            case EnemyIdentity.Centipede:
                return ((CentipedeAI)this.EnemyToPossess).GetSecondarySkillName();
            case EnemyIdentity.Baboon:
                return ((BaboonBirdAI)this.EnemyToPossess).GetSecondarySkillName();
            case EnemyIdentity.HoarderBug:
                return ((HoarderBugAI)this.EnemyToPossess).GetSecondarySkillName();
            case EnemyIdentity.Jester:
                return ((JesterAI)this.EnemyToPossess).GetSecondarySkillName();
            case EnemyIdentity.Nutcracker:
                return ((NutcrackerEnemyAI)this.EnemyToPossess).GetSecondarySkillName();
            case EnemyIdentity.Puffer:
                return ((PufferAI)this.EnemyToPossess).GetSecondarySkillName();
            case EnemyIdentity.Sandworm:
                return ((SandWormAI)this.EnemyToPossess).GetSecondarySkillName();
            case EnemyIdentity.MouthDog:
                return ((MouthDogAI)this.EnemyToPossess).GetSecondarySkillName();

[winstxnhdw]

I am still looking really superficially. Give me a while to dig into it.

[winstxnhdw]

It would've probably been better to use a IController interface since every extension has a common method. We'll see.

[xAstroBoy]

You could probably just do this.

T IdentifyEnemy(EnemyAI enemy) where T : EnemyAI {
   ...
}

without casting the enemy over and over?

Also, you are doing a lot of casting anyways.

            case EnemyIdentity.Centipede:
                return ((CentipedeAI)this.EnemyToPossess).GetSecondarySkillName();
            case EnemyIdentity.Baboon:
                return ((BaboonBirdAI)this.EnemyToPossess).GetSecondarySkillName();
            case EnemyIdentity.HoarderBug:
                return ((HoarderBugAI)this.EnemyToPossess).GetSecondarySkillName();
            case EnemyIdentity.Jester:
                return ((JesterAI)this.EnemyToPossess).GetSecondarySkillName();
            case EnemyIdentity.Nutcracker:
                return ((NutcrackerEnemyAI)this.EnemyToPossess).GetSecondarySkillName();
            case EnemyIdentity.Puffer:
                return ((PufferAI)this.EnemyToPossess).GetSecondarySkillName();
            case EnemyIdentity.Sandworm:
                return ((SandWormAI)this.EnemyToPossess).GetSecondarySkillName();
            case EnemyIdentity.MouthDog:
                return ((MouthDogAI)this.EnemyToPossess).GetSecondarySkillName();

yes, but the switch case casts only based off which enemy is, and the enum acts as a identity

[winstxnhdw]

Have you seen IEnemyPrompter?

[xAstroBoy]

Have you seen IEnemyPrompter?

nvm

[xAstroBoy]

@winstxnhdw another issue is seconday attack while you are dead and controlling a enemy you risk of starting a vote for the ship to leave early

[winstxnhdw]

@winstxnhdw another issue is seconday attack while you are dead and controlling a enemy you risk of starting a vote for the ship to leave early

Good catch. We can patch that out.

[xAstroBoy]

@winstxnhdw i want to use Update() and add a better primary and secondary since with your input system sometimes it fails and it does not hold on the attacks.
Also a way to set the enemy walk/run speed would be handy as well, since controlcompany does that to make it as natural as possible.

[winstxnhdw]

i want to use Update() and add a better primary and secondary since with your input system sometimes it fails and it does not hold on the attacks.

Can you tell me how it fails? You might be implementing it wrongly. Hold commands must be set within the update loop like this.

    void Update() {
        InputListener.onShiftButtonHold?.Invoke(Keyboard.current[Key.LeftShift].isPressed);
        InputListener.onFButtonHold?.Invoke(Keyboard.current[Key.F].isPressed);
        InputListener.onRButtonHold?.Invoke(Keyboard.current[Key.R].isPressed);
        InputListener.onEButtonHold?.Invoke(Keyboard.current[Key.E].isPressed);

        foreach ((Func<bool> keyPressed, Action eventAction) in this.InputActions) {
            if (!keyPressed()) continue;
            eventAction();
        }
    }

[xAstroBoy]

i want to use Update() and add a better primary and secondary since with your input system sometimes it fails and it does not hold on the attacks.

Can you tell me how it fails? You might be implementing it wrongly. Hold commands must be set within the update loop like this.

    void Update() {
        InputListener.onShiftButtonHold?.Invoke(Keyboard.current[Key.LeftShift].isPressed);
        InputListener.onFButtonHold?.Invoke(Keyboard.current[Key.F].isPressed);
        InputListener.onRButtonHold?.Invoke(Keyboard.current[Key.R].isPressed);
        InputListener.onEButtonHold?.Invoke(Keyboard.current[Key.E].isPressed);

        foreach ((Func<bool> keyPressed, Action eventAction) in this.InputActions) {
            if (!keyPressed()) continue;
            eventAction();
        }
    }

yeah, the secondary needs to be a hold attack, because you need to keep pressing the button to make the jester crank and when it pops out the secondary has to be released, ill investigate better, but im not sure about that, as well we need to disable ping scan until we stop possessing the enemy

[winstxnhdw]

Also a way to set the enemy walk/run speed would be handy as well, since controlcompany does that to make it as natural as possible.

Would subscribing to the onLeftBracketPress and onRightBracketPress events to change speed work?

[Totoqoe]

There's a bug where if an enemy despawns or you get kicked while possessing it, then you're unable to to possess any other enemies until you restart.

[D1GQ]

There's also issue where the door gets absolutely spammed trying to open and close it as an enemy. And it seems disabling The death HUD doesn't work when possessing.

[xAstroBoy]

@winstxnhdw a health checker to see if a possessed enemy is dead to eject PossessionMod and a way to give AI the enemy control and override it would be nice.
Haven't figured how to fix the door issue .

[winstxnhdw]

AI the enemy control and override it would be nice

What?

[winstxnhdw]

Haven't figured how to fix the door issue .

Just use TriggerMod interact??

[xAstroBoy]

Haven't figured how to fix the door issue .

Just use TriggerMod interact??

yeah triggermod can do it, but when you are dead is not working to open/close doors.

[winstxnhdw]

Nope, it still works.

[xAstroBoy]

Nope, it still works.

not dedicated to the enemy itself tho?
for example if you possess a enemy and you want to make it leave the facility that wouldn't work.

[winstxnhdw]

Oh, you mean the entrances? Honestly, there is nothing wrong with noclipping outside. Let's figure it out next time.

[winstxnhdw]

I have been wondering what's the point of this? Do we just let the Nutcracker move on its own?

[winstxnhdw]

I just spent 7 hours on this. This is the last time I will be accepting a PR of such dubious quality. The next time I see a PR that will take more more than 2 hours to fix, I am just closing it. I can't waste so much time anymore.

[winstxnhdw]

This was absolute mental abuse.