良好编程习惯中已发送消息的唯一标识符

Question

系统发送 xml 警告消息。为了将 xml 文件放在一起，创建了一个类，以便可以在将其转换为 xml 之前生成警告。每条消息都必须有一个唯一的 ID。 有多独特？ ID 从 3400 到 3500。如果达到最大值，则重置 ID。

#ifndef WARNINGMESSAGE_H
#define WARNINGMESSAGE_H
#include <ctime>

static unsigned int control_ID = 3399;
//Effect Codes:
const unsigned int TRAFFIC_FREE = 2;
//(...)
class WarningMessage{
 public:
  //setters
  void setActionID();

  //getters
  //(...)
 private:
  unsigned int actionID;    //to be controlled by control_ID
 };
#endif // WARNINGPARAMVEHICLE_H

而在cpp文件中，当需要设置消息的id时，执行如下：

#include "project/include/warningmessage.h"

//setters
void WarningParamVehicle::setActionID(){
 if(control_ID == 3500){
  control_ID = 3399;
 }
 control_ID++;
 actionID = control_ID;
}

另一个重要的细节是，该类消息的内存空间只给出一次。每次必须发送另一条消息时，实例都会被删除并重新创建：

void WarningDetector::createWarning(){
 delete currentWarningMessage;  // delete previous warning instance
 currentWarningMessage = new WarningMessage();
 (...)
}

最后，currentWarningMessage 在警告检测器中被声明为一个指针：

WarningMessage* currentWarningMessage;

我可以注意到在头文件中和类范围之外声明变量 control_ID 的不良编程习惯，但是，给定这样的场景编码看起来已优化。有没有更好的方法来解决这里看到的这个唯一标识符问题？提前非常感谢

Answer 1

良好的编程实践可以包括单一职责。例如，为当前消息设置动作 id 可以与决定下一条消息的动作 id 分离。这可以在构建 WarningMessage 期间发生异常时实现一致的行为（如果您有机会修复异常的原因，您可能希望再次尝试创建 WarningMessage 并期望相同的 actionId）。

要解决这个问题以及上面的其他 cmets，您可以：

• 将 control_ID 设为 WarningMessage 的私有静态成员（setActionID 当前是 WarningMessage 的成员）
• 移除 public setter setActionID 并在初始化列表中初始化 actionID
• 管理control_ID，构造函数中的最后一件事（以防出现异常）

在标题中：

#ifndef WARNINGMESSAGE_H
#define WARNINGMESSAGE_H
#include <ctime>

//Effect Codes:
const unsigned int TRAFFIC_FREE = 2;
//(...)
class WarningMessage{
public:
  WarningMessage();
  //remove public setters if your use case doesn't require them
  //getters
  //(...)
private:
  // deconstructing control_ID
  static const unsigned ID_COUNT = 100;
  static const unsigned ID_START = 3400;
  static unsigned idOffset = 0;
  const unsigned int actionID;    //to be controlled by control_ID
};
#endif // WARNINGPARAMVEHICLE_H

在cpp文件中：

#include "project/include/warningmessage.h"

unsigned WarningMessage::idOffset = 0;

WarningMessage::WarningMessage()
: actionId(ID_START + idOffset) /* ... */
{
    // increment idOffset only when you are sure that it is safe
    ++idOffset;
    idOffset %= ID_COUNT;
}

请注意，这不适用于多线程环境，但这似乎不是您目前关心的问题

Answer 2

我将创建 2 个常量 IDMIN = 3499 和 IDMAX = 3500 以避免在实现文件中出现幻数。这些常量应该是类WarningMessage 的静态常量成员。

我会在方法WarningParamVehicle::setActionID 中将control_ID 设置为静态变量（不是WarningMessage::setActionID 吗？）。然后它会给出：

class WarningMessage{
 public:
  static const int IDMIN = 3499;
  static const int IDMAX = 3500;
  //setters
  void setActionID();
  ...

及以后的实施：

void WarningParamVehicle::setActionID(){
 static int control_ID = WarningMessage::IDMIN;

 if(control_ID == IDMAX){
  control_ID = IDMIN;
 }
 control_ID++;
 actionID = control_ID;
}