使裝飾者的代碼更清晰嗎？

Question

例如，我有這樣的功能，它執行了一些有用的工作（事件驅動仿真）：

int function() 
{ 
    do_useful_work(); 
    return 0; 
} 

如果我需要做我應該做的這個useful_work的性能測量：

int function() 
{ 
    count_time(time_before); 
    count_X_metrics(X_before); 

    do_useful_work(); 

    count_time(time_after); 
    count_X_metrics(X_after); 

    return 0; 
} 

這種方法使代碼更笨拙。有沒有辦法，撇開int function()以外的這些計數來使代碼更清晰？

Answer 1

你可以創建自己的裝飾像下面這樣：

#include<functional> 
#include <iostream> 

void count_time() {}; 
void count_X_metrics() {}; 

void decorator(std::function<void()> work) 
{ 
    count_time(); 
    count_X_metrics(); 

    work(); 

    count_time(); 
    count_X_metrics(); 
} 


void do_work_1() { 
    std::cout << "Hello, World 1!" << std::endl; 
} 

void do_work_2() { 
    std::cout << "Hello, World 2!" << std::endl; 
} 

int main() { 
    decorator(do_work_1); 
    decorator(do_work_2); 
} 

編輯：我不知道你的count_time和count_X_metrics功能是如何工作的，但如果你需要更復雜的東西，還是有辦法來跟蹤的狀態，你可以創建一個對象來爲你工作。這肯定是不同的比你需要的，但希望它傳達了我想說明一點：

#include<functional> 
#include <iostream> 

int current_time() { return 0; } 
int x_metric() { return 0; } 

class Timer { 
    public: 
    void time(std::function<void()> work) { 
     // Capture state before 
     int starttime = current_time(); 
     int startmetric = x_metric(); 

     work(); 

     // Capture state after 
     int endtime = current_time(); 
     int endmetric = x_metric(); 

     // Update results 
     ellapsed = endtime - starttime; 
     metric = endmetric - startmetric; 

     // Possibly do something with the metrics here. 
     // ... 
    } 

    int get_ellapsed() { return ellapsed; } 
    int get_metric() { return metric; } 

    private: 
    int ellapsed; 
    int metric; 
}; 

void do_work_1() { 
    std::cout << "Hello, World 1!" << std::endl; 
} 

void do_work_2() { 
    std::cout << "Hello, World 2!" << std::endl; 
} 

int main() { 
    Timer t; 
    t.time(do_work_1); 

    // Possibly do something with the metrics here. 
    // cout << t.get_ellapsed(); 

    t.time(do_work_2); 
}