首先，咱们先聊聊现有代码的问题：

• 缺少范围校验：这是最核心的问题！isdigit()只能判断输入是纯数字字符串，但完全没考虑端口号的合法范围是1-65535。比如你输入0或者65536，isdigit()都会返回True，但这两个都不是有效端口。
• 虽然你的测试里-1被正确识别，但那只是因为负号不属于数字字符，isdigit()直接返回False，但如果输入的是超出范围的正整数，代码就会误判。

接下来是优化方案，我们需要确保输入是1到65535之间的整数，可以用异常处理来覆盖非整数输入的情况，再加上范围判断：

try:
    # 尝试把输入转成整数，失败会触发ValueError
    port_number = int(input("Enter a port number: "))
    # 校验端口的合法范围
    if 1 <= port_number <= 65535:
        print("This is a VALID port number.")
    else:
        print("This is NOT a valid port number.")
except ValueError:
    # 输入无法转成整数的情况（比如字母、小数、特殊符号）
    print("This is NOT a valid port number.")

优化后的代码能覆盖所有无效场景： #

• 输入非整数（比如a、12.3、!@#）→ 触发异常，返回无效
• 输入小于1的整数（比如0、-5）→ 范围判断不通过，返回无效
• 输入大于65535的整数（比如65536、100000）→ 范围判断不通过，返回无效
• 输入1-65535之间的整数（比如8、8080、65535）→ 返回有效

如果你还想额外处理带前导零的输入（比如0008），虽然转成整数后是合法的，但如果业务上不允许这种格式，可以在转整数前加个判断：

input_str = input("Enter a port number: ")
try:
    port_number = int(input_str)
    # 额外判断是否有多余前导零
    if len(input_str) > 1 and input_str.startswith('0'):
        print("This is NOT a valid port number.")
    elif 1 <= port_number <= 65535:
        print("This is a VALID port number.")
    else:
        print("This is NOT a valid port number.")
except ValueError:
    print("This is NOT a valid port number.")

不过一般来说，端口号是数值型的，前导零不影响实际功能，所以这个判断属于可选需求。

内容的提问来源于stack exchange，提问作者user9013730