ООП C # - шаблон проектирования для симулятора механики MMO

Один из вариантов - передать crit конструктору Abillity.

Другой вариант, если к плееру всегда подключен Ability. Сделайте свойство крита публичным с частным набором:

public class Player 
{
    private double crit { get; private set;}
    ...
}

Затем передайте Player конструктору Ability и удерживайте его там. Однако обратите внимание, что это увеличит связь.

Это можно было сделать так:

public class Ability
{
    protected Player _player;
    public Ability(Player player)
    {
        _player = player;
    }
}

вы также захотите изменить класс Headshot, производный от Ability

public class Headshot : Ability
{
    public Headshot(Player player) : base(player)
    {
        ...
    }
}

Вместо того, чтобы выполнять ротацию способностей напрямую, сохраните способности в списке. Таким образом, у вас может быть метод настройки в Player, который вводит игрока во все способности по очереди. Добавление условий и какой-то «используемой способности», которая отменяет следующую способность, на самом деле является еще одной причиной для того, чтобы выразить ротацию в виде какого-то списка. Таким образом, вам не нужно дублировать проверку «использованных способностей» повсюду, а иметь ее в одном месте. Что-то вроде этого:

public class Player 
{
    private string name {get;set;}
    private string job {get;set;}
    private string STR;
    private string DEX;
    private string VIT; 
    //etc..

    public struct RotationAbility
    {
        public RotationAbility(Func<bool> cond, Ability ability)
        {
            this.cond = cond;
            this.ability = ability;
        }

        public Func<bool> cond;
        public Ability ability;
    }

    private List<RotationAbility> rotation = new List<RotationAbility>();

    public void execute()
    {
        foreach (var ab in rotation)
        {
            if (ab.cond())
                ab.ability.execute();
        }
    }

    public void setUpRotation()
    {
        setUpRotation(rotation);
        foreach (var ab in rotation)
        {
            ab.ability.Player = this;
        }
    }

    protected virtual void setUpRotation(List<RotationAbility> rotation) { }
}

//I want to make the program a bit modular for jobs (roles/classes)
//So.. 

public class Bard : Player
{
    protected override void setUpRotation(List<RotationAbility> rotation)
    {
        rotation.Add(new RotationAbility(()=>buff>0, new Heavyshot());
        //etc.
    }

    public class Heavyshot : Ability
    {
        public Heavyshot()
        {
            name = "Heavy Shot";
            potency = 150;
            dotPotency = 0;
            recastTime = 2.5;
            TPcost = 60;
            animationDelay = 0.8;
            abilityType = "Weaponskill";
            castTime = 0.0;
            duration = 0.0;
        }

        public override void impact()
        {
            //add heavier shot buff activation here
            base.impact();
        }
    }
}

    public class Ability{
        public Player Player { get; set; }

        public int cooldown;
        public int cost;
        public virtual void impact() 
        {
            //Deal some damage.
            // !! - the key problem is here, i want to initiate a roll to compare to the players CRIT rating versus the roll to determine the bonus damage. But I can't access the initiated players crit from here. The rating may change depending on abilities used so I can't create a new object. I know i need an object reference but I can't figure it out...
            log(time.ToString("F2") + " - " +  name + 
                " Deals " + potency + 
                " Potency Damage. Next ability at: " + nextability);
        }
    }
}