开发者您好。在今天的文章中，我将分享如何花费我一个小时来查找和修复一行代码的故事。让我们开始吧。
该项目最初是在spring boot 2.6.4开发，目前运行在spring boot 3.2.3版本。我将版本升级到了spring boot 3.2.3，因为我遇到了一些问题，这些问题阻碍并延迟了即将推出的功能。我会在另一篇文章中讲述升级之旅。请务必关注以获取有关即将发布的文章的通知。
问题
我在一家新公司开始了我的se之旅，并加入了一个有趣的项目。该项目包含多个调度程序，每个调度程序以不同的时间运行，例如每 10 秒、30 秒、30 分钟等。如果您有此类项目的经验，那么您已经知道发现问题有多么困难。
好吧，让我们直接进入花费我几个小时调试的那行。
public void sendchatrequest(long chatid) {
chatservice.findchatbyname("demo-chat").orelse(save(new chat("demo-chat")));
// other codes ommited...
}
你能发现问题吗？
无论聊天是否存在，每次都会调用 save() 方法。这是因为 orelse() 方法急切地求值。另一方面，orelseget() 方法是延迟计算的，只有当optional为空时才会执行。
为了了解它们之间的区别，请在 ide 中执行以下代码并查看日志。
首先，我们将返回空的optional，看看结果会是什么。
public class orelseandorelsegetmethods {
record chat(string name) {
}
class chatservice {
public optional findchatbyname(string name) {
return optional.empty();
}
}
private final chatservice chatservice = new chatservice();
public static void main(string[] args) {
var main = new orelseandorelsegetmethods();
main.testorelse();
system.out.println("*************************");
main.testorelseget();
}
public void testorelse() {
system.out.println("testing orelse() method");
chatservice.findchatbyname("demo-chat")
.orelse(save(new chat("demo-chat")));
}
public void testorelseget() {
system.out.println("testing orelseget() method");
chatservice.findchatbyname("demo-chat")
.orelseget(() -> save(new chat("demo-chat")));
}
public chat save(chat chat) {
system.out.println("saving " + chat);
// saving chat to database
return chat;
}
}
上述代码片段的输出如下所示：
testing orelse() method saving chat[name=demo-chat] ********** testing orelseget() method saving chat[name=demo-chat]
正如您在上面看到的，在两个方法（orelse() 和 orelseget()）中都执行了 save() 方法。
让我们看看如果optional 不为空会发生什么。
public class orelseandorelsegetmethods {
record chat(string name) {
}
class chatservice {
public optional findchatbyname(string name) {
// searching inside db
var foundedchat = new chat(name);
return optional.of(foundedchat);
}
}
private final chatservice chatservice = new chatservice();
public static void main(string[] args) {
var main = new orelseandorelsegetmethods();
main.testorelse();
system.out.println("**********");
main.testorelseget();
}
public void testorelse() {
system.out.println("testing orelse() method");
chatservice.findchatbyname("demo-chat")
.orelse(save(new chat("demo-chat")));
}
public void testorelseget() {
system.out.println("testing orelseget() method");
chatservice.findchatbyname("demo-chat")
.orelseget(() -> save(new chat("demo-chat")));
}
public chat save(chat chat) {
system.out.println("saving " + chat);
// saving chat to database
return chat;
}
}
上面的整个代码片段与之前相同，只有第 11 行不同。返回的不是optional.empty()，而是optional.of(foundedchat)。
来看看结果吧
testing orelse() method saving chat[name=demo-chat] ********** testing orelseget() method
正如你在输出中看到的，在执行 orelse() 方法时调用了 save() 方法，但是，这并没有发生在 orelseget() 方法上。
为什么要花这么多时间来检测和解决问题？
我没有看到一些意外情况，但 20 分钟后我意识到，数据库上的聊天请求比预期的要多。当时是测试环境，当时我的队友正在服务器上工作。但他们告诉我他们那天没有碰服务器上的任何东西。
调试会话开始了，有两件事让我很挣扎。
调度程序太多了。
缺乏日志记录，尤其是发生更改的方法。
为了解决这个问题，我将方法从 orelse() 更改为 orelseget()。
该问题之后项目的改进
第一个改进是将调度程序数量从 5 个减少到 1 个。我观察并发现这些调度程序几乎具有相同的目的，因此可以将它们合并到一个调度程序操作中。另外，其中一篇已过时并立即删除。
第二个改进是在方法中添加了不言自明的日志语句，以方便调试过程。
该问题之后的经验教训
当问题根源找到时，我意识到我已经看了这个类两次但第一次找不到。这是因为我很着急并试图尽快解决问题。然而，这花了我更多的时间来找出问题所在。
下次遇到这种问题的时候，大声读每一行或者自言自语，这样代码就会变得更清晰。
当你开始将上述策略应用到你的工作中时，你会发现很多问题或写得不好的代码。这将使您有优势找到需要重构的地方。
这是上述示例：
- You: Okay, in the first line of the method, it do null checks - You: if it is null, then return immediately, otherwise - You: send the method parameter to method findByName() - You: and findByName() returns Optional.
你明白这个想法。当你做到以上的时候，不要着急，否则不会有那么大的影响。要有耐心，慢慢来。
文章到此结束，喜欢文章的话别忘了分享和评论哦
如果您有任何疑问，可以通过 linkedin 联系我。
终于在 github 上提供了代码。
