C / C++ / MFC

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

9-Oct-08 19:54

Thanks for help.It's working

Total days in a year !

Le@rner6-Oct-08 23:26

Le@rner

6-Oct-08 23:26

Hi all,

PLease tell me how can i get Total days of a Year.(wheter 365 or 366)

thanks in advance.

IN A DAY, WHEN YOU DON'T COME ACROSS ANY PROBLEMS - YOU CAN BE SURE THAT YOU ARE TRAVELLING IN A WRONG PATH

Roger Stoltz6-Oct-08 23:31

Roger Stoltz

6-Oct-08 23:31

If the year is a multiple of 4, it's a leap year, i.e. there are 366 days.

Exception #1:
If the year is dividable by 100 it's not a leap year.

Exception #2:
If the year is dividable by 400 it's a leap year anyway.

Hint: use the modulus operator.

"It's supposed to be hard, otherwise anybody could do it!" - selfquote
"High speed never compensates for wrong direction!" - unknown

Le@rner6-Oct-08 23:32

Le@rner

6-Oct-08 23:32

Thanks

IN A DAY, WHEN YOU DON'T COME ACROSS ANY PROBLEMS - YOU CAN BE SURE THAT YOU ARE TRAVELLING IN A WRONG PATH

PravinSingh7-Oct-08 0:41

PravinSingh

7-Oct-08 0:41

Wow, I didn't know about exception #2, I always thought scientists decide after a few centuries whether this century year will be a leap year or not. Cool | :cool:

Thanks.

It's better to know some of the questions than all of the answers.

Pravin.

Michael Schubert6-Oct-08 23:36

6-Oct-08 23:36

You could use COleDateTime() or COleDateTimeSpan()

SandipG 6-Oct-08 23:37

SandipG

6-Oct-08 23:37

Check this thread Leap Year[^]

Regards,
Sandip.

Michael Schubert7-Oct-08 1:11

7-Oct-08 1:11

bool isLeapYear(int year )
{
  return  year % 400 ? year % 100 ? year % 4 ? false : true : false : true;
}

There should be a forum for really cool code snippets like this! Nicely done Carlo!

SandipG 7-Oct-08 1:14

SandipG

7-Oct-08 1:14

Michael Schubert wrote:
There should be a forum for really cool code snippets like this! Nicely done Carlo!
Quote Selected Text

No worries!! We have the magic site of carlo.. Big Grin | :-D

Regards,
Sandip.

Michael Schubert7-Oct-08 1:14

7-Oct-08 1:14

I'm serious!