Problemas con un condicional

[aguml]

¿hay diferencia entre hacerlo de una manera o de otra a la hora de la comparación?
Me refiero a hacerlo asi:

#define ON_WRITE     288 // (PAGE_EXECUTE_READ | PAGE_GUARD)

o asi:

#define ON_WRITE PAGE_EXECUTE_READ | PAGE_GUARD

¿funciona de manera diferente a la hora de la comparación? El caso es que en el depurador, al parar en el condicional, podia ver como ON_WRITE tenia el valor correcto que debia tener pero no procesaba bien el valor. ¿en tu caso si lo haria? ¿acaso es un problema del #define que no trabaja bien como yo lo hice?

[ecfisa]

Hola aguml.

Empezado por aguml

¿hay diferencia entre hacerlo de una manera o de otra a la hora de la comparación?

Depende como hayas declarado la macro.

De este modo:

#define ON_WRITE PAGE_EXECUTE_READ | PAGE_GUARD

podría arrojar resultados no deseados debido a la precedencia de operadores. Declarada de ese modo, al expandirse en una evaluación tenemos que:

  if (Valor == ON_WRITE)

es equivalente a escribir:

if ( (Valor == PAGE_EXECUTE_READ) | PAGE_GUARD)

ya que el operador "==" tiene mayor precedencia que el operador "|". Y definitivamente no es lo que buscas...

La declaración que corresponde a tu propósito es:

#define ON_WRITE ( PAGE_EXECUTE_READ | PAGE_GUARD )

que al expandirse queda:

if (Valor == ( PAGE_EXECUTE_READ | PAGE_GUARD ) )

Comprobación:

#define ON_WRITE1  288
#define ON_WRITE2 (PAGE_EXECUTE_READ | PAGE_GUARD)

void __fastcall TForm1::Button1Click(TObject *Sender)
{
  if ( ON_WRITE1 == ON_WRITE2)
    ShowMessage("ON_WRITE1 es igual a ON_WRITE2");
  else
    ShowMessage("ON_WRITE1 es distinto a ON_WRITE2");
}

Saludos

[aguml]

pues no sabia que podia usarse parentesis en un #define y no habia pensado que pudiera ser problema de precedencia de operadores. Mañana hago la prueba y cambio el codigo. Muchisimas gracias por las aclaraciones, sois un foro cojonudo.