Django SQL Explorer中日志计数显示问题的分析与修复

在Django SQL Explorer项目中，开发者发现了一个关于查询日志清理功能的日志输出问题。该问题出现在explorer.tasks.snapshot_queries函数中，具体表现为日志信息未能正确显示将被删除的QueryLog对象数量。

问题现象

在清理过期查询日志时，系统会记录一条日志信息，预期格式应为：

Deleting 100 QueryLog objects older than 90 days.

然而实际输出却变成了：

Deleting <bound method QuerySet.count of <QuerySet [<QueryLog: QueryLog object (2585296)>, ...]> QueryLog objects older than 60 days.

问题根源

经过分析，问题的根本原因在于日志语句中直接引用了qs.count方法而非调用该方法。在Python中，当直接引用方法而不加括号调用时，返回的是方法对象本身而非方法的执行结果。

原代码为：

logger.info(f"Deleting {qs.count} QueryLog objects older than {days} days.")

正确的写法应该是调用count()方法：

logger.info(f"Deleting {qs.count()} QueryLog objects older than {days} days.")

技术影响

这个问题虽然不会影响实际的删除操作功能，但会导致日志信息难以阅读和理解，特别是：

1. 无法直观看到将被删除的记录数量
2. 日志中包含了不必要的QuerySet详细信息
3. 不利于后续的日志分析和监控

解决方案

修复方案非常简单，只需在count后添加括号使其成为方法调用即可。这个修改：

1. 保持了原有功能的完整性
2. 提高了日志的可读性
3. 不影响性能，因为count()方法本身就是优化过的

最佳实践建议

在处理Django QuerySet时，有几个值得注意的点：

1. 方法调用与属性访问的区别要明确
2. 日志输出应保持简洁和可读
3. 对于可能返回大量数据的QuerySet，使用count()比len()更高效
4. 在日志中包含关键操作的数量信息有助于问题排查

这个问题的修复体现了良好的开发实践：即使是看似微小的日志输出问题，也应该及时修正以保证系统的可维护性。