Want to protect your cyber security and still get fast solutions? Ask a secure question today.Go Premium

x
  • Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 5318
  • Last Modified:

Help with error: Cannot implicitly convert type 'System.Collections.Generic.List<AnonymousType#1>' to 'System.Collections.Generic.List<string>'

Im getting a slight problem, ive changed the original:

lst_CounterfeitParts = (from d in context.QA_CounterfeitParts select (d.PartNumber)).ToList();

Open in new window


changed to

lst_CounterfeitParts = (from d in context.QA_CounterfeitParts select (d.PartNumber, d.Manufacturer)).ToList();

Open in new window


I thought I solved the problem with

lst_CounterfeitParts = (from d in context.QA_CounterfeitParts select d.ToString()).ToList();

Open in new window


This returns no errors, but when I run the app it returns nothing?!

Thanks,
Dean
0
deanlee17
Asked:
deanlee17
  • 11
  • 10
1 Solution
 
deanlee17Author Commented:
Sorry I should have included this:

private List<String> lst_CounterfeitParts = new List<string>();

Open in new window

0
 
Angelp1ayCommented:
Your list is typed as a list<String> but the return type of your middle block is returning a list of an anonymous type with 2 params (part number and manufacturer).

Change the declaration to this:
private var lst_CounterfeitParts;

Open in new window

0
 
deanlee17Author Commented:
So replacing

private List<String> lst_CounterfeitParts = new List<string>(); 

Open in new window


with

private var lst_CounterfeitParts; 

Open in new window

0
Keep up with what's happening at Experts Exchange!

Sign up to receive Decoded, a new monthly digest with product updates, feature release info, continuing education opportunities, and more.

 
Angelp1ayCommented:
Yes :)
0
 
deanlee17Author Commented:
The type or namespace name 'var' could not be found (are you missing a using directive or an assembly reference?)

The code site just after:

public partial class MainWindow : DXRibbonWindow
    {

Open in new window

0
 
Angelp1ayCommented:
Urgh sorry that's because it's defined as a field in the class not inside a method.

Ok... well instead let's just get explicit!

Create a private class:
        private class MyClass {
            public String PartNumber;
            public String Manufacturer;
        }

Open in new window


Declare your field member to be a list of that type:
List<MyClass> lst_CounterfeitParts = new List<MyClass>();

Open in new window


...and update your LINQ to this:
    this.lst_CounterfeitParts = (from d in context select new MyClass() { PartNumber = d.PartNumber, Manufacturer = d.Manufacturer }).ToList();

Open in new window

0
 
Angelp1ayCommented:
While we're talking about it, if you only need to enumerate then you could better define the member as an IEnumerable<MyClass>, and then you don't need to mess around with calling ToList(). I believe all LINQ methods operate on and return IEnumerables.
List<MyClass> lst_CounterfeitParts = new List<MyClass>();

...

lst_CounterfeitParts = (from d in context select new MyClass() { PartNumber = d.PartNumber, Manufacturer = d.Manufacturer });

Open in new window

0
 
deanlee17Author Commented:
haha ok now one error on:

Could not find an implementation of the query pattern for source type 'Ascent_ERP.AscentEntities'.  'Select' not found.
0
 
deanlee17Author Commented:
Sorry I hadn't seen your post above, will try that now.
0
 
deanlee17Author Commented:
Still get that error
0
 
Angelp1ayCommented:
Are you missing a reference?
using System.Linq;

Open in new window

0
 
Angelp1ayCommented:
...actually I think that would fail earlier.

Did you perhaps declare your field as just IEnumerable instead of IEnumerable<MyClass> ?
0
 
deanlee17Author Commented:
Sorry I missed this before I left work, I shall reply Monday morning.
0
 
Angelp1ayCommented:
If it still doesn't work can you post your code as complete as possible so I can check for any issues?
0
 
deanlee17Author Commented:
Ok I think ive missed the IEnumerable. So far I have the following:

List<MyClass> lst_CounterfeitParts = new List<MyClass>();

Open in new window


lst_CounterfeitParts = (from d in context select new MyClass() { PartNumber = d.PartNumber, Manufacturer = d.Manufacturer });

Open in new window


        private class MyClass
        {
            public String PartNumber;
            public String Manufacturer;
        } 

Open in new window



So should the top part be:

List IEnumerable <MyClass> lst_CounterfeitParts = new List<MyClass>();

Open in new window


Thanks for the help so far :)
0
 
Angelp1ayCommented:
The List<T> class implements the IEnumerable<T> interface, i.e. a list is an enumerable but an enumerable is not necessarily a list.

Top part should be:
IEnumerable<MyClass> lst_CounterfeitParts = new List<MyClass>();

Open in new window


LINQ methods tend to return generic IEnumerables instead of more specific Lists so if enumerables are sufficient for your needs it's less effort and more flexible to work entirely with them.
0
 
deanlee17Author Commented:
Could not find an implementation of the query pattern for source type 'Ascent_ERP.AscentEntities'.  'Select' not found.

on this line:

lst_CounterfeitParts = (from d in context select new MyClass() { PartNumber = d.PartNumber, Manufacturer = d.Manufacturer });

Open in new window


I thought maybe it could not find the table inside the entity framework so I changed it to

lst_CounterfeitParts = (from d in context.QA_CounterfeitParts select new MyClass() { PartNumber = d.PartNumber, Manufacturer = d.Manufacturer });

Open in new window


This seems to work in that the grid displays the correct number of rows, but the data in each cell is 'Ascent_ERP.MainWindow+MyClass'
0
 
Angelp1ayCommented:
Ok, that looks like progress!

I think the final step is that you're binding the entire MyClass to a field that is expecting a string. The data in the cell is the result of a call to ToString on the class itself (it's trying it's best to get the string).

I imagine in your grid markup you have some line that binds <? =Item ?> or something. Change this to Item.PartNumber or Item.Manufacturer as needed.
0
 
deanlee17Author Commented:
All done.

Thanks very much for all the help.

Just out of interest, the way we have done this using a class, is that common practise and a good template for me to use?
0
 
Angelp1ayCommented:
Yea I think so.

It's quite explicit which means you can benefit from static analysis (e.g. when it won't compile it you catch bugs early!)

I typically use this pattern for converting (projecting) lists between 2 pre-existing types (List<InputType> --> List<OutputType>).
0
 
deanlee17Author Commented:
Thanks
0

Featured Post

Get expert help—faster!

Need expert help—fast? Use the Help Bell for personalized assistance getting answers to your important questions.

  • 11
  • 10
Tackle projects and never again get stuck behind a technical roadblock.
Join Now