Dynamic Treeview using C# - C# Discussion Boards

No, because GetPixel and SetPixel are part of the Bitmap class, and Controls are not derived from Bitmap.

What are you trying to achieve that you think you need this?

Bad command or file name. Bad, bad command! Sit! Stay! Staaaay...

Not to mention that SetPixel and GetPixel are really slow...

You're going to have to explain what you're trying to do and how these functions play into this goal.

A guide to posting questions on CodeProject

Click this: Asking questions is a skill.
Seriously, do it.

Dave Kreskowiak

Assuming you can get a bitmap of the current visual state of a Control, which you can do in WinForms pretty easily, well ...

Yes, you could manipulate that bitmap with Get/Set/Pixel, and then re-draw it onto the surface of the Control.

But, I can't think of a single-case where that would be a wise thing to do.

What you can do, and has been done, frequently, is to sub-class a Control in WinForms, like a 'Button, and then make it owner-drawn, and handle all the drawing yourself.

But, as comments here have told you, you are going to take big hit on performance using Get/Set/Pixel.

«Tell me and I forget. Teach me and I remember. Involve me and I learn.» Benjamin Franklin

As other have mentioned, it's rather pointless to use SetPixel/GetPixel on controls (that is, getting a Handle, getting a DC, and converting that to a bitmap).

What are you trying to achieve? Pretty sure you will end up using something else.

Best,
John

-- LogWizard - a Log Viewer that is easy and fun to use!

Please Help
How can I replace all text in a listBox1 that matches "some text " with "other text "?
I tried, but does not work...thanks.

private void button1_Click_1(object sender, EventArgs e)
{
listBox1.Items = listBox1.Items.ToString.Replace("some text ", "other text ")

}

I would approach this by:

// defining the ListBox Items in code   
private List<string> listItemData = new List<string>
{
    "one","two","three","two","five","six","seven","two","nine","ten"
};

// setting the DataSource property of the ListBox to use the List<string>
private void Form1_Load(object sender, EventArgs e)
{
    listBox1.DataSource = listItemData;
}

// creating a method to do substitution
private void ReplaceInListBox(string toFind, string toReplace, bool replaceOneItemOnly)
{
    int ndx;
    string itm;

    for (int i = 0; i < listItemData.Count; i++)
    {
        itm = listItemData[i];

        if (itm == toFind)
        {
            ndx = listItemData.IndexOf(itm);
            listItemData.RemoveAt(ndx);
            listItemData.Insert(ndx, toReplace);

            if (replaceOneItemOnly) break;
        }
    }

    // necessary to reset the DataSource
    // in order to refresh it
    listBox1.DataSource = null;
    listBox1.DataSource = listItemData;
    listBox1.Invalidate();
}

// test like this:
private void SomeButton_Click(object sender, EventArgs e)
{
    ReplaceInListBox("two", "twenty", false);
}

By the way, it would be interesting to know why it is you want to dynamically change the ListBox Items; that's something I would avoid unless it was absolutely necessary.

«Tell me and I forget. Teach me and I remember. Involve me and I learn.» Benjamin Franklin

I am dynamically binding a tree-view.I have multiple root nodes and multiple child nodes.OnSelectedNodeChanged event is firing only for childnodes of first root node.If I click on child nodes of second root node,i am getting root node value only in Treeview.Selectednode.Text. Can anyone help?this is my code..

this is my code..

C#

private void FillCategories(string item, TreeNode parent)
    {
        string[] category1 = item.Split(',');
        TreeNode tnParent = new TreeNode();
        tnParent.Text = category1[1];
        tnParent.Value = category1[0];
        tnParent.ToolTip = category1[1];
        tnParent.SelectAction = TreeNodeSelectAction.SelectExpand;
        tnParent.CollapseAll();
        tnParent.Selected = true;
        parent.ChildNodes.Add(tnParent);
        FillMetadata(tnParent, tnParent.Value);
    }

    private void FillMetadata(TreeNode parent, string categoryId)
    {
        DataSet ds = PDataset("SELECT MetadataID,MetadataName FROM Metadata WHERE MetadataID IN (SELECT MetadataID FROM CategoryMetadata WHERE CategoryID =" + categoryId + ") and ParentMetadataID IS NULL");
        parent.ChildNodes.Clear();
        foreach (DataRow dr1 in ds.Tables[0].Rows)
        {
            TreeNode child = new TreeNode();
            child.Text = dr1["MetadataName"].ToString().Trim();
            child.Value = dr1["MetaDataID"].ToString().Trim();
            child.ToolTip = child.Text;
            child.SelectAction = TreeNodeSelectAction.Select;
            child.CollapseAll();
            parent.ChildNodes.Add(child);
            FillSubMetadata(child, child.Value);
        }
    }

private void FillSubMetadata(TreeNode child, string metadataId)
    {
        DataSet ds = PDataset("SELECT MetadataID,MetadataName FROM Metadata WHERE ParentMetadataID =" + metadataId + " ");
        child.ChildNodes.Clear();
        foreach (DataRow dr1 in ds.Tables[0].Rows)
        {
            TreeNode subchild = new TreeNode();
            subchild.Text = dr1["MetadataName"].ToString().Trim();
            subchild.Value = dr1["MetaDataID"].ToString().Trim();
            subchild.ToolTip = subchild.Text;
            subchild.SelectAction = TreeNodeSelectAction.Select;
            subchild.CollapseAll();
            child.ChildNodes.Add(subchild);
        }
    }

Well that was confusing ... until I realised that this was ASP.NET.
Just so you're aware there is a separate forum for Web development - ASP.NET Discussion Boards - CodeProject[^]

Your code is vulnerable to SQL Injection[^].

NEVER use string concatenation to build a SQL query. ALWAYS use a parameterized query.

"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer

Does anyone know how to make a countdown timer?
I have tried but no methods I try work.

Try this: Timer Class (System.Timers)[^].

I searched for a solution on the internet but no luck.

I am using windows form with Serial Port control and richTextBox. I am trying to read the phone number from caller ID device (CTI comet caller ID) which decodes the Caller's telephone number from the telephone line and presenting it on a virtual COM5 Port of my PC because the Caller ID is a COM device.

The caller ID is connected to my PC via USB to COM Adapter because my PC doesn't have physical COM ports. Also the USB to COM Adapter driver is installed well .Note that the telephone line supports the caller ID feature.

When the phone rings the phone number should show up in richTextBox, I tried the following code but nothing happens (phone number did not appear) Am I missing something?.

My question:

How can I read data from virtual COM5 port in C#? Please help me to modify this code to make it work. Thank you

C#

public partial class Form1 : Form
    {
      public  SerialPort mySerialPort = new SerialPort("COM5");

    public Form1()
    {
        InitializeComponent();

        mySerialPort.DataReceived += new SerialDataReceivedEventHandler(DataR);
        mySerialPort.Open();
     }

    public void DataR(object sender, SerialDataReceivedEventArgs e)

    {

        if (richTextBox1.InvokeRequired)
         {
             richTextBox1.Invoke(new SerialDataReceivedEventHandler(DataR), sender, e);
         }

         else
         {

            richTextBox1.AppendText(mySerialPort.ReadExisting()); //recieve data in real time.

          }
      }


    private void serialPort1_DataReceived(object sender, SerialDataReceivedEventArgs e)
    {

    }
}

Start by making sure that you can communicate with the device at all, by using Hyperterminal or similar.
Until you have established that you have the right port, speed, pbc, parity, stop bits, and that any flow control is working fine, you are just guessing with "home brewed" code. So when it doesn't work, it could quite literally be anything causing it.

Bad command or file name. Bad, bad command! Sit! Stay! Staaaay...

@ OriginalGriff .

I will try your idea and see what happens, I will let you know . thank you

How do you know that you're not receiving data and that the "display" part is the problem?

Read your port into a string and check the contents before trying to display something that may or may not be there.

(That's why you don't chain methods so early on in development).

@

Gerry Schmitz

. I tried to read the port into a string but the string still empty, I debugged the program and I noticed that the DataReceived event did not got fired. Does it mean the port did not receive any data ? plseae help

naouf10 wrote:
Does it mean the port did not receive any data ?

Yes... Exactly that...

Bad command or file name. Bad, bad command! Sit! Stay! Staaaay...

@ OriginalGriff. I tested the caller Id in different PC (PC2) which has EPOS and this EPOS could display the phone number so it means the port receives data when phone rings therefor I tested my C# code in PC2 and still my app didn't receive data from COM port although EPOS can read data from same port. any idea?

I've "heard" the DataReceived event doesn't always work (so I avoid it).

You first need to confirm that your SerialPort (class) settings match what's in Control Panel AND match what the device expects.

Also insure your "timeout" settings are high enough.

When you believe there is data on the serial port, you can do a .BytesToRead() call on the serial port to see if there is data waiting. You should "sleep" some amount of time before trying.

If you never have any "bytes to read" > 0, then you're still not configured / hooked-up properly.

(You should build a dialog (button) so that you can read / query the port on demand for testing. You cannot restart a program and serial port and expect data to persist).

@ Gerry Schmitz . Thank you for your good information.

yes both the SerialPort (class) and the Caller ID have the same settings.

maybe your ideas will work but I have to modify my code first. Can you please tell me how to do the following in my code:
-

C#

.BytesToRead() call

- how to sleep before trying?
- how to insure my "timeout" settings are high enough. I don't have TimeOut in my code.

Please help me so I can modify my code. Thank you

C#

int bytesToRead = 0;

for ( int i = 0; i < 5; i++ ) {
   bytesToRead = _serialPort.BytesToRead;

   if ( bytesToRead > 0 ) {
      break;
   }

   Thread.Sleep( 50 );

}  // end for.

Setting timeouts is trial-and-error; you just need to insure they are "long enough".

I've been in situations where the timeouts needed varying from 250ms up to 5000ms; depending on the environment (i.e. network).

Another thing: Serial devices don't typically just "send"; they respond to requests. i.e. You send data to the device; it responds ... even if it is just with an error.

If there is no "command" that you can send to the device and for which you can expect a response (like "current status"), then you are missing something (like RTFM).

Hi,

serial input on a Windows PC (or on any device for that matter) tends to be tricky, here are some pointers:

1. The code you have shown does NOT read any data, your DataReceived handler is empty.

2. I/O operations occur in "real time", making it hard to observe them correctly; typical debug operations (single-stepping and the like) don't work, the world doesn't stop when you stop typing the keyboard. My solution to this is: use logging. A simple Console.WriteLine() or a File.WriteLine() could do, prefixing a time stamp always proves useful. I tend to also log to a listbox, with auto-scroll, and obviously with the appropriate Invoke stuff built-in so it works from any thread that happens to call it. And no, you don't need a logging package for this, all the logging infrastructure requires is about 10 lines of code.

3. Advice on all software development: don't build your entire app, then test and watch it fail. Develop one basic operation at a time, get it to work properly, then proceed. Hence, do not touch WinForms Controls before you must.

4. Tricky bits part 1: AFAIK there is no decent description as to when an I/O event is called; for DataReceived it does not mean: "when my data is in" (the computer does not know what you consider "your data"), and we can only hope and observe it does not mean for each and every byte. What typically happens is the operating system collects incoming bytes, and signals an input operation when it has a reasonable amount of data (say 5 bytes), which probably is half a message in your application domain, so you get an event for half a message; and then the remaining bytes possibly aren't enough to cause a second signal, so you don't get them at all.
(Exception is when you read a text string and tell the driver what constitutes an end-of-message, by default that could be a CARRIAGE RETURN, a LINEFEED or both). Conclusion: you most often are better off not using those events, as they often do not connect well to your application. Yes they work, but not the way you would dream they work.

5. Advice on I/O: for simple things (and also when starting complex things), first use operations you perform explicitly AND synchronously (i.e. actual Read, ReadLine and the like, without events), avoid asynchronous stuff (such as the DataReceived operation).

6. Tricky bits part 2: how does one synchronize the external device and the code that reads data? the start of transmission is no good, reading a message often makes sense only when the entire message is available.
I see four useful schemes:
a) if you can choose the message format, use text, the proper end-of-line marker, and ReadLine; easy, but seldom possible;
b) otherwise, when an end-of-transmission event exists, use it and read the entire message.
c) otherwise, when a transmission-has-started event exists, and bandwidth and latency aren't real issues, then use that signal to start a delay, after the delay read the entire message. The delay must be long enough to transmit the longest possible message at the current (or the slowest expected) baud rate. This approach wastes both time (hence latency) and bandwidth. As you want to display incoming data, I expect latency and bandwidth aren't an issue, so this would be the right choice for you.
d) if the above schemes don't exist or aren't acceptable, construct your own data buffer, feed it with incoming data as and when it comes, then detect message starts and ends, and create a signal to your actual business logic as appropriate.

7. What I would do first is have a button that reads from the port and logs everything it got. Only proceed when that works. Next step could be: create a timer (I mean a Windows.Forms.Timer, which ticks on the main thread) that periodically reads the serial port.
Then start looking into the synchronization problem I mentioned under 6.

8. A number of I/O and timer operations run on a different thread, so you can't directly access WinForms Controls from there; that is where Control.Invoke() comes in. This applies to almost all events on incoming data (SerialPort.DataReceived, and most kinds of timers).

I hope this helps, and I wish you good luck!

Smile | :)

PS: The following might be useful: Article: A simple logging scheme[^], Article: Invalid cross-thread operations[^], Article: Asynchronous operations run on ThreadPool threads[^], Article: Timer surprises, and how to avoid them[^]

Luc Pattyn [My Articles] Nil Volentibus Arduum

modified 24-Dec-15 11:41am.

Thank you alot, I will consider what you have said.

Excellent post! Thumbs Up | :thumbsup:

You have just been Sharapova'd.