Question

我有一个问题，似乎无法看到我出错的地方。

我有这个代码，用于遍历文本文件，并查看该行的最后一个单词，以查看该行开头的用户是否应该获得对其指定笔记本电脑的提升访问权限。

然而，代码只运行一次，然后退出。

@echo off
cls
echo.
echo The following script will process the entire list of students, giving each 
echo student administration rights to the laptop and set the laptop description 
echo where necessary.
echo.
pause
cls
FOR /F "tokens=1-6 delims=," %%a IN (accesselevations.txt) DO (
set School1=%%a
set LaptopID=%%b
set Model1=%%c
set Serial1=%%d
set User1=%%e
set Access1=%%f
If /i "%%f" EQU "Admin" goto Elevate2
If /i "%%f" EQU "NoAdmin" goto NoElevate2

:Elevate2
set Desc1=!School1! - !LaptopID! - !Model1! - !Serial1! - !User1!
echo.
echo Now creating local admin account for !user! on !LaptopID!
echo.
echo Description: !Desc1!
echo.
pause
psexec \\!LaptopID! -u GBN\!ocusr! -p !ocpw! -n 10 -e net localgroup Administrators "GBN\!User1!" /add
psexec \\!LaptopID! -u GBN\!ocusr! -p !ocpw! -n 10 -e net config server /srvcomment:"!Desc1!"

:NoElevate2
cls
Echo.
Echo User !user1! is not allowed Local Administrator rights on !LaptopID!.
pause
)
pause
:End

AccessElevations文件中包含机密数据，所以不幸的是我无法发布，但它包含类似的内容

School,WorkstationID,LaptopModel,LaptopSerial,StudentUsername,AdminOrNot
School,WorkstationID,LaptopModel,LaptopSerial,StudentUsername,AdminOrNot
School,WorkstationID,LaptopModel,LaptopSerial,StudentUsername,AdminOrNot
School,WorkstationID,LaptopModel,LaptopSerial,StudentUsername,AdminOrNot

......等等。

我希望我已经提供了足够的信息，我在这里很新。

提前感谢您在此问题上可以传播的任何亮点。

托比

Answer 1

使用setlocal enabledelayedexpansion，否则您将无法对变量使用感叹号。
在FOR

Elevate2

更改此内容：

If /i "%%f" EQU "Admin" goto Elevate2

If /i "%%f" EQU "NoAdmin" goto NoElevate2

要：

If /i "%%f" EQU "Admin" (CALL:Elevate2) else (CALL:NoElevate2)

使用CALL命令，在完成提升操作后，您的脚本将返回到FOR循环的末尾，GOTO命令无法执行此操作。

将exit/b放在:NoElevate2之前，否则您的脚本将无法返回CALL命令点。

您的代码应该是这样的：

@echo off
setlocal enabledelayedexpansion
cls
echo.
echo The following script will process the entire list of students, giving each 
echo student administration rights to the laptop and set the laptop description 
echo where necessary.
echo.
pause
cls
FOR /F "tokens=1-6 delims=," %%a IN (accesselevations.txt) DO (
set School1=%%a
set LaptopID=%%b
set Model1=%%c
set Serial1=%%d
set User1=%%e
set Access1=%%f
If /i "%%f" EQU "Admin" (CALL:Elevate2) else (CALL:NoElevate2))
:Elevate2
set Desc1=!School1! - !LaptopID! - !Model1! - !Serial1! - !User1!
echo.
echo Now creating local admin account for !user! on !LaptopID!
echo.
echo Description: !Desc1!
echo.
pause
psexec \\!LaptopID! -u GBN\!ocusr! -p !ocpw! -n 10 -e net localgroup Administrators "GBN\!User1!" /add
psexec \\!LaptopID! -u GBN\!ocusr! -p !ocpw! -n 10 -e net config server /srvcomment:"!Desc1!"
exit/b
:NoElevate2
cls
Echo.
Echo User !user1! is not allowed Local Administrator rights on !LaptopID!.
pause
exit/b

:End

Answer 2

@echo off
setlocal
cls
echo.
echo The following script will process the entire list of students, giving each 
echo student administration rights to the laptop and set the laptop description 
echo where necessary.
echo.
pause
cls
FOR /F "tokens=1-6 delims=," %%a IN (accesselevations.txt) DO (
set School1=%%a
set LaptopID=%%b
set Model1=%%c
set Serial1=%%d
set User1=%%e
set Access1=%%f
If /i "%%f" EQU "Admin" CALL :Elevate2
If /i "%%f" EQU "NoAdmin" CALL :NoElevate2
)
GOTO :EOF

:Elevate2
set Desc1=%School1% - %LaptopID% - %Model1% - %Serial1% - %User1%
echo.
echo Now creating local admin account for %user1% on %LaptopID%
echo.
echo Description: %Desc1%
echo.
ECHO psexec \\%LaptopID% -u GBN\%ocusr% -p %ocpw% -n 10 -e net localgroup Administrators "GBN\%User1%" /add
ECHO psexec \\%LaptopID% -u GBN\%ocusr% -p %ocpw% -n 10 -e net config server /srvcomment:"%Desc1%"
GOTO :eof

:NoElevate2
Echo.
Echo User %user1% is not allowed Local Administrator rights on %LaptopID%.
GOTO :EOF

注意：

setlocal以确保环境不会受到运行污染
设置Access1似乎没有多大意义，因为它没有被使用。
CALL :routine而不是goto - 对于goto内的cmd而言，这不是一个想法，因为某些%var%版本的工作方式不同。用于运行内部子程序

在内部子程序中，可以使用!var!，因为它们在自己的上下文中运行。

setlocal enabledelayedexpansion语法仅在psexec调用时有效（未在原始批处理中执行。）

ECHO命令只是echo。需要删除前面的psexec才能执行!user!

“Nodamin”不应该使用“删除权限”例程吗？

%user1%已更改为:elevate2中的user1，因为user是由循环建立的，而不是School1,Workstation1ID,Laptop1Model,Laptop1Serial,Student1Username,Admin School2,Workstation2ID,Laptop2Model,Laptop2Serial,Student2Username,NoAdmin School3,Workstation3ID,Laptop3Model,Laptop3Serial,Student3Username,Admin School4,Workstation4ID,Laptop4Model,Laptop4Serial,Student4Username,NoAdmin。

用于测试的合适数据文件是：

{{1}}

并不难构建 - 可以通过替换名称和其他敏感信息轻松完成更漂亮的版本。

循环不在批处理文件中循环

2 个答案: