❔ Use `this` inside method decorator

Tried to follow https://stackoverflow.com/a/2966758

    [AttributeUsage(AttributeTargets.Method)]
    private class GUIEffect : Attribute
    {
        public void PerformCall(Action action)
        {
            action.Invoke();
            //action.somehowGetThis.UpdateGUI()
        }
    }

    [AttributeUsage(AttributeTargets.Method)]
    private class GUIEffect : Attribute
    {
        public void PerformCall(Action action)
        {
            action.Invoke();
            //action.somehowGetThis.UpdateGUI()
        }
    }

Can't figure out how to access this using method decorator

Stack Overflow

Can I use the decorator pattern to wrap a method body?

I have a bunch of methods with varying signatures. These methods interact with a fragile data connection, so we often use a helper class to perform retries/reconnects, etc. Like so: MyHelper.

vi•415d ago

side note I was like "how hard can it be I will just factor out the simple side effect code into a decorator and have it up front" oh how naive @nekodjin ^u^

sampersand•415d ago

lol

vi•415d ago

oh damn you too huh yeah I actually thought decorators of all thing would be a small cost abstraction to make my code a little prettier I'm f*cking stupid sigh but alas, I still want my pretty [Effect] boxes so, here we are

nekodjin•415d ago

C# doesn't have decorators

Angius•415d ago

Attributes are just metadata For some other code to find and do something with They're not like C++ macros or anything of the sort

vi•415d ago

but you can sandwich the original method in extra stuffs right?

nekodjin•415d ago

no.. if so it'd be a decorator :p

Angius•415d ago

Only in a very roundabout way. The attribute can contain a Pre and Post methods, then you can get a method with this attribute using reflections, and execute Pre() then ActualMethod() then Post() Attributes themselves don't ever execute Something has to execute them

nekodjin•415d ago

~~you could do some build-time codegen shit~~

Angius•415d ago

You can, of course, do something like

int Wrapper(Func<int, int, int> func)
{
    // pre code
    var res = func(2, 2);
    // post code
    return res;
}

int Wrapper(Func<int, int, int> func)
{
    // pre code
    var res = func(2, 2);
    // post code
    return res;
}

and call it with

Wrapper((a, b) => a + b)

Wrapper((a, b) => a + b)

vi•415d ago

mmm I did an inefficient but it makes my brain happy

nekodjin•415d ago

the long and short of it, vi, is give up the decorator idea if i know anything about what you might be trying to use it for lol

vi•415d ago

    [AttributeUsage(AttributeTargets.Method)]
    private class GUIEffect : Attribute
    {
        public void PerformCall(Action action)
        {
            action.Invoke();
            GameLogicSingletonScript.Instance.UpdateGUI();
        }
    }
    public static GameLogicSingletonScript Instance
    {
        get
        {
            return GameObject
                .FindGameObjectWithTag("GameController")
                .gameObject
                .GetComponent<GameLogicSingletonScript>();
        }
    }

    [AttributeUsage(AttributeTargets.Method)]
    private class GUIEffect : Attribute
    {
        public void PerformCall(Action action)
        {
            action.Invoke();
            GameLogicSingletonScript.Instance.UpdateGUI();
        }
    }
    public static GameLogicSingletonScript Instance
    {
        get
        {
            return GameObject
                .FindGameObjectWithTag("GameController")
                .gameObject
                .GetComponent<GameLogicSingletonScript>();
        }
    }