一行代码花了我一个小时来修复-java教程-PHP中文网

一行代码花了我一个小时来修复

WBOY

发布： 2024-07-11 11:52:16

转载

610人浏览过

开发者您好。在今天的文章中，我将分享如何花费我一个小时来查找和修复一行代码的故事。让我们开始吧。

该项目最初是在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<chat> 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<chat> 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 上提供了代码。

以上就是一行代码花了我一个小时来修复的详细内容，更多请关注php中文网其它相关文章！