Разрабатываю библиотеку по работе со схемой, где доменная логика следующая:
- есть схема;
- схема может содержать либо таблицу или картинку, но не оба контента одновременно.
Если разработчик, который будет использовать мою либу попытаеться обратиться к таблице (Scheme.Table.Rows
), а у схемы ее нет, то вместо NullReferenceException
, я хочу, чтобы выбрасывалась более внятная ошибка. И для было решено реализовать паттерн Null Object
.
Интерфейс
Сейчас это сделано через интерфейс и вот как это выглядит в проекте:
public class Scheme
{
public ITable Table { get; }
public IPicture Picture { get; }
/// <summary>
/// Creates a new <see cref="Scheme"/> instance with table content.
/// </summary>
public Scheme(ITable table)
{
Table = table;
Picture = new NoPicture();
}
/// <summary>
/// Creates a new <see cref="Scheme"/> instance with picture content.
/// </summary>
public Scheme(IPicture picture)
{
Table = new NoTable();
Picture = picture;
}
}
public interface ITable
{
IEnumerable<Row> Rows { get; }
}
public class Table : ITable
{
public IEnumerable<Row> Rows { get; }
}
public class NoTable : ITable
{
public IEnumerable<Row> Rows => throw new Exception("Scheme does not contain a table.");
}
public interface IPicture
{
Image Image { get; }
}
public class Picture : IPicture
{
public Image Image { get; }
}
public class NoPicture : IPicture
{
public Image Image => throw new Exception("Scheme does not contain a picture.");
}
Абстрактный класс
А вот как бы это выглядело на абстрактных классах:
public class Scheme
{
public AbstractTable Table { get; }
public AbstractPicture Picture { get; }
/// <summary>
/// Creates a new <see cref="Scheme"/> instance with table content.
/// </summary>
public Scheme(AbstractTable table)
{
Table = table;
Picture = new NoPicture();
}
/// <summary>
/// Creates a new <see cref="Scheme"/> instance with picture content.
/// </summary>
public Scheme(AbstractPicture picture)
{
Table = new NoTable();
Picture = picture;
}
}
public abstract class AbstractTable
{
public abstract IEnumerable<Row> Rows { get; }
}
public class Table : AbstractTable
{
public override IEnumerable<Row> Rows { get; }
}
public class NoTable : AbstractTable
{
public override IEnumerable<Row> Rows => throw new Exception("Scheme does not contain a table.");
}
public abstract class AbstractPicture
{
public abstract Image Image { get; }
}
public class Picture : AbstractPicture
{
public override Image Image { get; }
}
public class NoPicture : AbstractPicture
{
public override Image Image => throw new Exception("Scheme does not contain a picture.");
}
ВОПРОС
Это очередной случай, когда одно и тоже можно реализовать с помощью интерфейса или абcтрактного класса. Но с точки зрения логики и дизайна, что правильнее?
Я обычно использую абстрактный класс, чтобы вынести общий код в отдельное место и самое главное, чтобы отношение между сущностями было типа IS, но что-то затрудняюсь сказать, что NoTabel
is AbstractTable
.
public IAbstractTable Table => table ?? throw new Exception("Scheme does not contain a table.")
. Как этим пользоваться? Везде писатьif (scheme.Table is NoTable) { ... }
?Scheme.HasTable => _table is Table
.