Как заменить это утверждение GOTO, чтобы было понятно - PullRequest
Новая
       28

Как заменить это утверждение GOTO, чтобы было понятно

0 голосов
/ 24 марта 2019

Я использую FREERTOS и tring для реализации мьютекса. Я хотел бы знать, как переписать его, чтобы мне не понадобился GOTO (потому что это плохая практика), или это действительный случай GOTO в США. Спасибо 

void mainThread(void const * argument) {

    uint8_t buff[100];

    writeMutex = xSemaphoreCreateMutex();
    if (writeMutex != NULL) {
        osThreadDef(compassReadThread, compassReadThread, osPriorityNormal, 0, 128);
        compassReadTaskHandle = osThreadCreate(osThread(compassReadThread), NULL);
    } else {
        HAL_GPIO_WritePin(GPIOA, GPIO_PIN_15, 1);
    }

    for (uint16_t i = 0; i < 50; i++) {
        if (xSemaphoreTake(writeMutex, (TickType_t) 100) == pdTRUE) {
            counter++;
            xSemaphoreGive(writeMutex);
            osDelay(10);
        }
    }


        ///////////////// THE UGLY GOTO PART /////////////////////////
    here:
    if (xSemaphoreTake(writeMutex, (TickType_t) 100) == pdTRUE) {
        if (counter != 110) {
            xSemaphoreGive(writeMutex);
            osDelay(1);
            goto here;
        }
    }
        //////////////////////////////////////////////////////////////

    snprintf((char*) buff, 100, "%d\n\r", (int) counter);
    HAL_UART_Transmit(&huart2, buff, strlen((char*) buff), 1000);
}

void compassReadThread(void const * argument) {
    for (uint16_t i = 0; i < 60; i++) {
        if (xSemaphoreTake(writeMutex, (TickType_t) 100) == pdTRUE) {
            counter++;
            xSemaphoreGive(writeMutex);
            osDelay(10);
        }
    }
}

1 Ответ

2 голосов
/ 24 марта 2019

Вы можете использовать цикл while так: 

while (xSemaphoreTake(writeMutex, (TickType_t) 100) == pdTRUE && counter != 110) {
    xSemaphoreGive(writeMutex);
    osDelay(1);
}
...