【バッチファイル】コードを綺麗に整理したい！

Question

下記のようなバッチファイルを作成しました。
機能上は問題ないですが、見栄えが気になります。

実行内容はそのままに綺麗にするのは、
どんなコードがおすすめでしょうか？

【コード原文】

:top

timeout 1
rem

pushd C:\Users\mika.hattori\Desktop\bat

for %%F in (*.csv) do for /f "tokens=8 delims=," %%A in (%%F) do if "%%A" == "1" move %%F target

popd

goto top

:top

timeout 2
rem

robocopy C:\ML C:\Users\mika.hattori\Desktop\bat\backup /E /DCOPY:DAT

goto top

Answer 1

topというラベルが2つあるとか、前半で無限ループなので後半には行かないとかが、おかしい点ですが、
それ以外だと、改善すべき点は、pushd/popdをループの外に出すくらいでしょうか。

機能面としては、
「機能上は問題ない」とお書きなので大丈夫なのでしょうが、もし対象ファイル中に第8カラムが1のものが複数行あった場合には、moveが複数回実行されて、2回目以降はファイルが無いというエラーになりますね。

Comments

[退会済みユーザー]

ご回答ありがとうございます。

コードの通り、2種類の動作を違うタイミングでループさせたいと思いました。
試運転では問題なさそうに感じましたが、改善が必要みたいですね・・・

pushd/pupdをループ外にし、
後半moveの複数回実行エラーについては再度検証します。

[退会済みユーザー]

下記のように修正しました。
欲を言えば、2種類の動作実行のタイミングを異なる時間にしたいです。

もし何かご教示頂けることがありましたら宜しくお願い致します。

【修正後】
pushd C:\Users\mika.hattori\Desktop\bat

:top

timeout 10
rem

for %%F in (*.csv) do for /f "tokens=8 delims=," %%A in (%%F) do if "%%A" == "1" move %%F target

robocopy C:\ML C:\Users\mika.hattori\Desktop\bat\backup /E /DCOPY:DAT

goto top

popd

[otn]

> 欲を言えば、2種類の動作実行のタイミングを異なる時間にしたいです。

意味がよく分かりませんが、別々に処理したいのなら、別々に処理すれば良いだけでは？