Имам система за верижна атака за игра, но тя става много сложна много бързо

Вместо това бих измислил някакъв рекурсивен модел:

enum Outcomes { Crit, DoubleCrit, FireDMG, Burn, NoEffect }

abstract class Attack 
{ 
    public Attack() { Child = null; }

    List<Outcomes> GetOutcomes(); 
    protected virtual Attack Child; 
}
class Melee : Attack 
{ 
    public Melee() : base() { }
    public Melee(Attack child) : base() { Child = child; }

    List<Outcomes> GetOutcomes()
    {
        List<Outcomes> ret = new List<Outcomes>();
        if(Child != null) ret.Add(Child.GetOutcomes());

        if(ret.Contains(Outcomes.Crit))
            ret.Add(Outcomes.DoubleCrit);
        else
            ret.Add(Outcomes.Crit);

        return ret;
    }
}
class Fire : Attack 
{ 
    public Fire() : base() { }
    public Fire(Attack child) : base() { Child = child; }

    List<Outcomes> GetOutcomes()
    {
        List<Outcomes> ret = new List<Outcomes>();
        if(Child != null) ret.Add(Child.GetOutcomes());

        List<Outcomes> PossibleOutcomes = new List<Outcomes>();        

        PossibleOutcomes.Add(Outcomes.FireDMG);
        PossibleOutcomes.Add(Outcomes.Burn);

        if(ret.Contains(Outcomes.Burn)) PossibleOutcomes.Add(Outcomes.Fireball)
        if(ret.Contains(Outcomes.FireDMG)) PossibleOutcomes.Add(Outcomes.NoEffect);

        // Use some randomization method to select an item from PossibleOutcomes
        int rnd = 2; // Totally random number.
        ret.Add(PossibleOutcomes[rnd]);

        return ret;
    }
}

Тогава за верижни атаки просто използвайте:

Attack ChosenAttack = new Melee(new Fire(new Melee()));

Или ако приемем, че потребителят избира всеки клон, вие просто трябва да запазите последния и непрекъснато да го добавяте като дете към следващата атака, която избере.

Attack ChosenAttack = new Melee();

// Some events occur...

ChosenAttack = new Fire(ChosenAttack);

// Some more...

ChosenAttack = new Melee(ChosenAttack);

Извинявам се, ако не разбирам напълно проблема и предложението ми няма да работи.

Звучи сякаш наистина искате да имате възможността да персонализирате какви ефекти ще имат различните избори за играча, като същевременно избягвате да пишете целия код.

Един от начините да направите това е да създадете структура от данни, която ще обработва резултата от всяка комбинация и ще направи задачата за създаване на нови комбинации по-лесна за справяне с потенциално големия брой възможности в нещо по-лесно за промяна от кода трябва да прекомпилирате като него в XML и да заявите резултат въз основа на избора на потребителите.

Може да имате клас нещо като следното, за да съхраните резултата.

public class AttackResult
{
    public int ChanceToCrit { get;set; }
    public int ChanceToBurn { get;set; }
    public int FireDmg { get;set; }
}

Що се отнася до XML, вероятно ще имате нещо подобно,

<Options>
    <Attack>
        <AttackResult>
            <ChanceToCrit>5</ChanceToCrit>
        </AttackResult>
        <Attack>
            <AttackResult>
                <ChanceTCrit>10</ChanceToCrit>
            </AttackResult>
        </Attack>
    </Attack>
    <Fire>
    </Fire>
</Options>

Схванахте идеята. Когато обработвате избора на вашите потребители (само предполагам тук), можете да съхранявате какво правят в Enum например. Така че, за да използвате XML с това, можете да направите нещо подобно на

public enum ActionType
{
    Attack,
    Fire,
    None
}

allActions ще трябва да са в съответствие, за да може този пример да работи с вашата идея за игра според мен.

public ActionResult ProcessActions(List<ActionType> allActions)
{
    string xpathquery = "Options";

    foreach(var playerAction in allActions)
    {
        if(playerAction == ActionType.None)
           break;
        xpathquery += "/" + playerAction.ToString();
    }

    //Query xmldocument of all options
    var result = Singleton.ActionsDoc.SelectSingleNode(xpathquery);
    string attackRes = result.InnerXml;
    XmlSerializer serializer = new XmlSerializer(typeof(ActionResult))
    //Might need to prepend something to attackRes before deserializing, none of this is tested
    byte[] allBytes = Encoding.UTF8.GetBytes(attackRes);
    MemoryStream ms = new MemoryStream(allBytes);

    ActionResult result = (ActionResult)serializer.Deserialize(ms);

    return result;
}

Тази опция има своите недостатъци, като например това, че XmlSerializer е доста бавен, но това означава, че можете да създадете прост инструмент, който да ви помогне да генерирате xml, необходим за всички ваши опции, и да променяте стойностите съвсем просто за балансиране на играта.

Един вариант на масата. Надявам се това да помогне.