Что такое хорошая практика для генерации подробного вывода? - PullRequest
13 голосов
/ 10 августа 2009

Что такое хорошая практика для генерации подробного вывода? В настоящее время у меня есть функция

bool verbose;
int setVerbose(bool v)
{
    errormsg = "";
    verbose = v;
    if (verbose == v)
        return 0;
    else
        return -1;
}

и всякий раз, когда я хочу генерировать вывод, я делаю что-то вроде

if (debug)
     std::cout << "deleting interp" << std::endl;

однако, я не думаю, что это очень элегантно. так что мне интересно, что было бы хорошим способом реализовать этот переключатель многословия?

Ответы [ 5 ]

12 голосов
/ 10 августа 2009

Самый простой способ - создать небольшой класс следующим образом (здесь есть версия Unicode, но вы можете легко изменить ее на однобайтовую версию):

#include <sstream>
#include <boost/format.hpp>
#include <iostream>
using namespace std;

enum log_level_t {
    LOG_NOTHING,
    LOG_CRITICAL,
    LOG_ERROR,
    LOG_WARNING,
    LOG_INFO,
    LOG_DEBUG
};

namespace log_impl {
class formatted_log_t {
public:
    formatted_log_t( log_level_t level, const wchar_t* msg ) : fmt(msg), level(level) {}
    ~formatted_log_t() {
        // GLOBAL_LEVEL is a global variable and could be changed at runtime
        // Any customization could be here
        if ( level <= GLOBAL_LEVEL ) wcout << level << L" " << fmt << endl;
    }        
    template <typename T> 
    formatted_log_t& operator %(T value) {
        fmt % value;
        return *this;
    }    
protected:
    log_level_t     level;
    boost::wformat      fmt;
};
}//namespace log_impl
// Helper function. Class formatted_log_t will not be used directly.
template <log_level_t level>
log_impl::formatted_log_t log(const wchar_t* msg) {
    return log_impl::formatted_log_t( level, msg );
}

Вспомогательная функция log был сделан шаблон для получения хорошего синтаксиса вызова. Тогда его можно использовать следующим образом:

int main ()
{
    // Log level is clearly separated from the log message
    log<LOG_DEBUG>(L"TEST %3% %2% %1%") % 5 % 10 % L"privet";
    return 0;
}

Вы можете изменить уровень многословия во время выполнения, изменив глобальную GLOBAL_LEVEL переменную.

9 голосов
/ 10 августа 2009
int threshold = 3;
class mystreambuf: public std::streambuf
{
};
mystreambuf nostreambuf;
std::ostream nocout(&nostreambuf);
#define log(x) ((x >= threshold)? std::cout : nocout)

int main()
{
    log(1) << "No hello?" << std::endl;     // Not printed on console, too low log level.
    log(5) << "Hello world!" << std::endl;  // Will print.
    return 0;
}
3 голосов
/ 20 января 2012

1. Если вы используете g ++, вы можете использовать флаг -D, это позволит компилятору определить макрос по вашему выбору.

Определение

Например:

#ifdef DEBUG_FLAG
 printf("My error message");
#endif

2. Я согласен, что это тоже не элегантно, поэтому, чтобы сделать его немного лучше:

void verbose(const char * fmt, ... )
{
va_list args;  /* Used as a pointer to the next variable argument. */
va_start( args, fmt );  /* Initialize the pointer to arguments. */

#ifdef DEBUG_FLAG
printf(fmt, &args);  
#endif
/*This isn't tested, the point is to be able to pass args to 
printf*/
}

То, что вы можете использовать как printf:

verbose("Error number %d\n",errorno);

3. Третье решение проще, и больше похоже на C ++ и Unix - передать аргумент вашей программе, который будет использоваться - как макрос ранее - для инициализации конкретной переменной (которая может быть глобальным постоянным).

Пример: $ ./myprogram -v

if(optarg('v')) static const verbose = 1;
3 голосов
/ 10 августа 2009

Вы можете обернуть свою функциональность в класс, который поддерживает оператор <<, который позволяет вам делать что-то вроде </p>

class Trace {
   public:
      enum { Enable, Disable } state;
   // ...
   operator<<(...)
};

Тогда вы можете сделать что-то вроде

trace << Trace::Enable;
trace << "deleting interp"
3 голосов
/ 10 августа 2009

Вы можете использовать log4cpp

...