C / C++ / MFC

Re: problem with using program Creating a Serial communication on Win32

12-Aug-13 1:19

dear eng:Jochen Arndt

i am trying your idea now

but the problem still happen

would you please give me hand ?

Jochen Arndt16-Aug-13 2:52

: problem with using program Creating a Serial communication on Win32

16-Aug-13 2:52

Sorry for my late reply.

I can't help without seeing your code. I suggest to write a new question showing the relevant code part and explaining the problem. So others may help also.

sindbad116-Sep-13 23:29

Re: : problem with using program Creating a Serial communication on Win32

6-Sep-13 23:29

i am trying to cut string from a pattern like this

@) 1450kg @) 1450kg @) 1450kg

i m trying to cut from the 4th chatacter to the 10th chatacter

var mess contain the chatacter string

i enter the code like this

char mess[MAX_MESSAGE];
unsigned int lenBuff = MAX_MESSAGE;
unsigned long lenMessage;
static CString outPut;
outPut =mess ;
std::string str1 ;
str1=outPut ;

str1 = str1.substr (4,10); /////
CString cs(str1.c_str());
outPut=cs ;

strcpy(mess, outPut);

is this code part is true ???????????????

Jochen Arndt6-Sep-13 23:45

Re: : problem with using program Creating a Serial communication on Win32

6-Sep-13 23:45

Why do you copy the data using so many vars? You should really learn about string operations in C/C++.

The CString class has a constructor that accepts a pointer to a char* string and a length. So you can simply copy the required number of characters:

C++

CString strCut(mess + 3 , 6);

This will copy 6 characters from offset 3 of mess to strCut. Note that offset 3 points to the 4th character.

sindbad117-Sep-13 0:08

Re: : problem with using program Creating a Serial communication on Win32

7-Sep-13 0:08

thanks for reply eng.Jochen Arndt

in this example if i want to cut from 4th character to the kg position which can be dynamic

@) 1450kg @) 1450kg @) 1450kg

how can i get the position of kg

i use this

outPut =mess ; //////
//------------------
std::string str1 ;
str1=outPut ;
unsigned pos = str1.find("KG");

str1 = str1.substr (4,pos - 1);

CString cs(str1.c_str());
outPut=cs ;

strcpy(mess, outPut);

is this code is true

Jochen Arndt7-Sep-13 0:28

Re: : problem with using program Creating a Serial communication on Win32

7-Sep-13 0:28

Still wrong. As I told you, indexes are zero-based. So you must use 3 as start index. To get the length, subtract the start position from the position of 'kg'. Also there is no need to mix CString and std::string:

C++

CString cs(mess); // copy mess to CString object
int nStart = 3; // start at 4th character
int nStop = cs.Find("kg"); // get position of 'kg'
cs = cs.Mid(nStart, nEnd - nStart); // extract the weight value

If you want to get numeric values from strings, use the atoi() function:

C++

int nWeight = atoi(mess + 3);

The atoi() function will parse the string stopping at the first non-digit char and return the converted value.

sindbad117-Sep-13 0:56

Re: : problem with using program Creating a Serial communication on Win32

7-Sep-13 0:56

i tryed this code

char mess[MAX_MESSAGE];
unsigned int lenBuff = MAX_MESSAGE;
unsigned long lenMessage;
static CString outPut;

CString cs(mess); // copy mess to CString object
int nStart = 3; // start at 4th character
int nStop = cs.Find("kg"); // get position of 'kg'
cs = cs.Mid(nStart, nStop - nStart);
//////--------------------------------------------

outPut=cs ;

strcpy(mess, outPut); /////

i finally put the outPut string in the mess character

is this code true

Jochen Arndt7-Sep-13 2:31

PlaySoundA does not return within injected DLL

7-Sep-13 2:31

Again: Why did you use the std::string outPut object?
You can simply copy the extracted sub-string from the CString to mess.

Overall, I can't say if the code is 'true'. You must know what should be done. Check the extracted value using a debugger or printing it out. If it is what you want, you are finished. Otherwise change your code until you get what you want.

Coding guidelines C

topcatalpha27-May-13 2:56

topcatalpha

27-May-13 2:56

Hi,

I like to gather some coding guideline tools for C. More specific for integration into a CI system showing us some statistics of the code quality.

Can anyone post some suggestions, eventually with feedback or experience about the tool?

regards.

T.

Re: Coding guidelines C

Maximilien27-May-13 7:08

Maximilien

27-May-13 7:08

topcatalpha wrote:
More specific for integration into a CI system

What's a CI system ?

Anyway, coding guidelines are quite subjective, google for a few of them and pick whatever looks nice and applicable to your organization (or yourself).

For example, the IBM[^] one looks good.

Nihil obstat

Re: Coding guidelines C

topcatalpha28-May-13 20:41

topcatalpha

28-May-13 20:41

CI = Continious Integration.

Well we reviewed some tools but it would be nice if we had some feedback from users who are actually using them.

thx,

K

Furblog Sidloth26-May-13 0:42

Furblog Sidloth

26-May-13 0:42

Hi, i have the following problem:

I have a process which uses PlaySoundA[^] WINAPI function to play a specific sound (let's be it "c:\test.wav"). So the process has already loaded the needed "winmm.dll"

I want to play the same sound when the DLL is injected into the process, so the code looks like:

C++

#define WINVER 0x600
#define _WIN32_WINDOWS 0x600
#define _WIN32_WINNT 0x600

#include <windows.h>

typedef bool __stdcall (*psfunc)(LPCTSTR pszSound, HMODULE hmod, DWORD fdwSound);

void play() {
    psfunc ps;
    HMODULE winmm = GetModuleHandle("winmm");;
    ps = (psfunc)GetProcAddress(winmm, "PlaySoundA");
    ps("c:\\test.wav", 0, SND_FILENAME | SND_ASYNC);
}

BOOL __stdcall DllMain (HANDLE hDll, DWORD dwReason, LPVOID lpReserved)
{
    switch (dwReason)
    {
        case DLL_PROCESS_ATTACH:
            OutputDebugString("play before");
            play();
            OutputDebugString("play after");
            break;

        case DLL_PROCESS_DETACH:
            break;

        case DLL_THREAD_ATTACH:
            break;

        case DLL_THREAD_DETACH:
            break;
    }
    return TRUE;
}

The dll is injected correctly via CreateRemoteThread & LoadLibrary.
But the sound is never played and the dll is blocking at the call of

C++

ps("c:\\test.wav", 0, SND_FILENAME | SND_ASYNC);

Trying to exit the process freezes too because of the blocking thread.
Why does it never return? Procss Monitor doesn't show any error.
How can I solve this?

Richard MacCutchan26-May-13 0:54

26-May-13 0:54

~~Are you certain that GetProcAddress is returning a valid handle?~~
See the warning at http://msdn.microsoft.com/en-us/library/windows/desktop/ms682583(v=vs.85).aspx[^]. I do not think you should be calling your play function inside DllMain.

Use the best guess

modified 26-May-13 7:20am.

Furblog Sidloth26-May-13 20:39

Furblog Sidloth

26-May-13 20:39

Oh, I didn't see that.
So I called the function via CreateRemoteThread too and it's working. Thank you!
It would be nice to know what things are allowed/disallowed in DllMain.

Richard MacCutchan26-May-13 21:38

26-May-13 21:38

Furblog Sidloth wrote:
It would be nice to know what things are allowed/disallowed in DllMain.

I agree, the documentation could be better. However, my understanding is that you should not do anything more than initialise the odd static variable or class, and save a handle to the DLL itself. Anything that takes you out of DllMain is not allowed and can lead to problems - as you already discovered.

Use the best guess

Erudite_Eric27-May-13 5:48

Erudite_Eric

27-May-13 5:48

Not only that he has no error checking....

==============================

Nothing to say.

Richard MacCutchan27-May-13 5:53

27-May-13 5:53

Seems to be fairly standard these days.

Use the best guess

Erudite_Eric27-May-13 10:54

Erudite_Eric

27-May-13 10:54

Exception handling will clean up the crap.....

==============================

Nothing to say.

Richard MacCutchan27-May-13 20:59

Modeless Dialog CricheditCtrl Issues

27-May-13 20:59

I sometimes wonder if some of the course being taught these days even mention error checking.

Use the best guess

ForNow25-May-13 19:34

ForNow

25-May-13 19:34

Hi,

I am having a number of problems Getting my richedit to work with my Modless Dialog Box

it started with scrolling as I was unable to get the Current Line and then I while trying to debug it

I noticed the I was I using DDX to connect my crichedit class with the resource defined window

which the documentation says only works for modal however when I comment out the UpdateData

and used the CRichEdtCtrl::Create to connect my windows class to the resource window none the Streamed in data is displayed I'll display the code along with the resource statements

The Create of the modeless Dialog

C++

ret =  progdbg->Create(IDD_PROGDBG,(CWnd *)this);

C++

BOOL CprogDebug::OnInitDialog()
{
  int FirstVisible; 
  long start, end, theline;
  myedit = new CProgedit;       // allocate the richedit
		CRect myrect(17,39,735,273);
		myedit->SetOptions(ECOOP_OR, ECO_AUTOVSCROLL |  ECO_AUTOHSCROLL);
  //  myedit->Create(WS_CHILD| ES_AUTOHSCROLL | ES_AUTOVSCROLL | ES_MULTILINE,myrect,this,IDC_RICHEDIT22);
	 UpdateData(FALSE);  // connect to dilog object
         EDITSTREAM es;
           

   // Set the ENM_REQUESTRESIZE event flag
 //  myedit->SetEventMask( ENM_REQUESTRESIZE ); // set notificate to keep number of charcters in view

   // Force the control to issue a EN_REQUESTRESIZE notification
 //  myedit->RequestResize( );


   CStdioFile *fileptr = new CStdioFile;
   fileptr->Open(proglisting,CFile::typeBinary);
           es.dwCookie = (DWORD) fileptr;
        es.dwError = 0;
        es.pfnCallback = (EDITSTREAMCALLBACK) readlisting;
         myedit->StreamIn(SF_TEXT, es);
		  myedit->SetFocus();
	//	 myedit->GetSel(start,end);
		  theline = myedit->LineFromChar(myedit->LineIndex(-1));
	//	 FirstVisible = myedit->GetLineCount();
		 
//		FirstVisible = myedit->LineIndex(-1);
		 myedit->LineScroll(-theline,0);
		 myedit->HideSelection(TRUE,FALSE);
			        // Call MFC Oninit
          ShowWindow(SW_SHOW);
      //  progeditptr = new CProgedit();
		CDialog::OnInitDialog(); 	
        return TRUE;
}

C++

void CprogDebug::DoDataExchange(CDataExchange* pDX)
 {

//
//        //should precede the DDX and DDV routines
   CDialog::DoDataExchange(pDX);
        // Attach
        DDX_Control(pDX,IDC_RICHEDIT22,(CWnd &)*myedit); // connect rich edit
 //       DDX_Control(pDX,IDC_DESCRIP,descrip);

 //  DDX_Text(pDX, (int) IDC_DESCRIP, mytcpipmess);

 }

Richard MacCutchan26-May-13 0:45

26-May-13 0:45

This is not the correct way to use a dialog; switch it to a modal type.

Use the best guess

ForNow26-May-13 2:29

ForNow

26-May-13 2:29

meaning I can't use I cant use RichEdit with modeless

Richard MacCutchan26-May-13 2:32

26-May-13 2:32

Why would you want to? It offers no advantages and just makes your code far more complex than it needs to be. Use dialogs the way they are meant to be used and your life will be so much easier.

Use the best guess