How do I return textbox in loop from class using ASP.NET C#

Question

0.00/5 (No votes)

See more:

Hey All
I am returning text box from class but when i use while loop and return textbox within loop, i am getting error as follows
not all code paths return a value
need help

What I have tried:

C#

public class FormDesign
{
public TextBox txtbox(string textid)
        {
            TextBox txt = new TextBox();
            int i = 1;
            while (i<3)
            {
txt.ID = textid+i.ToString();
            txt.Text = "inserted from class"; 
            return txt;
        }
                                                                                                                                    }

}

Posted 18-Sep-20 4:22am

Aftab Iqbal Clips

Updated 19-Sep-20 19:33pm

Add a Solution

3 solutions

Solution 1

The error means what it says: there is at least one route through the method that could result in nothing being returned.
In practice, it can't happen, but ... if I fix your indentation:

C#

public TextBox txtbox(string textid)
    {
    TextBox txt = new TextBox();
    int i = 1;
    while (i<3)
       {
       txt.ID = textid+i.ToString();
       txt.Text = "inserted from class"; 
       return txt;
       }
    }

The system doesn't work out that i will always be less than 3, so it assumes that there are cases when the loop is not entered at all, and in that case there is no return value.

But ... you can't return multiple values, and if you use return inside a loop, the method exits immediately and does no more - so your loop (which would never exit if it wasn't for the return statement is pretty much pointless as your whole method evaluates to this:

C#

public TextBox txtbox(string textid)
    {
    TextBox txt = new TextBox();
    txt.ID = textid + "1";
    txt.Text = "inserted from class"; 
    return txt;
    }

If you want to return multiple values, you need to change the method signature so it returns a collection : a TextBox[] or a List<TextBox> perhaps, and then assemble the collection before you exit.

Posted 18-Sep-20 4:44am

OriginalGriff

Updated 18-Sep-20 4:45am

v2

Solution 2

So, follow the code. What happens when the code drops out of the while loop when i is equal to 3? That's what the error is talking about.

Posted 18-Sep-20 4:52am

Dave Kreskowiak

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)

Aftab Iqbal Clips · Accepted Answer · 2020-09-19T19:33:00

i figured it out myself
if you want to return multiple values using loop from function then you will have to create an array function instead of normal function

C#

public TextBox[] txtbox(int count)
        {
                                    TextBox[] txtbox;
            txtbox = new TextBox[count];
            int i = 0;
            while(i<txtbox.Length)
                        {
                TextBox t = new TextBox();
                txtbox[i] = t;
                txtbox[i].Text= "inserted textbox array"+i.ToString();
                mypanel.Controls.Add(txtbox[i]);
                i++;
            }
            return txtbox;
        }