你提供的代码是一个
使用 Flask 框架和自定义 `
MongoUtil` 类来从
MongoDB 查询问题与答案并渲染到前端页面的简单 Web 应用。下面我将：
1. **解释这段代码的功能**
2. **指出潜在的问题或可改进之处**
3. **给出一个完整、健壮的示例实现（包括 `
MongoUtil` 的可能实现）**
4. **提供相关建议和最佳实践**
---
### ✅ 代码功能解析
```
python
from your_code_here.
MongoUtil import
MongoUtil
from flask import Flask, render_template
app = Flask
(__name__
)
mongo =
MongoUtil
() # 初始化数据库连接工具
@app.route
('/'
)
def index
():
question_list =
mongo.query_question
()
return render_template
('index.html', question_list=question_list
)
@app.route
('/question/<question_
id>'
)
def question_detail
(question_
id=None
):
if question_
id:
question_answer_dict =
mongo.query_answer
(question_
id)
return render_template
('answer_list.html', question_answer_dict=question_answer_dict
)
if __name__ == "__main__":
app.run
()
```
#### 功能说明：
-
使用 Flask 创建一个简单的 Web 服务。
- `/` 路由：获取所有问题列表，传给 `index.html` 显示。
- `/question/<question_
id>` 路由：根据
ID 获取某个问题及其答案，传给 `answer_list.html` 渲染。
-
使用了 `
MongoUtil` 封装对
MongoDB 的操作。
---
### ⚠️ 存在的问题与风险
1. **缺少错误处理**：
- 如果 `question_
id` 不存在或格式错误，`query_answer
()` 可能抛出异常。
- `
mongo.query_answer
(question_
id)` 在 `if question_
id:` 后执行，但即使有
ID，查询也可能失败。
2. **未验证 `question_
id` 是否为有效
ObjectId**（如果是
MongoDB 默认 `_
id`）。
3. **`
MongoUtil` 实现未知**，需确保其正确初始化数据库连接。
4. **模板中变量假设存在**，若查询为空未做处理，可能导致模板渲染出错。
5. **没有 404 处理**：当问题不存在时应返回 404 页面。
6. **硬编码导入路径**：`your_code_here.
MongoUtil` 是相对路径，部署时容易出错。
---
### ✅ 改进后的完整示例
#### 1. 实现 `
MongoUtil.py`
```
python
#
mongo_util.py
from
pymongo import
MongoClient
from bson import
ObjectId
import os
class
MongoUtil:
def __init__
(self
):
# 从环境变量读取配置更安全
self.client =
MongoClient
(os.getenv
("
MONGO_URI", "
mongodb://localhost:27017/"
))
self.db = self.client["qa_database"] # 数据库名
self.questions_col = self.db["questions"]
def query_question
(self
):
"""查询所有问题"""
questions = []
for q in self.questions_col.find
({}, {"title": 1, "_
id": 1}
):
questions.append
({
"
id": str
(q["_
id"]
),
"title": q["title"]
}
)
return questions
def query_answer
(self, question_
id):
"""根据 question_
id 查询问题及答案"""
try:
obj_
id =
ObjectId(question_
id)
except Exception:
return None # 非法
ID
doc = self.questions_col.find_one
({"_
id": obj_
id}
)
if not doc:
return None
return {
"question": doc.get
("title"
),
"answers": doc.get
("answers", []
)
}
```
#### 2. 更新主应用文件（`app.py`）
```
python
# app.py
from flask import Flask, render_template
from
mongo_util import
MongoUtil # 修改导入路径
app = Flask
(__name__
)
mongo =
MongoUtil
()
@app.route
('/'
)
def index
():
try:
question_list =
mongo.query_question
()
if question_list is None:
question_list = []
except Exception as e:
print
(f"Error fetching questions: {e}"
)
question_list = []
return render_template
('index.html', question_list=question_list
)
@app.route
('/question/<question_
id>'
)
def question_detail
(question_
id):
if not question_
id:
return render_template
("answer_list.html", question_answer_dict=None
), 400
question_answer_dict =
mongo.query_answer
(question_
id)
if not question_answer_dict:
return render_template
("404.html", message="Question not found"
), 404
return render_template
('answer_list.html', question_answer_dict=question_answer_dict
)
# 错误处理
@app.errorhandler
(404
)
def page_not_found
(e
):
return render_template
("404.html", message="Page not found"
), 404
@app.errorhandler
(500
)
def internal_error
(e
):
return render_template
("500.html", message="Internal server error"
), 500
if __name__ == "__main__":
app.run
(debug=True
) # 开发模式开启 debug
```
#### 3. 示例 HTML 模板
##### `templates/index.html`
```html
<!DOCTYPE html>
<html>
<head><title>Questions</title></head>
<body>
<h1>All Questions</h1>
<ul>
{% for q in question_list %}
<li><a href="/question/{{ q.
id }}">{{ q.title }}</a></li>
{% endfor %}
</ul>
</body>
</html>
```
##### `templates/answer_list.html`
```html
<!DOCTYPE html>
<html>
<head><title>Answer Detail</title></head>
<body>
{% if question_answer_dict %}
<h1>{{ question_answer_dict.question }}</h1>
<ul>
{% for ans in question_answer_dict.answers %}
<li>{{ ans }}</li>
{% endfor %}
</ul>
{% else %}
<p>No answer available.</p>
{% endif %}
<a href="/">Back to list</a>
</body>
</html>
```
##### `templates/404.html`
```html
<h1>404 - Not Found</h1>
<p>{{ message }}</p>
<a href="/">Go Home</a>
```
---
### ✅ 最佳实践建议
| 建议 | 说明 |
|------|------|
| ✅
使用环境变量管理数据库连接 | 如 `
MONGO_URI=
mongodb://user:pass@host:port/dbname` |
| ✅ 添加日志记录 |
使用 `logging` 替代 `print` |
| ✅
使用蓝本（Blueprints）拆分路由 | 便于大型项目维护 |
| ✅ 添加输入验证 |
使用 `werkzeug` 或 `marshmallow` 校验参数 |
| ✅
使用连接池和上下文管理 | 确保连接复用，避免频繁创建 |
| ✅ 添加单元测试 | 测试 `
MongoUtil` 和视图函数 |
---
###
本文介绍了如何在Python中使用ObjectId，一种用于MongoDB的对象标识符。通过从bson库导入ObjectId，可以创建和操作MongoDB的文档ID。
扫一扫
1465
被折叠的 条评论
为什么被折叠?
到【灌水乐园】发言
