Struct에 대한 Accessor를 두기??

      1
      2 #include <stdio.h>
      3
      4 #ifdef USE_TRACE
      5 #define Trace(fmt, args...) \
      6     printf("[" __FILE__ ":%d:%s] " fmt, __LINE__, __func__, args)
      7 #else // !USE_TRACE
      8 #define Trace(fmt, args...)
      9 #endif // USE_TRACE
     10
     11 typedef struct {
     12     int data;
     13 } Context;
     14
     15 Context gvContext;
     16
     17 #ifdef USE_ACCESSOR
     18 Context*
     19 App_GetContext() {
     20     static Context context;
     21     return &context;
     22 }
     23
     24 #define Context_SetData(value) \
     25     Trace("Context_SetData(%d)\n", (App_GetContext()->data = (value)))
     26 #define Context_GetData() (App_GetContext()->data)
     27
     28 void
     29 example() {
     30     Context_SetData(1234);
     31     Trace("Context_GetData() = %d\n", Context_GetData());
     32 }
     33
     34 #else // !USE_ACCOSSOR
     35 void
     36 example() {
     37     gvContext.data = 1234;
     38 }
     39 #endif // USE_ACCESSOR
     40
     41 int
     42 main() {
     43     example();
     44     return 0;
     45 }

여러 사람이 같이 작업하는 embeded 환경의
프로젝트를 하다 보니
전역변수를 여기 저기 사용하게 되어 버렸습니다.
주로 printf로 찍히는 메세지로 간단한 디버깅을 하거나
T32로 디버깅을 하거나 합니다.
그래서 전역 변수를 위와 같이 accessor로 접근하게 하여
전역 변수를 수정하는 경우에 출력되게 하려고 하는데
괜찮은 방법인지 궁금합니다.

이게 다른 소스에서 보고 따라해 본건데 다른
장단점이 있는 지 궁금합니다.