Python编写密码生成器调用upper()无法生成大写字母如何解决

• 你写的letter_def函数根本没用上你传进去的参数：不管你调用的时候传了什么，函数内部都会根据随机生成的1-26的数字，给letter重新赋值成小写的a到z，你传的Capital_1.upper()直接被覆盖了，当然不会生成大写字母。
• 你还有个隐藏坑：用集合{}存密码字符，集合会自动去重，要是生成的字符有重复，集合长度就不够10，后面转列表按索引取的时候肯定会报索引越界的错。

不需要单独写大写字母生成函数，两种修改方案都能解决问题：

方案1：修改letter_def函数，增加大小写控制参数 #

同时可以把原来手写26个条件判断的冗余逻辑直接简化，用Python内置的字符库实现：

import random
import string

def letter_def(is_uppercase: bool = False) -> str:
    if is_uppercase:
        # 直接从大写字母库随机选字符
        return random.choice(string.ascii_uppercase)
    # 默认返回小写字母
    return random.choice(string.ascii_lowercase)

调用的时候要生成大写字母就传True参数即可：

# 生成大写字母
letter_def(is_uppercase=True)
# 生成小写字母
letter_def()

方案2：不修改原函数，调用时对返回值转大写 #

如果不想改动原有的letter_def逻辑，直接把upper()方法移到函数调用的外侧，对函数返回的小写字母转大写即可：

letter_def(Capital_1).upper(), letter_def(Capital_2).upper()

另外建议把存储密码的集合改成列表，避免去重导致的长度不足问题，输出密码时也不用逐个取索引拼接：

# 用[]代替{}存字符，不会去重
password = [symbol_def(symbol_1), symbol_def(symbol_2), number_def(number_1), number_def(number_2),
            letter_def(is_uppercase=True), letter_def(is_uppercase=True), letter_def(letter_1),
            letter_def(letter_2), letter_def(letter_3), letter_def(letter_4)]
# 直接拼接列表所有元素
print("your password is: " + "".join(password))

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