Big code refactoring to improve readability and maintenance

ArrangePictures.ps1

@@ -1,5 +1,4 @@
-# Fonction pour récupérer une Array des extensions d'images et vidéos depuis un json
-function Get-ImageVideoExtensions {
+function Get-APImageVideoExtensions {
     param (
         [string] $sourceUrl = "https://cdn.jsdelivr.net/gh/jshttp/mime-db@v1.52.0/db.json"
     )
@@ -7,7 +6,7 @@ function Get-ImageVideoExtensions {
     $extensions = @()
 
     foreach ($item in $jsonData) {
-        if ($item.Name -like "image*" -or $item.Name -like "video*") {
+        if ($item.Name -match "^(image|video)") {
             $extensions += $item.Value.extensions
         }
     }
@@ -16,113 +15,88 @@ function Get-ImageVideoExtensions {
     return $extensions
 }
 
-# Fonction pour décompresser les archives
-function Decompress-ArchiveIfNeeded {
-    param(
-        [string]$ArchivePath,
-        [string]$ExtractPath
+function Get-APFileDateTaken {
+    param (
+        [string] $FilePath
     )
+    $Folder = (New-Object -ComObject Shell.Application).NameSpace((Split-Path $FilePath))
+    $PropIndex = 12  # Date Taken Property Index
+    $CharWhiteList = '[^: \w\/]'
+
+    $DateTaken = $Folder.GetDetailsOf(
+        $Folder.ParseName((Split-Path -Leaf $FilePath)),
+        $PropIndex
+    ) -replace $CharWhiteList
+
     
-    if (Test-Path $ArchivePath -PathType Leaf) {
-        $tempDir = Join-Path -Path $ExtractPath -ChildPath ([System.IO.Path]::GetRandomFileName())
-        while (Test-Path $tempDir) {
-            $tempDir = Join-Path -Path $ExtractPath -ChildPath ([System.IO.Path]::GetRandomFileName())
-        }
-        New-Item -Path $tempDir -ItemType Directory | Out-Null
-        Expand-Archive -Path $ArchivePath -DestinationPath $tempDir
-        return $tempDir
-    }
-    return $null
+    $DateTaken = try { [DateTime]($DateTaken) } catch { $null }
+    return $DateTaken
+}
+
+function Get-APTemporaryDirectory {
+    do {
+        $TempDir = Join-Path -Path $env:TEMP -ChildPath ([System.IO.Path]::GetRandomFileName())
+    } while (Test-Path $TempDir)
+    return $TempDir
 }
 
-# Fonction pour obtenir la date de création d'une image/vidéo
-function Get-OriginalDate {
+function Get-APCreationDateSubDirectory {
     param (
-        $file
+        [string] $filePath
     )
-    # Tentative de lecture des données EXIF
-    $exif = Try { [System.Drawing.Image]::FromFile($file.FullName) } Catch { $null }
 
-    if ($exif) {
-        $exifDate = $exif.PropertyItems | Where-Object { $_.Id -eq 0x9003 } | ForEach-Object {
-            [System.Text.Encoding]::ASCII.GetString($_.Value).TrimEnd([char]0)
-        }
-        if ($exifDate) {
-            $date = [datetime]::ParseExact($exifDate, "yyyy:MM:dd HH:mm:ss", $null)
-        } else {
-            $date = $file.CreationTime
-        }
-        $exif.Dispose()
+    $originalDate = Get-APFileDateTaken -FilePath $filePath
+
+    if ($originalDate) {
+        $year = $originalDate.Year
+        $month = $originalDate.Month.ToString("D2")
+        $monthName = Get-Culture | ForEach-Object { $_.DateTimeFormat.GetMonthName($month) }
+
+        return "$year\$month - $monthName"
     } else {
-        $date = $file.CreationTime
+        return "Non daté"
     }
 }
 
-# Demande des paramètres si non fournis
 if (-not $args[0] -or -not $args[1]) {
-    $SourceDir = Read-Host "Entrez le chemin du dossier source"
-    $DestinationDir = Read-Host "Entrez le chemin du dossier destination"
+    $SourceDir = Read-Host "Veuillez entrer le chemin du dossier source (ou une archive .zip)"
+    $DestinationDir = Read-Host "Veuillez entrer le chemin du dossier destination"
 } else {
     $SourceDir = $args[0]
     $DestinationDir = $args[1]
 }
 
-# Vérifie si le dossier source est une archive
-if ($SourceDir -match "\.zip$") {
-    $SourceDir = Decompress-ArchiveIfNeeded -ArchivePath $SourceDir -ExtractPath $env:TEMP
-    if (-not $SourceDir) {
+if ($SourceDir -imatch "\.zip$") {
+    $TempDir = Get-APTemporaryDirectory
+
+    try {
+        Expand-Archive -Path $SourceDir -DestinationPath $TempDir
+        $SourceDir = $TempDir
+    }catch{
         Write-Host "L'archive n'a pas pu être décompressée."
         return
     }
 }
 
-$validExtensions = Get-ImageVideoExtensions
-
-# Parcourt tous les fichiers récursivement dans le dossier source
+$ValidExtensions = Get-APImageVideoExtensions
 Get-ChildItem -Path $SourceDir -Recurse -File | ForEach-Object {
-    $file = $_
-    $extension = $file.Extension.ToLower().TrimStart('.')
-
-    if ($extension -in $validExtensions) {
-        # Tentative de lecture des données EXIF
-        $exif = Try { [System.Drawing.Image]::FromFile($file.FullName) } Catch { $null }
-
-        if ($exif) {
-            $exifDate = $exif.PropertyItems | Where-Object { $_.Id -eq 0x9003 } | ForEach-Object {
-                [System.Text.Encoding]::ASCII.GetString($_.Value).TrimEnd([char]0)
-            }
-            if ($exifDate) {
-                $originalDate = [datetime]::ParseExact($exifDate, "yyyy:MM:dd HH:mm:ss", $null)
-            } else {
-                $originalDate = $file.CreationTime
-            }
-            $exif.Dispose()
-        } else {
-            $originalDate = $file.CreationTime
-        }
-        
-        if (-Not $originalDate) {
-            $targetFolder = Join-Path -Path $DestinationDir -ChildPath "Non daté"
-        } else {
-            # Créer le dossier de destination basé sur l'année et le mois
-            $year = $originalDate.Year
-            $monthNum = $originalDate.Month.ToString("00")
-            $monthName = $originalDate.ToString("MMMM", [System.Globalization.CultureInfo]::GetCultureInfo("fr-FR"))
-            $targetFolder = Join-Path -Path $DestinationDir -ChildPath "$year\$monthNum - $monthName"
-        }
+    $File = $_
+    $FileExtension = $File.Extension.ToLower().TrimStart('.')
+
+    if ($ValidExtensions -contains $FileExtension) {
+        $subFolder = Get-APCreationDateSubDirectory -filePath $File.FullName
+        $targetDir = Join-Path -Path $DestinationDir -ChildPath $subFolder
 
-        if (-Not (Test-Path $targetFolder)) {
-            New-Item -Path $targetFolder -ItemType Directory | Out-Null
+        if (-not (Test-Path $targetDir)) {
+            New-Item -Path $targetDir -ItemType Directory | Out-Null
         }
 
-        # Copier le fichier dans le dossier de destination
-        Copy-Item -Path $file.FullName -Destination $targetFolder
+        Copy-Item -Path $file.FullName -Destination $targetDir
     } else {
         Write-Host "Fichier ignoré: $($file.FullName)"
     }
 }
 
-# Suppression du dossier temporaire si nécessaire
 if ($SourceDir -like "$env:TEMP\*") {
     Remove-Item -Path $SourceDir -Recurse -Force
 }