C / C++ / MFC

Richard MacCutchan8-Jul-18 7:23

8-Jul-18 7:23

In case 1 you have not shown the value of pData. In case 2 you have explicitly set its value to NULL.

_Flaviu8-Jul-18 19:57

_Flaviu

8-Jul-18 19:57

In case 1, the trace show me that pData is not NULL, and after GetData call, I set up nothing, I only display if pData is NULL or not ... but is NULL ...

Richard MacCutchan9-Jul-18 3:01

9-Jul-18 3:01

Your test does not show what you think. It shows the values that you set in pData before the call to GetData, which does not change it. You need to reread the documentation, and understand the difference between a pointer and a reference.

CPallini8-Jul-18 20:59

CPallini

8-Jul-18 20:59

Because the CData pointer is passed by value. If you need to modify the pointer then you have to pass it by reference (or use a double pointer).

MFC Static Library Project

_Flaviu9-Jul-18 5:45

_Flaviu

9-Jul-18 5:45

I used double pointer and goes perfectly. Kindly thank you all of you (including Richard)!

Richard Andrew x647-Jul-18 16:04

Richard Andrew x64

7-Jul-18 16:04

In VS 2017, is it possible to create a static library project that exports classes the same way an MFC Extension DLL does?

I know I can create a regular static library project, but how would I make it possible to use MFC classes like CString inside that project?

The difficult we do right away...
...the impossible takes slightly longer.

Re: MFC Static Library Project

Victor Nijegorodov7-Jul-18 21:48

Victor Nijegorodov

7-Jul-18 21:48

Yes. it is possible. See also mfc as static library: is option gone? - Developer Community
What problems do you have exactly?

Re: MFC Static Library Project

Richard Andrew x648-Jul-18 3:26

Richard Andrew x64

8-Jul-18 3:26

Thank you for your response!

I just figured out how to do it.

First, create a static library project, then include all of the MFC headers in the stdafx.h file.

Thanks again. Smile | :)

The difficult we do right away...
...the impossible takes slightly longer.

Using fstream with USB Serial Port

OscardelaGrouch6-Jul-18 20:53

6-Jul-18 20:53

Hello everyone,
I'm kinda new to writing application programs for a PC, a Linux one at that, so I hope this isn't a stupid question. Wink | ;)

I am rather experienced at writing C programs for Microchip PICs. I need a PC application to interface with a PIC project, it will up load information to the PIC from the PC using a pair of Wixel modules. They are basically a USB to RF or RF to Serial bridge. I have them working (one connected to the PC, one connected to the PIC) using a terminal program. I can send data back and forth just fine.
I am now trying to send and receive data using a C++ application. The sending from the PC to the PIC works just fine also. The problem I am having is getting data from the PIC to the PC Application.

This is the application code ..

#include <cstdlib>
#include <iostream>
#include <fstream>
#include <sstream>

using namespace std;

/*
 * 
 */
int main(int argc, char** argv)
{
    fstream COMPORT;

    char line[256];
    streamsize lim = 256;
    char c;

    COMPORT.open("/dev/ttyACM0");
    
    while(1)    // only works for first input. run again and missed one reply
    {
        if (COMPORT.is_open())
        {
            cout << "Hit <Enter> to send command, <S> to Stop .." << endl;
            cin.get(c);
            if ((c == 's' || c == 'S')) 
            {
                COMPORT.close();
                return 0;
            }
            COMPORT << "UPLOAD NEW MAP\n";
            COMPORT.getline(line, lim);
            cout << "Line =" << line;
            cout << "Response=" << c;
            cout << endl;
        }
    }
    COMPORT.close();
    return 0;
}

The PIC code, if that helps ..

void APP_Wixel_IO_Test_Tasks(void)
{
    APP_ESUART_Tasks();
    if (CmdReady == 1)
    {
        CmdReady = 0;                               // reset flag
        RXBufCnt = 0;                     // reset RX Buffer count command
        utoa(TXBuf, Cnt, 10);
        strcat(TXBuf, "\n");
        WriteESUARTDataMsg(TXBuf, strlen(RXBuf));   // send command back to ESUART
        Cnt++;                                      // increment cnt
    }
}

Basically all the PIC code does is send back the 'cnt' variable when it gets data from the PC.
The

COMPORT.getline(line, lim);

doesn't work right. The first time I run it, it kinda works, but resending the data back to the PIC gets no response ..

Hit <Enter> to send command, <S> to Stop ..

Response=4
Hit <Enter> to send command, <S> to Stop ..

Response=

Hit <Enter> to send command, <S> to Stop ..

Response=

Hit <Enter> to send command, <S> to Stop ..

Response=

Hit <Enter> to send command, <S> to Stop ..

I've tried everything I can think of. Like I said, data out

COMPORT << "UPLOAD NEW MAP\n";

works fine. Data in does not.
Thanks Wink | ;)

OtG

Richard MacCutchan6-Jul-18 23:25

6-Jul-18 23:25

C++

WriteESUARTDataMsg(TXBuf, strlen(RXBuf));

Should that not be strlen(TXBuf) ?

OscardelaGrouch7-Jul-18 6:39

7-Jul-18 6:39

Yes, it should. I changed that but I get the same response from the application ..

Response=�
Hit <Enter> to send command, <S> to Stop ..

Response=

Hit <Enter> to send command, <S> to Stop ..

Response=

Hit <Enter> to send command, <S> to Stop ..

Response=

Hit <Enter> to send command, <S> to Stop ..
^C
RUN FINISHED; Interrupt; real time: 11s; user: 0ms; system: 0ms

I am pretty sure the PIC is doing it's job and replying to the message from the application (PC) correctly.
Thanks for spotting that error.
OdlG

Richard MacCutchan7-Jul-18 6:52

7-Jul-18 6:52

I have never tried to read a serial port via a stream class. Serial ports tend to be slow, and are really character oriented. You could try just using a basic file handle and reading as many characters as you can, gradually filling a buffer until you see a newline character. I know the ifstream should work that way but ...

OscardelaGrouch7-Jul-18 6:55

7-Jul-18 6:55

Thanks for the reply. I did try using get(c); but that didn't seem to work either. I will look at that some more.
OdlG

OscardelaGrouch7-Jul-18 8:21

7-Jul-18 8:21

Any advise on how to do that?
Thanks.

Richard MacCutchan wrote:
You could try just using a basic file handle and reading as many characters as you can, gradually filling a buffer until you see a newline character

mo14927-Jul-18 12:59

7-Jul-18 12:59

I am not familiar with Linux but I have done some programming with usb to rs232 dongles in Windows and your symptom is familiar. For what it's worth, are you sure your serial port parameters match?
If not, you may get partial data and the port on the computer can error. Can you check for error
conditions like parity error? I do not know if you are sending a lot of data, but if you are,
you will have to consider serial handshaking as well or you will get buffer overruns.

OscardelaGrouch7-Jul-18 13:18

7-Jul-18 13:18

speedbump99 wrote:
For what it's worth, are you sure your serial port parameters match?

Yes. I can send data back and forth with a terminal program.
I have modified the programs like this ..

void APP_Wixel_IO_Test_Tasks(void)
{
    APP_ESUART_Tasks();
    if (CmdReady == 1)
    {
        CmdReady = 0;                               // reset flag
        RXBufCnt = 0;                               // reset RX Buffer count
        utoa(TXBuf, Cnt, 10);                       // convert cnt into a string
        strcat(TXBuf, "\n");                        // add NULL to end of string
        WriteESUARTDataMsg(TXBuf, strlen(TXBuf));   // send command back to ESUART
        Cnt++;                                      // increment cnt
    }
}

and this ..

int main(int argc, char** argv)
{
    fstream COMPORT;
    int number;
    int x = 0;
    char c;

    while(1)    // only works for first input. run again and missed one reply
    {
        COMPORT.open("/dev/ttyACM0");
        if (COMPORT.is_open())
        {
            cout << "Hit <Enter> to send command, <S> to Stop .." << endl;
            cin.get(c);
            if ((c == 's' || c == 'S')) 
            {
                COMPORT.close();
                return 0;
            }
            COMPORT << "UPLOAD NEW MAP\n";
            COMPORT >> number;
            cout << "number = " << number;
            cout << endl;
        }
        COMPORT.close();
    }
    return 0;
}

I get this response ..

Hit <Enter> to send command, <S> to Stop ..

number = 0
Hit <Enter> to send command, <S> to Stop ..

number = 2
Hit <Enter> to send command, <S> to Stop ..

number = 4
Hit <Enter> to send command, <S> to Stop ..

number = 6
Hit <Enter> to send command, <S> to Stop ..

number = 8
Hit <Enter> to send command, <S> to Stop ..
s

I cannot figure why the cnt goes to two times instead of one.
I would like to not be limited to just sending numbers back from the PIC.
I think I am doing things "pretty' much correctly, but??
Any thoughts?
OdlG

mo14927-Jul-18 14:51

7-Jul-18 14:51

If it works in a terminal program I really don't have any help for you except this.
I don't know if it is this issue but these lines could be a potential problem
depending on how COMPORT functions:

COMPORT << "UPLOAD NEW MAP\n";
COMPORT >> number;

There might be a timing issue if all data has not been received from the pic when
COMPORT >> number; is called. As a test you might want to put a delay in to give it time.

COMPORT << "UPLOAD NEW MAP\n";
delay?
COMPORT >> number;

OscardelaGrouch7-Jul-18 15:46

7-Jul-18 15:46

Thanks for the reply and suggestion.
I think that a delay might help if I am trying to get a string from the PIC to the PC app.
I would HOPE that the Getline function would do the waiting, but ??

speedbump99 wrote:
As a test you might want to put a delay in to give it time.

I've thought about trying that, I don't see a delay function in C++ though. Is there one?
In PIC C there is a __delay_us(10); function that would delay, in this case 10 microseconds based on the current clock speed.

Seems that C++ ought to have one. Maybe have to just do a loop?
OdlG

mo14927-Jul-18 16:10

7-Jul-18 16:10

I kind of doubt that getline will wait for data as it has no way of knowing how
much data to expect.

A quick search for Linux and Delay turned up a sleep() command that may be what you need.

Also a search of "Linux synchronous serial port PIC" showed several of examples where they went through the process of setting the parameters for the serial port and running a loop to retrieve data.

Something like (pseudo code):
while (1)
{
read(1 char);
if char == "\n"
break;
}
You would probably read the text and convert back to integer.

You might try that search.
Hope this helps.

OscardelaGrouch8-Jul-18 8:08

8-Jul-18 8:08

Thanks for the reply,

speedbump99 wrote:
I kind of doubt that getline will wait for data as it has no way of knowing how
much data to expect.

I think it will. In fact there are several versions of the getline function. In one you can specify a char array, max number of chars and a delimiter char. The default delim is '\n'.

I have tried doing it char by char and didn't have any better results. I may try that again as it seems the operator >> only inputs numbers (of various types) and not strings. I think getline must be used for that.

For right now I'm going to get my scope out and see why my reply jumps by two's instead of one with each command sent to the PIC.

I will keep this thread updated.
Thanks all,
OdlG

mo14928-Jul-18 11:55