delteing a file...I got it just need help...

Alright I have delted a single file but I want to try and delete a whole directory with files in it.

Here is the code I got so far...

Code:

---

//Lots o headers please don't comment on them

#include <iostream.h>
#include <conio.h>
#include <windows.h>
#include <iomanip>
#include <fstream.h>
#include <stdlib.h>
#include <stdio.h>
#include <math.h>
#include <shellapi.h>

using namespace std;



int main()
{   
    typedef struct _SHFILEOPSTRUCT    //SHFileOperation structure?
    {
        HWND hwnd;
        UINT wFunc;
        LPCTSTR pFrom;
        LPCTSTR pTo;
        FILEOP_FLAGS fFlags;
        BOOL fAnyOperationsAborted;
        LPVOID hNameMappings;
        LPCTSTR lpszProgressTitle;
    } SHFILEOPSTRUCT, *LPSHFILEOPSTRUCT;

   
    _SHFileOperation(_SHFILEOPSTRUCT);    //deletes file
}

---

yes yes I know tons of headers not sure wich ones I need. (Basicly my test headers)

ok so I got alot of non filled out junk. Can someone help me with that?


Edit: I really don't need them filled. I just need maybe 1 or 2 filled out to see how to properly fill them out.

First, don't put the leading underscore. The function is called SHFileOperation, according to MSDN and Google. Second, you shouldn't need to define the struct yourself.

Ok. According to MSDN (found by searching on Google), what you want is:

wFunc = FO_DELETE
(?)hwnd = NuLL
pFrom = "path\filename.ext\0path2\filename2.ext\0path3\*.* \0";
pTo = NULL
etc.

It's the same as filling out any struct; read which member does what, fill it with what you need, and pass it to the function.

>> please don't comment on them
What if I just want to comment on the fact that you're mixing standard and non-standard C++ headers and that it's sure to cause you problems?

Ok, I won't.

gg

ok alright first time I'm doing something like this.

So far this is what I got...

Code:

---

//Lots o headers please don't comment on them

#include <iostream.h>
#include <conio.h>
#include <windows.h>
#include <iomanip>
#include <fstream.h>
#include <stdlib.h>
#include <stdio.h>
#include <math.h>
#include <shellapi.h>

using namespace std;



int main()
{   
    typedef struct SHFILEOPSTRUCT    //SHFileOperation structure?
    {
        UINT wFunc = FO_DELETE
        (?)hwnd = NuLL
        LPCTSTR pFrom = "path\filename.ext\0path2\filename2.ext\0path3\*.*\0"
        LPCTSTR pTo = NULL
        FILEOP_FLAGS fFlags;
        BOOL fAnyOperationsAborted;
        LPVOID hNameMappings;
        LPCTSTR lpszProgressTitle;
    } SHFILEOPSTRUCT, *LPSHFILEOPSTRUCT;

   
    SHFileOperation(SHFILEOPSTRUCT);    //deletes file
}

---

the headers never caused probeblems for me yet when I do the testing stage. But anways. Can I leave some of the things just like they are.

Like one of them only returns true if so and so happens. Do I do anything with them?


and here is my error log. (I get 2 errors)


C:\Documents and Settings\Alex\Desktop\test.cpp In function `int main()':
22 C:\Documents and Settings\Alex\Desktop\test.cpp syntax error before `?' token
25 C:\Documents and Settings\Alex\Desktop\test.cpp:23 [Warning] unknown escape sequence '\*'
32 C:\Documents and Settings\Alex\Desktop\test.cpp syntax error before `)' token


Edit: the error before `)' is in this line: SHFileOperation(SHFILEOPSTRUCT); //deletes file


Thank you for all of your help.

you have more then 2 errors if you look at it this way
treat warnings as if they were errors and get rid of them :)

sorry wasted post but its a good habit.

lol no prob but the warning as I can I see is not fixable sence that is how I am suposed to put the lines like that. But the other 2 I need fixed.

actually from what i see the warning is the path up there, and
instead of \ you should have \\

Rune: My post was intended to let you know what goes in what member, i.e. how to fill the struct in. You weren't supposed to copy and paste, as you can probably tell by now. In fact, if you just look at what I posted, the (?) means "I'm not sure about this one", you'll notice that I made a typo and put "NuLL" instead of "NULL", didn't put semicolons on the ends of the lines.

And, looking at your 'updated' code again, I see that you're redefining the structure still, and also you're trying to initialize the structure members in the prototype, which you can't do (you can only do it in a constructor). And you still shouldn't be redefining the structure, because then you're essentially giving the function something that it wasn't meant to take. What you want is something more like this:

Code:

---

SHFILEOPSTRUCT sfos;
sfos.wFunc = FO_DELETE;
sfos.hwnd = NULL; //(?)
etc.
 
SHFileOperation(&sfos);

---

[edit]And yes, the \ should be \\. Oversight on my part.

ok but first I did copy and paste. But I modified. Becuase when I put the semicolens in it has that error were I tried to initialize. That is why they are not in. And yes I do look and feel like an idoit but I gotta start somewere. I might start looking for a nice book on amazon.com or something today so that shoudl help me lesson the questions and stupidity.

Have you learned about structures yet?
The tutorial has a section on them, if you haven't read about them yet. 'sfos' is short for SHFileOpStruct, and is an object of type SHFileOpStruct, which is already defined in another header - which is why I said you shouldn't be defining it yourself.

>>error were I tried to initialize
Once you've read about structures, you'll (probably) know why you got these errors. In case you don't, it's because you can't initialize variables in a structure definition - either make a constructor for the structure and do initialization in that, or manually fill the members in each object you create. In this case, it's the latter, since you're not defining the structure yourself.

Well yes I have read all the tuts on this site. Well the bsaics anyways. (ok not the last 2 or so but that has notin to do with this yet).

Ok I am really close now!


First off here is my updated code tat compilers and "works"...

Code:

---

#include <iostream.h>
#include <conio.h>
#include <windows.h>
#include <iomanip>
#include <fstream.h>
#include <stdlib.h>
#include <stdio.h>
#include <math.h>
#include <shellapi.h>

using namespace std;



int main()
{   
    SHFILEOPSTRUCT sfos;
    sfos.hwnd = NULL;
    sfos.wFunc = FO_DELETE;
    sfos.pFrom = "C:\\test\\*.*";
    sfos.pTo = NULL;
    sfos.fFlags = NULL;
    sfos.fAnyOperationsAborted = NULL;
    sfos.hNameMappings = NULL;
    sfos.lpszProgressTitle = NULL;
   
    SHFileOperation(&sfos);

}

---

Now the only error I get is when I run the program it gives an error saying it can read file or folder from disk. I know why that happens: they folders are read only.

And when I un check the read only button -> hit apply -> hit ok, it stays as read only. I can't remember my trick on how to make it stay that way but do anyone else know?

I've never had problems with folders making themselves read-only automatically...

oh never mind that. But yes that is crazy but then again our computer does some wierd things. But anyways my probeblem was the NULL turminating character wich I need after the path I deinated.

sfos.pFrom = "C:\\test\\*.*"; should be sfos.pFrom = "C:\\test\\*.*\0";


It works now and I finnaly learned how to use files and folders now.

alright sence it is working now I am actually makeing a really program. All this does is delete the cookies and stuff. But I want to be able to exclude a file extension but still be able to use the * for a wild card. Is this possible?

As far as I know, there's no such functionality built into the API. But you can generate a list of possible extensions (*.aaa, *.aab, *.aac, etc.), and delete all except the one you want to keep. Of course, that will probably be horribly inefficient, so you could try a 'cheating' way: Make backup copies of all the files you want to keep, and store them in another folder; then delete all files in the target folder, and move the backups back into the now-empty folder.

dang it! Well ok how do I set more then one flag then? Notin I did worked and I need to use mroe then 1 of em?

>> Well ok how do I set more then one flag then? <<

Use the bitwise or operator.

Code:

---

sfos.fFlags = FOF_ALLOWUNDO | FOF_NOCONFIRMATION;

---

>> But I want to be able to exclude a file extension but still be able to use the * for a wild card. <<

Code:

---

// Pseudo code only
// Delete all files which don't have the .doc extension.
for (each file in folder)
{
    if (!PathMatchSpec(file, "*.doc"))
    {
        DeleteFile(file);
    }
}

---

well actually I don't need the exclude thing no more because I now know how to use more then 1 flag. Altho it would be nice to exclude files, I don't under stand that code to well.

Thanks alot tho!

Code:

---

sfos.fFlags = FOF_ALLOWUNDO & FOF_NOCONFIRMATION;

---

didn't work :'(


Edit: In fact non of your codes worked. :'( :'(

Quote:

---

Originally Posted by Rune Hunter

Code:

---

sfos.fFlags = FOF_ALLOWUNDO & FOF_NOCONFIRMATION;

---

didn't work :'(


Edit: In fact non of your codes worked. :'( :'(

---

Sorry, had a "brain mis-fire", you want the bitwise or operator! See original post. The other code is pseudo-code, it aims to demonstrate the concept but will not compile.

oh I see... I ain't that good at c++ so... I am going to try some more stuff.

Thanks.