Lokalizacja Enum

2018-07-12T15:23:07+02:00

wow bardzo fajny artykuł, dużo nowych rzeczy się dowiedziałem.

2018-07-12T15:44:38+02:00

Bardzo mnie to cieszy, zapraszam częściej

2018-07-14T14:53:25+02:00

Niestety Twoje pomysły to próba wymyślania koła od nowa.
1.Rozbijanie plików resource tylko dla kilku wartości zabija to co w Resource-ach jest najistotniejsze, czyli wydajność i prostotę. Tutaj polecam zapoznać się z tym czym właściwie są resource i jak działają m.in. jak działa w nich cache.
2.Atrybut na wartościach enum-a są po to, żeby można je niezależnie konfigurować, ponieważ „słownik tłumaczeń” jest w zasadzie „słownikiem” i w ten sposób mamy pewność w kodzie, że dany klucz się nie powtarza, albo, że używamy danego klucza wielokrotnie świadomie.
2.ToDisplayString przez to, że korzysta z refleksji jest strasznie niewydajne i w przypadku błędów dostajemy błąd z klasy z której jej nie oczekujemy. Nie mówiąc już o tym, że są tutaj błędy np. Twój atrybut można nadać wielokrotnie, a i tak weźmiemy pierwszą wartość.

Odpowiedz

2018-07-15T06:52:11+02:00

Hej, dzięki za komentarz!

A co masz na myśli o wymyślaniu koła od nowa? Chodzi Ci, że jest już coś gotowego, co można użyć? Kilka lat temu, kiedy potrzebowałem stworzyć taki automatyczny mechanizm nic niestety gotowego nie było.

Oczywiście można też to zorganizować trochę inaczej. Sam kiedyś w niektórych aplikacjach używałem podobnego mechanizmu tylko, że miałem jeden plik resourców dla wszystkich enumów, z którego metoda ToDispalyString wyciągała wartość już bez użycia refekcji. Ale niestety takie podejście na dłuższą metę było problematyczne, w szczególności, gdy aplikacja była z budowana z modułów, które były dynamiczne ładowane z wykorzystaniem MEFa, gdzie klient mógł dodawać własne rozszerzenia do aplikacji. Więc nie dało się korzystać z jednego pliku resources dla enumów.

A mógłbyś rozwinąć kwestię z punkut 1? Przyznam się szczerze, że jak kiedyś interesowałem się tematem, to właśnie ludzie zalecali, aby korzystać z wielu plików resource (np. https://stackoverflow.com/questions/8792180/1-resource-file-vs-many lub https://stackoverflow.com/questions/19910926/multiple-resource-files-versus-single-resource-file). U nas w projektach duże pliki resourców średnio się sprawdzały. Masz rację, że jeden plik będzie wydajniejszy ponieważ tylko raz go się wczytuje z dysku. A nie jest tak, że mechanizm cache spowoduje, że kolejne odczytanie wartości będzie już bez tego narzutu? W wolnej chwili z ciekawości sprawdzę jaki faktycznie jest narzut.

Co do punktu 2 to myślę, że to zależy jak organizujesz sobie jako całoś korzystaniem z resourców. Jak pisałem w wpisie, sam kiedyś korzystałem z atrybutu Display na każdej wartości w enum, ale takie rozwiązaniem powodowało, że w definicji enuma było sporo zduplikowanego kodu, który w przypadku, gdy mam jeden plik resources per enum był nie do końca potrzebny. np.:

enum OrderStatus
{
[Display(ResourceType = typeof(OrderStatusResources), Name = „Created”)]
Created,
[Display(ResourceType = typeof(OrderStatusResources), Name = „Processed”)]
Processed
}

Masz rację, atrybuty na poziomie wartości są bardziej elastyczne, gdy na przykład korzystasz z jednego pliku resources, gdzie wartości są współdzielone przez różne elementy aplikacji.

Natomiast co do metody ToDisplayString to masz rację, że w tym wydaniu nie jest efektywna. Dzięki Twojemu komentarzowy zauważyłem, że podczas edycji wpisu uciekł mi jeden akapit związany właśnie z tą kwestią. Pod koniec miesiąca planuje dodać wpis na temat BenchmarkDotNet, gdzie właśnie chciałem użyć tej metody jako przykładu do analizy wydajności.

Jeszcze raz wielkie dzięki zakomentarz!

Odpowiedz

	public enum OrderStatus
	{
	Created,
	Completed,
	Canceled
	}

	public class EnumDescriptionAttribute : Attribute
	{
	public Type ResourceType { get; set; }

	public EnumDescriptionAttribute(Type resourceType)
	{
	ResourceType = resourceType;
	}
	}

	[EnumDescription(typeof(OrderStatusResources))]
	public enum OrderStatus
	{
	Created,
	Completed,
	Canceled
	}

	public static class EnumExtensions
	{
	public static string ToDisplayString(this Enum value)
	{
	if (value == null)
	{
	throw new ArgumentNullException("value");
	}

	string description = value.ToString();
	EnumDescriptionAttribute[] attributes = (EnumDescriptionAttribute[])value.GetType().GetCustomAttributes(typeof(EnumDescriptionAttribute), false);

	if (attributes != null && attributes.Length > 0)
	{
	Type resourceType = attributes[0].ResourceType;

	if (resourceType != null)
	{
	var property = resourceType.GetProperty(description);
	if (property != null)
	{
	var propertyValue = property.GetValue(null);

	if (propertyValue != null)
	{
	description = propertyValue.ToString();
	}
	}
	}
	}
	return description;
	}
	}

	public class EnumsProfile : Profile
	{
	public EnumsProfile()
	{
	CreateMap<Enum, string>()
	.ConvertUsing<EnumToStringConverter>();
	}
	}

PROGRAMUJE .NET – Daniel Plawgo

Lokalizacja Enum

Wprowadzenie

Status zamówienia

Użycie view modeli

Atrybut EnumDescription

ToDisplayString – extension method

Darmowy kurs Visual Studio

Quiz C#

Automapper oraz TypeConverter

Przykład

Podsumowanie

Przeczytaj również:

5 thoughts on “Lokalizacja Enum”

Dodaj komentarz Anuluj pisanie odpowiedzi

	public class Order
	{
	public string Number { get; set; }

	public OrderStatus Status { get; set; }
	}

	public class OrderViewModel
	{
	public string Number { get; set; }

	public string Status { get; set; }
	}

	var status = OrderStatus.Completed;

	var display = status.ToDisplayString();

	public class EnumToStringConverter : ITypeConverter<Enum, string>
	{
	public string Convert(Enum source, string destination, ResolutionContext context)
	{
	return source.ToDisplayString();
	}
	}

	public class OrdersProfile : Profile
	{
	public OrdersProfile()
	{
	CreateMap<Order, OrderViewModel>();
	}
	}

	public class OrdersController : Controller
	{
	private static IEnumerable<Order> _orders;

	static OrdersController()
	{
	int orderNumber = 0;

	_orders = new Faker<Order>()
	.StrictMode(true)
	.RuleFor(u => u.Status, (f, u) => f.PickRandom<OrderStatus>())
	.RuleFor(u => u.Number, (f, u) => (++orderNumber).ToString())
	.Generate(10);
	}

	private Lazy<IMapper> _mapper;

	protected IMapper Mapper
	{
	get { return _mapper.Value; }
	}

	public OrdersController(Lazy<IMapper> mapper)
	{
	_mapper = mapper;
	}

	// GET: Orders
	public ActionResult Index()
	{
	var viewModel = new IndexViewModel();

	viewModel.Items = Mapper.Map<IEnumerable<OrderViewModel>>(_orders);

	return View(viewModel);
	}
	}