今天在整理我的个人博客网站时，发现存在一个逻辑错误：点击“上一篇博客”链接时，跳转的不是上一篇文章，而是直接跳到了第一篇博客。 经过排查，问题很快定位到 mapper 层的排序逻辑错误上。 为了修复此问题，我动手修改了代码。同时，考虑到这种情况在开源项目中很常见，我整理了这篇关于如何发现问题并提交 Pull Request（PR）的实践指南，希望对大家有所帮助。

1. 发现问题与复现过程

在参与开源项目时，若遇到 Bug 或 功能缺陷，一般流程如下：

1.1 问题描述

• 明确指出问题发生的具体场景。

• 说明是哪个页面、哪个功能模块出现异常。

1.2 复现步骤

• 列出清晰、简洁的复现步骤，使他人在相同环境下能快速重现问题。

例如：

1. 打开任意一篇博客文章详情页（例如ID=3）。
2. 点击页面底部的“上一篇”链接。
3. 观察跳转结果。

1.3 期望结果 vs 实际结果

• 期望：跳转到 ID=2 的文章（正确的上一篇）。

• 实际：跳转到 ID=1 的第一篇文章（逻辑错误）。

2. 本地 Fork 并修复问题

确认问题后，即可着手修复。

2.1 Fork并克隆仓库

git clone https://github.com/你的用户名/项目名.git
cd 项目名
git remote add upstream https://github.com/原作者用户名/项目名.git

2.2 创建新分支

git checkout -b fix-prev-article-link

2.3 定位并修改代码

找到 ArticleMapper.xml（或对应SQL文件），将原有的：

ORDER BY article_id
LIMIT 1

修改为：

ORDER BY article_id DESC
LIMIT 1

这样能够正确地选取当前文章 ID 更小的上一篇文章。

2.4 本地测试

• 重新编译、重启项目服务。

• 访问博客详情页，验证“上一篇”链接跳转是否符合预期。

3. 提交并发起 Pull Request

确认本地修改无误后，可以将变更提交至远程仓库并创建 PR。

3.1 提交改动

git add src/main/resources/mapper/ArticleMapper.xml
git commit -m "fix: 修复上一篇文章跳转逻辑，改为 DESC 排序"

3.2 推送分支

git push origin fix-prev-article-link

3.3 在 GitHub 创建 Pull Request

• 进入自己的仓库页面，点击 "Compare & pull request" 按钮。

• 填写规范的 PR 标题和详细描述。

示例：

• 标题：fix: 修复上一篇文章跳转逻辑

• 描述：

  ## 描述
  在博客文章详情页中，点击“上一篇”按钮时跳转到了第一篇文章。
  ​
  ## 修改
  修改了 SQL 查询中的排序逻辑，改为 `ORDER BY article_id DESC`，确保跳转到正确的上一篇文章。
  ​
  ## 测试
  已在本地环境测试，跳转逻辑正常。

提交后，等待原作者审阅并合并。

4. 通知原作者

除了自动生成的通知外，为了提高合并效率，可以在 PR 或 Issue 区域礼貌留言提醒，例如：

Hi @原作者用户名，已在 PR #5 修复了上一篇链接跳转问题，烦请审阅并合并，谢谢！

礼貌的沟通能让合作更加顺畅，也展现出你的专业素养。

5. 小结

总结一下流程：

1. 确认并复现问题，确保修改是必要且准确的。

2. Fork → 创建分支 → 修改 → 本地验证，保持主分支整洁。

3. 规范提交 Commit 与 PR，让变更一目了然。

4. 礼貌沟通，简洁明了地说明修改内容。

只要按照这套流程操作，不论是修复 Bug，还是新增功能，都能高效、专业地参与到任何开源项目中！ 希望这篇分享对你有所帮助，也欢迎大家一同投入开源，共同成长。