cpp codeabschnitt Review

Hallo Zusammen,

folgende codeabschnitt möchte ich Reviewen

Variante:

void *m_pvBclass; // Bclass ist einen class
void *m_pvDclassPool; // DclassPool ist einen class
void *m_pvEclassHold; //EclassHold ist einen class

#define pmaGetBclass() ((Bclass m_pvBclass)
#define pmaGetDclassPool() ((DclassPool ) m_pvDclassPool) // soll ich so
#define pmaGetEclassHold(a) (&((EclassHold) m_pvEclassHold) [static_cast <int>(a)])

Variante:

Header File

Bclass* m_Bclass;
DclassPool* m_DclassPool;
EclassHold* m_EclassHold;

Bclass* pmaGetBclass();
DclassPool* pmaGetDclassPool();
EclassHold* pmaGetEclassHold( int a );

Cpp File

Bclass* pmaGetBclass(){
  return m_pvBclass;
}

DclassPool* pmaGetDclassPool(){
  return m_pvDclassPool;
}

EclassHold* pmaGetEclassHold( int a ){
  return &pvEclassHold[a];
}

Eure Meinung wurde mich interessieren
Danke in voraus

reicht es jetzt langsam?

www.mikrocontroller.net/topic/393349

https://www.c-plusplus.net/forum/337243-full

das wars mal: http://stackoverflow.com/questions/36093300/feedback-on-my-spaghetti-code

http://codereview.stackexchange.com/questions/123526/how-can-i-write-the-macro-method-better-please-feadback

http://pastebin.com/pp6JciVk

Sorry für die Umstände aber ist das ein Problem hier zu fragen?

theta

Was gibt's da zu reviewen?

Was ist eure Meinung nach dieses Codeabschnitt:

VarianteEins:

void *m_pvBclass; // Bclass ist einen class
void *m_pvDclassPool; // DclassPool ist einen class
void *m_pvEclassHold; //EclassHold ist einen class

#define pmaGetBclass() ((Bclass *) m_pvBclass)
#define pmaGetDclassPool() ((DclassPool *) m_pvDclassPool) // soll ich so
#define pmaGetEclassHold(a) (&((EclassHold*) m_pvEclassHold) [static_cast <int>(a)])

oder diese:
2) VarianteZwei:

Header File

Bclass* m_Bclass;
DclassPool* m_DclassPool;
EclassHold* m_EclassHold;

Bclass* pmaGetBclass();
DclassPool* pmaGetDclassPool();
EclassHold* pmaGetEclassHold( int a );

Cpp File

Bclass* pmaGetBclass(){
  return m_pvBclass;
}

DclassPool* pmaGetDclassPool(){
  return m_pvDclassPool;
}

EclassHold* pmaGetEclassHold( int a ){
  return &pvEclassHold[a];
}

Es ist lästig und dafür entschuldige ich mich

Sorry für die Umstände aber ist das ein Problem hier zu fragen?

an sich nicht - ABER dein stoisches Posten der immer gleichen Frage - obwohl schon Diskussion zu deiner Fragen (auf anderen Plattformen) gelaufen sind - das ist ein Problem

1. was bedeutet für dich Review in diesem Kontext - ist doch viel zu wenig
Code für eine "Review" - was sind deine Review-Ziele?

2. versteht du den Code nicht?

3. versteht du den Inhalt der Diskussionen auf den anderen Plattformen nicht?

und,und,und...

dein Fragenziel ist einfach unklar? Wiederholung macht es nicht besser

Mechanics

Wieso schreibst du Kommentare überhaupt auf deutsch, wenns nicht deine Muttersprache ist? Oder sind die einfach so schlampig geschrieben wie der Code?