Design and Architecture

Troy.compdr wrote:

how far an application should go to allow users to address possible errors?

Troy.compdr wrote:

at what point does the error checking and correction begin to affect
application build costs and performance.

interface BoosterInterface {
    Booster getBooster();
    Boostertype getBoosterType();
    Boostertype getBoosterPower();
    void initBooster(Boostertype bt, BoosterPower bp);
    void setBooster(Booster bl);
    void setBoosterType(Boostertype bt);
    void setBoosterPower(BoosterPower bp);
}

class Rocket
        impliments BoosterInterface {
    // ya da ya
}

Nagy Vilmos wrote:

If the rocket must have a booster, then I agree it must be a part of the
constructor to ensure its presence.

public class Rocket
{
  public Rocket()
  {
  }

  public void PreLaunchTests()
  {
    OnPreLaunchTests();
  }

  public void Launch()
  {
  }

  protected virtual void OnPreLaunchTests()
  {
  }

  protected virtual void OnLaunch()
  {
  }
}

public class SingleBoosterRocket : Rocket
{
  public List<IBooster> Boosters { get; private set; }

  public SingleBoosterRocket()
  {
    Boosters = new List<IBooster>();
  }

  public void AddBooster(IBooster booster)
  {
    Boosters.Add(booster);
  }

  protected override void OnPreLaunchTests()
  {
    // Do some pre launch tests.
  }

  protected override void OnLaunch()
  {
    if (Boosters.Count == 0)
      throw new NotEnoughBoostersException();
    // Do some other work.
  }
}

public class DualBoosterRocket : SingleBoosterRocket
{

  public DualBoosterRocket() : base()
  {
  }

  protected override void OnPreLaunchTests()
  {
    // Do some pre launch tests.
  }

  protected override void OnLaunch()
  {
    if (Boosters.Count < 2)
      throw new NotEnoughBoostersException();
    // Do some other work.
  }
}

Pete O'Hanlon wrote:

Funnily enough, that was the gist of the answer that I've just added above.

Pete O'Hanlon wrote:

We internally have a DI engine that is only 60 lines or so of code

VuNic wrote:

The Boosters.GetBooster() can be applied for both the scenarios?