На файловой системе хранятся отчеты школ. Есть отчеты персональные – персональные для каждой школы (класс School
). И есть отчеты общие – отчеты, которые содержат общую информацию/статистику по школам региона.
Метаданные (информация об отчетах) хранится в классе ReportMeta
и они одинаковы, как для персональных, так и для общих отчетов:
public class ReportMeta
{
public string Name { get; set; }
public DateTime PublishDate { get; set; }
public string OnlineLink { get; set; }
//...
}
Алгоритмы получения ReportMeta
для персональных и общих отчетов разные. Причем код получения ReportMeta
для персональных постоянен, а вот код для общих отчетов в будущем может претерпеть изменения, т.к. структура хранение этих (общих отчетов) будет иная и еще планируется добавить другие типы отчетов. Поэтому я сделал разъединения:
public interface IReportMeta
{
IEnumerable<ReportMeta> GetReportMetas();
}
public class PrivateReportMeta : IReportMeta
{
private School school;
public PrivateReportMeta(School school)
{
this.school = school;
}
public IEnumerable<ReportMeta> GetReportMetas()
{
//…
return new List<ReportMeta>();
}
}
public class PublicReportMeta : IReportMeta
{
public IEnumerable<ReportMeta> GetReportMetas()
{
//...
return new List<ReportMeta>();
}
}
Клиентскому коду необходимо получить списки ReportMeta
в общей куче и поэтому предполагается следующее:
static void Main(string[] args)
{
School currentSchool = new School();
IReportMeta privateReportMetas = new PrivateReportMeta(currentSchool);
IReportMeta publicReportMetas = new PublicReportMeta();
var allReportMetas = privateReportMetas.GetReportMetas().Union(publicReportMetas.GetReportMetas());
//...
}
Вопрос
- Соблюдены ли в коде принципы SOLID?
- Есть ли грубое нарушение в проектировании приложения?
Буду благодарен за предложение другой архитектуры.