How to query linq for list in where condition?

Question

1.00/5 (2 votes)

See more:

i have method like this

C#

public List<cil> Accounts( List<string> CL)
{
List<cil> cil = new List<cil>();
cil =(from CI in context.CIs join V1 in context.V1 on CI.Id equals V1.Id where CL.Contains(CI.ID)
select new CIL()
{
Name = V1.Name
}).ToList();
}
return cil;
}

my method parameter will return List of CL like below
1111
2222
In my db having
CI table having
ID
Row1: 1111
Row2: 2222

now above query cil returning null
Actually it suppose to return
1111
2222
incase my method parameter will return List of CL like below only one row
1111
its returning one row
that is cil returning 1111
Note: All same Id checked in on condition equals

What I have tried:

public List<cil> Accounts( List<string> CL)
{
List<cil> cil = new List<cil>();
cil =(from CI in context.CIs join V1 in context.V1 on CI.Id equals V1.Id where CL.Contains(CI.ID)
select new CIL()
{
Name = V1.Name
}).ToList();
}
return cil;
}

Any Answers would be appreciated.

Posted 7-Oct-18 6:44am

Salt Pepper

Updated 8-Oct-18 1:02am

v7

Add a Solution

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Andy Lanng · Accepted Answer · 2018-10-08T01:02:00

Have you tried using the IEnumerable extensions instead?  I find it much easier:

<pre lang="c#">public List<cil> Accounts( List<string> CL){

  var cil= context.CIs
               .Join(
                  context.V1,
                  ci=>ci.Id,
                  v1=>v1.Id
                  (ci,v1)=>new{ci, v1})
               .Where(a => CL.contains(a.ci.ID))
               .Select(a=>a.v1.Name)
               .ToList();
               
}

You can then break down the query to see what's wrong.

Also, if that "context" is entity framework then the query is not run until you call ToList().

Take a look at this:

C#

public List<cil> Accounts( List<string> CL){

  var query= context.CIs
               .Join(
                  context.V1,
                  ci=>ci.Id,
                  v1=>v1.Id
                  (ci,v1)=>new{ci, v1})
               .Where(a => CL.contains(a.ci.ID))
               .Select(a=>a.v1.Name);
  
  var queryString = query.ToString();

  return query.ToList(); 
}

That querystring will be the SQL that will be run on the DB (without the variables). Take a look and see if your logic is wrong somewhere. Maybe a join is wrong or the data isn't as you expect it ?

Hope that helps
Andy