I this correct?

David Crow

I have no idea if this is the right answer to the OP's question, Richard, but the fact that you were able to ascertain something from it is impressive. :thumbsup:

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"You can easily judge the character of a man by how he treats those who can do nothing for him." - James D. Miles

Lost User

Sir, errors are quiet similar to my initial work. i.e initially when I use system(store) it shows me an error the same error is showing when I use system(sysCommand); Error:

Severity Code Description Project File Line
Error C2664 'int system(const char *)': cannot convert argument 1 from 'wchar_t [128]' to 'const char *' CG C:\Users\User\Desktop\Analyser tools\Sub-Project\Sub-Project\USB informer.cpp 52

Thank you for your kind help

Lost User

Thanks, it was somewhat easier than many of the questions we are seeing.

David Crow

This is a Unicode issue.

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"You can easily judge the character of a man by how he treats those who can do nothing for him." - James D. Miles

Lost User

Sorry, my mistake, you need to use _wsystem. See the link I provided in a previous message.

Lost User

To sir DavidCrow: I am using Unicode characet set what should I do now? I am using only unicode and not multibyte Thank you for your kind help

David Crow

You can't send a wchar_t array to system(). It's expecting a char pointer/array instead.

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"You can easily judge the character of a man by how he treats those who can do nothing for him." - James D. Miles

Lost User

Sir, It works now Thank you for you kind,quick and constant help!

Lost User

Sir, I have completed everything and when I run my code the command window disappears immediately what should I do now? Thank you

Lost User

The system process will only run the command that you send it, and then terminate. There are alternative ways of doing this: see the exec commands at Process and Environment Control[^].

Lost User

Sir, I referred here Execute a Program with C++[^] and did this

ShellExecute(
NULL,
_T("open"),
_T("cmd.exe"),
_T("attrib"),
(show),
SW_SHOW);

it is showing the show (I mean the directory) like this K:\ but not ececuting the attrib.how could I solve this? Thank you

David Crow

If you are wanting to get the attributes of a file/folder, why do it the way we were doing it 30 years ago? See GetFileAttributes() instead.

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"You can easily judge the character of a man by how he treats those who can do nothing for him." - James D. Miles

Lost User

Your parameters are not correct, and it will not work just by opening cmd.exe. You need to add the option to tell cmd to execute the sub-command in the parameter list automatically like:

PWSTR show = L"/k attrib D:\\";
ShellExecute(
NULL,
L"open",
L"cmd.exe",
show,
NULL,
SW_SHOW);

Of course, the attrib command will not operate on a drive letter, and as David mentions, there are better ways of getting this information.

Lost User

Sir, Is there any other way to execute some commands like this to a specific drive by giving the drive string at the run time? because I have to define the drive name here? I have referred to create process, shell execute etc., attributes is one of my functions I need to execute several different commands like this on a specific drive how could one do it? Thank you

David Crow

VISWESWARAN1998 wrote:

...I need to execute several different commands like this on a specific drive...

Be specific.

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"You can easily judge the character of a man by how he treats those who can do nothing for him." - James D. Miles

Lost User

Sir, my program will not do anything on its own. I have a added GetLogicalDriveStrings in a combobox which will show the list of drive present in the PC. Then if the user wishes to add a directory by pressing a push button, then cmd function should execute and create a directory or hiding folders and so on on the drive specified. I know it is much easier to do this manually but I am learning windows API so it will be helpful for my future use. Thank you

David Crow

VISWESWARAN1998 wrote:

...but I am learning windows API...

Then start using it. Trying to do all of this via system() and ShellExecute() is all but completely wrong.

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"You can easily judge the character of a man by how he treats those who can do nothing for him." - James D. Miles

Lost User

Sir, what to use? could you please explain me briefly?

David Crow

See here.

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"You can easily judge the character of a man by how he treats those who can do nothing for him." - James D. Miles

Lost User

I showed you how to do it in my correction of your code.