add node to treeview

Question

2.00/5 (3 votes)

See more:

Hi, I'm trying in this code to add node from textbox to treeview if this node equal to node that is found in treeview the new node must be next old node and have color yellow the problem when I click to the button nothing happen

C#

for (int i = 0; i < treeView1.Nodes.Count; i++)
           {
               string p = treeView1.Nodes[i].Text;
               if (p == textBox1.Text)
               {
                   treeView1.SelectedNode.NextNode.Nodes.Add(textBox1.Text);
                   treeView1.SelectedNode.BackColor = Color.Yellow;

               }
           }

Posted 28-Nov-15 8:56am

_Tuba

Updated 28-Nov-15 21:48pm

v2

Add a Solution

Comments

PIEBALDconsult 28-Nov-15 15:00pm

Use the debugger to inspect the value of p.
Additionally, if you need to do that a lot, then I recommend you use a Dictionary as well as a Treeview and don't iterate the Nodes.

[no name] 28-Nov-15 15:18pm

I don't think that nothings happens, but it Looks like you add the node to the "root" and not to the node "p ==...". Have a look to this:http://c-treeview.com/[^]

_Tuba 28-Nov-15 15:30pm

for loop does not complete just int i = 0 and ended

PIEBALDconsult 28-Nov-15 15:34pm

Are there any existing Nodes?

_Tuba 28-Nov-15 15:39pm

yes,I'm adding at design time root and sub child

PIEBALDconsult 28-Nov-15 18:34pm

Seriously; look at the value of p.

Sergey Alexandrovich Kryukov 29-Nov-15 4:01am

TreeView? Which one? Full type name, please.
—SA

_Tuba 29-Nov-15 5:12am

Sorry What do you mean?

Sergey Alexandrovich Kryukov 29-Nov-15 11:10am

What do I mean? a type name is a type name, full. As a developer, first thing you need to know is the types and their names. You need to know them well before getting to UI. "TreeView" is not a full name; there are different (unrelated) types under this name, so we need to know which one, exactly, do we have to look at.
—SA

PIEBALDconsult 29-Nov-15 12:26pm

I use the standard .net System.Windows.Forms.TreeView, and maybe you do too, but not everyone does, there are others out there so you should tell us exactly which one you are using.

1 solution

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)

BillWoodruff · Answer 1 · 2015-11-28T16:37:00

Solution 1

Instead of using:

string p = treeView1.Nodes[i].ToString();

use:

string p = treeView1.Nodes[i].Text;

Calling 'ToString on a 'TreeNode will return something like:

"TreeNode: Node0"

I suggest you revise your question and clarify:

0. you say you have sub-nodes, but the 'for loop in your code only enumerates the root-level nodes: is that your intent ?

1. where exactly do you want to insert the new TreeNode ?

2. why would you want to have two TreeNodes with the same visible 'Text property ?

3. what is the role of the 'SelectedNode here, and why do you changes its 'BackColor ?

Posted 28-Nov-15 16:37pm

BillWoodruff

Comments

_Tuba 29-Nov-15 3:46am

thanks can you see my update question

BillWoodruff 29-Nov-15 21:34pm

Yes, I read your updated question, and it is still unclear to me what you want.

Try answering the three specific questions I asked you in the solution I posted.

_Tuba 29-Nov-15 3:49am

but I Don't know how can I visit child without Recursive

PIEBALDconsult 29-Nov-15 12:02pm

You don't want to search the Tree. When you add a Node to the Tree also add it to a Dictionary < string , TreeNode > , then you search the Dictionary instead, it's much faster.

_Tuba 29-Nov-15 12:24pm

if the node found as child how can I add new node that has the same text as NextNode ? and my code doesn't work :(

BillWoodruff 29-Nov-15 21:35pm

You need to distinguish between a TreeNode which is "found" by your searching the TreeView in code at run-time, and a TreeNode which is selected by a user at run-time.

Make it clear what you are talking about.