组织批处理代码

Question

我尝试过的所有搜索，但我不能孤单你的代码清晰易读。

目前我的编码要么是尝试缩进的随机数组，要么是完全没有缩进的平线编码。

有没有人知道任何关于如何停止意大利面条式编码和单调编码的链接、一般规则或示例？

Answer 1

几年前我学到的一件事是对你们的 cmets 慷慨。不仅适用于您自己，也适用于任何将审查您的代码的人。我通常只会在文件的开头插入一个注释来描述批处理文件的主要功能是什么，以及一些功能，只是为了解释将要发生的事情或正在发生的事情。至于缩进，通常应该缩进诸如 IF 或 FOR 之类的东西。这是一个简单的例子。 :: 检查当前登录用户的用户名。

@ECHO OFF

:: Declare variables
SET Something=abc
SET SomethingElse=def
SET FinalNumber=20

:: Start of batch file

:GetUsername
:: Get username of currently logged in user, and output to file
CLS
ECHO %USERNAME%>Test.txt

:CheckAdmin
:: Check if username = Admin, Echo numbers in a FOR loop
CLS
IF "%USERNAME%" EQU "Admin" (
    ECHO Admin is currently logged in
    ECHO Printing numbers 0-%FinalNumber%
    FOR /L %%I IN (0,1,%FinalNumber%) DO (
        ECHO %%I
        ECHO %%I>>Test.txt
    )
) ELSE (
    ECHO Admin is not logged in
    ECHO Printing numbers %FinalNumber%-0
    FOR /L %%I IN (%FinalNumber%,-1,0) DO (
        ECHO %%I
        ECHO %%I>>Test.txt
    )
)
:: Wait 5 seconds, than loop through again, because why not?
ECHO %Something%
ECHO %SomethingElse%
TIMEOUT>NUL /T 5
GOTO :GetUsername

虽然我不是“专家”，但希望这有助于为您解决一些问题。如您所见，对于 cmets，我更喜欢使用 :: 而不是 REM，并且我还在各种功能上设置了标签 :GetUsername。有人会说这是对的，这是错的，我觉得随心所欲。

Answer 2

我想说，你能做的，是你可以把一些部分作为一个位，像这样：

@echo off
echo Hello. Type in your name.
set /p "name"
if %name%==bob123 goto login
if %name%==joe321 goto login

然后，在另一部分，更多的部分。