hiob08
Goto Top

Probleme mit For-Schleife und Variablen

Hallo zusammen,

ich hab ein kleines Problem mit meinem Skript.
Zunächst zum Hintergrund:
Ich prüfe einen Ordner auf eine txt Datei. Ist diese vorhanden, prüfe ich noch ob die gleichnamigen pdf und
doc Dateien vorhanden sind. Ist dies der Fall, werden die Dateien gemoved.
Sofern sie nicht vorhanden sind, werden alle vorhandenen Dateien in ein Error Verzeichnis gemoved.

Hier der Quelltext:

@echo off

--- Hier werden die Ordnervariablen Error, Target , Log und Workdir gesetzt ----

for %%i in ("C:\Test\*.TXT") do (  
    if exist "%workdir%\%%~ni.pdf" (set pdf_ja="x")  
    if exist "%workdir%\%%~ni.doc" (set doc_ja="x")  

    if "%pdf_ja%"=="x" (  
	if "%doc_ja%"=="x" (  
  		 move %%~ni.pdf %target%
   		 move %%~ni.doc %target%
   		 move %%i %target%

			    ))

     else ( 
   		 echo %date% - %time:~0,8% : ERROR : Dateitrippel unvollständig  >> %log%
		 move %%i %error%
		 move %%~ni.pdf %error%
		 move %%~ni.doc %error%


 	  ) 		
)

Komischerweise landet das Skript immer im ELSE Zweig, auch wenn das Trippel vollständig ist. Die Dateien werden dann in das Error Verzeichnis verschoben.


Ich glaub ich seh einfach vor lauter Bäumen den Wald nicht face-smile
Hoffe, ihr könnt mir helfen

Content-Key: 112390

Url: https://administrator.de/contentid/112390

Printed on: April 19, 2024 at 14:04 o'clock

Member: Biber
Biber Mar 25, 2009 at 19:53:05 (UTC)
Goto Top
Moin hiob08,

für die CMD.exe ist alles, was innerhalb einer FOR-Anweisung oder auch innerhalb einer IF ..ELSE -Struktur passiert eine Zeile, eine Anweisung, auch wenn es aus Lesbarkeitsgründen auf 20 zeilen auseinadergezogen wird.
Und in einer Anweisung werden %variablen% eben nur einmalig aufgelöst (=durch Werte ersetzt), nämlich beim Parsen der Syntax VOR der eigentlichen Ausführung.

Wenn man/frau es anders haben will, also möchte, dass die Variablen nicht alle vor Beginn der Ausführung der ganzen Anweisung aufgelöst werden sollen, sondern innerhalb der Anweisung aktualisierte Werte zugewiesen bekommen, dann muss "DelayedExpansion" = "verzögerte Variablenauflösung" angekündigt werden.

@echo off & setLocal EnableDelayedExpansion

--- Hier werden die Ordnervariablen Error, Target , Log und Workdir gesetzt ----

for %%i in ("C:\Test\*.TXT") do (  
    if exist "%workdir%\%%~ni.pdf" (set pdf_ja="x")  
    if exist "%workdir%\%%~ni.doc" (set doc_ja="x")  

    if "!pdf_ja!"=="x" (  
	if "!doc_ja!"=="x"   
  		 move %%~ni.pdf %target%
   		 move %%~ni.doc %target%
   		 move %%i %target%

			    ))

     else ( 
   		 echo %date% - %time:~0,8% : ERROR : Dateitrippel unvollständig  >> %log%
		 move %%i %error%
		 move %%~ni.pdf %error%
		 move %%~ni.doc %error%


 	  ) 		
)
Außer der Ankündigung "setlocal EnableDelayedExpansion" sind die Variablen, die erst unmittelbar beim In-die-Hand-nehmen aufzulösen sind mit der Schreibweise !variable! statt %variable% zu schreiben.

Grüße
Biber
Member: hiob08
hiob08 Mar 26, 2009 at 08:48:16 (UTC)
Goto Top
Danke für deine Antwort..
Leider wird auch so immer der Else-Zweig gewählt..
Bin langsam ratlos face-sad
Member: Biber
Biber Mar 26, 2009 at 13:44:04 (UTC)
Goto Top
Moin hiob,

sorry, ich hatte gestern zu flüchtig gelesen.

Das nächste Problem ist natürlich, dass die Variablen anders gesetzt sind als sie geprüft werden.

Wenn Du setzt:
set pdf_ja="x"
... dann müsstest Du ja prüfen auf
"!pdf_ja!"==""x""

---> was es natürlich nicht lesbarer macht.

Bitte ändere bei den beiden SETs von pdf_ja und doc ja die Anführungszeichen so:

...
...
if exist "%workdir%\%%~ni.pdf" set "pdf_ja=x"  
if exist "%workdir%\%%~ni.doc" set "doc_ja=x"  

Grüße
Biber
Member: hiob08
hiob08 Mar 26, 2009 at 15:46:38 (UTC)
Goto Top
Danke für deine Hilfe... Jetzt klappt es super!

Aber ich verstehe nicht, warum er mir immer die beiden echos aus dem ELSE Zweig in die Logdatei schreibt, auch wenn das Trippel vollständig ist!?
Member: Biber
Biber Mar 26, 2009 at 16:37:48 (UTC)
Goto Top
Moin hiob08,

dann fasst die cmd.exe wohl die IF..ELSE-Konstrukte logisch anders auf als du sie optisch formatiert und auch gemeint hast.

Aber dieses Problem des Aneinandervorbeiredens ist nichts batchspezifisches...
Das hab ich jeden gottverdammten Tag mit meinen Kollegen auch ...
@echo off & setLocal EnableDelayedExpansion

--- Hier werden die Ordnervariablen Error, Target , Log und Workdir gesetzt ----

for %%i in ("C:\Test\*.TXT") do (  

    if exist "%workdir%\%%~ni.pdf"  (  
	if exist "%workdir%\%%~ni.doc" (   
  		 move %%~ni.pdf %target%
   		 move %%~ni.doc %target%
   		 move %%i %target%
                                         )
                       )  else ( 
   		 echo %date% - %time:~0,8% : ERROR : Dateitrippel unvollständig  >> %log%
		 move %%i %error%
		 move %%~ni.pdf %error%
		 move %%~ni.doc %error%

 	  ) 		
)
Vielleicht so geklammert?

Grüße
Biber
Member: hiob08
hiob08 Mar 27, 2009 at 07:35:06 (UTC)
Goto Top
Super, so klappt es!!!
Danke dir, hast mir sehr geholfen!

P.S.: Lass dich nicht unterkriegen face-wink