Using lambda expression to get values in an array

Question

Alright, here's the deal. I am doing a data conversion where I'm taking data from two databases and putting it into another. I'm adding a list of contacts, and then I'm adding a list of communication records. In order to simplify the process, I made a small array of all of the communication records with the household address of the contacts. Now I'm trying to use a lambda expression to sort out email addresses from the array, but I'm having a problem. The code so far is as follows:

            DataRow[] Comms = dtComms.Select("household_id = " + previousID);

            if (Comms.Where(x => x.Field<string>("communication_type") == "Home Phone").Count() > 0)
            {
                string HomePhone = rNDigits.Replace(Comms[0].Field<string>("communication_value").ToString().Trim(), "");
                if (HomePhone.Length > 6)
                    oAddress._Phone = HomePhone;
            }
            if (Comms.Where(x => x.Field<string>("communication_type") == "Email").Count() > 0)
            {
                string FamilyEmail = rNDigits.Replace(Comms[0].Field<string>("communication_value").ToString().Trim(), "");
                if (FamilyEmail.Contains('@') && FamilyEmail.Contains('.'))
                    oAddress._FamilyEmail = FamilyEmail;
            }

The problem is that obviously, this always will return the first value in the array, which might not always be the one that I want. How can I change the code so that it selects only the value from the array that matches the entry containing the email? Or, is there a better way to search through values in an array?

Specifically, I want to know what I'm supposed to replace (Comms[0].Field<string>... with. — ijb109
– ijb109, Commented Sep 17, 2012 at 19:04

MichaelT · Accepted Answer · 2012-09-17 19:03:57Z

2

I suggesting to use a simple for or foreach loop in this case, LINQ can't modify data only select it.

answered Sep 17, 2012 at 19:03

MichaelT

7,9548 gold badges37 silver badges47 bronze badges

Sign up to request clarification or add additional context in comments.

5 Comments

Servy Over a year ago

foreach makes more sense than for for most LINQ queries. Additionally it's not so much that it "can't" as it "shouldn't".

MichaelT Over a year ago

Yes you right about shouldn't. It can but in not straight forward way

ijb109 Over a year ago

Alright, looks like I'm going to rewrite a foreach loop in place of the LINQ query. Thanks, guys.

Ian Dallas Over a year ago

LINQ can modify data, see: stackoverflow.com/questions/807797/….

ijb109 Over a year ago

I had to wait till the time was up :)

Collectives™ on Stack Overflow

Using lambda expression to get values in an array

1 Answer 1

5 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

5 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related