Предисловие. Чтобы как-то описать проблему я написал консольное приложение (см. код ниже).
Суть задачи. Пусть есть два протокола передачи сообщений: St
и Gv
. Есть два абстрактных типа:
- StMessage
- GvMessage
Нужно реализовать конвертацию сообщений типа StMessage
в GvMessage
. Реализации StMessage
содержат уникальный набор полей для своего типа сообщений. После конвертации необходимо вывести сообщение в консоли.
Вот, что у меня получилось. Кода достаточно много, для удобства разбил на регионы, поэтому лучше посмотреть его в редакторе кода. Код:
using System;
namespace TestHelloWorld.ConsoleApp
{
internal static class Program
{
private static void Main(string[] args)
{
LaunchTestOfStatusMessageConverting();
LaunchTestOfSpeedMessageConverting();
}
private static void LaunchTestOfStatusMessageConverting()
{
var stMessage = new StMessageStatus
{
Status = "Pending"
};
var converter = GetConverterForMessage(stMessage);
var gvMessage = converter.Convert(stMessage);
var printer = new ConsoleMessagePrinter();
printer.PrintMessage(gvMessage);
}
private static void LaunchTestOfSpeedMessageConverting()
{
var stMessage = new StMessageSpeed
{
Speed = 42
};
var converter = GetConverterForMessage(stMessage);
var gvMessage = converter.Convert(stMessage);
var printer = new ConsoleMessagePrinter();
printer.PrintMessage(gvMessage);
}
private static IMessageConverter GetConverterForMessage(StMessage stMessage)
{
switch (stMessage.MessageType)
{
case StMessageType.Speed:
return new SpeedMessageConverter();
case StMessageType.Status:
return new StatusMessageConverter();
default:
throw new ArgumentOutOfRangeException();
}
}
}
#region [ GvMessages ]
public enum GvMessageType
{
Speed,
Status
}
public abstract class GvMessage
{
public abstract GvMessageType MessageType { get; }
}
public class GvMessageSpeed : GvMessage
{
public override GvMessageType MessageType => GvMessageType.Speed;
public int Speed { get; set; }
}
public class GvMessageStatus : GvMessage
{
public override GvMessageType MessageType => GvMessageType.Status;
public string Status { get; set; }
}
#endregion
#region [ StMessages ]
public enum StMessageType
{
Speed,
Status
}
public abstract class StMessage
{
public abstract StMessageType MessageType { get; }
}
public class StMessageSpeed : StMessage
{
public override StMessageType MessageType => StMessageType.Speed;
public int Speed { get; set; }
}
public class StMessageStatus : StMessage
{
public override StMessageType MessageType => StMessageType.Status;
public string Status { get; set; }
}
#endregion
#region [ Converters ]
public interface IMessageConverter
{
GvMessage Convert(StMessage stMessage);
}
public class StatusMessageConverter : IMessageConverter
{
public GvMessage Convert(StMessage stMessage)
{
var stMessageStatus = stMessage as StMessageStatus;
if (stMessageStatus == null)
{
throw new ArgumentException($"Message is not of {nameof(StMessageStatus)} type", nameof(stMessage));
}
var gvMessageStatus = new GvMessageStatus
{
Status = stMessageStatus.Status
};
return gvMessageStatus;
}
}
public class SpeedMessageConverter : IMessageConverter
{
public GvMessage Convert(StMessage stMessage)
{
var stMessageSpeed = stMessage as StMessageSpeed;
if (stMessageSpeed == null)
{
throw new ArgumentException($"Message is not of {nameof(StMessageStatus)} type", nameof(stMessage));
}
var gvMessageSpeed = new GvMessageSpeed
{
Speed = stMessageSpeed.Speed
};
return gvMessageSpeed;
}
}
#endregion
#region [ Message printers ]
public interface IMessagePrinter
{
void PrintMessage(GvMessage message);
}
public class ConsoleMessagePrinter : IMessagePrinter
{
public void PrintMessage(GvMessage message)
{
string messageAsText;
switch (message.MessageType)
{
case GvMessageType.Speed:
var speedMessage = (GvMessageSpeed) message;
messageAsText = $"Gv speed message. Speed={speedMessage.Speed}";
break;
case GvMessageType.Status:
var statusMessage = (GvMessageStatus)message;
messageAsText = $"Gv status message. Status={statusMessage.Status}";
break;
default:
throw new ArgumentOutOfRangeException();
}
Console.WriteLine(messageAsText);
}
}
#endregion
}
Не могу понять, насколько правильно я решал эту задачу с помощью средств языка C#. Мне кажется, здесь что-то не так. Как минимум, мне не нравятся обратные преобразования объекта из базового типа в конкретный.
Прошу проинспектировать мой подход к решению данной задачи и, если что-то не так, предложить свой вариант решения.
GvMessageType
иStMessageType
, - у которых одинаковые значения? Нельзя ли их заменить одним общим? | Зачем вообще эти перечисления, если есть строгие типы? СвойстваMessageType
дублируют тип самих классов.