c++ - 基本的 C++ 设计

Question

我对我的 C++ 类设计有疑问。我经常遇到这样的小问题，我希望得到一些关于什么更容易被接受的建议。

我有一些类通过 UDP 监视某些设备的温度。如果设备收到一个数据包，它会打印“x\n”到标准输出以表明它收到了一些东西。然后是检查那个数据包中的数据，验证数据没有显示设备温度过高。如果太高，我必须调用一些函数。如果不是，我必须调用其他函数。

我不确定我是否应该这样做:

enum temperature {TEMPERATURE_FINE, TEMPERATURE_EXCEEDED};

int main(int argc, char* argv[])
{
    std::vector<std::string> args(argv+1, argv + argc);

    if(!args.size())
        cout << "No parameters entered.\n";
    else
    {
        CTemperatureMonitor tempMonitor(args);

        if(tempMonitor.MonitorTemperature() == TEMPERATURE_EXCEEDED)
            tempMonitor.ActivateAlarm();
        else
            tempMonitor.DisableAlarm();
    }

    return 0;
}

其中 tempMonitor.MonitorTemperature() 调用 std::cout << "x\n" .所以std::cout << "x\n"内置于类中。

或者:

enum temperature {TEMPERATURE_FINE, TEMPERATURE_EXCEEDED};

int main(int argc, char* argv[])
{
    std::vector<std::string> args(argv+1, argv + argc);

    if(!args.size())
        cout << "No parameters entered.\n";
    else
    {
        CTemperatureMonitor tempMonitor(args);

        temperature tempExceeded = tempMonitor.MonitorTemperature();
        std::cout << "x\n";
        if(tempExceeded == TEMPERATURE_EXCEEDED)
            tempMonitor.ActivateAlarm();
        else
            tempMonitor.DisableAlarm();
    }

    return 0;
}

哪里std::cout << "x\n"不属于该类。

std::cout << "x\n"必须在调用 CTemperatureMonitor::ActivateAlarm() 之前发生和 CTemperatureMonitor::DisableAlarm() .

我知道这可能看起来非常简单和简单，但我经常想知道类(class)中究竟应该包含哪些内容。该类是否应该输出到标准输出？我做一个或另一个有什么区别吗？我是不是太迂腐了？

另外，顺便说一句，我知道全局变量被认为是不好的做法。我在主类和类中都使用温度枚举。我应该声明它两次，一次在 main 中，一次在 CTemperatureMonitor 类中，还是一次全局？虽然这个问题看起来很具体，但它实际上会为我解决很多其他问题。

谢谢。

Answer 1

首先，我想指出项目的规模各不相同，根据规模(和关键程度)的不同，建议实际上也会有所不同。所以首先是一条经验法则:

The size of the "framework" that you put in place (Logger, Option Parser, etc...) should probably not exceed 10% of the total program. After this point, it's just overkill. Unless it's the goal of the exercise!

话虽如此，我们可以开始查看您的实际问题。

---

Also, as an aside, I know global variable are considered poor practice. I use the temperature enum in both the main and the class. Should I declare it twice, once in main and once in the CTemperatureMonitor class, or once globally?

您实际上在此处弄错了变量 和类型。 temperature 是一种类型(枚举类型)。

通常，类型用作程序各个部分之间的桥梁，为此，所有这些部分共享相同的类型定义很重要。因此，对于类型，实际上声明它两次是不好的做法。

此外，并不是所有的全局变量都是邪恶的。全局变量是(共享状态)，但全局常量很好，并且通常扮演类似于类型的角色。

---

I know this might seem really minor and simplistic, but I'm often wondering what exactly should be part of the class. Should the class be outputting to stdout? Does it make any difference whether I do one or the other? Am I being to pedantic about this?

有两种输出:

• 日志输出，用于在遇到问题时进行诊断
• 真实输出，即程序所做的

根据程序的不同，您可能有两者之一，也可能没有。

从迂腐的角度来看，您通常不希望将它们混合在一起。例如，您可以完美地将日志记录发送到一个文件，或者在严重时发送 stderr，并使用 stdout 作为“有用”的东西。

这实际上在某种程度上插入了设计，因为您需要两个接收器:每个输出一个。

因为你有一个非常简单的程序，最简单的方法可能是在构造时简单地将两个不同的 std::ostream& 传递给你的类。或者，更简单，只有两个通用函数并使用(邪恶的)全局变量。

在较大的程序中，您可能会设计一个 Logger 类，它具有各种日志级别并提供特定的宏来注册(自动)日志行的函数名、文件名和行号。您还可能拥有一个轻量级日志记录机制，允许您在发布版本中禁用日志记录 DEBUG/DEV 级别跟踪。