Jak zastąpić rozbudowany switch w aplikacji

Pingback: dotnetomaniak.pl

2018-12-12T11:49:59+01:00

Swietny artykul. Dzieki bardzo Daniel! 🙂

Odpowiedz

2018-12-12T19:24:47+01:00

Super, że się podoba 🙂

Odpowiedz

2018-12-13T10:11:41+01:00

Można też trzymać stałą liste z formaterami

Odpowiedz

2018-12-13T19:05:20+01:00

Można, tylko, że takie rozwiązanie na dłuższą metę jest mniej elastyczne. Szczególnie, gdy lista obiektów zależy od jakieś logiki (np. uprawnień użytkownika lub za co zapłacić klient).

Odpowiedz

2018-12-16T22:29:05+01:00

Nie jedna osoba straciła zdrowie przy mega-switchu:) Dwa spostrzeżenia do arykułu:

1) zamiast listy, która ma liniowy czas przeszukiwania O(n) (użycie FirstOrDefault) warto rozważyć użycie słownika Dictionary, który ma (zazwyczaj) stały czas przeszukiwania O(1), Szczególnie, jeśli switch jest bardzo rozbudowany i mamy duże n 🙂

2) switch, jest brzydki, ale generuje tzw. tablicę skoków („jump table” albo „branch table”), która może być znacznie wydajniejsza niż wyszukiwanie na liście a potem wykonywanie metod interfejsu (dispatch metody wirtualnej, itp.). Przy kodzie, który jest zorientowany na wydajność, zachęcałbym do napisania benchmarku w BenchmarkDotNet. Oczywiście, dla tradycyjnego kodu biznesowego, te zyski będą nieznaczące

Odpowiedz

2018-12-17T05:22:31+01:00

Dzięki za spostrzeżęnia!

Odpowiedz

	public class XmlFormatter : IXmlFormatter
	{
	public void Export(Report report)
	{
	Console.WriteLine($"XmlFormatter.Export: {report.Name}");
	}
	}

	public interface IXmlFormatter
	{

	}

	public class CsvFormatter : ICsvFormatter
	{
	public void Export(Report report)
	{
	Console.WriteLine($"CsvFormatter.Export: {report.Name}");
	}
	}

	public interface ICsvFormatter
	{
	}

	public class ReportProcessorWithSwitch : IReportProcessor
	{
	private IXmlFormatter _xmlFormatter;

	private ICsvFormatter _csvFormatter;

	public ReportProcessorWithSwitch(IXmlFormatter xmlFormatter,
	ICsvFormatter csvFormatter)
	{
	_xmlFormatter = xmlFormatter;
	_csvFormatter = csvFormatter;
	}

	public void Process(Report report)
	{
	switch (report.ExportFormat)
	{
	case ExportFormat.Xml:
	_xmlFormatter.Export(report);
	break;
	case ExportFormat.Csv:
	_csvFormatter.Export(report);
	break;
	default:
	throw new ArgumentOutOfRangeException();
	}
	}
	}

	public class XmlFormatter : IXmlFormatter
	{
	public ExportFormat Format { get; } = ExportFormat.Xml;
	public void Export(Report report)
	{
	Console.WriteLine($"XmlFormatter.Export: {report.Name}");
	}
	}

	public interface IXmlFormatter : IFormatter
	{

	}

	public class CsvFormatter : ICsvFormatter
	{
	public ExportFormat Format { get; } = ExportFormat.Csv;

	public void Export(Report report)
	{
	Console.WriteLine($"CsvFormatter.Export: {report.Name}");
	}
	}

	public interface ICsvFormatter : IFormatter
	{
	}

PROGRAMUJE .NET – Daniel Plawgo

Jak zastąpić rozbudowany switch w aplikacji

Wprowadzenie

Problem

Darmowy kurs Visual Studio

Quiz C#

A co z dodaniem nowego formatera?

Rozwiązanie

Co dalej?

Przykład

Podsumowanie

.NET 5 Web App w Azure

Przeczytaj również:

7 thoughts on “Jak zastąpić rozbudowany switch w aplikacji”

Dodaj komentarz Anuluj pisanie odpowiedzi

	public class Report
	{
	public string Name { get; set; }

	public ExportFormat ExportFormat { get; set; }
	}

	public interface IFormatter
	{
	ExportFormat Format { get; }

	void Export(Report report);
	}

	public class ReportProcessorWithoutSwitch : IReportProcessor
	{
	private IEnumerable<IFormatter> _formatters;

	public ReportProcessorWithoutSwitch(IEnumerable<IFormatter> formatters)
	{
	_formatters = formatters;
	}

	public void Process(Report report)
	{
	var formatter = _formatters.FirstOrDefault(f => f.Format == report.ExportFormat);

	if (formatter == null)
	{
	throw new Exception("Not supported report formatter.");
	}

	formatter.Export(report);
	}
	}

	public enum ExportFormat
	{
	Xml,
	Csv
	}