Skip to content
  • Categories
  • Recent
  • Tags
  • Popular
  • Users
  • Groups
  • Search
  • Get Qt Extensions
  • Unsolved
Collapse
Brand Logo
  1. Home
  2. Qt Development
  3. General and Desktop
  4. problem with writing on serial port
Qt 6.11 is out! See what's new in the release blog

problem with writing on serial port

Scheduled Pinned Locked Moved Unsolved General and Desktop
36 Posts 4 Posters 5.4k Views
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • N nanor

    @JonB I removed the while loop and mythread.cpp is now:

    #include "mythread.h"
    
    MyThread::MyThread(QObject *parent) : QThread(parent)
    {
    
    }
    
    void MyThread::run()
    {
        serial = new QSerialPort();
        serial->setPortName("COM1");
    
        for(const auto &serialPortInfo : QSerialPortInfo::availablePorts())
        {
           qDebug() << "find serial port: " << serialPortInfo.portName() ;
        }
    
        serial->open(QIODevice::ReadWrite);
    
    
        if(serial->isOpen())
        {
            serial->setBaudRate(QSerialPort::Baud9600);
            serial->setDataBits(QSerialPort::Data8);
            serial->setParity(QSerialPort::NoParity);
            serial->setStopBits(QSerialPort::OneStop);
            serial->setFlowControl(QSerialPort::NoFlowControl);
    
    
    
            QByteArray data = serial->read(1);
            for(int i=0; i<data.size(); i++)
             {
                    qDebug() << data[i];
             }
    
    
    
    
        }
    
        else
        {
          qDebug() << "can't open the port";
        }
    
    }
    
    

    But still I can't get the messages. I send data from hercules.

    JonBJ Offline
    JonBJ Offline
    JonB
    wrote on last edited by JonB
    #26

    @nanor
    Start by putting in qDebug() statements to see for yourself where you are actually getting to, plus more error checking.

    I don't know if any serial data is available to read at the instant you open the port. And you only try to read one byte once.

    N 1 Reply Last reply
    0
    • JonBJ JonB

      @nanor
      Start by putting in qDebug() statements to see for yourself where you are actually getting to, plus more error checking.

      I don't know if any serial data is available to read at the instant you open the port. And you only try to read one byte once.

      N Offline
      N Offline
      nanor
      wrote on last edited by
      #27

      @JonB Thank you for mentioning using debug. I added the line qDebug() << "inside the run function"; inside the run function, but after running, this message isn't shown. But I have started the thread inside the mainwindow class. I can't understand why this is happened!

      JonBJ jsulmJ 2 Replies Last reply
      0
      • N nanor

        @JonB Thank you for mentioning using debug. I added the line qDebug() << "inside the run function"; inside the run function, but after running, this message isn't shown. But I have started the thread inside the mainwindow class. I can't understand why this is happened!

        JonBJ Offline
        JonBJ Offline
        JonB
        wrote on last edited by
        #28

        @nanor
        My own view, often times expressed, is why are you going anywhere near QThread at all? Every time it causes posters problems. Qt framework is asynchronous, so why do you need any threads when you have signals?

        jsulmJ 1 Reply Last reply
        0
        • N nanor

          @JonB Thank you for mentioning using debug. I added the line qDebug() << "inside the run function"; inside the run function, but after running, this message isn't shown. But I have started the thread inside the mainwindow class. I can't understand why this is happened!

          jsulmJ Offline
          jsulmJ Offline
          jsulm
          Lifetime Qt Champion
          wrote on last edited by
          #29

          @nanor I gave you a link to an example which uses the synchronous API. Apparently you did not really read it carefully, did you? In your loop you should use waitForReadyRead(...) as is done in the example. This makes sure that the event loop can actually do its job.

          if (serial.waitForReadyRead(currentWaitTimeout)) {
              QByteArray responseData = serial.readAll();
          

          https://forum.qt.io/topic/113070/qt-code-of-conduct

          1 Reply Last reply
          2
          • JonBJ JonB

            @nanor
            My own view, often times expressed, is why are you going anywhere near QThread at all? Every time it causes posters problems. Qt framework is asynchronous, so why do you need any threads when you have signals?

            jsulmJ Offline
            jsulmJ Offline
            jsulm
            Lifetime Qt Champion
            wrote on last edited by
            #30

            @JonB I asked already why threads. Seems to be an exercise.

            https://forum.qt.io/topic/113070/qt-code-of-conduct

            J.HilkJ 1 Reply Last reply
            1
            • jsulmJ jsulm

              @JonB I asked already why threads. Seems to be an exercise.

              J.HilkJ Offline
              J.HilkJ Offline
              J.Hilk
              Moderators
              wrote on last edited by
              #31

              @jsulm said in problem with writing on serial port:

              I asked already why threads. Seems to be an exercise.

              possibly, but the OP only states the while loop as a task. Not threads specifically.

              @nanor can you word exactly, what you want to do ?


              Be aware of the Qt Code of Conduct, when posting : https://forum.qt.io/topic/113070/qt-code-of-conduct


              Q: What's that?
              A: It's blue light.
              Q: What does it do?
              A: It turns blue.

              N 1 Reply Last reply
              1
              • J.HilkJ J.Hilk

                @jsulm said in problem with writing on serial port:

                I asked already why threads. Seems to be an exercise.

                possibly, but the OP only states the while loop as a task. Not threads specifically.

                @nanor can you word exactly, what you want to do ?

                N Offline
                N Offline
                nanor
                wrote on last edited by nanor
                #32

                @J-Hilk Hi
                I have a button that when a user pushes it, "salam" is printed. At the same time, I want to constantly read from and write to a port (I want to read byte by byte and print them(debug them)). Now I know that I have to declare a class inheritted from qThread class and inside its run function, read and write. But I don't know if I have to declare while loop inside the run function in order to read constantly or not. In the code I posted, I have:

                
                        QByteArray data = serial->read(1);
                        for(int i=0; i<data.size(); i++)
                        {
                                qDebug() << data[i];
                        }
                

                Is this part reading the datas byte by byte and store them inside an array and print the datas?

                JonBJ J.HilkJ 2 Replies Last reply
                0
                • N nanor

                  @J-Hilk Hi
                  I have a button that when a user pushes it, "salam" is printed. At the same time, I want to constantly read from and write to a port (I want to read byte by byte and print them(debug them)). Now I know that I have to declare a class inheritted from qThread class and inside its run function, read and write. But I don't know if I have to declare while loop inside the run function in order to read constantly or not. In the code I posted, I have:

                  
                          QByteArray data = serial->read(1);
                          for(int i=0; i<data.size(); i++)
                          {
                                  qDebug() << data[i];
                          }
                  

                  Is this part reading the datas byte by byte and store them inside an array and print the datas?

                  JonBJ Offline
                  JonBJ Offline
                  JonB
                  wrote on last edited by JonB
                  #33

                  @nanor said in problem with writing on serial port:

                  Now I know that I have to declare a class inheritted from qThread class and inside its run function,

                  Why? Is this a required exercise to use a thread, or are you just doing it because you think you need to??

                  Oh, now I see

                  @jsulm I was asked to do this. I was asked to send a text in while(1) loop.

                  EDIT OK, I have seen now you have answered that you must do it this way. Seems terrible to me! But OK :)

                  N 1 Reply Last reply
                  0
                  • JonBJ JonB

                    @nanor said in problem with writing on serial port:

                    Now I know that I have to declare a class inheritted from qThread class and inside its run function,

                    Why? Is this a required exercise to use a thread, or are you just doing it because you think you need to??

                    Oh, now I see

                    @jsulm I was asked to do this. I was asked to send a text in while(1) loop.

                    EDIT OK, I have seen now you have answered that you must do it this way. Seems terrible to me! But OK :)

                    N Offline
                    N Offline
                    nanor
                    wrote on last edited by
                    #34

                    @JonB This is a required exersize

                    1 Reply Last reply
                    1
                    • N nanor

                      @J-Hilk Hi
                      I have a button that when a user pushes it, "salam" is printed. At the same time, I want to constantly read from and write to a port (I want to read byte by byte and print them(debug them)). Now I know that I have to declare a class inheritted from qThread class and inside its run function, read and write. But I don't know if I have to declare while loop inside the run function in order to read constantly or not. In the code I posted, I have:

                      
                              QByteArray data = serial->read(1);
                              for(int i=0; i<data.size(); i++)
                              {
                                      qDebug() << data[i];
                              }
                      

                      Is this part reading the datas byte by byte and store them inside an array and print the datas?

                      J.HilkJ Offline
                      J.HilkJ Offline
                      J.Hilk
                      Moderators
                      wrote on last edited by
                      #35

                      @nanor said in problem with writing on serial port:

                      I want to constantly read from and write to a port (

                      is it specified how you have to constantly read from and write to a port or is that implementation detail up to you?

                      because while- loops and threads I the least optimal way to do it


                      Be aware of the Qt Code of Conduct, when posting : https://forum.qt.io/topic/113070/qt-code-of-conduct


                      Q: What's that?
                      A: It's blue light.
                      Q: What does it do?
                      A: It turns blue.

                      N 1 Reply Last reply
                      0
                      • J.HilkJ J.Hilk

                        @nanor said in problem with writing on serial port:

                        I want to constantly read from and write to a port (

                        is it specified how you have to constantly read from and write to a port or is that implementation detail up to you?

                        because while- loops and threads I the least optimal way to do it

                        N Offline
                        N Offline
                        nanor
                        wrote on last edited by
                        #36

                        @J-Hilk It is specified to use while(1) loop

                        1 Reply Last reply
                        0

                        • Login

                        • Login or register to search.
                        • First post
                          Last post
                        0
                        • Categories
                        • Recent
                        • Tags
                        • Popular
                        • Users
                        • Groups
                        • Search
                        • Get Qt Extensions
                        • Unsolved