C++ cout 方法良好的编程风格

Question

你有什么问题，我应该在方法中使用cout << 吗？或者我应该返回一个数组？因为我需要不止一个返回value。

我有课程SGetraenkeAutomat，这是.h 文件

class SGetraenkeAutomat
{
    public:
        // this method
        void DisplayInventory();

        SGetraenkeAutomat();
        SGetraenkeAutomat(int nColaAnzahl, int nSpriteAnzahl, int nFantaAnzahl);
        virtual ~SGetraenkeAutomat();
    private:
        int m_nColaAnzahl;
        int m_nSpriteAnzhal;
        int m_nFantaAnzahl;
};

SGetraenkeAutomat.cpp中的方法void SGetraenkeAutomat::DisplayInventory()

void SGetraenkeAutomat::DisplayInventory(){
    std::cout << m_nColaAnzahl;
    std::cout << m_nSpriteAnzahl;
    std::cout << m_nFantaAnzahl;
}

这是一种好的编程风格吗？

-> 我不这么认为，但也许你们中的某个人可以解释一下。

Answer 1

或者我应该返回一个数组？

没有。该类有 3 个private 成员，最好不要将它们暴露在外面，这在大多数情况下会导致紧密耦合。

我应该在方法中使用 cout

如果它满足请求，应该没问题。

Answer 2

我宁愿将其重命名为 PrintInventory 并提供一个可选参数：

    void PrintInventory(std::ostream &output = std::cout);

通过这种方式，您可以将库存输出到其他流，同时您不必在每次要打印到标准输出时都指定它。

void SGetraenkeAutomat::PrintInventory(std::ostream &output){
    output << m_nColaAnzahl;
    output << m_nSpriteAnzahl;
    output << m_nFantaAnzahl;
}

这样你可以做到：

automat.PrintInventory(); // prints to stdout

或类似的东西

automat.PrintInventory(std::cerr); // prints to stderr, for example

Answer 3

在这种情况下，因为该方法被称为DisplayInventory，所以人们不会期望它返回任何东西，但实际上以某种方式向用户显示库存。这符合Tell, Don't Ask“规则”，即您应该告诉您的对象该做什么，而不是向他们询问有关其状态的问题。

在一个简单的应用程序中，使用cout与用户交互是完全可以接受的；另一种选择是使用 GUI 来显示信息。

您也可以这样看待：要将库存显示到标准输出，您必须使用coutsomewhere。 （在我看来）将它封装在一个类方法中肯定比用一个 getter 暴露所有数据（从而打破“告诉，不要问”）并在类之外打印它更好。

如果您想要更花哨并获得可定制的输出，您可以创建某种格式化程序类并将其传递给显示函数（Sergey 的回答是该原理的变体）。

Answer 4

答案取决于您的任务。如果您需要显示变量而不是使用一些流来显示。如果您的程序中的其他类需要来自您的 SGetraenkeAutomat 类的数据，那么您应该为每个私有成员提供公共 getter。例如：

int get_ColaAnzahl()
{
 return  m_nColaAnzahl;
}

正如我所见，在您的情况下创建一些基类会更好（让我们将其命名为 Drink）：

struct Drink {
int count;
std::string name;
}

然后将您的 SGetraenkeAutomat 类修改为：

class SGetraenkeAutomat
{
    public:
        // this method
        void DisplayInventory() {
           for (auto drink: drinks) {
              std::cout << drink.name << ": " << drink.count << std::endl;
           }
        }

        void addDrink(const Drink& drink) {
          drinks.push_back(drink);
        };
        SGetraenkeAutomat();
        virtual ~SGetraenkeAutomat();
    private:
        std::vector<Drink> drinks;
};

我想你理解我上面描述的想法。