在构造函数上传递参数或在每个方法上传递它们

Question

我正在学习 C++，我有这个头文件：

#ifndef _ROBOT_MOVE_H__
#define _ROBOT_MOVE_H__ 1

#include "ros/ros.h"

class RobotMove
{
private:
    double current_linear;
    double current_angular;
public:
    void TurnLeft(const ros::Publisher& publisher, ros::Rate& loop_rate);
    void TurnRight(const ros::Publisher& publisher, ros::Rate& loop_rate);
    void TurnAround(const ros::Publisher& publisher, ros::Rate& loop_rate);
    void Forward(const ros::Publisher& publisher, ros::Rate& loop_rate);
    void Backward(const ros::Publisher& publisher, ros::Rate& loop_rate);
    void Stop();

    RobotMove();
    ~RobotMove();
}


#endif

我不知道最好在构造函数上传递一次const ros::Publisher& publisher, ros::Rate& loop_rate，还是每次调用其他方法时传递它们。

在构造函数中传递这两个参数并在类中存储一个指针更好还是在每个方法上传递它们？

顺便说一下，这两个参数总是相同的对象。

Answer 1

顺便说一下，这两个参数总是相同的对象。

对我来说，这表明它们应该是类的成员变量。使它们成为类的成员变量遵循Don't Repeat Yourself (DRY) principle。因此，在构造函数中传递它们并让对象将它们存储为成员变量是有意义的。

Answer 2

这是一个很好的基本问题。

要回答这个问题，你可以问自己are these parameters specific to each action(method) or are they objects used by an instance of the class in general ?

在您的示例中，给出了您的类和方法的名称，事实上有这么多方法使用它们并且您还指定它们不会改变，我建议将它们传递给构造函数而不是每次调用。

但这并不总是正确的答案，您必须根据您认为这些参数可能更改的频率以及您希望赋予类的语义意义及其与这些参数的关系来判断和平衡。

我知道这听起来很浮夸，但设计选择通常取决于上下文。

Answer 3

我想说的是，从编程的角度来看，您需要一次又一次地复制和粘贴相同的代码越少越好。这就是编程的目的，让计算机为你重复一些事情。我会将这些参数传递给构造函数，然后将它们存储为属性。我相信你会像下面这样实现它：

class RobotMove
{
private:
    double current_linear;
    double current_angular;
    const ros::Publisher * publisher;
    ros::Rate * loop_rate;

public:
    RobotMove(const ros::Publisher& publisher, ros::Rate& loop_rate)
    {
        this->publisher = &publisher;
        this->loop_rate = &loop_rate;
    }
};