Можно ли получить тип интерфейса, переданного в дженерик-метод и правильно ли использовать его таким способом?

Question

[romaro]

В моем приложении есть сервис, который отвечает за печать этикеток. Всего несколько видов этикеток и для каждого свой набор параметров. Однако для некоторых этикеток параметры полностью совпадают. Я определил единственный универсальный метод для интерфейса сервиса:

public interface IPrintService
    {
        public void PrintLabel<T>(T labelParams) where T: IPrintService_AbstractLabelParams;
    }

И, соответственно, описал интерфейсы для объектов параметров каждого вида этикеток:

public interface IPrintService_AbstractLabelParams
    {
        string PrinterName { get; init; }
        int Copy { get; init; }
    }

    public interface IPrintService_OLabelParams : IPrintService_AbstractLabelParams
    {
        string ProductName { get; init; }
    }

    public interface IPrintService_WLabelParams : IPrintService_AbstractLabelParams
    {
        string ProductName { get; init; }
    }

    public interface IPrintService_IncomeProductLabelParams : IPrintService_AbstractLabelParams
    {
        public int ProductId { get; init; }
        public int Sku { get; init; }
    }

Теперь я реализую метод на уровне класса самого сервиса:

public class PrintService : IPrintService
    {
        public void PrintLabel<T>(T labelParams) where T : IPrintService_AbstractLabelParams
        {
            ...
        }
    }

Это метод-роутер, который, в зависимости от типа этикетки, должен вызывать соответствующий внутренний метод библиотеки принтера.

Но как мне определить тип этикетки?

Можно, конечно, добавить в IPrintService_AbstractLabelParams обязательное свойство LabelType и по его значению делать switch. Но наверняка существует способ получить значение типа интерфейса, который был передан в обобщенный метод. Оператор is тут не подойдет, т.к. для некоторых этикеток могут полностью совпадать параметры.

То есть насколько правильно (и возможно ли) использовать интерфейс в качестве сепаратора типа этикетки?

Comments

[сергей кузьмин]

технически можно узнать имя типа переменной

var check = labelParams.GetType().Name;
			Console.Error.WriteLine("type: " + check );

[Василий Банников]

сергей кузьмин, только так ты узнаешь конкретный тип, а не тип интерфейса.

[сергей кузьмин]

это верно но для логики сойдет и быстрее чем каст к соответствию интерфейсу верно ?

Answer 1

[Василий Банников]

Ну ты можешь сделать так:

switch(parameter) {
  case IPrintService_IncomeProductLabelParams incomeProductLabel: {
    // ...
    break;
  }
  case IPrintService_WLabelParams wLabel: {
    // ...
    break;
  }
  // итд
  default:
    // ...
  break;
}

Но мне не понятен смысл женерика и интерфейсов с пропертями.
Ты собираешься делать параметры ещё и структурами?
В принципе тут можно избавиться от интерфейсов вообще и оставить только конкретные классы, а в IPrintService сделать несколько перегрузок, например.
Или сделать по интерфейсу для каждого типа этикеток.

Вообще, я у тебя тут вижу три разные вещи у этикеток:
1. Какой принтер используется (PrinterName)
2. Параметры задания на печать (Copy)
3. Параметры самой этикетки. Её тип и, например, SKU товара.
Попробуй подумать с этой стороны.
Например ты можешь сделать такой IPrinter, который будет принимать в себя PrintTask, у которого будет написано количество нужных копий и Payload который будет уже самой этикеткой.
Конкретная реализация IPrinter может быть привязана к одному физическому принтеру.

А Payload ты можешь формировать отдельно от сервиса печати. Например у тебя может быть какая-нибудь "служба шаблонов этикеток", в которую ты передаёшь объект с параметрами этикетки, а она отдаёт тебе уже сформированный объект этикетки, который пригоден для печати (Что там вообще принтер принимает? Картинку?)

Comments

[romaro]

Спасибо за подробный ответ! Я действительно решил обойтись классами для типовых заданий на печать этикеток и сделать перегрузки для PrintLabel.

Answer 2

[freeExec]

Тут то всё просто, кастуешь к интерфейсу и проверяешь получилось что или пусто.
Но в данном примере вообще не вижу какой либо выгоды от интерфейсов, если всё равно надо делать развилки руками.

Comments

[romaro]

freeExec , то есть примерно такой код? https://dotnetfiddle.net/PdkvDu

using System;
					
public class Program
{
	public static void Main()
	{
		var labelOne = new LabelOne()
		{
			ProductName = "Test 1"
		};
		
		var labelTwo = new LabelTwo()
		{
			ProductName = "Test 2"
		};
		
		var printer = new PrintService();
		printer.PrintLabel<ILabelTwo>(labelTwo);
	}
}

public class PrintService {
	public void PrintLabel<T>(T obj) {
		if (obj as ILabelOne is not null) {
			PrintLabelOne(obj as ILabelOne);
		} else if (obj as ILabelTwo is not null) {
			PrintLabelTwo(obj as ILabelTwo);
		} else {
			Console.WriteLine("Unknown");
		}
	}
	
	private void PrintLabelOne(ILabelOne obj) {
		Console.WriteLine($"Let's go print Label One with product name {obj.ProductName}");
	}
	
	private void PrintLabelTwo(ILabelTwo obj) {
		Console.WriteLine($"Let's go print Label Two with product name {obj.ProductName}");
	}
}

public interface IBaseLabel {
	public string PrinterName { get; init; }
}

public interface ILabelOne : IBaseLabel {
	public string ProductName { get; init; }
}

public interface ILabelTwo : IBaseLabel {
	public string ProductName { get; init; }
}

struct LabelOne : ILabelOne
{
	public string PrinterName { get; init; }
	public string ProductName { get; init; }
}

struct LabelTwo : ILabelTwo
{
	public string ProductName { get; init; }
	public string PrinterName { get; init; }
}

Это наверняка можно записать короче, но у меня пока мало опыта в C#. То есть когда мне необходимо дать задание на печать этикетки, я создаю экземпляр структуры, которая наследует два интерфейса: абстрактный интерфейс этикетки, а так же интерфейс, в котором описана конкретная этикетка.

Затем я вызываю обобщенный метод (собственно, интерфейс позволяет мне ограничиться одним джереник-методом, а не создавать перегруженные для каждого вида этикетки). В этом методе я использую оператор as, т.к. он не выбрасывает исключений.

В этом варианте, наверное, можно было бы отказаться от интерфейсов, которые реализуют структуры и передавать в обобщенный метод объекты этикеток, унаследованные от абстрактного класса этикетки. Но оптимально ли это по производительности?

[freeExec]

romaro, А что у вас принтер печатает быстрее чтения из памяти. Ваш код плохо читаем, со всеми этими интерфейсами, которые по факту и не использовались. Напишите с начала красивую архитектуры, когда тормоза вам будут мешать, начнёте думать о производительности.

Answer 3

[crokocat]

typeof(MyType).GetInterfaces().Contains(typeof(IMyInterface))