重构常规c++代码模式

Refactor regular C++ code pattern

本文关键字：模式 代码 c++ 常规 重构      更新时间：2023-10-16

总结:我想看看我是否可以重构一些有规则模式的c++代码，使其更容易更新和维护。

:

我有一些代码创建线程本地计数器来跟踪程序执行期间的统计信息。目前，当一个统计数据被添加到源代码时，有5个东西需要更新:计数器线程局部声明，计数器总数声明，重置线程计数器的函数，将线程计数器添加到总数的函数，以及打印函数。

代码如下:

// Adding a statistic named 'counter'
// Declaration of counter
__thread int counter = 0;
int total_counter = 0;
// In reset function
counter = 0;
// In add function
total_counter += counter;
// In print function
printf("counter value is: %dn", total_counter);

我可以看到如何为计数器的声明创建宏，这样做:

#define STAT(name) __thread int name; 
                   int total_##name;

但是我还没有想到如何扩展它来更新add和reset函数。理想情况下，我想输入像STAT(counter)这样的东西，并有管理统计数据的所有声明和函数。

编辑:

我已经有了用于更新代码中的统计信息的宏。像STAT_INC(counter)这样的东西会增加本地计数器的值。然后，当线程完成执行时，它的线程本地值被添加到总体总数中。所以每个统计数据的名称很重要，这就是为什么数组不适合我。因为真正的计数器名称是像cache_hit这样的东西，它比counter[2]更有意义，而且我不想失去为创建的统计数据使用任意名称的能力。如果可能的话，只是为了简化在声明统计数据时必须编写的代码量。

这将使您在问题中描述的内容或多或少地封装在模板类中:

enum StatNames {
    STAT_rx_bytes,
    STAT_tx_bytes,
    //...,
};
template <StatNames SN>
class Stat {
    static const char *name_;
    static __thread int x_;
    static int total_;
public:
    Stat(const char *name) { name_ = name; }
    static void reset () { x_ = 0; }
    static void add () { total_ += x_; }
    static void print () {
        std::cout << name_ << " value is: " << total_ << "n";
    }
    static int & x () { return x_; }
    static int total () { return total_; }
};
template <StatNames SN> const char * Stat<SN>::name_;
template <StatNames SN> __thread int Stat<SN>::x_;
template <StatNames SN> int Stat<SN>::total_;
#define STAT(name) Stat<STAT_##name> name(#name)

然后可以编写如下代码:

STAT(rx_bytes);
void * test (void *)
{
    rx_bytes.x() += 4;
    rx_bytes.add();
    std::cout << pthread_self() << ": " << rx_bytes.x() << "n";
    return 0;
}
int main ()
{
    pthread_t t[2];
    pthread_create(&t[0], 0, test, 0);
    pthread_create(&t[1], 0, test, 0);
    pthread_join(t[0], 0);
    pthread_join(t[1], 0);
    rx_bytes.print();
}

(7分钟后无人接听…我收到礼物了!)

所以基本上你不需要五个独立的命名变量。可以使用数组或向量:

int counters[5];

那么更新某个计数器就很容易了:

class Counter {
    int counters[5];
    void update_nth(int n)
    {
        counters[n]++;
    }
};

与所有其他变量类似。

接着H2CO3的回答，有一个常见的成语，看起来像这样:

enum CounterEnums {
    MyFirstCounter,
    MySecondCounter,
    // ... add new counter names here ...
    NumCounters
};
class Counter {
    int counters[NumCounters];
public:
    void update(int n) { counters[n]++; }
};

现在，您可以轻松地添加另一个计数器，只需将其放在 NumCounters之前。现在，您可以声明您的实例，类似于:

Counter totals; // global
boost::thread_specific_pointer<Counter> counters; // per-thread
使用

和

counters->update(MyFirstCounter);

(你还需要一些方法来更新totals和零你的每线程计数器，但我会…

相关文章：