Внутри Case не присваивается значение глобальной переменной, При вызове функции внутри Case Опции

[Димон Безпарольн...]

Весьма запутано, постараюсь объяснить.

Код

            case(OpenTCP):                        //Открытие TCP
                {                                //
                    printf("\rOpen TCP Session\r");//    
                    GSM_State_Mashine = ConnMQTT;//Успешно
                    GSM_Mashine_Errors = 0;        //
                    int T =    GPRSConnect1(TCP, MQTT_BROKER_SERVER, PORT, DEFAULT_TIMEOUT*8, DEFAULT_INTERCHAR_TIMEOUT*5);
                    if(!T)                         //
                        {                        //Не успешно
                            printf("\rError TCP Open... Try Again...\r");//    
                            int T =    GPRSClose();//
                            GSM_Mashine_Errors++;//
                            break;                //
                        }                        //
                    GSM_State_Mashine = ConnMQTT;//Успешно
                    GSM_Mashine_Errors = 0;        //
                    printf("\rTCP Session Open\r");//
                    break;                        //

Если ставить GSM_State_Mashine = ConnMQTT после вызова GPRSConnect1, то в переменной GSM_State_Mashine остается старое значение и этот case выполняется вечно.

Сама функция GPRSConnect1 и вызываемые ей функции никакого отношения к переменной GSM_State_Mashine не имеет. Причем перед выходом из функции, содержащей этот switch (GSM_State_Mashine) присвоение есть, но за ней (снаружи этой функции) эта переменная имеет старое значение!

Пока вышел из положения присваивая значение GSM_State_Mashine = ConnMQTT; до вызова функции. Так работает. После вызова - нет. Что ж за глюк такой?

Переменная объявлена глобальной volatile unsigned int GSM_State_Mashine = 0;//

Сообщение отредактировал Димон Безпарольный - Jul 31 2017, 13:10

[Димон Безпарольн...]

Не оптимизация и ничего таккого. В функции GPRSConnect есть вызов itoa1(port, num);.

Поскольку этой функции нет в библиотеке, я взял что - то подобное в Сети. С ней что - то не так, поскольку как только я блокирую этот вызов (не забыв в GPRSConnect поставить выход с положительным результатом return 1), машина состояний исправно переходит на следующий шаг.

Вот с этой функцией что - то не так:

Код

//---------------------------------------------------------------------------------------------------
void reverse(char s[])                            //reverse:  переворачиваем строку s на месте
{                                                //
    int i, j;                                    //
    char c;                                        //
    for (i = 0, j = strlen(s)-1; i<j; i++, j--)
        {                                        //
            c = s[i];                            //
            s[i] = s[j];                        //
            s[j] = c;                            //
        }                                        //
}                                                 //
//---------------------------------------------------------------------------------------------------
void itoa1(int n, char s[])                        //itoa:  конвертируем n в символы в s
{                                                //
    int i, sign;                                //
    if ((sign = n) < 0)                            //записываем знак
    n = -n;                                        //делаем n положительным числом
    i = 0;                                        //
    do                                            //генерируем цифры в обратном порядке
        {                                        //
            s[i++] = n % 10 + '0';                //берем следующую цифру
        }while ((n /= 10) > 0);                    //удаляем
    if (sign < 0)                                //
    s[i++] = '-';                                //
    s[i] = '\0';                                //
    reverse(s);                                    //
}                                                //

Попробую другие варианты из

https://electronix.ru/forum/index.php?showt...=142582&hl=

С ней все так. Эта функция преобразует номер порта 1883 в текст. Так вот, в вызове itoa1(port, num); под нее объявлен массив char num[4];
А нужно 5. Вероятно не хватило места для нуля. При объявлении char num[5]; все заработало исправно.



Сообщение отредактировал Димон Безпарольный - Aug 6 2017, 15:11