[solved] Slow medkits action

[azz]

Good day! Is there a way to make a medikits stretched over time, for example 1 health per second?

[Matt]

Would something like this help? (untested, apologies for typos)

Code: Select all

class LeakyMedikit:Inventory{
    int ticker;
    override void DoEffect() //this happens every tic while carried
    {
        if(!owner)return; //null check
        if(amount<1) //remove if run out
        {
            destroy();
            return;
        }
        ticker--; //35 tick = 1 second
        if(ticker<1)
        {
            ticker=35; //reset the counter
            amount--; //expend 1 unit of itself
            owner.GiveBody(1); //heal owner 1 health
        }
    }
    default
    {
        inventory.pickupmessage "Picked up a medikit.";
        inventory.amount 25;
        inventory.maxamount 100;
    }
    states
    {
    spawn:
        MEDI A -1;
        stop;
    }
}

EDIT: changed goawayanddie to destroy

[azz]

Thank's, how i can use it? It must be in file? I am nube in modes dev

[ramon.dexter]

This is the right place for you:

https://zdoom.org/wiki/Main_Page

[Zen3001]

I made this for you, don't know how much you allready know about gzdoom but if you're just getting into modding...
use slade to edit wads and pk3 files and read something about DECORATE.
there's more than just decorate but that's where you should start I guess, if you allready know about everything I just wrote then just read arround the wiki ramon mentioned.

[Apeirogon]

What difference between goawayanddie() and destroy()?

[Matt]

Destroy() destroys the actor immediately, while GoAwayAndDie() waits at least 1 tick (and does some other stuff too, I assume, but I don't know what that stuff would be).

I only used GoAwayAndDie() here on the assumption that that function does something important that owned inventory actors should be doing before they are destroyed. If I'm wrong, Destroy() would be better since it ends up with less weird side effects when interacting with other inventory manipulation.

[Graf Zahl]

GoAwayAndDie is meant for removing items during pickup actions because those need to do some checks for item respawning settings. In DoEffect you should use Destroy instead.

[azz]

I made this for you, don't know how much you allready know about gzdoom but if you're just getting into modding...
use slade to edit wads and pk3 files and read something about DECORATE.
there's more than just decorate but that's where you should start I guess, if you allready know about everything I just wrote then just read arround the wiki ramon mentioned.

Very big thank's! I'll be learn it.

[azz]

One problem, stimpak & medikit now is always pickup. I try to add "-INVENTORY.ALWAYSPICKUP" to "default" section, but it isn't work.

[Apeirogon]

You need to override trypickup virtual.
Here how it works in regular medikit

But you need something like

Code: Select all

override bool TryPickup (in out Actor other)
{
	int PrevHealth = other.player != NULL ? other.player.health : other.health;

	if (other.GiveBody(Amount, MaxAmount))
	{
		GoAwayAndDie();
		return true;
	}
	return false;
}

And also you must add flag INVENTORY.ISHEALTH into default block, to let gzdoom know that this item is a health item. Its important if you play with cvar "respavn healing item" set to true, so gzdoom can properly respawn this item.