Find the current port and open it

[deleted286]

@jsulm but it was selected

[jsulm]

@suslucoder I mean: is a port selected in combo box (or what ever widget ui->ports is)?

[deleted286]

@jsulm yes it was selected

[jsulm]

@suslucoder How do you know that ui->ports->currentText() returns an empty string?

[deleted286]

@jsulm from here

if (serial->open(QIODevice::ReadWrite))
{
    qDebug() << currentPortName << "is Open ";
    serial->clear();
    write_Json();
    connect(serial, &QSerialPort::readyRead, this, &MainWindow::on_Serial_Read);
}

[Christian Ehrlicher]

@suslucoder You don't use ui->ports->currentText() anywhere in your code you show us here ...

[jsulm]

@suslucoder said in Find the current port and open it:

currentPortName

If you mean this one, then this is wrong as you only set it once when nothing is selected! Use ui->ports->currentText():

qDebug() << ui->ports->currentText() << "is Open ";

[deleted286]

@Christian-Ehrlicher im using it to open the port?

serial->setPortName(ui->ports->currentText());

[deleted286]

@jsulm it returns empty again

[jsulm]

@suslucoder said in Find the current port and open it:

it returns empty again

How did you verify that?
Please add

qDebug() << ui->ports->currentText();

and see what it prints out.

[deleted286]

@jsulm said in Find the current port and open it:

qDebug() << ui->ports->currentText();

I've add it. It returns empty string.
Like ""
Why you didnt believe me?

[jsulm]

@suslucoder said in Find the current port and open it:

Why you didnt believe me?

Because it was not clear how you actually verified that it is indeed empty. Often people simply assume something, that's why it's worth asking.

[Christian Ehrlicher]

Please output also the count and the current index of the combobox. Is it really filled? Do you create it two times by accident?