SHFileOperation problem
-
Chris' observation is correct. To remedy this, I suggest using this class from Michael Dunn. It takes care of those
\0oddities!
"Take only what you need and leave the land as you found it." - Native American Proverb
Actually CString does handle multiple \0. Otherwise it won't work for pTo field of the structure. And even if it doesn't it still doesn't explain the fact why if I convert CString to char[] (using the way described in MSDN aka using strcpy) and add a \0 to the end it still doesn't work. Oh and I would really like to avoid using any wrapper classes. The project is already bloated (at least for the purpose it serves) and I don't want to bloat it any more.
-
Actually CString does handle multiple \0. Otherwise it won't work for pTo field of the structure. And even if it doesn't it still doesn't explain the fact why if I convert CString to char[] (using the way described in MSDN aka using strcpy) and add a \0 to the end it still doesn't work. Oh and I would really like to avoid using any wrapper classes. The project is already bloated (at least for the purpose it serves) and I don't want to bloat it any more.
Alex Orovetskiy wrote:
Actually CString does handle multiple \0.
actually, it doesn't. when doing an assignment to a CString, it checks the length of the new (right hand side) string by scanning for the first \0. then it reallocs and memcpy's strlen bytes from the new string to the destination string. it will never scan past the first \0, never count anything past the \0 towards the new string length, never alloc space for anything past the \0, and never copy anything past the \0. step into the CString assignment operator and watch.
CString test;
test = "A\0\0";
ASSERT(test.GetLength()==1);CString isn't going to get you where you want to go. Cleek | Image Toolkits | Thumbnail maker -- modified at 15:07 Tuesday 22nd November, 2005
-
Actually CString does handle multiple \0. Otherwise it won't work for pTo field of the structure. And even if it doesn't it still doesn't explain the fact why if I convert CString to char[] (using the way described in MSDN aka using strcpy) and add a \0 to the end it still doesn't work. Oh and I would really like to avoid using any wrapper classes. The project is already bloated (at least for the purpose it serves) and I don't want to bloat it any more.
Alex Orovetskiy wrote:
Oh and I would really like to avoid using any wrapper classes.
Why?
Alex Orovetskiy wrote:
The project is already bloated (at least for the purpose it serves) and I don't want to bloat it any more.
Bloated? With data, possibly. With code, hardly ever. You are making the common mistake of assuming more code = more bloat. This is hardly ever the case, especially without any supporting metrics. Whether you put together the 30 lines of code to deal with
SHFileOperation()and its nuances, or you use a class that wraps those same 30 lines of code, the net result is the same.
"Take only what you need and leave the land as you found it." - Native American Proverb
-
Alex Orovetskiy wrote:
Actually CString does handle multiple \0.
actually, it doesn't. when doing an assignment to a CString, it checks the length of the new (right hand side) string by scanning for the first \0. then it reallocs and memcpy's strlen bytes from the new string to the destination string. it will never scan past the first \0, never count anything past the \0 towards the new string length, never alloc space for anything past the \0, and never copy anything past the \0. step into the CString assignment operator and watch.
CString test;
test = "A\0\0";
ASSERT(test.GetLength()==1);CString isn't going to get you where you want to go. Cleek | Image Toolkits | Thumbnail maker -- modified at 15:07 Tuesday 22nd November, 2005
Chris Losinger wrote:
CString isn't going to get you where you want to go.
OK...but why does it work for pTo field which also should be double-zero terminated ? Actually it working for pTo and not working for pFrom got me to beleive that it is not double-zero problem...
-
Alex Orovetskiy wrote:
Oh and I would really like to avoid using any wrapper classes.
Why?
Alex Orovetskiy wrote:
The project is already bloated (at least for the purpose it serves) and I don't want to bloat it any more.
Bloated? With data, possibly. With code, hardly ever. You are making the common mistake of assuming more code = more bloat. This is hardly ever the case, especially without any supporting metrics. Whether you put together the 30 lines of code to deal with
SHFileOperation()and its nuances, or you use a class that wraps those same 30 lines of code, the net result is the same.
"Take only what you need and leave the land as you found it." - Native American Proverb
DavidCrow wrote:
Bloated? With data, possibly. With code, hardly ever. You are making the common mistake of assuming more code = more bloat.
Well yeah, I am making this assuption. I am basing it on one thing: when I'm not doing any coding on the project and than return only to spend around 10 mins figuring out what does what. But that's pretty much irrelevant to the subject. Just tried the following (note that szFrom contains \*.*):
char buf[MAX_PATH]; strcpy(buf,szFrom); strcat(buf,"\0");It didn't work. I will try using the demo of that wrapper class to see if it comes out with the same results or no. -
DavidCrow wrote:
Bloated? With data, possibly. With code, hardly ever. You are making the common mistake of assuming more code = more bloat.
Well yeah, I am making this assuption. I am basing it on one thing: when I'm not doing any coding on the project and than return only to spend around 10 mins figuring out what does what. But that's pretty much irrelevant to the subject. Just tried the following (note that szFrom contains \*.*):
char buf[MAX_PATH]; strcpy(buf,szFrom); strcat(buf,"\0");It didn't work. I will try using the demo of that wrapper class to see if it comes out with the same results or no.Alex Orovetskiy wrote:
strcat(buf,"\0");
any of the string functions are going to recognize '\0' as an empty string and basically do nothing at all: "append nothing to a string? consider it done!". try : buf[szFrom.GetLength() + 1] = 0; Cleek | Image Toolkits | Thumbnail maker -- modified at 15:35 Tuesday 22nd November, 2005
-
DavidCrow wrote:
Bloated? With data, possibly. With code, hardly ever. You are making the common mistake of assuming more code = more bloat.
Well yeah, I am making this assuption. I am basing it on one thing: when I'm not doing any coding on the project and than return only to spend around 10 mins figuring out what does what. But that's pretty much irrelevant to the subject. Just tried the following (note that szFrom contains \*.*):
char buf[MAX_PATH]; strcpy(buf,szFrom); strcat(buf,"\0");It didn't work. I will try using the demo of that wrapper class to see if it comes out with the same results or no.Alex Orovetskiy wrote:
It didn't work.
What do you mean by "It didn't work?" After
strcpy()returns,bufandszFromshould contain identical strings. Are you saying this is not the case?Alex Orovetskiy wrote:
strcat(buf,"\0");
Given that
strcat()operates on nul-terminated strings, this effectively does nothing. What were you expecting?
"Take only what you need and leave the land as you found it." - Native American Proverb
-
Alex Orovetskiy wrote:
strcat(buf,"\0");
any of the string functions are going to recognize '\0' as an empty string and basically do nothing at all: "append nothing to a string? consider it done!". try : buf[szFrom.GetLength() + 1] = 0; Cleek | Image Toolkits | Thumbnail maker -- modified at 15:35 Tuesday 22nd November, 2005
If I knew how to build statues I would have built one for you. It works. It perfectly works. No words can express my gratitude. Thank you ! P.S. And like always the solution was really simple. I remember putting a wrong comma once...
-
Alex Orovetskiy wrote:
It didn't work.
What do you mean by "It didn't work?" After
strcpy()returns,bufandszFromshould contain identical strings. Are you saying this is not the case?Alex Orovetskiy wrote:
strcat(buf,"\0");
Given that
strcat()operates on nul-terminated strings, this effectively does nothing. What were you expecting?
"Take only what you need and leave the land as you found it." - Native American Proverb
DavidCrow wrote:
Given that strcat() operates on nul-terminated strings, this effectively does nothing. What were you expecting?
Expecting the wrong thing it seems :) Actually I didn't think it would turn this way. Oh well...it is working now. Thank you guys for spending your time on me.
-
If I knew how to build statues I would have built one for you. It works. It perfectly works. No words can express my gratitude. Thank you ! P.S. And like always the solution was really simple. I remember putting a wrong comma once...
Alex Orovetskiy wrote:
Thank you
you're welcome. come again soon, we're open 24/7. Cleek | Image Toolkits | Thumbnail maker