Implicit conversion error in LINQ

Question

I have list of all ID's.

//Code

List<IAddress> AllIDs = new List<IAddress>();
AllIDs= AllIDs.Where(s => s.AddressId.Length >= s.AddressId.IndexOf("_"))
              .Select(s => s.AddressId.Substring(s.AddressId.IndexOf("_")))
              .ToList();

I am using the above LINQ query but getting compilation error:

//Error

Cannot implicitly convert type System.Collections.Generic.List to System.Collections.Generic.List

I want to to substring operation on a member field AddressId based on a character "_".

Where am I wrong?

You're trying to assign a List<string> to List<IAddress>... — Patryk Ćwiek
– Patryk Ćwiek, Commented Apr 19, 2013 at 7:09

Preet Sangha · Accepted Answer · 2013-04-19 21:52:02Z

You find the addresses you want with the where but then you select some strings from the id.

s.AddressId.Substring(s.AddressId.IndexOf("_")) is string

ie Select(s => s.AddressId.Substring(s.AddressId.IndexOf("_"))).ToList(); returns a list of substrings

Just remove it and use

AllIDs= AllIDs.Where(s => s.AddressId.Length >= s.AddressId.IndexOf("_")).ToList()

as

Where(s => s.AddressId.Length >= s.AddressId.IndexOf("_"))

filters the list of AllIDs but keeps them as IAddresss

if you rewrite is like this you should be able to see what the problem is

you said

var items  = from addr in AllIds 
             where addr.AddressId.Length >= addr.AddressId.IndexOf("_") // filter applied
             select addr.AddressId.Substring(s.AddressId.IndexOf("_")); // select a string from the address

AllIDs = items.ToList(); // hence the error List<string> can't be assigned to List<IAddress>

but you wanted

var items  = from addr in AllIds 
             where addr.AddressId.Length >= addr.AddressId.IndexOf("_") // filter applied
             select addr;                        // select the address

AllIDs = items.ToList(); // items contains IAddress's so this returns a List<IAddress>

Cyril Gandon · Accepted Answer · 2013-04-19 07:23:26Z

1

If you want to update AddressId with a Linq query, you can do it this way:

AllIDs.Where(s => s.AddressId.Length >= s.AddressId.IndexOf("_"))
      .ToList()
      .ForEach(s => s.AddressId = s.AddressId.Substring(s.AddressId.IndexOf("_")));

Note that .ForEach() is not a Linq extension, but a method of the class List< T >.

Since IndexOf could be time consuming, think about caching the value:

AllIDs.Select(s => new { Address = s, IndexOf_ = s.AddressId.IndexOf("_") })
      .Where(s => s.Address.AddressId.Length >= s.IndexOf_ )
      .ToList()
      .ForEach(s => s.Address.AddressId = s.Address.AddressId.Substring(s.IndexOf_ ));

edited Apr 19, 2013 at 7:23

answered Apr 19, 2013 at 7:17

Cyril Gandon

17.1k15 gold badges84 silver badges124 bronze badges

Comments

Dirk · Accepted Answer · 2013-04-19 07:11:06Z

0

Your select operation .Select(s => s.AddressId.Substring(s.AddressId.IndexOf("_"))) does not modify your objects, it projects each object to a substring. Thus .ToList() returns a List<string>.

answered Apr 19, 2013 at 7:11

Dirk

11k2 gold badges38 silver badges50 bronze badges

Collectives™ on Stack Overflow

Implicit conversion error in LINQ

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related