Better code or can any one solve this?

Question

0.00/5 (No votes)

See more:

hi m getting one error

SQL

Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index

C#

protected void ChangeNodeColorOfTreeView(TreeView treeViewWorkflow, String DocTypeText, Boolean isRecreate)
    {
        ChangeNodeColorOfTreeView(treeViewWorkflow.Nodes, DocTypeText, isRecreate);

        /*Root Node - Create*/
        {
            string varText = TreeView1.Nodes[0].Text;//this line error
            varText = varText.Replace("<span style='color: blue;'>", "<span style='color: blue;'>");
            varText = varText.Replace("<span style='color: black;'>", "<span style='color: blue;'>");
            TreeView1.Nodes[0].Text = varText;
        }
        /**/
    }

while clicking workflow button m getting this error..

and m using dynamic in my project
here is work flow class

C#

public void WorkFlowSelection(bool IsAction)
    {
        WorkflowBLL objWFBLL = new WorkflowBLL();
        DataTable dt = new DataTable();
        dt = objWFBLL.GetWorkFlowRights(SessionManager.CurrentDocType);

        if (dt.Rows.Count > 0)
        {
            if (dt.Rows[0]["dtWorkFlow"].ToString() != string.Empty)
            {
                if (dt.Rows[0]["dtWorkFlow"].ToString() == "M")
                {
                    toolBar.ShowManualWorkFlowButton = true;
                    if (IsAction)
                    {
                        toolBar.ShowForwardButton = true;
                    }


                    toolBar.ShowForwardButton = false;
                }
            }
            if (dt.Rows[0]["dtWorkFlow"].ToString() == "P")
            {
                if (IsAction)
                {
                    toolBar.ShowPredefinedWorkFlowButton = false;
                    toolBar.ShowForwardButton = false;
                }
                else
                {
                    toolBar.ShowPredefinedWorkFlowButton = true;
                }
            }
        }


    }

can any one solve this?thanks
or suggest me.

Posted 17-Apr-12 0:24am

ythisbug

Updated 17-Apr-12 0:27am

phil.o

v2

Add a Solution

5 solutions

Solution 1

You say you're getting the error here:

C#

string varText = TreeView1.Nodes[0].Text;//this line error

Which means quite clearly that your TreeView1 has no nodes. Either add a check around your faulty code that checks if TreeView1 has any nodes, or add some nodes to TreeView1.

Posted 17-Apr-12 0:29am

jim lahey

Solution 3

Please add check for treeview's node count as you did for datatable rows and check if error still persist.

Regards
Praveen

Posted 17-Apr-12 0:30am

Praveen Meghwal

Solution 4

C#

string varText=string.Empty;
if(TreeView1.Nodes.Count > 0)
{
varText=TreeView1.Nodes[0].Text;//this line error
}

Posted 17-Apr-12 0:30am

bbirajdar

Solution 5

If the line that is giving you the error is

C#

TreeView1.Nodes[0].Text;//this line error

then this is telling you that you have no items in TreeView1.Nodes. If you put a breakpoint on this line, and debug your application, when you reach this line, right click on the Nodes element and select QuickWatch from the context menu. You can then inspect what's in TreeView1.Nodes.

Posted 17-Apr-12 0:30am

Pete O'Hanlon

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)

phil.o · Accepted Answer · 2012-04-17T00:30:00

Solution 2

I would try to put something like

C#

if (TreeView1.Nodes.count > 0)

in place of

C#

/*Root Node - Create*/

and see what happens. The problem seems to be that there isn't any node in your TreeView1 object (hence the message saying that index was out of range).

Posted 17-Apr-12 0:30am

phil.o

Updated 17-Apr-12 0:31am

v2

Comments

ythisbug 17-Apr-12 7:22am

thanks dude