Javascript - Правильный способ использования класса - PullRequest
Новая
       22

Javascript - Правильный способ использования класса

0 голосов
/ 05 мая 2020

Я написал клиент mqtt. js, но понял, что мне нужно несколько экземпляров этого скрипта, поэтому я переписал его в класс. Теперь у меня появилось много ссылок на this.foo для моих переменных, что заставило меня задуматься, является ли это правильный способ использования классов в javascript?

Вот как часть моего класса выглядит так: 

      this.client = mqtt.connect("mqtts://" + this.host, options); // mqtts for tls connection

      // client event handling
      this.client.on("connect", () => {
        log.write("LOG_INFO", "PubClient connected to Broker: " + this.host + ":" + this.settings.port);
        this.client.publish("client/publisher/status", "online", { qos: 1, retain: true });
        this.clientToSocket.pubConnect(true);
      });
      this.client.on("close", () => {
        //log.write("LOG_INFO", "PubClient close");
      });
      this.client.on("disconnect", (packet) => {
        log.write("LOG_INFO", "PubClient disconnected. Reasoncode: " + packet.reasoncode);

      });
      this.client.on("offline", () => {
        log.write("LOG_INFO", "PubClient offline");
        this.clientToSocket.pubConnect(false);
      });
      this.client.on("error", (err) => {
        //log.write("LOG_ERR", "PubClient Error: " + err.message);
      });
      this.client.on("packetreceive", (packet) => {
        //log.write("LOG_INFO", "PubClient packet received: " + packet.cmd);
      });
    }
  }

  publish(topic, qos, msg) {
    if(this.client !== undefined) {
      this.client.publish(topic, msg, { qos: parseInt(qos), retain: true });
      //log.write("LOG_INFO", "Publishing Message with: topic: " + topic + " payload: " + msg);
    } else {
      log.write("LOG_ERR", "Error: Publish not possible because client undefined");
    }
  }

  close() {
    if(this.client !== undefined) {
      this.client.end();
      log.write("LOG_INFO", "PubClient closing");
    }
  }
}

1 Ответ

0 голосов
/ 05 мая 2020

Вы можете упростить это, используя локальные переменные. Когда работа сделана, можно назначить на это. Вы также можете использовать реструктуризацию для использования переменных.

Local Var 

constructor() {
  const client = mqtt.connect("mqtts://" + host, options); // mqtts for tls connection

  client.on("close", () => {
    //log.write("LOG_INFO", "PubClient close");
  });

  this.client = client;
}

De-Structure: 

doSomthingOnClient() {
  const {client} = this // destrcuture
  client.get() // use it 
}

Весь класс: 

class Client {
  constructor() {
    const client = mqtt.connect("mqtts://" + host, options); // mqtts for tls connection
    client.on("connect", () => {
      log.write(
        "LOG_INFO",
        "PubClient connected to Broker: " + host + ":" + settings.port
      );
      client.publish("client/publisher/status", "online", {
        qos: 1,
        retain: true,
      });
      clientToSocket.pubConnect(true);
    });
    client.on("close", () => {
      //log.write("LOG_INFO", "PubClient close");
    });
    client.on("disconnect", (packet) => {
      log.write(
        "LOG_INFO",
        "PubClient disconnected. Reasoncode: " + packet.reasoncode
      );
    });
    this.client = client;
  }
  getClient() {
    return this.client;
  }
  doSomthingOnClient() {
    const {client} = this // destrcuture
    client.get() // use it 
  }
}
...