Checking QByteArray for certain values

[-Kyr0s-]

Unless I completely misunderstand the QBA, I receive precisely what I expected. I've done as you adviced, and the following printed out in my output textbox:

01020003000100
01020003000101

01020003000100
01020003000101

To copy to clipboard, switch view to plain text mode 

I wrote to the DIO (output) first to turn it on, and then read it (input) to see the values in the textbox. The first line is DIO turned on, the second is off.

So, assuming the returned value doesn't include anything beside that in a secretive manner, the reading works as it is supposed to. :S

For debug purposes, I disabled the function calls for the other 3 checkboxes, it is still the same as before.

Attachment 7035

[Spitfire]

Try that:

void EcoDIOManager::connection_established()
{
    QString strSuccess = "Successfully connected to ",
            device_address = this->ui->txtbxIP->text();
 
    this->enableObjects();
    this->ui->txtbxSuccess->setText(strSuccess%device_address);
    this->ui->chkDIO0->setChecked(this->isOn(0));
    this->ui->chkDIO1->setChecked(this->isOn(1));
    this->ui->chkDIO2->setChecked(this->isOn(2));
    this->ui->chkDIO3->setChecked(this->isOn(3));
}
 
QByteArray EcoDIOManager::read_bytes(quint16 dio_n)
{
    QByteArray bytes;
    bytes.push_back(1);
    bytes.push_back(2);
    bytes.push_back(NULLPTR);
    bytes.push_back(1);
    bytes.push_back(dio_n);
 
    client.write(bytes);
 
    return client.read(7);
}
 
bool EcoDIOManager::isOn( quint16 dio_n )
{
    bool isOn = false;
    const char ON = 0x00;
    const int switchIndex = 6;
 
    QByteArray readBytes = this.read_bytes(dio_n);
 
    if(readBytes.size() > switchIndex && readBytes[switchIndex] == ON)
        isOn = true;
 
    return isOn;
}

void EcoDIOManager::connection_established()
{
    QString strSuccess = "Successfully connected to ",
            device_address = this->ui->txtbxIP->text();

    this->enableObjects();
    this->ui->txtbxSuccess->setText(strSuccess%device_address);
    this->ui->chkDIO0->setChecked(this->isOn(0));
    this->ui->chkDIO1->setChecked(this->isOn(1));
    this->ui->chkDIO2->setChecked(this->isOn(2));
    this->ui->chkDIO3->setChecked(this->isOn(3));
}

QByteArray EcoDIOManager::read_bytes(quint16 dio_n)
{
    QByteArray bytes;
    bytes.push_back(1);
    bytes.push_back(2);
    bytes.push_back(NULLPTR);
    bytes.push_back(1);
    bytes.push_back(dio_n);

    client.write(bytes);

    return client.read(7);
}

bool EcoDIOManager::isOn( quint16 dio_n )
{
    bool isOn = false;
    const char ON = 0x00;
    const int switchIndex = 6;

    QByteArray readBytes = this.read_bytes(dio_n);

    if(readBytes.size() > switchIndex && readBytes[switchIndex] == ON)
        isOn = true;
        
    return isOn;
}

To copy to clipboard, switch view to plain text mode 

[-Kyr0s-]

Thanks for taking your time to assist me, once again much appreciated.

Seems that it still is refusing to work. I've put a breakpoint in bool EcoDIOManager::isOn(quint16 dio_n), and the following were the results once it returned at the end:

ON: 0 '\0'
dio_n: 0
isOn: false
readBytes: ""
switchIndex: 6
this: @0x28fe48

ON: 0 '\0'
dio_n: 0
isOn: false
readBytes: ""
switchIndex: 6
this: @0x28fe48

To copy to clipboard, switch view to plain text mode 

I guess readBytes isn't supposed to be an empty string?

[Spitfire]

No it should not

To make sure that everything else works just your example values into the array and see if checkbox is set correctly.
then investigate around that place, this array should contain exacly what you expect to see.

[-Kyr0s-]

Hey again,

I've been pulling my hair out for these past few days, but it seems I just can't find the problem.

readBytes is still an empty string after this bit:

bool EcoDIOManager::isOn(quint16 dio_n)
{
    bool isOn = false;
    const char ON = 0x00;
    const int switchIndex = 6;
 
    QByteArray readBytes = this->read_bytes(dio_n);
 
    if(readBytes.size() > switchIndex && readBytes[switchIndex] == ON)
    {
        isOn = true;
    }
    else
    {
        isOn = false;
    }
 
    return isOn;
}

bool EcoDIOManager::isOn(quint16 dio_n)
{
    bool isOn = false;
    const char ON = 0x00;
    const int switchIndex = 6;

    QByteArray readBytes = this->read_bytes(dio_n);

    if(readBytes.size() > switchIndex && readBytes[switchIndex] == ON)
    {
        isOn = true;
    }
    else
    {
        isOn = false;
    }

    return isOn;
}

To copy to clipboard, switch view to plain text mode 

I suspected that the problem lies with the call of the read_bytes(dio_n) function.

QByteArray EcoDIOManager::read_bytes(quint16 dio_n)
{
    /* *******************************
    * 1st byte: Command number, 1
    * 2nd byte: Version, 2
    * 3rd byte: Response only, any
    * 4th byte: Data length, 1
    * 5th byte: DIO channel
    ********************************/
    QByteArray bytes;
    bytes.push_back(1);
    bytes.push_back(2);
    bytes.push_back(NULLPTR);
    bytes.push_back(1);
    bytes.push_back(dio_n);
 
    client.write(bytes);
 
    this->ui->txtbxOutput->append(client.read(7).toHex());
 
    return client.read(7);
}

QByteArray EcoDIOManager::read_bytes(quint16 dio_n)
{
    /* *******************************
    * 1st byte: Command number, 1
    * 2nd byte: Version, 2
    * 3rd byte: Response only, any
    * 4th byte: Data length, 1
    * 5th byte: DIO channel
    ********************************/
    QByteArray bytes;
    bytes.push_back(1);
    bytes.push_back(2);
    bytes.push_back(NULLPTR);
    bytes.push_back(1);
    bytes.push_back(dio_n);

    client.write(bytes);

    this->ui->txtbxOutput->append(client.read(7).toHex());

    return client.read(7);
}

To copy to clipboard, switch view to plain text mode 

I've also tried return client.read(7).toHex();.

The function on itself parses the values (the string of bytes) properly in the textbox (I can see the values I need without any issues), yet it cannot be used in EcoDIOManager::isOn().

Any suggestions as to why this might occur at all? It's driving me crazy. I've nearly not had as many problems figuring out how to write as opposed to apply a process to reading.

[-Kyr0s-]

Never mind, false alarm with this post. Wasn't called twice. >_<

[Spitfire]

What happens if you call client.read(7) twice in a row?
Do you get the same output twice?

Instead of returning the output straight away create temporary variable and return that variable and see what happens.
IMHO it should make no difference.

[-Kyr0s-]

I've tried what you suggested, but to no avail.

QByteArray bytes;
 
    bytes.push_back(1);
    bytes.push_back(2);
    bytes.push_back(NULLPTR);
    bytes.push_back(1);
    bytes.push_back(dio_n);
 
    client.write(bytes);
    buffer = client.read(7);
    this->ui->txtbxOutput->append(buffer.toHex());
 
    return buffer;

QByteArray bytes;

    bytes.push_back(1);
    bytes.push_back(2);
    bytes.push_back(NULLPTR);
    bytes.push_back(1);
    bytes.push_back(dio_n);

    client.write(bytes);
    buffer = client.read(7);
    this->ui->txtbxOutput->append(buffer.toHex());

    return buffer;

To copy to clipboard, switch view to plain text mode 

Buffer is a QByteArray.

Also, maybe worth noting, is that the textbox won't show any output, until I click one of the checboxes. Oddly enough, when the application starts, it does go through read_bytes() and thus, is supposed to print out the string. It doesn't do that, only after having clicked a checkbox it would, whilst it goes through the same function as start-up.

I don't know why it doesn't immediately parse the output in the textbox, as it's pretty much the very same functions..

[-Kyr0s-]

Hello,

I've decided to redo the application to see where it goes wrong. It seems everything works fine; it writes and reads the appropriate number of bytes. Reading is supposed to send 5 bytes, and receive 7 in return.

However, once it accesses the isOn() method, these bytes turn into 20.

I suspect it's due to this:

this->ui->chkDIO0->setChecked(this->isOn(0));
    this->ui->chkDIO1->setChecked(this->isOn(1));
    this->ui->chkDIO2->setChecked(this->isOn(2));
    this->ui->chkDIO3->setChecked(this->isOn(3));

this->ui->chkDIO0->setChecked(this->isOn(0));
    this->ui->chkDIO1->setChecked(this->isOn(1));
    this->ui->chkDIO2->setChecked(this->isOn(2));
    this->ui->chkDIO3->setChecked(this->isOn(3));

To copy to clipboard, switch view to plain text mode 

Each time it goes through reading bytes, it sends 5 bytes, so 5x4 is 20. I guess that makes sense, but even if it didn't, readBytes in isOn() remains to be an empty string.

QByteArray DIOMoxaBoard::read_bytes(quint16 dio_n)
{
    /* *******************************
       * 1st byte: Command number, 1
       * 2nd byte: Version, 2
       * 3rd byte: Response only, any
       * 4th byte: Data length, 1
       * 5th byte: DIO channel
       ********************************/
 
    QByteArray bytes;
 
    bytes.push_back(1);
    bytes.push_back(2);
    bytes.push_back(_NULL);
    bytes.push_back(1);
    bytes.push_back(dio_n);
 
    client.write(bytes); // successfully writes 5 bytes if it doesn't access isOn()
 
    return client.read(7).toHex(); //successfully reads 7 bytes if it doesn't access isOn()
}
 
bool DIOMoxaBoard::isOn(quint16 dio_n)
{
    bool isOn = false;
    const char ON = 0x00;
    const int switchIndex = 6;
 
    QByteArray readBytes = this->read_bytes(dio_n); //readBytes remains ""
 
    if(readBytes.size() > switchIndex && readBytes[switchIndex] == ON)
        isOn = true;
    else
        isOn = false;
 
    return isOn;
}

QByteArray DIOMoxaBoard::read_bytes(quint16 dio_n)
{
    /* *******************************
       * 1st byte: Command number, 1
       * 2nd byte: Version, 2
       * 3rd byte: Response only, any
       * 4th byte: Data length, 1
       * 5th byte: DIO channel
       ********************************/

    QByteArray bytes;

    bytes.push_back(1);
    bytes.push_back(2);
    bytes.push_back(_NULL);
    bytes.push_back(1);
    bytes.push_back(dio_n);

    client.write(bytes); // successfully writes 5 bytes if it doesn't access isOn()

    return client.read(7).toHex(); //successfully reads 7 bytes if it doesn't access isOn()
}

bool DIOMoxaBoard::isOn(quint16 dio_n)
{
    bool isOn = false;
    const char ON = 0x00;
    const int switchIndex = 6;

    QByteArray readBytes = this->read_bytes(dio_n); //readBytes remains ""

    if(readBytes.size() > switchIndex && readBytes[switchIndex] == ON)
        isOn = true;
    else
        isOn = false;

    return isOn;
}

To copy to clipboard, switch view to plain text mode 

Any idea how to avoid sending 20 bytes due to calling isOn() four times for each checkbox (I've tried pause/sleep already), as well as being able to pass on the buffer values to the new variable readBytes, instead of having it remain as an empty string?


Thanks in advance!

[gouyoku]

Hi

I am no expert but here are some of my thoughts:
You shouldn't use client.read() right after client.write(). Getting data over the network is asynchronous so you need to give it time to arrive.
If I were to guess, you are using QAbstractSocket as your client. If so, you should use its signal readyRead() to know when to read the answer.
This would explain why your program prints out your string only after you click a checkbox. At first pass of read_bytes() you don't have anything to read() as the answer have not arrived yet.
To set your checkboxes you will need to implement some queue, for example: isOn(0); readyRead()->setchecked()->isOn(1); readyRead()->setChecked()->isOn(2); and so on.

Hope it helps.

[Lesiok]

In line 21 You are converting binary data to hex (that is to say hexadecimal character notation). So method read_bytes() returns more when 2 x 7 bytes. Next in line 32 You compare this character to binary value ON. This have no sense.