Question

I have two LINQ table objects calls Orders and ImportedOrders which have exact the same properties but ImportedOrders has one more property. After import an Order gets an ImportedOrder and so I have to copy values into a new ImportedOrders object.

I have this overly ugly method I want to get rid of – preferably without reflection. I think it can be done with expressions but my using of expression trees is more than rusty.

    private static ImportedOrders MapNewOrder(Order order, int uebergabeNr)
    {
        return new ImportedOrders
        {
            DFÜ_PARTNER_ID = order.DFÜ_PARTNER_ID,
            KD_BESTELL_NR = order.KD_BESTELL_NR,
            BEST_DAT = order.BEST_DAT,
            LIEF_DAT = order.LIEF_DAT,
            ILN_KUNDE = order.ILN_KUNDE,
            KUNDE_NR = order.KUNDE_NR,
            ILN_RG_EMPFG = order.ILN_RG_EMPFG,
            RG_EMPFG_NR = order.RG_EMPFG_NR,
            ILN_LIEFERANT = order.ILN_LIEFERANT,

            // ... 30 more properties ...

            Uebergabe_Nr = uebergabeNr // < the only difference
        };
    }

EDIT:

After using AutoMapper as suggested by Amittai Shapira, the relevant logic now looks like this:

Mapper.CreateMap<Order, ImportedOrders>()
    .ForMember(o => o.Uebergabe_Nr, mo => mo.UseValue(uebergabeNr));

That’s what I call an improvement 🙂

Answer 1

Try AutoMapper, I think this is what you’re after…