Powershell script for zipping up old filesRegular backup/snapshotsService class for Archiving and Mirroring Files and DirectoriesCopying files in HaskellBacking up single-user PC dataRenaming PDF files based on given rules in a text fileZip the contents of subfolders, conditionallyIdentify files within folder structure that have common file sizesUsing build event to run Powershell script to run built DLLXignCode Bypasser installer for Blade & SoulBash on Synology - Delete recycle bin entries over X days old

Pre-Employment Background Check With Consent For Future Checks

What is the meaning of "You've never met a graph you didn't like?"

How to make money from a browser who sees 5 seconds into the future of any web page?

ContourPlot — How do I color by contour curvature?

How to understand "he realized a split second too late was also a mistake"

How do you justify more code being written by following clean code practices?

Would a primitive species be able to learn English from reading books alone?

Do you waste sorcery points if you try to apply metamagic to a spell from a scroll but fail to cast it?

Why is participating in the European Parliamentary elections used as a threat?

What should be the ideal length of sentences in a blog post for ease of reading?

Can I cause damage to electrical appliances by unplugging them when they are turned on?

Anime with legendary swords made from talismans and a man who could change them with a shattered body

Would this string work as string?

Should I assume I have passed probation?

Why does a 97 / 92 key piano exist by Bösendorfer?

Did I make a mistake by ccing email to boss to others?

Given this phrasing in the lease, when should I pay my rent?

Is there a distance limit for minecart tracks?

If Captain Marvel (MCU) were to have a child with a human male, would the child be human or Kree?

Isometric embedding of a genus g surface

Confusion over Hunter with Crossbow Expert and Giant Killer

Storage of electrolytic capacitors - how long?

Is there anyway, I can have two passwords for my wi-fi

What does the word 'upstream' mean in the context?



Powershell script for zipping up old files


Regular backup/snapshotsService class for Archiving and Mirroring Files and DirectoriesCopying files in HaskellBacking up single-user PC dataRenaming PDF files based on given rules in a text fileZip the contents of subfolders, conditionallyIdentify files within folder structure that have common file sizesUsing build event to run Powershell script to run built DLLXignCode Bypasser installer for Blade & SoulBash on Synology - Delete recycle bin entries over X days old













0












$begingroup$


I have a script for zipping up old files. I know using Windows Compression isn't ideal, so I will make the script run using 7-Zip later on. For now though, I just want see how I can make my current script better. By better I mean, how could I make this code cleaner or neater? How could I make the script more effective?
Here is the code:



Write-Output("Beginning script....")

#File path of file to be cleaned
$File_Path = "C:UsersAdministratorDownloadsTesting*"

#Location of ZIP file
$Send_To = "C:UsersAdministratorDocumentsARCHIVE2"

#Location of old files before being zipped
$Old_Files = "C:UsersAdministratorDocumentsOLD_FILES"

#Time frame for files
$Days = "-65"
$now = Get-Date
$last_Write = $now.AddDays($Days)

#Filtering files according to time parameters
$Filter = Get-ChildItem -Path $File_Path | Where-Object $_.LastWriteTime -lt $last_Write

#Moving old files to destination folder
if (!$Filter)
Write-Host "Variable is null"

else
Move-Item $Filter -Destination $Old_Files
Write-Output("Moving files....")




#Compressing destination folder if there is folders in $Old_Files folder
if (!(Test-Path $Old_Files))
Write-Output("No old files found")

else
Compress-Archive -Path $Old_Files -DestinationPath $Send_To -Update
Write-Output("Old files zipped!")
Remove-Item -Path $Old_Files -Force


Write-Output("Script is finished")








share







New contributor




techguy1029 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.







$endgroup$
















    0












    $begingroup$


    I have a script for zipping up old files. I know using Windows Compression isn't ideal, so I will make the script run using 7-Zip later on. For now though, I just want see how I can make my current script better. By better I mean, how could I make this code cleaner or neater? How could I make the script more effective?
    Here is the code:



    Write-Output("Beginning script....")

    #File path of file to be cleaned
    $File_Path = "C:UsersAdministratorDownloadsTesting*"

    #Location of ZIP file
    $Send_To = "C:UsersAdministratorDocumentsARCHIVE2"

    #Location of old files before being zipped
    $Old_Files = "C:UsersAdministratorDocumentsOLD_FILES"

    #Time frame for files
    $Days = "-65"
    $now = Get-Date
    $last_Write = $now.AddDays($Days)

    #Filtering files according to time parameters
    $Filter = Get-ChildItem -Path $File_Path | Where-Object $_.LastWriteTime -lt $last_Write

    #Moving old files to destination folder
    if (!$Filter)
    Write-Host "Variable is null"

    else
    Move-Item $Filter -Destination $Old_Files
    Write-Output("Moving files....")




    #Compressing destination folder if there is folders in $Old_Files folder
    if (!(Test-Path $Old_Files))
    Write-Output("No old files found")

    else
    Compress-Archive -Path $Old_Files -DestinationPath $Send_To -Update
    Write-Output("Old files zipped!")
    Remove-Item -Path $Old_Files -Force


    Write-Output("Script is finished")








    share







    New contributor




    techguy1029 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
    Check out our Code of Conduct.







    $endgroup$














      0












      0








      0





      $begingroup$


      I have a script for zipping up old files. I know using Windows Compression isn't ideal, so I will make the script run using 7-Zip later on. For now though, I just want see how I can make my current script better. By better I mean, how could I make this code cleaner or neater? How could I make the script more effective?
      Here is the code:



      Write-Output("Beginning script....")

      #File path of file to be cleaned
      $File_Path = "C:UsersAdministratorDownloadsTesting*"

      #Location of ZIP file
      $Send_To = "C:UsersAdministratorDocumentsARCHIVE2"

      #Location of old files before being zipped
      $Old_Files = "C:UsersAdministratorDocumentsOLD_FILES"

      #Time frame for files
      $Days = "-65"
      $now = Get-Date
      $last_Write = $now.AddDays($Days)

      #Filtering files according to time parameters
      $Filter = Get-ChildItem -Path $File_Path | Where-Object $_.LastWriteTime -lt $last_Write

      #Moving old files to destination folder
      if (!$Filter)
      Write-Host "Variable is null"

      else
      Move-Item $Filter -Destination $Old_Files
      Write-Output("Moving files....")




      #Compressing destination folder if there is folders in $Old_Files folder
      if (!(Test-Path $Old_Files))
      Write-Output("No old files found")

      else
      Compress-Archive -Path $Old_Files -DestinationPath $Send_To -Update
      Write-Output("Old files zipped!")
      Remove-Item -Path $Old_Files -Force


      Write-Output("Script is finished")








      share







      New contributor




      techguy1029 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.







      $endgroup$




      I have a script for zipping up old files. I know using Windows Compression isn't ideal, so I will make the script run using 7-Zip later on. For now though, I just want see how I can make my current script better. By better I mean, how could I make this code cleaner or neater? How could I make the script more effective?
      Here is the code:



      Write-Output("Beginning script....")

      #File path of file to be cleaned
      $File_Path = "C:UsersAdministratorDownloadsTesting*"

      #Location of ZIP file
      $Send_To = "C:UsersAdministratorDocumentsARCHIVE2"

      #Location of old files before being zipped
      $Old_Files = "C:UsersAdministratorDocumentsOLD_FILES"

      #Time frame for files
      $Days = "-65"
      $now = Get-Date
      $last_Write = $now.AddDays($Days)

      #Filtering files according to time parameters
      $Filter = Get-ChildItem -Path $File_Path | Where-Object $_.LastWriteTime -lt $last_Write

      #Moving old files to destination folder
      if (!$Filter)
      Write-Host "Variable is null"

      else
      Move-Item $Filter -Destination $Old_Files
      Write-Output("Moving files....")




      #Compressing destination folder if there is folders in $Old_Files folder
      if (!(Test-Path $Old_Files))
      Write-Output("No old files found")

      else
      Compress-Archive -Path $Old_Files -DestinationPath $Send_To -Update
      Write-Output("Old files zipped!")
      Remove-Item -Path $Old_Files -Force


      Write-Output("Script is finished")






      file-system powershell





      share







      New contributor




      techguy1029 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.










      share







      New contributor




      techguy1029 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.








      share



      share






      New contributor




      techguy1029 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.









      asked 7 mins ago









      techguy1029techguy1029

      11




      11




      New contributor




      techguy1029 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.





      New contributor





      techguy1029 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.






      techguy1029 is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.




















          0






          active

          oldest

          votes











          Your Answer





          StackExchange.ifUsing("editor", function ()
          return StackExchange.using("mathjaxEditing", function ()
          StackExchange.MarkdownEditor.creationCallbacks.add(function (editor, postfix)
          StackExchange.mathjaxEditing.prepareWmdForMathJax(editor, postfix, [["\$", "\$"]]);
          );
          );
          , "mathjax-editing");

          StackExchange.ifUsing("editor", function ()
          StackExchange.using("externalEditor", function ()
          StackExchange.using("snippets", function ()
          StackExchange.snippets.init();
          );
          );
          , "code-snippets");

          StackExchange.ready(function()
          var channelOptions =
          tags: "".split(" "),
          id: "196"
          ;
          initTagRenderer("".split(" "), "".split(" "), channelOptions);

          StackExchange.using("externalEditor", function()
          // Have to fire editor after snippets, if snippets enabled
          if (StackExchange.settings.snippets.snippetsEnabled)
          StackExchange.using("snippets", function()
          createEditor();
          );

          else
          createEditor();

          );

          function createEditor()
          StackExchange.prepareEditor(
          heartbeatType: 'answer',
          autoActivateHeartbeat: false,
          convertImagesToLinks: false,
          noModals: true,
          showLowRepImageUploadWarning: true,
          reputationToPostImages: null,
          bindNavPrevention: true,
          postfix: "",
          imageUploader:
          brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
          contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
          allowUrls: true
          ,
          onDemand: true,
          discardSelector: ".discard-answer"
          ,immediatelyShowMarkdownHelp:true
          );



          );






          techguy1029 is a new contributor. Be nice, and check out our Code of Conduct.









          draft saved

          draft discarded


















          StackExchange.ready(
          function ()
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f215876%2fpowershell-script-for-zipping-up-old-files%23new-answer', 'question_page');

          );

          Post as a guest















          Required, but never shown

























          0






          active

          oldest

          votes








          0






          active

          oldest

          votes









          active

          oldest

          votes






          active

          oldest

          votes








          techguy1029 is a new contributor. Be nice, and check out our Code of Conduct.









          draft saved

          draft discarded


















          techguy1029 is a new contributor. Be nice, and check out our Code of Conduct.












          techguy1029 is a new contributor. Be nice, and check out our Code of Conduct.











          techguy1029 is a new contributor. Be nice, and check out our Code of Conduct.














          Thanks for contributing an answer to Code Review Stack Exchange!


          • Please be sure to answer the question. Provide details and share your research!

          But avoid


          • Asking for help, clarification, or responding to other answers.

          • Making statements based on opinion; back them up with references or personal experience.

          Use MathJax to format equations. MathJax reference.


          To learn more, see our tips on writing great answers.




          draft saved


          draft discarded














          StackExchange.ready(
          function ()
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f215876%2fpowershell-script-for-zipping-up-old-files%23new-answer', 'question_page');

          );

          Post as a guest















          Required, but never shown





















































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown

































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown







          Popular posts from this blog

          瀋陽號驅逐艦 目录 接收與服役 配置反潛直升機 武進三型性能升級 歷史 除役 參考資料 外部連結 导航菜单Taiwan Air Power海疆老兵-陽字號驅逐艦沿革World Navies Today: Taiwan (Republic of China)DD-839 USS POWER

          波兰旗帜列表 目录 国旗 军旗 其他制服部门旗帜 特别国家机构船只 参考文献 外部链接 导航菜单Polskie flagi, chorągwie, bandery... [波兰旗帜、条幅、船旗等]原始内容Ustawa z dnia 31 stycznia 1980 r. o godle, barwach i hymnie Rzeczypospolitej Polskiej oraz o pieczęciach państwowychZarządzenie Ministra Obrony Narodowej z dnia 14 grudnia 2005 r. zmieniające zarządzenie w sprawie szczegółowych zasad używania znaków Sił Zbrojnych Rzeczypospolitej Polskiej oraz ustalenia innych znaków używanych w Siłach Zbrojnych Rzeczypospolitej PolskiejZarządzenie Ministra Obrony Narodowej z dnia 29 stycznia 1996 r. w sprawie szczegółowych zasad używania znaków Sił Zbrojnych Rzeczypospolitej Polskiej oraz ustalenia innych znaków używanych w Siłach Zbrojnych Rzeczypospolitej PolskiejUstawa z dnia 19 lutego 1993 r. o znakach Sił Zbrojnych Rzeczypospolitej PolskiejHistoria Marynarki Wojennej RP [波兰海军史]Rozporządzenie Ministra Spraw Wewnętrznych i Administracji z dnia 12 kwietnia 2002 r. w sprawie wzoru flagi oraz oznakowania jednostek pływających i statków powietrznych Straży GranicznejRozporządzenie Ministra Spraw Wewnętrznych i Administracji z dnia 18 kwietnia 2005 r. w sprawie wzoru flagi oraz oznakowania jednostek pływających i statków powietrznych PolicjiRozporządzenie Ministra Infrastruktury z dnia 21 października 2005 r. w sprawie wzorów flag dla statków morskich na oznaczenie pełnionej specjalnej służby państwowej oraz okoliczności i warunków ich podnoszenia波兰旗帜波兰

          Indenting and Dedenting ASP code with Python