Visual C# Delegate Array

Question

Okay. So what I want to do... I'm trying to store a list of status effects as delegates in an array. The statuses will act like the statuses in the pokemon games.. (Stun makes you lose a turn etc).

I have this so far...

public class Statuses : Chara{
        public static void para(){
            this.health -= 10;
        }
    }

     status[] statuses = new status[]{
        new status(Statuses.para)
    };

It's complaining about this not being a static property, I was wondering how I should proceed.

Thanks heaps.

It's not clear what you're trying to do, where you're trying to declare statuses, what the status type is, or what the Chara type is. I would also strongly advise that you start following .NET naming conventions. — Jon Skeet
– Jon Skeet, Commented Aug 2, 2011 at 6:15

Simon Fischer · Accepted Answer · 2011-08-02 07:01:24Z

1

The compiler error you most likely are getting when compiling the Statuses class says it all: “Keyword 'this' is not valid in a static property, static method, or static field initializer”: You are not allowed to reference “this” in a static method. If your health variable is static you can do like this:

private static int health;
public static void para() 
{
    health -= 10; 
}

If health is not static you will get this compiler error “An object reference is required for the non-static field, method, or property 'Statuses.health'.

Another error is that your para is not a property but a method. Since the code you have posted are very out of context a number of different errors could be present at well.

answered Aug 2, 2011 at 7:01

Simon Fischer

3,9063 gold badges26 silver badges32 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

George Duckett · Accepted Answer · 2011-08-02 07:10:00Z

0

The problem the compiler is complaining about is that you've marked the method Para as static. You're then trying to access the health property of the current instance using this, which doesn't make sense, given that the containing method is static.

You should read up on the static keyword and its usage.

I think what you wanted to do was to create a delegate that reduces the instance's health, along the lines of (assuming you have a type called pokemon, with a property health):

public class Statuses : Chara{
    public static Action<Pokemon> para =
        (pokemonInstance) => { pokemonInstance.Health -= 10; };
}

Action<Pokemon>[] statuses = new Action<Pokemon>[]{
    Statuses.para
};

Read up on Action<T> and Anonymous Methods.

edited Aug 2, 2011 at 7:10

answered Aug 2, 2011 at 6:58

George Duckett

32.6k12 gold badges101 silver badges167 bronze badges

2 Comments

Corey Over a year ago

I know - but delegates can only be used with static methods can't they?

George Duckett Over a year ago

You can also assign a delegate, or in my case Action<T> in-place. In my above code i'm creating a method that has a single parameter, the pokemonInstance. The method will reduce the health by 10. I'm assigning that method to the para static variable.

Collectives™ on Stack Overflow

Visual C# Delegate Array

2 Answers 2

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related