質問編集履歴

誤字修正

2018/09/02 07:38

投稿

minhouse10

スコア41

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -78,7 +78,7 @@
-**satocha様にご指摘の箇所に関して：**
+**satocha様からご指摘の箇所に関して：**
@@ -168,7 +168,7 @@
-**satocha様にご指摘の箇所に関して　その２：**
+**satocha様からご指摘の箇所に関して　その２：**

修正版のフルコードを更新

2018/09/02 07:38

投稿

minhouse10

スコア41

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -163,3 +163,173 @@
     + CategoryInfo          : NotSpecified: (:) [Copy-Item], NotSupportedException
     + FullyQualifiedErrorId : System.NotSupportedException,Microsoft.PowerShell.Commands.CopyItemCommand
+**satocha様にご指摘の箇所に関して　その２：**
+satocha様にご指摘頂きました通り、dateの変数のフォーマットが問題でした。
+以下のように変更したところスクリプトが意図した通りの結果を出しました！
+$date = Get-Date -Format "-yyyyMMdd-HH-mm-ss"
+ただしコロンを含んでいてもTrueが返りました。
+PS C:\Users\Desktop\PS_Folder> test-path -isvalid "C:\Users\Desktop\PS_Folder\CC\A\web.config-20180902-11:18:47 "
+**True**
+全体的にまだまだ改善の余地の多い汚いコードかと思いますが、修正版フルコードは以下の通りとなります。
+```Powershell
+$date = Get-Date -Format "-yyyyMMdd-HH-mm-ss"
+$homeDir = 'C:\Users\Desktop\PS_Folder'
+$conStrFrom = 'Server1'
+$conStrTo = 'Server2'
+$targetFile = 'web.config'
+Set-Location $homeDir
+function Get-ScriptPath {
+    Split-Path $MyInvocation.ScriptName
+}
+$CntReplaced = 0
+if ((Get-ScriptPath) -eq $homeDir) {
+    echo "The Path is correct: $homeDir"
+    echo "`r`n"
+    }
+else {
+    echo "--Script is not in the correct path!!--"
+    exit
+}
+    $web_configfiles = Get-ChildItem $homeDir -recurse -include $targetFile
+    $filesCnt = ($web_configfiles).Count
+    echo ("There are $filesCnt `"$targetFile`" Files`r`n")
+    $backUp = Read-Host "Do You Want to Take Backups of $targetFile ? Please enter Y or N"
+    if ($backUp -eq "Y"){
+        echo "--Start Backuping $targetFile--`r`n"
+        $web_configfiles | foreach-object{
+            $filePath = $_.fullname
+            $parent = split-path $filePath
+            $fileBase = [io.path]::getFileNameWithoutExtension( $filePath )
+            $ext = [io.path]::getExtension( $filePath )
+            $sourcePath = Join-Path $parent $fileBase$ext
+            $destPath = Join-Path $parent $fileBase$ext$date
+            Copy-Item -Path $sourcePath -Destination $destPath
+            echo "Backup was successful to create in : $parent"
+        }
+    }
+    elseif ($backUp -eq "N") {
+        echo "--User didn't choose to take backups--`r`n"
+    }
+    else {
+        echo "--Please enter Y or N--"
+        exit
+    }
+    if ($web_configfiles) {
+        foreach ($file in $web_configfiles) {
+            $cnt = (Get-Content $file |Select-String $conStrFrom).Count
+            $fname = Split-Path $file -Leaf
+            echo ("$fname contains Target text in `"$cnt`" location")
+            if ($cnt -gt 0) {
+                (Get-Content $file) | Foreach-Object { $_ -creplace $conStrFrom, $conStrTo } | Set-Content $file
+                $CntReplaced = $CntReplaced + 1
+            }
+            else {
+                echo ("--The File doesn't contain Target Text--")
+            }
+        }
+    }
+echo `r`n
+echo "`"$CntReplaced`" $fname files of text `"$conStrFrom`" were replaced to `"$conStrTo`""
+```

修正版コードと新たなエラーを更新

2018/09/02 07:17

投稿

minhouse10

スコア41

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -95,3 +95,71 @@
 ----                -------------         ------ ----
 -a----         9/1/2018   8:54 PM           5308 web.config
+-----
+```Powershell
+        $web_configfiles | foreach-object{
+            $filePath = $_.fullname  #ファイルのフルパスを文字列として取得
+            #echo $filePath
+            $parent = split-path $filePath #親ディレクトリ
+            #echo $parent
+            $fileBase = [io.path]::getFileNameWithoutExtension( $filePath ) #拡張子を除いたファイル名
+            #echo $fileBase
+            $ext = [io.path]::getExtension( $filePath ) #拡張子（.を含む）
+            #echo $fileBase$ext
+            #あとは新パスを適当に組み立ててコピー
+            $sourcePath = Join-Path $parent $fileBase$ext
+            $destPath = Join-Path $parent $fileBase$ext$date
+            echo $sourcePath
+            echo $destPath
+            Copy-Item -Path $sourcePath -Destination $destPath
+        }
+```
+Copy元とコピー先のそれぞれ対象ディレクトリのパスとファイル名は意図したものとなっているがエラーとなってしまう。
+**echo $sourcePath**
+ C:\Users\Desktop\PS_Folder\CC\A\web.config
+**echo $destPath**
+ C:\Users\Desktop\PS_Folder\CC\A\web.config-20180902-11:18:47
+Copy-Item : The given path's format is not supported.
+At C:\Users\Desktop\PS_Folder\change_connstr.ps1:51 char:13
++             Copy-Item -Path $sourcePath -Destination $destPath
++             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+    + CategoryInfo          : NotSpecified: (:) [Copy-Item], NotSupportedException
+    + FullyQualifiedErrorId : System.NotSupportedException,Microsoft.PowerShell.Commands.CopyItemCommand

satocha様のご指摘に関する修正について

2018/09/02 03:22

投稿

minhouse10

スコア41

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -16,13 +16,13 @@
     + CategoryInfo          : NotSpecified: (:) [Copy-Item], NotSupportedException
-    + FullyQualifiedErrorId : System.NotSupportedException,Microsoft.PowerShell.Commands.CopyItemCommand
+    + FullyQualifiedErrorId : System.NotSupportedException,Microsoft.PowerShell.Commands.CopyItemCommand
-**バックアップをとる部分のスクリプトは以下になります。**
+**バックアップをとる部分の修正前スクリプトは以下になります。**
-```ここに言語を入力
+```Powershell
 #複数のサブディレクトリに同じ名前のファイルがあるので再帰的に取得
@@ -75,3 +75,23 @@
         exit
 ```
+**satocha様にご指摘の箇所に関して：**
+web_configfilesをechoした結果は以下のようにヒットした複数ファイル数分出力されます。今回はweb.configというファイルが９つ検索にヒットします。
+Directory: C:\Users\Desktop\PS_Folder\A
+Mode                LastWriteTime         Length Name
+----                -------------         ------ ----
+-a----         9/1/2018   8:54 PM           5308 web.config