装饰器必须显式返回被包装函数的执行结果，否则默认返回 none；同时需用 isinstance(value, bool) 严格判断布尔 false，避免将合法数值（如 0）误判为错误。
装饰器必须显式返回被包装函数的执行结果，否则默认返回 none；同时需用 isinstance(value, bool) 严格判断布尔 false，避免将合法数值（如 0）误判为错误。
在 Telegram Bot 开发中，使用装饰器统一处理用户输入校验（如 @response_checker）是一种常见且优雅的实践。但若装饰器的 wrapper 函数未显式 return 被装饰函数的返回值，Python 会默认返回 None——这不仅破坏了逻辑链路，还可能导致后续流程异常（例如无法正确传递校验结果或中断状态流转）。
关键问题在于原 wrapper 的结构：
def wrapper(*args):
if func(*args): # ✅ 执行了 func，但未保存其返回值
# ... 成功分支
else:
# ... 失败分支
# ❌ 缺少 return 语句 → 隐式返回 None这导致无论 check() 返回 0、5 还是 False，wrapper 最终都返回 None。而 my_bot.register_next_step_handler() 的行为并不依赖 wrapper 的返回值，它只负责注册回调；真正影响流程的是：当用户发送错误输入后，Bot 发送 'Error notification' 并调用 register_next_step_handler(msg, check)——此时 msg 是一条系统生成的提示消息对象，但 check 函数实际等待的是用户下一次手动输入的消息，这是因为 telebot 的 register_next_step_handler 机制本质是“监听用户对上一条 bot 消息的首次回复”，而非将 msg 对象本身作为参数传入 check。因此，msg 只是触发监听的锚点，check 接收到的 message 始终是用户新发的文本消息，而非 msg 对象本身——这解释了为何逻辑看似矛盾却能正常运行。
但修复装饰器仍至关重要。以下是修正后的专业实现：
def response_checker(func):
def wrapper(*args):
result = func(*args) # ✅ 显式捕获返回值
if isinstance(result, bool) and result is False: # ✅ 严格判 False，避免 0/[]/'' 等误判
msg = my_bot.send_message(client_id, '请输入 0–10 之间的整数')
my_bot.register_next_step_handler(msg, check)
else: # ✅ result 为 int（如 0, 3, 10）或其他真值
my_bot.register_next_step_handler(args[0], get_course_rating)
return result # ✅ 必须返回，确保调用链完整性
return wrapper
@response_checker
def check(message):
global course_rating
try:
course_rating = int(message.text)
if 0 <= course_rating <= 10:
get_course_rating()
my_bot.send_message(client_id, 'Ready to memorize')
return course_rating # ✅ 明确返回合法整数
except (ValueError, TypeError):
pass
return False # ✅ 仅在此处返回布尔 False注意事项：
- ✅ 始终用
isinstance(result, bool) and result is False判断失败，而非not result； - ✅
wrapper必须return result，否则上游逻辑（如条件分支、日志记录、状态机）将丢失关键数据； - ✅
check函数内应避免重复解析message.text（原代码中try块已赋值course_rating，else中又int(message.text)属冗余）； - ✅ 将硬编码
client_id改为从message.chat.id动态获取，提升健壮性与可复用性。
通过此修正，装饰器既保持了横切关注点（错误处理、流程跳转）的解耦性，又严格保障了业务逻辑的准确性与可预测性。
