C++温度控制器类中创建FreeRTOS任务的报错排查与原理疑问

Hey there! Let's walk through your questions and code issues clearly, since you're transitioning from C to C++ and working on a thermostat controller with FreeRTOS.

Why does the task function need to be static? #

The root issue here is how C++ handles non-static member functions:

• Every non-static member function has an implicit this pointer passed as its first argument behind the scenes. This means the actual signature of void Controller::normal_operation_task(void*) is more like void Controller::normal_operation_task(Controller* this, void* argument).
• FreeRTOS's xTaskCreate expects a function pointer of type void(*)(void*)—a plain function that takes only one void* parameter. The non-static member function's signature doesn't match this, hence the compiler error about "invalid use of non-static member function".
• Static member functions, on the other hand, don't have an implicit this pointer. Their signature aligns perfectly with what xTaskCreate requires, which is why marking the task function static fixes the compilation issue.

Why do we need to pass the controller object pointer to access class members? #

• Static member functions belong to the class itself, not to any specific instance of the class. They don't have access to the implicit this pointer, so they can't directly reference non-static (instance-specific) members (like your private test1 variable).
• By passing the object pointer to the static task function, we give it a way to target a specific instance of Controller. Using that pointer, we can access all the instance's members (including private ones, since static functions are still part of the class and have access to private scope).

What's wrong with the alternative task implementation? #

Let's look at your problematic code snippet:

void Controller::begin(){
printf("Initialise the object with the default values \n");
xTaskCreate(this->normal_operation_task3,"thermostat normal operation task",10000,&controller_obj,1,NULL); 
}
void Controller::normal_operation_task3(void* parameters) {
Controller controller_obj = *((Controller*)parameters);
for(;;) {
printf("normal_operation_task\n");
vTaskDelay(1000/portTICK_RATE_MS);
}
}

This has two critical issues:

1. Signature mismatch (compiler error)
   normal_operation_task3 is a non-static member function, so it still has that implicit this pointer. Passing it directly to xTaskCreate will throw the same "invalid use of non-static member function" error you saw earlier—FreeRTOS can't accept a member function pointer here.

2. Unnecessary object copy (logic error)
   Even if you somehow got past the compiler error, Controller controller_obj = *((Controller*)parameters); creates a copy of your original controller_obj instance. Any changes you make to this copied object won't affect the original instance, which defeats the purpose of having a class-based controller. You should instead use a pointer or reference to the original object:

   // Corrected task logic (if we fixed the static issue first)
   static void Controller::normal_operation_task3(void* parameters) {
       Controller* controller_obj = static_cast<Controller*>(parameters);
       for(;;) {
           // Now you can access the original object's members, e.g.:
           // controller_obj->test1 = some_value;
           printf("normal_operation_task\n");
           vTaskDelay(1000/portTICK_RATE_MS);
       }
   }
   

内容的提问来源于stack exchange，提问作者TheBestPlayer