CODEWITHSUNDEEP
automapper.netautomapper-2
Sandeep Reddy Pinniti
Sandeep Reddy Pinniti

Reputation: 431

Automapper expression must resolve to top-level member

I am using automapper to map source and destination objects. While I map them I get the below error.

Expression must resolve to top-level member. Parameter name: lambdaExpression

I am not able resolve the issue.

My source and destination objects are:

public partial class Source
{
        private Car[] cars;

        public Car[] Cars
        {
            get { return this.cars; }
            set { this.cars = value; }
        }
}

public partial class Destination
{
        private OutputData output;

        public OutputData Output
        {            
            get {  return this.output; }
            set {  this.output= value; }
        }
}

public class OutputData
{
        private List<Cars> cars;

        public Car[] Cars
        {
            get { return this.cars; }
            set { this.cars = value; }
        }
}

I have to map Source.Cars with Destination.OutputData.Cars object. Could you please help me in this?

Upvotes: 36

Views: 31702

Answers (6)

Vijayanand Settin
Vijayanand Settin

Reputation: 1084

ForPath works for this exact scenario.

Mapper.CreateMap<Destination, Source>().ForPath(dst => dst.OutputData.Cars, e => e.MapFrom(src => src.Cars));

Upvotes: 8

Dominus.Vobiscum
Dominus.Vobiscum

Reputation: 847

This worked for me:

Mapper.CreateMap<Destination, Source>()
    .ForMember(x => x.Cars, x => x.MapFrom(y => y.OutputData.Cars))
    .ReverseMap();

Upvotes: 2

Kevat Shah
Kevat Shah

Reputation: 159

The correct answer given by allrameest on this question should help: AutoMapper - Deep level mapping

This is what you need:

Mapper.CreateMap<Source, Destination>()
    .ForMember(dest => dest.OutputData, opt => opt.MapFrom(i => i));
Mapper.CreateMap<Source, OutputData>()
    .ForMember(dest => dest.Cars, opt => opt.MapFrom(i => i.Cars));

When using the mapper, use:

var destinationObj = Mapper.Map<Source, Destination>(sourceObj)

where destinationObj is an instance of Destination and sourceObj is an instance of Source.

NOTE: You should try to move away from using Mapper.CreateMap at this point, it is obsolete and will be unsupported soon.

Upvotes: 0

Marcus.D
Marcus.D

Reputation: 769

You can do it that way:

// First: create mapping for the subtypes
Mapper.CreateMap<Source, OutputData>();

// Then: create the main mapping
Mapper.CreateMap<Source, Destination>().
    // chose the destination-property and map the source itself
    ForMember(dest => dest.Output, x => x.MapFrom(src => src));

That's my way to do that ;-)

Upvotes: 19

Patrick Desjardins
Patrick Desjardins

Reputation: 140983

You are using : 

 Mapper.CreateMap<Source, Destination>()
 .ForMember( dest => dest.OutputData.Cars, 
             input => input.MapFrom(i => i.Cars));

This won't work because you are using 2 level in the dest lambda.

With Automapper, you can only map to 1 level. To fix the problem you need to use a single level :

 Mapper.CreateMap<Source, Destination>()
 .ForMember( dest => dest.OutputData, 
             input => input.MapFrom(i => new OutputData{Cars=i.Cars}));

This way, you can set your cars to the destination.

Upvotes: 54

k0stya
k0stya

Reputation: 4315

  1. Define mapping between Source and OutputData.

    Mapper.CreateMap<Source, OutputData>();

  2. Update your configuration to map Destination.Output with OutputData.

    Mapper.CreateMap<Source, Destination>().ForMember( dest => dest.Output, input => 
    input.MapFrom(s=>Mapper.Map<Source, OutputData>(s)));

Upvotes: 17

Related Questions