Functions to replace the preprocessor with #defines

Hello everybody!!!
I need help... I think this could have a simple solution, but i'm not getting :(...
I have this following code:

Code:

---

#define PRINT_ME

//(...)

#ifdef PRINT_ME
#define PRINT1(s1,s2) printf(s1,s2)
#define PRINT2(s1,s2,s3) printf(s1,s2,s3)
#else
#define PRINT1(s1,s2)
#define PRINT2(s1,s2,s3)
#endif

---

this works fine when i need to use like this

Code:

---

void test()
{
        PRINT1("Hello number: %d", 1);
        PRINT2("Hello both: %d and %d", 1, 2);
}

---

But, now I need to forget the '#defined' and use a global variable to control the PRINT_ME state (because during the execution of my application, i need to change the state of the #define...therefore i can't use #define anymore...i need to use a global variable). And change the PRINT1, and PRINT2 to functions.
Something like this:

Code:

---

bool print_me = false;

void setPrint(bool status)
{
        print_me = status;
}

bool getPrint()
{
        return print_me;
}


template < typename T >
void PRINT1(char *s1, T s2)
{
        printf(s1,s2);
}

template <class T, class U>
void PRINT2(char *s1, T s2, U s3)
{
        printf(s1,s2,s3);
}


void test()
{
        if(getPrint() == true)
        {
                PRINT2("number: %d and %d", 1, 2); //don't work:-(
        }
}

---

I was trying to use templates...because the "printf" function allows any type of variable (%d, %s, %x, %f, etc)..

Any ideias, or a tutorial that can help me accomplish this?

Thanks :-)

What possible value is there in those defines? Instead of typing PRINT1, you could just type printf, with exactly the same result. I mean, you don't even save on keystrokes to do it.

Are you using C or C++?

Code:

---

template <class T, class U>

---

This is clearly not C.

Quote:

---

Originally Posted by KBriggs

What possible value is there in those defines? Instead of typing PRINT1, you could just type printf, with exactly the same result. I mean, you don't even save on keystrokes to do it.

---

I'm sorry. This is C++.
I really need to use this defines as i described..i can't just use printf. I need a global variable that control if I want printf's or not. As before, I use to have those defines, but now I need a global variable to control.
But I don't know how can I use a function that accept this arguments:
("number = %d", 1)...

thanks

Moved to C++ board

Quote:

---

Originally Posted by Quor

i can't just use printf. I need a global variable that control if I want printf's or not.

---

In other words, your example is wrong. You don't want to do this:

Code:

---

void test()
{
    if(getPrint() == true)
    {
        PRINT2("number: %d and %d", 1, 2); //don't work:-(
    }
}

---

Rather, you want to do this:

Code:

---

void test()
{
    print("number: %d and %d", 1, 2);
}

---

Then depending on the value of a global boolean flag, this may or may not print anything at all. Is that correct?

A solution here is to use a variable argument list. Templates will not offer much since you are going to use printf anyway, which uses a variable argument list. (But see Boost.Format)

That said, why do you want to do this? If this printing is for debugging, then it is probably fine. But if it is actually part of your program logic, then this is bad: you're using a global variable that can potentially confuse a maintainer.

You can't use #define's or templates in the way you want.

The reason is that both expansion of macro and template instantiation have finished when the compiler is finished with your code.

When the program is run, it is too late to change the results of the compilation process.

If you want the macro to have behaviour that depends on the value of a variable, the macro itself needs to test that variable at run time. For example;

Code:

---

#define PRINT1(s1, s2) do {if (print_me) printf(s1, s2);} while (0)

---

I'll leave it as an exercise to work out why I wrapped the logic in do {} while (0)