Ports name and description in combo box

Question

0.00/5 (No votes)

See more:

Hi,
I'm trying to display on a combo box all the ports that are available with their description. I did a code where I can just display the ports that have the description, but I can't view all the other ports that don't have the description.

Can you please help me? What am I missing in the code?

Thank you.

What I have tried:

using (var searcher = new ManagementObjectSearcher("SELECT * FROM WIN32_SerialPort"))
        {
            string[] portnames = SerialPort.GetPortNames();
            var ports = searcher.Get().Cast<ManagementBaseObject>().ToList();
            var tList = (from n in portnames
                         join p in ports on n equals p["DeviceID"].ToString()
                         select n + " - " + p["Caption"]).ToList();
            foreach (String A in tList)
            {
                ArduinoComCmbBox.Items.Add(A);
            }

Posted 5-Oct-22 1:33am

Member 15189683

Updated 5-Oct-22 22:46pm

Add a Solution

Comments

Richard MacCutchan 5-Oct-22 9:25am

The join in your LINQ is only capturing ports which match. So all the non-matched ones are ignored. So, either create a second list or add a clause to accept all ports in both lists.

Member 15189683 5-Oct-22 9:47am

How should I do it?

Richard MacCutchan 5-Oct-22 9:54am

You need to look at each list independently to see how you can get the ones that are not matching .I do not have any com ports to try it with.

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)

Richard Deeming · Accepted Answer · 2022-10-05T22:46:00

Solution 1

Try something like this:

C#

using (var searcher = new ManagementObjectSearcher("SELECT DeviceID, Caption FROM WIN32_SerialPort"))
{
    string[] portnames = SerialPort.GetPortNames();
    
    var ports = searcher.Get().Cast<ManagementBaseObject>()
        .ToDictionary(p => p["DeviceID"].ToString(), p => p["Caption"]);
    
    foreach (string name in portnames)
    {
        if (ports.TryGetValue(name, out var caption))
        {
            ArduinoComCmbBox.Items.Add($"{name} - {caption}");
        }
        else
        {
            ArduinoComCmbBox.Items.Add(name);
        }
    }

Posted 5-Oct-22 22:46pm

Richard Deeming

Comments

Member 15189683 6-Oct-22 8:43am

I tried, but I got this error when started:
'Arduino.exe' (CLR v4.0.30319: Arduino.exe): Loaded 'C:\WINDOWS\Microsoft.Net\assembly\GAC_MSIL\mscorlib.resources\v4.0_4.0.0.0_it_b77a5c561934e089\mscorlib.resources.dll'. Module was built without symbols.
Exception thrown: 'System.ArgumentException' in mscorlib.dll
An unhandled exception of type 'System.ArgumentException' occurred in mscorlib.dll
An item with the same key has already been added.

Richard Deeming 6-Oct-22 8:45am

Then check your management searcher results - you must be getting multiple serial ports with the same DeviceID.

Member 15189683 6-Oct-22 8:50am

I can't the error stop my code when try to do
var ports = searcher.Get().Cast<managementbaseobject>()
.ToDictionary(p => p["DeviceID"].ToString(), p => p["Caption"]);

Richard Deeming 6-Oct-22 8:53am

var rawPorts = search.Get().Cast<ManagementBaseObject>();
var ports = rawPorts.ToDictionary(p => p["DeviceID"].ToString(), p => p["Caption"]);

Set a breakpoint on the second line, and examine the rawPorts variable to look for duplicate device IDs.

Member 15189683 6-Oct-22 9:45am

The value in rawPorts is:
{System.Linq.Enumerable.<castiterator>d__97<system.management.managementbaseobject>}

Richard Deeming 6-Oct-22 9:47am

So expand the results view in the debugger and examine the list of values returned.

Member 15189683 6-Oct-22 10:13am

Ok, I found the error. Now it works. Thank you =)