AutoProperty over private var and public get
The following event could be made:
public class PlayerDamagedEvent : StaticContextEvent
{
public int Damage { get; }
public bool OverrideParry { get; }
public Monster Damager { get; }
public PlayerDamagedEvent(int damage, bool overrideParry, Monster damager)
{
Damage = damage;
OverrideParry = overrideParry;
Damager = damager;
}
}
However, Demmonic has created a class that uses the following syntax:
public class PlayerDamagedEvent : StaticContextEvent
{
private int damage;
private bool overrideParry;
private Monster damager;
public PlayerDamagedEvent(int damage, bool overrideParry, Monster damager)
{
this.damage = damage;
this.overrideParry = overrideParry;
this.damager = damager;
}
public int Damage { get { return damage; } }
public bool OverrideParry { get { return overrideParry; } }
public Monster Damager { get { return damager; } }
}
Using the first method (Auto-Properties) uses less variables and lines of code (half), and doesn't bother with returning the backing variable, but I've been told to keep everything consistent, so am therefore moving that private-read-set/public-read-only fields be made Auto-Properties.