Thread: Add a string to WinExec line - how?

Well, you'll probably have to create a string with what you want and then pass that string to WinExec. With C-style strings and functions, you could use something like this:

Code:

char str[100];
sprintf(str, "MKDIR C:\\Documents & settings\\%s\\New Dir", username);
WinExec(str);

With C++ strings, you could get away with

Code:

std::string str = "MKDIR C:\\Documents & settings\\ " + username + " \\New Dir";
WinExec(str.c_str());

Or you could use stringstreams. See the FAQ.

Well, first of all, you should not be using WinExec:

http://msdn2.microsoft.com/en-us/library/ms687393.aspx

Note This function is provided only for compatibility with 16-bit Windows. Applications should use the CreateProcess function.

Anyway, there are a few ways of handling the string. One is to use std::string and the other is the C approach with chars.

If you use the C approach you can allocate enough memory to hold the entire string, and then call sprintf() to write the string to the newly allocated block of memory. Then pass to the function.

If you use std::string, then it's a matter of adding the first part with the second part, etc. etc.. This is the preferred C++ approach. Something like this:

Code:

std::string path = "MKDIR C:\\Documents & settings\\";
std::string user = "John";

std::string full = path + user;

This is extremely simplified, however.

BTW, for WinExec, you need another argument. Something like SW_SHOW should be your second argument.

All in all, I would recommend you use CreateProcess() or some other appropriate function rather than WinExec().

Edit: Bah, beaten again. :'(

And the best option may be "ShellExecute" rather than "CreateProcess", as the former is a more "generic" function.

--
Mats

Probably because MOVE isn't a real program, but a shell command.

ShellExecute() may be what you want. Otherwise, some might argue the more "correct" approach would be to programatically move the file yourself by copying the file over, and then, upon verifying the copy worked successfully, you delete the original.

Or, use the MoveFile Win32 function, perhaps?

--
Mats

Check the return value of MoveFile() and call GetLastError() if it returns 0.

Although, tbh, I think you need to specify the actual name of the new file, not just the folder you want to move it to.

What does UserName contain? What do src and dst look like when you print them?

Code:

cout << "src: [" << src << "]\n";
cout << "dst: [" << dst << "]\n";

BTW, this

Code:

const char *p;
p=src.c_str();
    
const char *q;
q=dst.c_str();
    
MoveFile(p, q);

is equivalent to this:

Code:

MoveFile(src.c_str(), dst.c_str());