Apr 04, 2011, updated at Oct 18, 2012 (UTC)

8393

If, else problem in For Schleife

Bin leider trotz Tante Google und Onkel Forum Suche noch keinen deut schlauer

Hallo zusammen,

ich versuche mich gearde an einem kleinen Batch Programm um lokal .pst Dateien zu suchen und auf ein Netzlaufwerk zu kopieren.

irgendwie habe ich aber einen Wurm mit rein gepackt und werd einfach nicht schlau.

Hier mal ein Stückchen Code

@echo off
set dir=c:\
set file=*.pst
set pfad=K:\Mailarchiv

for /r "%dir%" %%i in ("%file%") do (if exist "%%i" (echo xcopy "%%i" %pfad%) else (goto fehler))             
echo kopiert
pause
exit

:fehler
echo kein Archiv
pause
exit

wenn das Script pst Dateien findet, ist alles ok, dann werden die Dateien (in meinem TestFall 2 Stück) auch angezeigt/kopiert und danach wird das echo "kopiert" ausgegeben.

Wenn jedoch keine pst Datei gefunden wird, passiert sekundenlang nichts, dann wird das echo kopieren ausgegeben.

Er soll aber doch in die Else Verzweigung rein springen.

Ich habe es glaube ich nicht richtig verstanden, kann es mir bitte jemand erklären?

Vielen Dank

Grüße

Hoffi

Please also mark the comments that contributed to the solution of the article

Content-Key: 163892

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

Printed on: April 23, 2024 at 11:04 o'clock

26 Comments

Latest comment

Aloha,

nimm doch mal die Klammer vor if exist und nach fehler) weg.

greetz André

edit: Ach, mal wieder nur halb gelesen (Montags ...) das eigentliche Problem wird so ja nicht behoben

Hallo,

Hmm, ist immernoch das selbe, er springt NICHT in den Else Zweig.

Habe aus der Endung zum testen .pts gemacht.

@echo off
set dir=c:\
set file=*.pts
set pfad=K:\Mailarchiv

for /r "%dir%" %%i in ("%file%") do if exist "%%i" (echo xcopy "%%i" %pfad%) else (goto :fehler)              
echo kopiert
pause
exit

:fehler
echo kein Archiv
pause
exit

es passiert 5 Sekunden nichts, dann kommt die Bildschirmausgabe "kopiert"

es gibt definitiv keine Dateien mit .pts, also müsste er doch in den Else Zweig springen ????

Ich raffs nicht, sorry

Grüße

Hoffi

Aloha,

ja, heute ist Montag, da bin ich noch nicht wach *gg*

Es gibt kein else bei if exist ... und ob %%i nicht existen kann ... ~~glaube ein if not defined müsste machbarer sein. (wobei das ginge ja nur, wenn eine Variable gesetzt ist~~ ... heute ist wirklich Montag).

greetz André

moin,

01. @echo off

ist bei Problembätchen immer der Sinnvollste Weg, nicht auf das Problem zu kommen....

for /r "%dir%" %%i in ("%file%") do if exist "%%i"

bedeutet doch - für jeden Treffer mache - wenn Treffer existiert..

also auf neudeutsch doppelt gemoppelt

for /r "%dir%" %%i in ("%file%") do echo xcopy "%%i" %pfad% && goto ready
echo fehler
:ready

Was wäre denn der Unterschied zu dem oben skizziertem Weg (außer das der funktioniert?)

Gruß

Zitat von @60730:

Was wäre denn der Unterschied zu dem oben skizziertem Weg (außer das der funktioniert?)

Aloha T-Mo,

ganz einfach: Der wäre zu einfach, Montags muss es aber kompliziert sein

greetz André

edit: Warum kann man eigentlich if "%1" == "" echo leer aber nicht if "%%i" == "" echo leer machen ... *grübel* - Die Schleife arbeitet wohl nicht weiter, wenn %%i leer ist, schade eigentlich ^_^

Hallo,

das ist zu einfach

Nee, Dein Code klappt in sofern, das bei Nichtvorhandensein nun in die Sprungmarke gesprungen wird und bei Vorhandensein, führt er den kopierbefehl genau einmal aus.

Es gibt aber zwei dateien, also müsste er auch 2 mal kopieren, oder?

Wie komme ich also aus der Sprungmarke wieder zurück mit Call ??

dann würde er doch den Rest der For Schleife abarbeiten, also echo fehler anzeigen ???

Ich werd wahnsinnig

Hi,

HiHi, Wink -> Zaunpfahl

aber auch das habe ich schon probiert:

for /r "%dir%" %%i in ("%file%") do if "%%i" == "" goto fehler  
echo xcopy "%%i" %pfad%  
echo fertig kopiert
pause
exit

Hier springt er NICHT in die Sprungmarke fehler.....

Aloha,

@echo off & setlocal
set dir=C:\
set file=*.pst
set pfad=K:\Mailarchiv
set "Z="  

for /r "%dir%" %%i in ("%file%") do call :Ablauf "%%i"  
if not defined Z goto :fehler
echo kopiert
pause
goto :eof

:Ablauf
set "Z=%1"  
echo xcopy %Z% "%pfad%"  
goto :eof

:fehler
echo fehler
pause
goto :eof

Mahlzeit, André..

Hallo,

auch das auslagern der If Abfrage und Aufruf über Call funktioniert nicht:

set dir=c:\
set file=*.pts
set pfad=K:\Mailarchiv

for /r "%dir%" %%i in ("%file%") do call :archiv "%%i"  
echo "%%i"  
echo fertig
pause
exit

:fehler
echo kein Archiv
pause
exit

:archiv
if not exist %1 goto fehler
if exist %1 echo xcopy %1 %Pafd%

Wenn pst Dateien vorhanden sind, will er sie kopieren, kein Thema, aber wenn die nicht vorhanden sind, arbeitet er einfach die rets Befehle ab (echo "%%i" und echo fertig, pause, exit)

Aloha,

ich versteh deine Logik nicht, habe meinen code oben jetzt gepostet - sollte doch so mehr als ausreichend sein oder nicht?

Du versuchst zu prüfen ob leer nicht existiert, was keinen Sinn macht, da leer gar nicht erst übergeben werden kann.
Oder anders gesagt:

... die Schleife nimmt nur existierende Dateien auf ... warum willst du prüfen, ob eine existierende Datei existiert oder nicht?

greetz André

moin.........

Zitat von @Hoffi:

Hi,

HiHi, Wink -> Zaunpfahl

aber auch das habe ich schon probiert:

for /r "%dir%" %%i in ("%file%") do if "%%i" == "" goto fehler  
echo xcopy "%%i" %pfad%  
echo fertig kopiert
pause
exit

Übersetze
das doch bitte mal.......

Hier springt er NICHT in die Sprungmarke fehler.....

für jeden Treffer, der sich aus pfad und suffix zusammensetzt - wenn dieser "" lautet - gehe zu Fehler?

DENKFEHLER

Nee, Dein Code klappt in sofern, das bei Nichtvorhandensein nun in die Sprungmarke gesprungen wird und bei Vorhandensein, führt er den kopierbefehl genau einmal aus.

NEIN

Ich werd wahnsinnig face-sad

Dann zieh mich da aber bitte nicht mit rein.

Hallo,

ich habe Timos Code ausgeführt.

Es wird nur 1 Datei kopiert, es gibt aber 2 Dateien.

Die For Schleife muss doch so lange durchlaufen werden, wie es Dateien gibt, oder?

Beim ersten Durchlauf springt er in die Sprungmarke, dort könnte ich ein echo ausgeben, aber was passiert mit der 2. Datei, oder vielleicht gibt es auch 3 oder 4 Dateien?

Wie komme ich denn wieder zurück in die For schleife, sodass er alle gefundenen Dateien kopiert und DANN ein Fertig ausgibt.

Wenn er keine Datei findet, soll er einen Fehler ausgeben und beenden

Vielen Dank für Eure Geduld aber ich glaub ich steh mächtig auf dem Schlauch....

Grüße

Hoffi

Zitat von @Skyemugen:

Aloha,

ich versteh deine Logik nicht, habe meinen code oben jetzt gepostet - sollte doch so mehr als ausreichend sein oder nicht?

Du versuchst zu prüfen ob leer nicht existiert, was keinen Sinn macht, da leer gar nicht erst übergeben werden
kann.
Oder anders gesagt:

... die Schleife nimmt nur existierende Dateien auf ... warum willst du prüfen, ob eine existierende Datei existiert oder
nicht?

[OT]
wie oft hatten wir das Thema denn schon
"Ich "sichere" eine pst Datei und kümmere mich nicht darum, das Ausguck diese hoffentlich in dem Moment auch nicht benutzt?
mindestens 2*.....
[/OT]

greetz André

retour

Zitat von @Hoffi:

Hallo,

ich habe Timos Code ausgeführt.

Es wird nur 1 Datei kopiert, es gibt aber 2 Dateien.

Nein - es wird garnix kopiert - nur ein echo....

Die For Schleife muss doch so lange durchlaufen werden, wie es Dateien gibt, oder?

dafür wurden die gemacht

Beim ersten Durchlauf ...

LESE DAS!!!!!!!!!!!!!!!!!!!!!!!!!!

Vielen Dank für Eure Geduld aber ich glaub ich steh mächtig auf dem Schlauch....

bitte mach das echo off raus - nimm ne leere DOSBOX und lasse das laufen!

Grüße

Hoffi

Aloha T-Mo,

[OT]
na dafür können wir ihm ja einen knallharten

set OL=outlook.exe
tasklist |find /i "%OL%" >nul
if %errorlevel% == 0 taskkill /IM %OL% /F /T

schenken, der bei falscher Anwendung zu Datenverlust führen kann *gg*
[/OT]

greetz André

@echo off
set dir=c:\
set file=*.pst
set pfad=K:\Mailarchiv

dir /B "%dir%"\%file% >NUL 2>NUL || goto:fehler  
for /r "%dir%" %%i in ("%file%") do echo xcopy "%%i" %pfad%             
echo kopiert
pause
exit

:fehler
echo kein Archiv
pause
exit

oder nur:

@echo off
set dir=c:\
set file=*.pst
set pfad=K:\Mailarchiv

xcopy "%dir%"\%file% %pfad%  
if %errorlevel% equ 1 echo kein Archiv
if %errorlevel% equ 0 echo kopiert
pause
exit

Hallo,

ich weis, wer den Schaden hat......

@echo off IST aus, Ausgabe in Dos Box sieht gut aus, kopiert wird trotzdem nur 1 Datei.

set dir=C:\
set file=*.pst
set pfad=K:\Mailarchiv

for /r "%dir%" %%i in ("%file%") do xcopy /E "%%i" %pfad%  && goto :Fertig  
echo Fehler
pause
exit


:Fertig
echo fertig
goto :eof

Dieser Code funktioniert bestens, wenn es 1 Datei gibt, bei 2 Dateien wird nur die erste kopiert !

@miniversum

Hallo,

Danke für Deinen Code...

da ich den genauen Pfad und Namen der Datei im Vorfeld nicht kenne (das Script soll auf User PC laufen) muss ich erst alle .pst Dateien suchen und dann nach einander kopieren.

das Suchen funktioniert, das kopieren leider nur bei einer Datei.

Vielen Dank

Gruß

Hoffi

Hallo,

Diese und andere eventualitäten habe ich bereits abgefangen !!

Mir geht es nur um diese bescheuerte For Schleife, die IMMER nur einmal ausgeführt wird !

Danke

Hallo,

Zitat von @Hoffi:
ich weis, wer den Schaden hat......

Jepp

@echo off IST aus, Ausgabe in Dos Box sieht gut aus, kopiert wird trotzdem nur 1 Datei.

for /r "%dir%" %%i in ("%file%") do xcopy /E "%%i" %pfad%  && goto :Fertig  

Wo hast du denn jetzt schon wieder das && hergeholt? Gabs das umsonst?

Dieser Code funktioniert bestens, wenn es 1 Datei gibt, bei 2 Dateien wird nur die erste kopiert !

Klar.

das Suchen funktioniert, das kopieren leider nur bei einer Datei.

Weder noch.

Gruß,
Peter

Hallo Peter,

for /r "%dir%" %%i in ("%file%") do echo xcopy "%%i" %pfad% && goto ready  
echo fehler
:ready

Diesen Code hatte Timo eingestellt.

Es soll die Platte C nach .pst dateien durchsucht werden und ALLE .pst dateien zum Pfad kopiert werden, DANACH wenn alles kopiert ist, soll eine Meldung ausgegeben werden.

Wenn keine .pst Datei gefunden wird, soll eine Meldung ausgegeben werden und das Script beendet.

Grüße

Hoffi

Hallo Hoffi!

Wenn ich das richtig verstehe und die Fehlermeldung nur ausgegeben werden soll, wenn keine einzige Datei gefunden wurde, sollte das (ungetestet) etwa so gehen:

@echo off
set dir=c:\
set file=*.pst
set pfad=K:\Mailarchiv

set "Gefunden="  
for /r "%dir%" %%i in ("%file%") do echo xcopy "%%i" %pfad% & set "Gefunden=True"             
if not defined Gefunden goto :fehler
echo kopiert
pause
goto :eof

:fehler
echo kein Archiv
pause
goto :eof

BTW: Ein "exit" ohne "/b" ist mE sinnlos ...

Grüße
bastla

moin,

ich nehm nix zurück - aber einen notwendigen klammersatz spendiere ich

das Suchen funktioniert, das kopieren leider nur bei einer Datei.

nein - wenn das kopieren, oder genauer - das anzeigen der kopieraktion erfolgreich war - dann springe zu ready - daher einen () drumherum.

Zitat von @Hoffi:

Hallo Peter,

for /r "%dir%" %%i in ("%file%") do (echo xcopy "%%i" %pfad%) && goto ready  
echo fehler
:ready

Diesen Code hatte Timo eingestellt.

und dabei die klammern um den do Teil verschlampert....

Es soll die Platte C nach .pst dateien durchsucht werden und ALLE .pst dateien zum Pfad kopiert werden,

yupp

DANACH wenn alles kopiert ist, soll eine Meldung ausgegeben werden.

Wenn keine .pst Datei gefunden wird, soll eine Meldung ausgegeben werden und das Script beendet.

Wie geschrieben - denk dran, dass pst ruckzuck in den Orkus fliegen, wenn Outlook geöffnet ist und man da was dran rumstrickt....

Grüße

Hoffi

Hallo,

Genau.

Vielen Dank,

es funktioniert.

Grüße

Hoffi

... ich sag dazu jetzt nichts ### 30 Zeichen

Hallo Timo,

Du wirst mich wahrscheinlich gleich lünchen

Dein Code:

@echo off
set dir=c:\
set file=*.pst
set pfad=K:\Mailarchiv

for /r "%dir%" %%i in ("%file%") do (echo xcopy "%%i" %pfad%) && goto ready  
echo fehler
pause
:ready
echo passt
pause

ergibt folgende Ausgabe:

xcopy "c:\Test_PST\PST_1.pst" K:\Mailarchiv  
passt
Drücken Sie eine beliebige Taste . . .

es gibt aber 2 Dateien PST_1 und PST_2 es wird aber nur 1 Datei "optisch" kopiert (echo)

und nun?

Grüße

Hoffi

Hallo Hoffi und T-Mo!

Natürlich wird nur eine Datei kopiert, da ja danach gleich die Schleife verlassen wird - daher wenn schon Klammern, dann am ehesten noch so (weitestgehend ungetestet):

(for /r "%dir%" %%i in ("%file%") do echo xcopy "%%i" %pfad%) && goto ready

[Edit] Nach einem kurzen Test würde ich sagen, der Errorlevel hängt davon ab, ob im letzten durchsuchten Ordner eine passende Datei gefunden wurde - daher wird das auch so nicht gehen ... [/Edit]

Grüße
bastla

German solved Question Batch, Shell Development

Hotly discussed

Check of ZFW Firewallgleixnerd - 5 Comments

Wireguard VPN on UDM Pro behind Fritzbox - Handshake did not completejstricker - 3 Comments

How to set up and configure a Linux GRE tunnelAlexWisha - 3 Comments

End of Support dates for Office 2016, 2019 Apps und Productivity ServersDani - 1 Comment