Node.js服务注册

Question

  register(name, version, ip, port) {
    this.cleanup();
    const key = name + version + ip + port;

    if (!this.services[key]) {
      this.services[key] = {};
      this.services[key].timestamp = Math.floor(new Date() / 1000);
      this.services[key].ip = ip;
      this.services[key].port = port;
      this.services[key].name = name;
      this.services[key].version = version;
      this.log.debug(`Added services ${name}, version ${version} at ${ip}:${port}`);
      return key;
    }
    this.services[key].timestamp = Math.floor(new Date() / 1000);
    this.log.debug(`Updated services ${name}, version ${version} at ${ip}:${port}`);
    return key;
  }

我觉得代码看上去有点业余。我不确定我是否在想象事情。难道没有更好的方法来分配值，以及键是如何生成的，我们不应该使用随机字符串生成器，或者在追加值之后至少对其进行编码吗？最好的做法是什么？

https://github.com/bluebrid/base-knowledge/blob/317bcdacd3a3806a6f98f05a7660dc97c506e8fb/javascript/advance/microservices/demo.1/service-registry/server/lib/ServiceRegistry.js

Answer 1

代码很清楚，我不认为是业余的。不过，可以做一些清理工作。

避免复制

时间戳代码是重复的，您可以为它创建一个函数：

const getTimestamp = () => {
    return Math.floor(new Date() / 1000)
}

您将在下面看到--我重构了寄存器方法，只设置了一次时间戳，但这仍然是一个一般性建议:不要重复相同的代码。

字符串构建

就我个人而言，我更喜欢使用反勾号从变量构建字符串：

const alpha = 'alpha';
const bravo = 'bravo';
const key = `${alpha}${bravo}`;

分配速记

ES6为对象属性分配提供了一个缩写。如果键的名称与作为属性传入的变量的名称相同，则只需指定键名。您的服务对象是这方面的最佳候选对象：

register(name, version, ip, port) {
    this.cleanup();

    const key = `${name}${version}${ip}${port}`;
    let operation = 'Updated'

    if (!this.services[key]) {
        operation = 'Added'
        this.services[key] = {
            name, version, ip, port
        };
    }
    this.services[key].timestamp = Math.floor(new Date() / 1000);

    this.log.debug(`${operation} services ${name}, version ${version} at ${ip}:${port}`);
    return key;
}