cpp codeabschnitt Review



  • Hallo Zusammen,

    folgende codeabschnitt möchte ich Reviewen

    1. 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)])

    1. 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





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



  • Was gibt's da zu reviewen?



  • Was ist eure Meinung nach dieses Codeabschnitt:

    1. 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



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


Log in to reply