Q&A
goto FOR01を’）’の前にもっていきましたが、1行目のみ処理してそのまま処理終了してしまいました。。
バッチスクリプトで共有設定されているフォルダを解除するツールをつくりたいのですが、永遠に1行目の共有名しか取得できず、2行目以降の共有フォルダを解除できません。
バッチスクリプトについて、あまり理解できていないので原因調査も行き詰っております。原因わかる方いらっしゃいましたら、ご教授ください。
@echo off echo 共有フォルダチェックを行います。 rem 共有フォルダをテキストに出力 net share | find /V "$" > C:\tmp\com.txt rem 出力テキストの4文字目以降を出力 more +4 C:\tmp\com.txt > C:\tmp\comn.txt pause rem 最初の一時ファイルを削除 del C:\tmp\com.txt setlocal ENABLEDELAYEDEXPANSION rem :FOR01 for /f "delims= " %%i IN (C:\tmp\comn.txt) DO ( echo "共有名 " %%i " が存在します" SET comn=%%i set /p ANSWER="共有解除してよろしいですか（Y/N）?" if /i {!ANSWER!}=={y} (goto :yes) if /i {!ANSWER!}=={yes} (goto :yes) if /i {!ANSWER!}=={n} (goto :no) if /i {!ANSWER!}=={no} (goto :no) if !comn!=="" EXIT /B ) :no echo !comn!の共有解除を中止しました。& pause exit :yes net share !comn! /delete echo !comn! の共有が解除されました。 SET comn="" goto FOR01 echo 一時ファイルを削除します。 del C:\tmp\comn.txt
回答3件
0
ベストアンサー
yesの場合にgotoでは無くcallにして :yes の処理をサブルーチンっぽくしてみてください。
if /i {!ANSWER!}=={y} (goto :yes) if /i {!ANSWER!}=={yes} (goto :yes)
を以下に変更
if /i {!ANSWER!}=={y} (call :yes) if /i {!ANSWER!}=={yes} (call :yes)
:yesの最後を
SET comn="" goto FOR01
から
exit /B
まとめるとこんなコードになります。
・・・略・・・ for /f "delims= " %%i IN (C:\tmp\comn.txt) DO ( if !comn!=="" goto :finish echo "共有名 " %%i " が存在します" SET comn=%%i set /p ANSWER="共有解除してよろしいですか（Y/N）?" if /i {!ANSWER!}=={y} (call :yes) if /i {!ANSWER!}=={yes} (call :yes) if /i {!ANSWER!}=={n} (goto :no) if /i {!ANSWER!}=={no} (goto :no) ) :finish if exist C:\tmp\comn.txt del C:\tmp\comn.txt EXIT /B :no echo !comn!の共有解除を中止しました。& pause exit :yes net share !comn! /delete echo !comn! の共有が解除されました。 exit /B ・・・略・・・
投稿2016/11/14 10:09
編集2016/11/14 15:08
総合スコア7918
0
yes のときの処理で、解除後にfor ループの手前に戻ってしまっています。
結果、forループをやり直しますから、先頭行だけが出てくることになります。
投稿2016/11/14 09:46
総合スコア13707
0
’SET comn=""
goto FOR01’しているけど、FOR01の位置が’）’の前では？
投稿2016/11/14 09:42
総合スコア3778
いえいえ'FOR01'だけを’）’の前。tacsheavenさんも回答されていますが、forの外では最初の行からになりますので。')'だと次の行に行きますよ。
:FOR01をfor分の中に入れるということですか？
はい。forの最後、つまり')'の前に。
フローチャートを描けば（って古いか？）、何がおかしかったのかわかりやすいかもしれませんね。
本来必要なことは、yesの処理が終わったら、forループの中の、yesを呼び出した部分の直後に戻ることです。
tacsheavenさん、そうですよね。goto で':yes'に飛んでるから、処理の後、forの次の為の場所に帰ればOK。
for文の最後に:FOR01を記載すると、「コマンドの使い方が誤っています」とエラーになってしまいます。
あ、そうか。yes の処理の中で comn をクリアしちゃってるから、単純に直後に戻ると
if !comn!=="" EXIT /B
に引っ掛かりますね。
ループ末端の方がいいか。
’:FOR01
for /f "delims= " %%i IN (C:\tmp\comn.txt) DO (
echo "共有名 " %%i " が存在します"
SET comn=%%i
set /p ANSWER="共有解除してよろしいですか（Y/N）?"
if /i {!ANSWER!}=={y} (goto :yes)
if /i {!ANSWER!}=={yes} (goto :yes)
if /i {!ANSWER!}=={n} (goto :no)
if /i {!ANSWER!}=={no} (goto :no)
if !comn!=="" EXIT /B
)
’これを
’
for /f "delims= " %%i IN (C:\tmp\comn.txt) DO (
echo "共有名 " %%i " が存在します"
SET comn=%%i
set /p ANSWER="共有解除してよろしいですか（Y/N）?"
if /i {!ANSWER!}=={y} (goto :yes)
if /i {!ANSWER!}=={yes} (goto :yes)
if /i {!ANSWER!}=={n} (goto :no)
if /i {!ANSWER!}=={no} (goto :no)
if !comn!=="" EXIT /B
:FOR01
)
これでエラー出ますか？
javabigineerさん、そしてtacsheavenさん、だから’）’の前です。（念のため前の行を改行したが）
）の使い方が間違えています。
と出ます。なんか違うところがおかしいんですかね？
んー…もしかして、for ループから goto で脱出してしまってるので、戻れないかも。
yes も no も for ループ内に処理持ってこないとダメかな？
for の部分を書き換えるとこんな感じになります。
for /f "tokens=1 delims= " %%i IN (C:\tmp\comn.txt) DO (
SET comn=%%i
if %%i=="" EXIT /B
echo "共有名 " %%i " が存在します"
set /p ANSWER="共有解除してよろしいですか（Y/N）?"
set yn = 0
if /i {!ANSWER!}=={y} (set yn=1)
if /i {!ANSWER!}=={yes} (set yn=1)
if /i {!ANSWER!}=={n} (set yn=2)
if /i {!ANSWER!}=={no} (set yn=2)
if {yn}=={1} (
net share !comn! /delete
echo !comn! の共有が解除されました。
) else if {yn}=={2} (
echo !comn!の共有解除を中止しました。
)
set ANSWER=""
)
echo 一時ファイルを削除します。
これで一応は動きますが、最後に「コマンドは正常に終了しました。」という存在しない共有名が出てきます（最初の net share | find /V "$" の結果から取り除いていないため）
あとこれ、管理者権限のコマンドプロンプトからでないと net share /DELETE が効かないかも
管理者権限で実行させるために、自動昇格ってできるんでしたっけ？
あなたの回答
tips
プレビュー
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
2016/11/14 10:14
2016/11/14 10:19
2016/11/14 10:26
2016/11/14 10:33
2016/11/14 10:35
2016/11/14 10:35
2016/11/14 10:39