C / C++ / MFC

phanindra varma18-Oct-08 4:04

phanindra varma

18-Oct-08 4:04

O.k Thanks for the Suggestion !!!!!

vijay_aroli7-Oct-08 6:26

vijay_aroli

7-Oct-08 6:26

In addition to what Rajesh said, I would also advice you not to use SMS text while posting messages here. Smile | :)

Regards,
Vijay.

David Crow7-Oct-08 3:36

David Crow

7-Oct-08 3:36

phanindra varma wrote:
It wasa giving an Error.

Turn your monitor around so we can see it. Roll eyes | :rolleyes:

"Love people and use things, not love things and use people." - Unknown

"The brick walls are there for a reason...to stop the people who don't want it badly enough." - Randy Pausch

Michael Schubert7-Oct-08 3:49

7-Oct-08 3:49

Rajesh R Subramanian7-Oct-08 5:10

Rajesh R Subramanian

7-Oct-08 5:10

How much I love your comments.

Many are stubborn in pursuit of the path they have chosen, few in pursuit of the goal - Friedrich Nietzsche

.·´¯`·->Rajesh<-·´¯`·.
[Microsoft MVP - Visual C++]

ThatsAlok7-Oct-08 19:27

ThatsAlok

7-Oct-08 19:27

Rajesh R Subramanian wrote:
How much I love your comments.

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow
Never mind - my own stupidity is the source of every "problem" - Mixture

cheers,
Alok Gupta
VC Forum Q&A :- I/IV
Support CRY- Child Relief and You/xml>

Hamid_RT7-Oct-08 5:13

Hamid_RT

7-Oct-08 5:13

You have powrful eyes. Big Grin | :-D

Timer Problem

MsmVc7-Oct-08 0:11

7-Oct-08 0:11

Hi All

I am useing timer.And that timer i am calling function FolderCopy().Which is working but after 2min code is hange.I havn't figure out what's a exactaly problem.Code is here.OnStartTimer i am calling in

OnInitDialog()

void CAbc::OnStartTimer() 
{

    SetTimer(1,10000, 0);
	
	
}

void CAbc::OnTimer(UINT_PTR nIDEvent)
{	
	FolderCopy();
	
	CDialog::OnTimer(nIDEvent);
}

void CAbc::FolderCopy()
{ 
CString finalString;
		 
		for(int aq=0;aq<20;aq++)
		{
		
	
		finalString=a1[aq];
				
		char * szMyString = (char *)(LPCTSTR)finalString;	
		
		PathRemoveFileSpec(szMyString);
		
		CString string = _T(szMyString);
				
		CString string1=string;
		string1 +='\0';
		int nLen = string1.GetLength();

		LPCSTR lpszBuf = string1.GetBuffer(nLen);

		string1.ReleaseBuffer();
	
		LPCSTR pF=lpszBuf;
		
		LPCSTR pT="c:\\Testing";
	
			
		SHFILEOPSTRUCT sf;
		memset(&sf,0,sizeof(sf));
		sf.hwnd = 0;
		sf.wFunc = FO_COPY;
		sf.pFrom = pF;
		sf.pTo = pT;
		sf.fFlags = FOF_NOCONFIRMATION | FOF_NOCONFIRMMKDIR | FOF_NOERRORUI;
 
		 int n = SHFileOperation(&sf);
		if( n == 0)
		{
			  AfxMessageBox("Success\n");
		 }
		else
		 {
			AfxMessageBox("Failed\n");
		 }

		}
		
		
}

Plz help me Rose | [Rose]

Cedric Moonen7-Oct-08 0:15

Cedric Moonen

7-Oct-08 0:15

The Ontimer function will still be called in the same thread, if the FolderCopy function is a lenghty operation, your GUI will freeze because it is not able to process messages anymore. You should use a worker thread for that task. Take a look at this great article[^].

Cédric Moonen
Software developer

Charting control [v1.5]
OpenGL game tutorial in C++

MsmVc7-Oct-08 0:40

7-Oct-08 0:40

Thank for reply.it's a good artical.Can you give me tips,hoe i use thread for do that task.

Thanks in advance.. Rose | [Rose]

Cedric Moonen7-Oct-08 1:00

Cedric Moonen

7-Oct-08 1:00

What do you mean ? Where are you stuck exactly ?

Cédric Moonen
Software developer

Charting control [v1.5]
OpenGL game tutorial in C++

MsmVc7-Oct-08 1:10

7-Oct-08 1:10

I havn't idea about Thread.So i want to what steps i use.Your artical is good. Plz help me

Michael Schubert7-Oct-08 1:14

7-Oct-08 1:14

Having looked at your previous posts I think you should get familiar with some C/C++ basics before you start with multi-threaded programming.

MsmVc7-Oct-08 1:29

7-Oct-08 1:29

Thanks for suggestion.Can you give me tips for do that task?

Michael Schubert7-Oct-08 1:35

7-Oct-08 1:35

I think this site is a good starting point (frequently recommended by Hamid):
http://www.cplusplus.com[^]
You should also consider getting some books.

Michael Schubert7-Oct-08 0:50

7-Oct-08 0:50

What a mess. Are you trying to copy the folders every 10 seconds?

Victor Nijegorodov7-Oct-08 2:08

Victor Nijegorodov

7-Oct-08 2:08

I agree with Michael Schubert: what you are doing is, at least, very strange.
But if you are sure you do need to periodically copy your folder(s), then call KillTimer before calling FolderCopy and then reset the timer ( call SetTimer) after FolderCopy returns.

David Crow7-Oct-08 3:38

David Crow

7-Oct-08 3:38

Try:

void CAbc::FolderCopy()
{
    if (! m_bBusy) // default m_bBusy to false in the constructor
    {
        m_bBusy = true;
 
        for (...)
        {
        }
 
        m_bBusy = false;
    }
}

"Love people and use things, not love things and use people." - Unknown

"The brick walls are there for a reason...to stop the people who don't want it badly enough." - Randy Pausch

MsmVc8-Oct-08 2:39

8-Oct-08 2:39

Hi All

After read artical i am useing this code.Can any one tell me i am wrong or right.

void CTestingDemoDlg::OnBnClickedButton1()
{
	running = TRUE;
	AfxBeginThread(sun, this);
	AfxBeginThread(sun1, this);
	
		
}

void CTestingDemoDlg::sun()
{
	OnStartTimer();

}
void CTestingDemoDlg::sun1()
{
	AfxMessageBox(_T("Hello"));
}

UINT CTestingDemoDlg::sun(LPVOID p)
{
	CTestingDemoDlg * me = (CTestingDemoDlg *)p;
	me->sun();
     return 0;

}
void CTestingDemoDlg::OnStop()
{
	running = FALSE;
}



UINT CTestingDemoDlg::sun1(LPVOID p)
{
	CTestingDemoDlg * me = (CTestingDemoDlg *)p;
	me->sun1();
     return 0;

}

void CTestingDemoDlg::OnTimer(UINT_PTR nIDEvent)
{
	// TODO: Add your message handler code here and/or call default

	m_test.SetWindowTextW(_T("Sun"));
	CDialog::OnTimer(nIDEvent);
}
void CTestingDemoDlg::OnStartTimer() 
{

    SetTimer(1, 5000, 0);
	
}

Plz help me

David Crow8-Oct-08 3:22

David Crow

8-Oct-08 3:22

MsmVc wrote:
Can any one tell me i am wrong or right.

What's wrong or right is ultimately up to you (since you are the only one that knows if your code produces the desired result). That said, what is your code (not) doing?

"Love people and use things, not love things and use people." - Unknown

"The brick walls are there for a reason...to stop the people who don't want it badly enough." - Randy Pausch

MsmVc9-Oct-08 19:54