C / C++ / MFC

acerunner316 wrote:

I need FunctTask1,2,&3 to be able to access other member variables and functions that aren't static, so making it static wont work.

BYTE CModule::FunctTask1(BYTE Param1, BYTE Param2)
{
    // access CModule variables and functions
}
 
static BYTE CModule::FunctTask1(BYTE Param1, BYTE Param2, CModule *pThis )
{
    // call the non-static version
    return pThis->FunctTask1(Param1, Param2);
}
 
void CModule::ManageTasks(BYTE Param1, BYTE Param2, BYTE (*ptrFunct)(BYTE, BYTE), CModule *pThis )
{   
    ... //unrelated code removed   
    switch (ptrFunction(1, 1, pThis))   
    {   
        case 0: // handle results   
        case 1: // handle results   
    }   
 
    ... //unrelated code removed
}
 
void CModule::OnSelchangeCombobox()
{   
    switch(m_Combobox->GetCurSel())   
    {   
        case 0: ManageTask(1, 1, FunctTask1, this); break;   
        case 1: ManageTask(1, 1, FunctTask2, this); break;   
        case 2: ManageTask(1, 1, FunctTask3, this); break;   
    }
}

DavidCrow wrote:

You're not out of luck just yet. Two plausible solutions still exist. One is to change the third parameter of ManageTasks() to include CModule. I don't have the syntax readily available, but it's not uncommon and can easily be searched.

acerunner316 wrote:

Having to overload every single one seems inefficient to me.

typedef unsigned char (CModule::*FUNCPTR)(unsigned char,unsigned char);
ManageTask(1,1,&CModule::FunctTask1);

(this->*ptrFunct)(0,0);<br />
 -- modified at 18:42 Thursday 1st November, 2007

CModule : public CDialog
{
// Construction
public:
   CModule (CWnd* pParent = NULL); // standard constructor
...
private:
   unsigned char FunctTask1(unsigned char, unsigned char);
   unsigned char FunctTask2(unsigned char, unsigned char);
   unsigned char FunctTask3(unsigned char, unsigned char);
   typedef unsigned char (CModule::*PFunctTask_t)(unsigned char, unsigned char);
   void ManageTasks(unsigned char, unsigned char, PFunctTask_t);
};

void CModule ::ManageTasks(unsigned char Param1, unsigned char Param2, unsigned char (*ptrFunct)(unsigned char, unsigned char)
{
   ... //unrelated code removed

   switch ((this->*ptrFunction)(1,1))
   {
   case 0:
      //handle results
   case 1:
      //handle results
   }

   ... //unrelated code removed
}

void CModule::OnSelchangeCombobox()
{
   switch(m_Combobox->GetCurSel())
   {
   case 0: ManageTask(1, 1, &CModule::FunctTask1); break;
   case 1: ManageTask(1, 1, &CModule::FunctTask2); break;
   case 2: ManageTask(1, 1, &CModule::FunctTask3); break;
   }
}

Peter Weyzen wrote:

Are you doing it this way because you don't want to use the support already in windows to do this?

It's fairly easy...

Peter Weyzen wrote:

You could have answered "no". But you had to get made and respond with 4 times as many words as I did.

You also were not clear in your intent.

I tried to help.

Lame-o!