Automapper help mapping a complex c# objects

Question

I am trying to map from Vehicle object to the Motor object using Automapper

public class Range<T>
{
    public T Min { get; set; }
    public T Max { get; set; }
}

public Enum SpeedType
{
    [Display(Name = "-")] Unknown = 0,
    [Display(Name = "M")] Manual= 1,
    [Display(Name = "A")] Automatic= 2
}

public class Vehicle
{
    public Range<string> Speed { get; set; }
}

public class Motor
{
    public Range<SpeedType?> Speed { get; set; }
}

I have tried using the MapFrom (reading the documentation) without any success. Can someone point me in the right direction. I am not even sure if this can be even mapped using Automapper. I have used automapper in the past for simple mappings.

Using Range<T> with an Enum as you've set it doesn't make sense. Why is Automatic max and Manual min or the other way around, There is no concept in the real world of one being greater than the other. — Erik Philips
– Erik Philips, Commented Dec 27, 2018 at 20:06
I have simplified the example Suppose for vehicle Min ="M", Max="A" then I want to convert it to Min=1,Max=2 — Lizzy
– Lizzy, Commented Dec 27, 2018 at 20:13
To be frank I'm not a big fan of AutoMapper. I had to use it in a project for more than a year but actually it is only good for making simple things more complicated than necessary. Some FromDto and ToDto [extension] methods are just simpler, faster, easier to debug and more reactive to code changes. Mapping between different class layouts often results practically as much code (or even more with tons of lambdas) as simply writing the mapping manually. See also: cezarypiatek.github.io/post/why-i-dont-use-automapper — György Kőszeg
– György Kőszeg, Commented Dec 27, 2018 at 20:22

Szabolcs Dézsi · Accepted Answer · 2018-12-28 03:04:28Z

2

This works for me:

Mapper.Initialize(cfg =>
{
    cfg.CreateMap<string, SpeedType?>().ConvertUsing(speed =>
    {
        switch (speed)
        {
            case "M": return SpeedType.Manual;
            case "A": return SpeedType.Automatic;
            default: return SpeedType.Unknown;
        }
    });

    cfg.CreateMap<Range<string>, Range<SpeedType?>>();
    cfg.CreateMap<Vehicle, Motor>();
});

var vehicle = new Vehicle
{
    Speed = new Range<string>
    {
        Min = "M",
        Max = "A"
    }
};

var motor = Mapper.Map<Vehicle, Motor>(vehicle);

answered Dec 28, 2018 at 3:04

Szabolcs Dézsi

8,84323 silver badges29 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Lizzy Over a year ago

Thanks for your reply. Learned that switch could be added to Automapper.

Arthur Attout Over a year ago

FYI, you can basically add anything you want inside the curly braces. This is a lambda expression, so any code is valid as long as all the code path returns something

Lizzy · Accepted Answer · 2018-12-28 17:00:54Z

0

I came up with the following mapping to solve my problem. I wrote a GetEnum custom method

CreateMap<Vehicle,Motor>()
.ForMember(g => g.Speed, opt => opt.MapFrom(u=> new Range<SpeedType?>
                {
                    Min = EnumHelper.GetEnum<SpeedType?>(u.Speed.Min),
                    Max = EnumHelper.GetEnum<SpeedType?>(u.Speed.Max),
                }))

answered Dec 28, 2018 at 17:00

Lizzy

1117 bronze badges

Collectives™ on Stack Overflow

Automapper help mapping a complex c# objects

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related