Convert Linq code for .net 2 app

Question

1.00/5 (1 vote)

See more:

Hi In this article, a method use Linq expressions. How convert this method that be usable in .net 2 without Linq?

The method:

XML

private static IEnumerable<Control> _get_all_controls(Control c)
    {
        return c.Controls.Cast<Control>().SelectMany(item =>
            _get_all_controls(item)).Concat(c.Controls.Cast<Control>()).Where(control =>
            control.Name != string.Empty);
    }

Posted 15-Aug-15 5:47am

maysamfth

Updated 15-Aug-15 5:53am

v3

Add a Solution

Comments

DamithSL 15-Aug-15 12:00pm

what have you tried so far?

maysamfth 15-Aug-15 12:12pm

i unable to work with linq unfortunatly!

2 solutions

Solution 1

You will have to look at eth documentation for each of the Linq methods, and replace them with the equivalent foreach loop.

We don't do that for you: we expect you to do at least some of the work yourself. All you have done so far (it would seem) is find some code that might do what you want, and ask us to make it do what you want.

It's pretty obvious what that code does: it returns a IEnumerable<T> collection of controls, which includes controls nested in the top level of the Controls collection.
So do it: it's only one nested loop!

Posted 15-Aug-15 6:24am

OriginalGriff

Comments

maysamfth 16-Aug-15 2:04am

thanks. i using list and write below code but this dont work corrctly!

private List<control> _get_all_controls(Control c)
{
List<control> lstC = new List<control>();

foreach (Control ctrl in c.Controls)
{
lstC.Add(ctrl);

_get_all_controls(ctrl);
}
return lstC;
}

OriginalGriff 16-Aug-15 2:37am

"this dont work corrctly"
Is not a helpful error report!
What did it do that you didn't expect, or not do that you did?

maysamfth 16-Aug-15 3:46am

Ok, i have a FORM with some controls like button, label, panel and groupbox. this code work for controls that parents are FORM and dont work for controls that parents are label or groupbox.

OriginalGriff 16-Aug-15 3:49am

And when you say "dont work", what do you mean?
I can't see your screen, access your HDD, or read your mind! :laugh:
I only get what you type to work with here...and there are generally a lot more ways that something might not work than do what is intended.

maysamfth 16-Aug-15 4:09am

Are you preacher?!!!
Question is clear, I want add all of controls in form and controls in panels and groupbox in a List<>.
If can help me thank you else goodbye!

OriginalGriff 16-Aug-15 4:17am

"Are you preacher?!!!"
No.
If you want help, you have to realise a couple of things:
1) We don't know what is happening at your end unless you tell us. Comments like "dont work" mean nothing - it's like you breaking down in the middle of nowhere and phoning the garage for help. When they answer, you say "dont work" and put the phone down. They can assume you have a car problem, but they don't know where you are, what car it is, what happened to it, or what equipment they would need to fix it. How long do you think you are going to be waiting until they turn up with the right kit to fix your car?
We are in the same position. Unless you tell us something about the problem, we don't know it. And "dont work" is meaningless unless you tell us what it does do, what it doesn't do. Are there error messages? I don't know. Does it ever exit? I don't know. You do - so tell us!

2) Don't get an attitude with people who are trying to help you: all it will normally do is annoy them, and stop them helping now or in the future.

maysamfth 16-Aug-15 4:27am

Oh...You are preacher!!!
Ok, i'm sorry if annoyed you.
Then my question is : want add all of controls in form and controls in panels and groupbox on form to a List<>.

and my code:

private List _get_all_controls(Control c)
{
List lstC = new List();

foreach (Control ctrl in c.Controls)
{
lstC.Add(ctrl);

_get_all_controls(ctrl);
}
return lstC;
}

this code return controls on form only and controls on panel or groupbox dont added to list.

OriginalGriff 16-Aug-15 4:33am

Well, what did you expect?
You create a new list of sub controls each time - and then throw it away.
Perhaps if you changed this:
_get_all_controls(ctrl);
to this:
lstC.AddRange(_get_all_controls(ctrl));
it might work a little better?

maysamfth 16-Aug-15 4:43am

So thanks, that is true.
I apologize again.

OriginalGriff 16-Aug-15 4:51am

That's ok - I'm pretty difficult to annoy!
Some people here however, are...um...rather less tolerant... :laugh:

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)

maysamfth · Accepted Answer · 2015-08-15T22:45:00

This code return a list of controls in a form:

C#

private List<control> _get_all_controls(Control c)
    {
        List<control> lstC = new List<control>();

        foreach (Control ctrl in c.Controls)
        {
            lstC.Add(ctrl);

            lstC.AddRange(_get_all_controls(ctrl));
        }
        return lstC;
    }</control></control></control>