使用BeautifulSoup提取HTML表格状态与注释的异常问题排查

我一眼就看出问题所在了——你犯了一个典型的「列表 vs 单个元素」的比较错误，这才导致所有评论都跑到BLOCKED列表里。

错误根源分析 #

先看你原来的核心代码：

self.status = [column.findAll('td')[5].get_text() for column in self.table]
if self.status == "PASS":
    self.pass_.append(self.Comment)
elif self.status == "FAIL":
    self.fail_.append(self.Comment)
else:
    self.blocked_.append(self.Comment)

这里的self.status是一个列表（你用列表推导式生成了所有表格的状态值），但你直接把这个列表和单个字符串"PASS"/"FAIL"做比较——列表和字符串永远不可能相等，所以代码直接走到了else分支。而且你还把整个self.Comment列表塞进了blocked_，这也不符合你想要的「单个评论对应单个状态」的需求。

修复方案 #

你需要逐个遍历每个表格对应的状态和评论，分别判断分类。下面是两种直观的修复方式：

方式1：直接遍历表格元素（推荐，更直观）

不需要提前生成三个独立列表，遍历每个表格时直接处理：

# 先初始化三个空列表（如果还没初始化的话）
self.pass_ = []
self.fail_ = []
self.blocked_ = []

self.table = self.soup_file.findAll(class_="table")
for column in self.table:
    # 提取当前表格的评论和状态，用strip()去除HTML里可能的空格/换行
    comment = column.findAll('td')[3].get_text().strip()
    status = column.findAll('td')[5].get_text().strip()
    
    # 逐个判断状态并添加到对应列表
    if status == "PASS":
        self.pass_.append(comment)
    elif status == "FAIL":
        self.fail_.append(comment)
    elif status == "BLOCKED":
        self.blocked_.append(comment)

方式2：保留原列表推导式，用zip遍历

如果你想保留之前生成的self.Macro、self.Comment、self.status列表，可以用zip把三个列表的元素一一对应遍历：

self.table = self.soup_file.findAll(class_="table")
self.Macro = [column.findAll('td')[1].get_text().strip() for column in self.table]
self.Comment = [column.findAll('td')[3].get_text().strip() for column in self.table]
self.status = [column.findAll('td')[5].get_text().strip() for column in self.table]

# 初始化空列表
self.pass_ = []
self.fail_ = []
self.blocked_ = []

# 逐个处理每组评论和状态
for comment, status in zip(self.Comment, self.status):
    if status == "PASS":
        self.pass_.append(comment)
    elif status == "FAIL":
        self.fail_.append(comment)
    elif status == "BLOCKED":
        self.blocked_.append(comment)

额外提醒 #

一定要用.strip()处理提取到的文本：HTML里的状态文本可能包含空格、换行符（比如实际是" PASS "而不是"PASS"），这会导致状态匹配失败，也是很多人容易踩的坑。

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