crit2.cs

public bool HasAdvantage()
{
    // Oh god, okay lets go fetch all our modifiers and check them...
    foreach( Modifier mod in _modifiers )
    {
        if( mod.GivesAdvantage() )
        {
            return true;
        }
    }

    // Oh maybe our current action will have something to say?
    if( _actionControl.currentAction.GivesAdvantage() )
    {
        return true;
    }

    // Don’t forget the perks!
    foreach( Perk perk in _perks )
    {
        if( perk.GivesAdvantage() )
        {
            return true;
        }
    }

    // …

    // On and on it goes…

    // …

    // ...aaaand our function is 100+ lines long.

    // See you next time the design calls for another addition to this function!

    return false;
}