代码走查

1.基础

---

python代码走查提示词优化

检查当前租户级诊断的实现，整体框架不变的情况下，评审代码。
1. python2 实现
2. 对边界场景，异常场景，数组越界，逻辑检查等场景做检查
3. 对整体的流程进行检查

---

2. AI优化

提示词名称：租户级诊断代码走查（Python2 实现）

审查背景
当前代码实现的是“租户级诊断”功能，整体框架保持不变的前提下，需要对代码质量、健壮性和正确性进行深度审查。项目使用 Python2 编写，请特别注意 Python2 特有的语法陷阱和运行时行为。

审查目标
在不重构整体架构的前提下，对现有实现进行逐行逐逻辑的评审，重点发现边界场景、异常处理、数组越界、逻辑缺陷、流程漏洞以及 Python2 相关风险。

审查范围

• 与租户级诊断直接相关的模块、函数、类方法
• 诊断数据的获取、处理、传递和输出环节
• 所有涉及列表/字典/字符串操作的代码路径
• 外部依赖调用、文件/网络 IO、数据库查询等可能出错的位置

---

一、核心检查维度

1. 边界场景与异常输入

• 所有输入参数是否校验了 None、空字符串 ""、空列表 []、空字典 {} 等空值？
• 数值参数是否处理了 0、负数、极大值 等边界？是否有除零风险？
• 字符串参数是否考虑 超长字符串、仅含空格/换行 的情况？
• 诊断对象（租户）不存在、被禁用、配置缺失时，代码路径是否可控？
• 集合操作（列表、字典）为空时，遍历、索引、切片是否安全？
• 递归或循环是否有明确的终止条件？是否会因异常数据导致死循环？

2. 异常处理与容错

• 是否所有可能抛出异常的操作都被 try...except 包裹？（尤其文件读写、网络请求、类型转换、字典取值等）
• except 是否捕获了具体异常类型，而非笼统的 except Exception 或裸 except:？
  （Python2 中裸 except: 会连 KeyboardInterrupt 和 SystemExit 一起吞掉，极易导致进程无法终止）
• 异常捕获后是否有足够的日志记录（包含上下文信息），避免问题难以追溯？
• 是否在 finally 中正确释放资源（文件句柄、连接、锁等）？Python2 中需注意 context manager 的可用性（with 支持有限）。
• 在诊断流程中，单个租户的异常是否会导致全局诊断中断？租户级隔离是否到位？

3. 数组越界与容器访问安全

• 列表/元组按索引访问前是否检查长度？如 lst[0] 或 lst[-1] 前是否有 if lst: 保护？
• 切片操作是否会因越界引起意外行为？（虽然 Python 切片不会抛异常，但可能返回意料之外的空结果）
• 遍历时修改被遍历对象是否可能引发错误？如删除列表元素导致索引错乱。
• 多维数据结构（列表嵌套）是否保证内层维度存在？
• 字符串切片、拼接是否考虑了多字节字符？（Python2 的 str 是字节串，需注意 unicode 与 str 混用）

4. 逻辑正确性与流程完整性

• 诊断步骤的顺序是否与业务预期一致？是否存在逻辑短路或提前返回导致关键检查被跳过？
• 所有 if-elif-else 分支是否覆盖所有可能状态？是否存在不可达代码或分支遗漏？
• 布尔判断是否正确？Python2 中 None、0、""、[]、{} 均被视为假，是否与业务意图一致？
• 比较运算符是否误用？如 is 与 == 混淆，Python2 中 is 比较整数或字符串可能因 intern 机制出现非确定性结果。
• 循环中累加、累乘、统计等操作，初始值是否正确？退出循环后变量状态是否符合预期？
• 多租户并发/顺序处理时，状态、变量是否隔离？是否存在全局变量或共享状态被意外篡改的风险？

5. Python2 特有陷阱检查

• 整数除法：3/2 结果为 1 而非 1.5。是否在有除法的地方引入 from __future__ import division 或显式转为 float？
• Unicode 处理：str 和 unicode 混用是否会导致 UnicodeDecodeError 或 UnicodeEncodeError？是否在字符串拼接/格式化时统一为 unicode？
• print 语句：若未来计划迁移至 Python3，是否应避免依赖 print 语句，使用 print() 函数并加 from __future__ import print_function？
• range/xrange：生成大列表时是否使用 xrange 以节省内存？
• 输入函数：若代码中使用了 input()，它会执行用户输入，是否存在严重安全隐患？应使用 raw_input()。
• 异常链：Python2 中捕获异常再抛出时，原始 traceback 可能丢失，是否使用 raise 或自定义保存？
• 比较任意对象：不同类型间的比较在 Python2 中不会抛出异常，可能返回稳定但无意义的顺序，是否可能导致隐蔽排序错误？
• 字典方法：.keys(), .values(), .items() 返回列表，大字典场景下是否有效率问题？可考虑 .iterkeys() 等迭代器版本。

6. 整体流程检查

• 诊断的入口和出口是否清晰？调用方是否能准确获取诊断成功/失败/部分成功的结果？
• 诊断结果的数据结构是否稳定？增加字段时是否会破坏旧版本兼容性？
• 是否有完整的日志链路，便于追踪单个租户的诊断过程？
• 对于长时间运行的诊断任务，是否有超时控制或进度上报机制？
• 诊断对下游系统（如数据库、配置中心）的依赖是否有重试、降级或熔断机制？

---

二、输出要求

请逐条列出发现的问题，并按以下格式说明：

• 文件:行号（若能定位）
• 问题类别（边界/异常/越界/逻辑/Python2陷阱/流程）
• 严重程度（致命/严重/一般/建议）
• 问题描述（具体的代码片段及风险）
• 修复建议（给出可操作的改进方案，并注意保持框架不变）

对于评审中未发现问题的检查项，可标注“通过”或简要说明。