New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Exclusion Paths #1
Comments
Greetings everyone, I just wanted to post a few musings about this issue and my understanding of it. Adding exclusion paths via GUI...Perhaps a new form should be opened when the user clicks on to add Exclusion Paths? Adding the paths themselves may be done via an openFileDialog, so the user can select the path via a file-dialog system and then the program could grab the path-string from the result. He could keep doing this and keep adding paths. A dataGridViewer may be used to display all the paths in a nice list format. Portability/Storage: I thought perhaps we could create a separate SQL table? The SQL table could store all the paths, but then the table would have to be installed alongside the program whenever the program is installed. When the scanner builds the directories to scan, there is a complexity problem. The biggest issue, thus far, is actually comparing the paths. I'm having a hard time finding some way of saying these paths are "equal", that C:\ umbrellas anything afterwards.. I'm researching this quite a bit, i'm sure i'll find some solution. I just wanted to provide an update on this issue. |
Good stuff here. We can discuss it more Tuesday as far as the GUI stuff. Some possible solutions might be from here http://stackoverflow.com/questions/2281531/how-can-i-compare-directory-paths-in-c I would say we do want to avoid and file I/O when comparing paths as that will get expensive. |
I was actually looking at that post on stackoverflow when you posted it haha. An update on what I'm doing: I spent most of Saturday researching some ways to exclude paths. This, as I mentioned, will include comparing paths. But anyway, After a bunch of time spent testing and researching, I found a nice and simple solution. The good news: I'm fairly confident I can exclude paths correctly in the code, all I need now is the team-okayed GUI design on how to get the paths from the user and the okay to start editing the code. |
An update: I believe I have successfully implemented the exclusion paths on top of the previous teams code. I started up the scanner and went to the exclusion paths. I excluded all the drives on my computer except C:. I then ran the scanner under Administrator mode. The scan actually COMPLETED successfully, scanning a total of 3431 files, 1674 hits, and taking about 10 minutes. I'm sure more testing is needed for greater robustness...But it actually scanned. So, i'm happy. Some notes: Future improvements: |
Except for more testing, general bug finding, and possibly GUI improvements.. I believe this issue may be closed. |
After discussion with James, we would like to be able to control/add exclusion paths so that the scan does not have to go through every file/folder.
Need more details on how we should implement this.
The text was updated successfully, but these errors were encountered: