Skip to content
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

Enhance Clear-DriveSpace PowerShell Script for Usability and Efficiency #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Paul1404
Copy link

Here's a summary of the changes and improvements I made to your PowerShell script:

  • Parameter Defaults: Changed the ComputerName parameter to default to 'localhost' if no argument is provided. This makes it easier to run the script on the current machine.

  • Parameter Validation: Added a ValidatePattern attribute to the ComputerName parameter to check if it's a valid hostname using a regex pattern.

  • Error Handling: Introduced the concept of error handling. The script now verifies the connection to the computer before running the cleanup operations, and it stops execution if the connection test fails.

  • Code Redundancy: Created a helper function, Remove-Files, to remove files from a specified directory. This function is called with various paths as an argument, rather than repeating similar code multiple times in the script. This makes the script more efficient, readable, and maintainable.

  • Script Modularity: Improved the modularity of your script by dividing different tasks into separate functions. This makes it easier to maintain and understand the code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant