Solved

Windows System Command (xcopy) in my c++ code.

Posted on 2009-05-06
21
705 Views
Last Modified: 2013-12-03
Hi,

I am facing problem of using Windows System Command (xcopy) in my c++ code.
Please let me know how to do it. Do I need to create new process?


Do I need to use system() api
Thanks....
0
Comment
Question by:palhade
  • 10
  • 10
21 Comments
 
LVL 40

Expert Comment

by:evilrix
ID: 24312456
Can you post your code please... we're good but no psychic :)
0
 

Author Comment

by:palhade
ID: 24312764
Following is the code i am trying to extecute.
It should override it if file already exist.
String            mExecutable = "xcopy";
String            mSourceFile = "C:\Temp\test.dat";
String            mDestDir = "C:\\";
String            str;
strArgs.format( "%s %s %s", mExecutable.Buffer(),mSourceFile.Buffer(),mDestDir.Buffer() );
system(str.Buffer());
0
 
LVL 6

Expert Comment

by:Mushq
ID: 24312946
>>Do I need to create new process?:
Yes I think you need to create a process there is an API CreateProcess for that. http://msdn.microsoft.com/en-us/library/ms682425.aspx

For simplicity you can use WinExec . Please check that.
http://msdn.microsoft.com/en-us/library/ms687393(VS.85).aspx

Regards,
Mushi
0
Courses: Start Training Online With Pros, Today

Brush up on the basics or master the advanced techniques required to earn essential industry certifications, with Courses. Enroll in a course and start learning today. Training topics range from Android App Dev to the Xen Virtualization Platform.

 
LVL 40

Expert Comment

by:evilrix
ID: 24312996
>> Yes I think you need to create a process
No, you don't the system() function takes care of all this for you. As long as the command line string is well formed it'll just work. Your parent application will block until the system executed command has terminated.

http://www.cplusplus.com/reference/clibrary/cstdlib/system/

>> For simplicity you can use WinExec
Since xcopy is a console command you probably want to call system().
0
 

Author Comment

by:palhade
ID: 24313088
Yes i want to execute "xopy" command but in differnet process.
 
But in system() it halt parent application. This i don't want. Please help me about how to create new process.
0
 
LVL 40

Expert Comment

by:evilrix
ID: 24313125
>> But in system() it halt parent application. This i don't want. Please help me about how to create new process.
Take a look at spawnlp,. which can be executed asynchronously
http://msdn.microsoft.com/en-us/library/t7y6z6fd.aspx
0
 

Author Comment

by:palhade
ID: 24313414
Hi,
 
Thanks for your reply.
 
Can you please give me c++ code for to create new process & to execute "xopy" command inside it.
 
I tried it but unable to do it.
 
Thanks...
0
 
LVL 40

Expert Comment

by:evilrix
ID: 24313883
>> I tried it but unable to do it.
Paste what you have and I'll see why it doesn't work :)
0
 

Author Comment

by:palhade
ID: 24313970

PROCESS_INFORMATION sProcessInfo;
STARTUPINFO sStartupInfo;
ZeroMemory(&sStartupInfo,sizeof(STARTUPINFO));
ZeroMemory(&sProcessInfo,sizeof(PROCESS_INFORMATION));
sStartupInfo.cb = sizeof( STARTUPINFO);

BOOL result = ::CreateProcess(mExecutable .c_substr(),
0, 0, 0, 0,
CREATE_NO_WINDOW|NORMAL_PRIORITY_CLASS,
0, 0,
&sStartupInfo,
&sProcessInfo);
 
0
 
LVL 40

Expert Comment

by:evilrix
ID: 24314000
Did you not look at spawnlp? CreateProcess is, IMHO, overkill for this. Can I suggest you look at that because using it is not much harder than system().
0
 

Author Comment

by:palhade
ID: 24314265
How to change above program into spawnlp().
 
please help me to change it
0
 
LVL 40

Expert Comment

by:evilrix
ID: 24314712
Something like this (you'll need to tweak it to suit your needs).
#include <process.h>
 
int main()
{
	intptr_t res = _spawnlp(
		_P_NOWAIT,
		"xcopy.exe",
		"C:\\Temp\\test.dat",
		"C:\\",
		0 
	);
}

Open in new window

0
 

Author Comment

by:palhade
ID: 24322373
But 4th parameter ("C:\\",)i want defalut directory means current application directory. I do not want to specify "C:\\", is there any way to do so
0
 
LVL 40

Expert Comment

by:evilrix
ID: 24322943
>> is there any way to do so
Have you tried just "." since the . is always current directory. Otherwise look at GetCurrentDirectory()
http://msdn.microsoft.com/en-us/library/aa364934(VS.85).aspx
0
 

Author Comment

by:palhade
ID: 24323144
hi,
i tried this but it is not working
intptr_t res = _spawnlp( _P_NOWAIT, "xcopy", "D:\tmep\temp.dat /Y", NULL, 0 );
0
 

Author Comment

by:palhade
ID: 24323208

hi,

i tried the way you told
intptr_t res = _spawnlp( _P_NOWAIT, "xcopy", "D:\tmep\temp.dat /Y", ".", 0 );
But i am getting following error
File cannot be copied onto itself
0 File<s> copied
 
0
 
LVL 40

Expert Comment

by:evilrix
ID: 24323231
intptr_t res = _spawnlp(
 _P_NOWAIT,
"xcopy",
"D:\tmep\temp.dat /Y",      // Is the source path valid? Also where is the destination?
NULL,                                 // You don't need NULL and 0, either will do -- it's just a way of terminating the varargs
0 );

spawnlp is a variable argument function, with 0 or NULL terminating the varargs. Each part of the xcopy <source>, <dest>, <params> should really be a separate argument. Something like below.


#include <process.h>
#include <windows.h>
 
int main()
{
	char szCurWorkingDir[MAX_PATH] = {0};
 
	GetCurrentDirectoryA(
		MAX_PATH,
		szCurWorkingDir
		);
 
 
	intptr_t res = _spawnlp(
		_P_NOWAIT,
		"xcopy",
		" /Y",
		"D:\temp\temp.dat",
		szCurWorkingDir,
		NULL 
	);
 
}

Open in new window

0
 
LVL 40

Expert Comment

by:evilrix
ID: 24323236
>> File cannot be copied onto itself
Check that the xcopy command you are passing to spawnlp is valid by typing it into a command line console in the same working directory as where you are executing your code.
0
 

Author Comment

by:palhade
ID: 24323297
I check xcopy command it is working fine there.
I tried your code also but it is giving following error
 File cannot be copied onto itself
0 File<s> copied
What is the purpose of following code
char szCurWorkingDir[MAX_PATH] = {0};

GetCurrentDirectoryA(
MAX_PATH,
szCurWorkingDir
);
 
0
 

Author Comment

by:palhade
ID: 24323306

I check xcopy command it is working fine there.
I tried your code also but it is giving following error
 File cannot be copied onto itself
0 File<s> copied
What is the purpose of following code
char szCurWorkingDir[MAX_PATH] = {0};
GetCurrentDirectoryA(
MAX_PATH,
szCurWorkingDir
);
 
 
0
 
LVL 40

Accepted Solution

by:
evilrix earned 500 total points
ID: 24323343
>> What is the purpose of following code
You said you wanted the current working directory right? This gets that.

>> File cannot be copied onto itself
The current working directory isn't the same directory as the file is in it is? This error suggests that is the case. What do you see if you do the following....?
#include <process.h>
#include <windows.h>
#include <iostream>
#include <cstdlib>
 
int main()
{
	char szCurWorkingDir[MAX_PATH] = {0};
	char szSource[] = "D:\\temp\\temp.dat";
 
	GetCurrentDirectoryA(
		MAX_PATH,
		szCurWorkingDir
		);
 
	// DEBUG
	std::cout << "Copying " << szSource << " to " << szCurWorkingDir << std::endl;
	system("pause");
	// DEBUG
 
	intptr_t res = _spawnlp(
		_P_NOWAIT,
		"xcopy",
		" /Y",
		szSource,
		szCurWorkingDir,
		NULL 
	);
 
}

Open in new window

0

Featured Post

Gigs: Get Your Project Delivered by an Expert

Select from freelancers specializing in everything from database administration to programming, who have proven themselves as experts in their field. Hire the best, collaborate easily, pay securely and get projects done right.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Suggested Solutions

Title # Comments Views Activity
C++ to C# code conversion issue 4 104
GUI: DIalog Stacking and Popping in MS C++ 4 74
Create a path if not exists 7 76
c++, dynamic object by json 1 42
Many modern programming languages support the concept of a property -- a class member that combines characteristics of both a data member and a method.  These are sometimes called "smart fields" because you can add logic that is applied automaticall…
With most software applications trying to cater to multiple user needs nowadays, the focus is to make them as configurable as possible. For e.g., when creating Silverlight applications which will connect to WCF services, the service end point usuall…
The viewer will be introduced to the technique of using vectors in C++. The video will cover how to define a vector, store values in the vector and retrieve data from the values stored in the vector.
The viewer will be introduced to the member functions push_back and pop_back of the vector class. The video will teach the difference between the two as well as how to use each one along with its functionality.

786 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question