多函数间输入传递问题：无法按预期输出字符串处理结果

你的核心问题是在capital()中调用main()导致重复触发用户输入，且未通过参数传递数据，导致效率低下、结果不符合预期。以下是满足你要求的重构方案：

重构思路 #

1. 拆分函数职责：让每个函数只专注一件事
   • 单独函数处理标点移除
   • 单独函数处理字符串转大写
   • main()统筹流程：获取输入、调用处理函数、输出结果
2. 通过参数传递数据，完全避免全局变量/常量

重构后的代码 #

def remove_punctuation(input_str, punctuation_chars):
    """移除输入字符串中的指定标点符号"""
    cleaned_str = ""
    for char in input_str:
        if char not in punctuation_chars:
            cleaned_str += char
    return cleaned_str

def convert_to_upper(input_str):
    """将字符串转为全大写（手动实现逻辑，也可直接用input_str.upper()）"""
    upper_str = ''
    for char in input_str:
        if 'a' <= char <= 'z':
            upper_str += chr(ord(char) - 32)
        else:
            upper_str += char
    return upper_str

def main():
    # 标点符号定义为局部变量，避免全局常量
    punctuation = "!@#$%^&*():<>?{}[]`\/~"
    # 仅获取一次用户输入
    user_input = input("Please enter a string: ")
    # 第一步：移除标点
    cleaned_string = remove_punctuation(user_input, punctuation)
    # 第二步：转大写
    upper_string = convert_to_upper(cleaned_string)
    # 按要求输出结果
    print(cleaned_string)
    print(upper_string)

if __name__ == "__main__":
    main()

关键改进点 #

• 仅触发一次用户输入，解决重复输入的问题
• 所有数据通过参数传递，无全局变量/常量
• 函数职责单一，代码更易维护和理解
• 可选优化：convert_to_upper函数可直接用Python内置方法简化：

  def convert_to_upper(input_str):
      return input_str.upper()
  

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