FOR me fait une boucle de trop

Bonjour j'ai un petit souci, j'ai fait un batch pour 7zipper des dossiers suivant une liste que je fourni.
J'ai reglé tous mes problemes ,surtout avec les noms contenant des espaces grace au parametre %%*, mais justement depuis que j'ai mis ce parametre, mon Batch me fait une boucle vide à la fin.
Une idée ?
Merci

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27

@ECHO OFF FOR /F "delims=" %%* IN (C:\Batch\liste.txt) Do call :ZIPER %%* :ZIPER set nomDuFichierArchive=%*% set folder=%nomDuFichierArchive% echo %folder% pause for %%a in ("\" ":" "/" " ") do call set "folder=%%folder:%%~a=%%" echo %folder% pause echo "Suppression de la precedente archive : \\serveur\7z\%folder%.7z" del \\serveur\7z\%folder%.7z /Q echo "Debut de compression de : %nomDuFichierArchive%" echo %time% > "C:\Batch\log\%folder%.log" "%ProgramFiles%\7-zip\7z.exe" a -t7z -mx5 "\\serveur\7z\%folder%.7z" "%nomDuFichierArchive%" >>"C:\Batch\log\%folder%.log" echo %time% > "C:\Batch\log\%folder%.log" echo "Fin de compression du fichier : %nomDuFichierArchive%.7z" GOTO FIN : FIN

---

ça doit etre a cause de la dernière ligne qui est vide probablement.

Merci pour la reponse.
J'ai mis à la fin :

Code:

---

1 2 3

rem GOTO FIN rem :FIN

---

mais ça change rien ça me fait toujours une boucle à la fin avec ma variable nomDuFichierArchive vide

Citation:

---

Envoyé par psychoboust

Merci pour la reponse.
J'ai mis à la fin :

Code:

---

1 2 3

rem GOTO FIN rem :FIN

---

mais ça change rien ça me fait toujours une boucle à la fin avec ma variable nomDuFichierArchive vide

---

je parlais de la dernière ligne de ton fichier d'entrée pas de ton script :D.

autrement tu peux tester le contenu du nom du fichier s'il est vide tu passe a la suite.

Ok j'avais pas compris, mais ça ne change rien.

C'est vrai que je peux rajouter une condition, mais ça devient l'usine à Gaz.
Je voudrais juste comprendre pourquoi il me fait un tour à vide.
Comme si mon fichier liste.txt avait une ligne vide.

salut tout le monde,
bienvenue psychoboust sur le forum,

Citation:

---

Envoyé par psychoboust

C'est vrai que je peux rajouter une condition, mais ça devient l'usine à Gaz.

---

En fait ça l'est déjà, ça risque même d'exploser.

Procède déjà à ce changement, et voie si ça t'aide à aller de l'avant:

Code:

---

1 2 3 4

for /f "tokens=*" %%i in (c:\batch\liste.txt) do call :ziper "%%i" goto fin :ziper

---

et pour info, y'a encore beaucoup à faire.

Citation:

---

Envoyé par maxim_um

salut tout le monde,
bienvenue psychoboust sur le forum,
.

---

Merci

Citation:

---

En fait ça l'est déjà, ça risque même d'exploser.

---

;)

Citation:

---

Procède déjà à ce changement, et voie si ça t'aide à aller de l'avant:

Code:

---

1 2 3 4

for /f "tokens=*" %%i in (c:\batch\liste.txt) do call :ziper "%%i" goto fin :ziper

---

.

---

Cool le

Code:

---

goto fin

---

a effectivement résolu mon probleme, c'etait vraiment tout bete.

Par contre le

Code:

---

for /f "tokens=*" %%i in (c:\batch\liste.txt) do call :ziper "%%i"

---

ça me met des guillements autour de ma variable et donc ça pete.
C'est ce que j'ai mis le plus de temps à trouver : ne pas avoir les guillements et garder les noms avec des espaces.

Citation:

---

et pour info, y'a encore beaucoup à faire.

---

Bah là c'est bon ça fonctionne nickel, tu voyais quoi pour optimiser ?

Merci pour votre aide. Vraiment tres reactif sur ce forum :ccool:

Citation:

---

Envoyé par psychoboust

tu voyais quoi pour optimiser ?

---

Ben je vois surtout des erreurs:

En ce qui concerne ton problème avec les guillemets, dans l'hypothèse où liste.txt n'en contient pas,

Code:

---

1 2 3 4 5 6 7 8 9 10

for /f "tokens=*" %%i in (c:\batch\liste.txt) do call :ziper "%%i" goto fin :ziper set fichier=%~1 .../... goto :eof :fin

---

ensuite plutôt que de supprimer les "/" dans le formatage du chemin autant les remplacer par "+" par exemple, ou encore ":" par "#", etc. Tu peux même générer un identifiant spécifique (qui réduirait la taille du nom) vu que tu as une traçabilité avec un log.

D'ailleurs dans ton script, à un moment tu le génères et ensuite tu l'écrases, je suppose que ce n'est pas intentionnel, donc c'est une erreur.

Enfin, l'intérêt d'avoir un log par fichier est plutôt discutable, le mieux serait d'avoir un seul log dans lequel tu consignerais toutes les données. L'avantage serait bien entendu la possibilité d'y effectuer des recherches, cela peut même en accroitre le potentiel et autoriser l'opération inverse, rapatrier des archives et les décompacter à la volée.

Aussi, je n'ai pas abordé la gestion des erreurs par le script ni même un système de suivi de la progression des opérations, mais comme tu le vois les possibilités de perfectionnement sont quasi infinies, à toi d'en définir le degré.

Bonne continuation.