c# - Converting dynamic into Lambda expression

Question

Coding Platform: ASP.NET C# 4.0

I have the following snippet

public string PageID { get { return "20954654402"; } }
dynamic accounts = fb.Get("me/accounts");
if (accounts != null)
{
    bool isFound = false;
    foreach (dynamic account in accounts.data)
    {
        if (account.id == PageID)
        {
            isFound = true;
            break;
        }
    }
    if (!isFound)
    {
        //  user not admin
    }
    else
    {

    }
}

Two questions

~~Why does (account.id == PageID) errors (PageID is a string property)~~ Update: That was a stupid unrelated error as I as calling all of these at PageMethods.
Is there a simpler and more C#4.0 like way to change the foreach loop?

Update:

Its the response from a call to Facebook API. A sample will be

{
    [{
        "name": "Codoons",
        "category": "Computers/technology",
        "id": "20954694402",
        "access_token": "179946368724329|-100002186424305|209546559074402|Hp6Ee-wFX9TEQ6AoEtng0D0my70"
    }, {
        "name": "Codtions Demo Application",
        "category": "Application",
        "id": "1799464329",
        "access_token": "179946368724329|-100002186424305|179946368724329|5KoXNOd7K9Ygdw7AMMEjE28_fAQ"
    }, {
        "name": "Naen's Demo Application",
        "category": "Application",
        "id": "192419846",
        "access_token": "179946368724329|61951d4bd5d346c6cefdd4c0.1-100002186424305|192328104139846|oS-ip8gd_1iEL9YR8khgrndIqQk"
    }]
}

Updated code also a little bit.

The intention is to get the account.id that matches with PageID and get the access_token associated with that account.id

Thank you for your time.

1. Try PageID.toString() 2. for each seems fine, don't see a direct way to write it shorter — Matthias
– Matthias, Commented May 13, 2011 at 13:10

Daniel Hilgarth · Accepted Answer · 2011-05-13 13:09:49Z

1

You can use the LINQ methods for an alternative to the foreach:

if(accounts.Any(a => a.id == PageID))
{
    //  user not admin
}
else
{

}

As to why it "errors": We can't say that, because we don't know what type id is of. But if id is of type int, this would explain an error.

answered May 13, 2011 at 13:09

Daniel Hilgarth

175k41 gold badges346 silver badges450 bronze badges

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

1 Comment

codeandcloud Over a year ago

sorry for the delayed response. was busy else where. daniel your solution wont work. we are talking about dynamic types

Ricibob · Accepted Answer · 2011-05-13 13:14:48Z

0

If the accounts collection can be modified (inserts, deletes) on another thread foreach will throw an exception when that happens (simple for loop won't).

answered May 13, 2011 at 13:14

Ricibob

7,7555 gold badges49 silver badges67 bronze badges

Comments

Drenai · Accepted Answer · 2011-05-13 13:23:55Z

0

When using == with a string i.e. PageID is a string, then the account.id should also be a string, not an int or float, maybe thats whats causing the error

answered May 13, 2011 at 13:23

Drenai

12.6k9 gold badges60 silver badges94 bronze badges

Comments

George Polevoy · Accepted Answer · 2011-05-13 13:38:38Z

0

accounts.data.Any(a => a.id.ToString() == PageID)

answered May 13, 2011 at 13:38

George Polevoy

7,7733 gold badges41 silver badges63 bronze badges

Comments

tucaz · Accepted Answer · 2011-05-13 13:55:16Z

0

You should use a dynamic predicate. Something like this:


var pagesWithId = (Predicate)((dynamic x) => x.id == PageId);
var pagesFound = accounts.FindAll(pagesWithId);

if(pagesFounds.Count() > 0)
 //do your thing

answered May 13, 2011 at 13:55

tucaz

6,6946 gold badges41 silver badges62 bronze badges

Collectives™ on Stack Overflow

c# - Converting dynamic into Lambda expression

5 Answers 5

1 Comment

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

1 Comment

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related