Гениальное открытие в синтаксисе языка Си, Не стесняемся в овациях :-) Опции

[GetSmart]

Только что изобрёл, пытаясь сэкономить код
Проверил - работает в CW и в IAR.

Код

//#define _DEBUG_LOOP_
#ifdef _DEBUG_LOOP_            /* для установки отладочных ловушек объявить этот символ */
#define DebugLoop(a)    a { DisableIRQ_FIQ(); while (1); }
#else
#define DebugLoop(a)    {}
#endif


int I2CEngine(uInt wrCnt, uInt rdCnt)
{
    long time_out = 100000;        // можно сделать зависимость от wrCnt и rdCnt

    I2CWrLength = wrCnt;
    I2CRdLength = rdCnt;
    I2CRdIndex = 0;
    I2CWrIndex = 0;
    if (I2CStart() == 0)    return I2C_ERROR;

    while (--time_out >= 0)
    {
        if (I2CMasterState >= I2C_SUCCESS)    // дальше I2C_SUCCESS все ошибки
        {
            DebugLoop(if (I2CMasterState != I2C_SUCCESS));            // <<<------------------------- обратите внимание на это
            return I2CMasterState;
        }
    }
    DebugLoop();
    return I2C_TIMEOUT;
}

Если это кто-то уже изобрёл до меня - дайте плиз ссылку. Хотелось бы убедиться

Сообщение отредактировал GetSmart - Sep 28 2009, 04:27

[GetSmart]

По поводу do {} while(0) в дефайнах. Кто-нить может объяснить чем это лучше простого {} ???

[Dima_G]

По поводу do {} while(0) в дефайнах. Кто-нить может объяснить чем это лучше простого {} ???

не ручаюсь за терминологию, но как-то так:
do {...} while (0) - выглядит один оператор
{...} - блок операторов

#define FUN do{...} while(0)
#define FUN2 {...}

Код

корректно:
if (..)
  FUN (x);
else
  FUN (2*x);

ошибка:
if (..)
  FUN2 (x);
else
  FUN2 (2*x);

Только в моей версии там не совсем параметр. И тем более не типизированный. Там может стоять любая строка сишного кода, в данном случае if (...)

Хоть убейте, не могу придумать задачу, где в параметре данного дефайна необходимо поставить что-то окромя булевского выражения

[GetSmart]

Хоть убейте, не могу придумать задачу, где в параметре данного дефайна необходимо поставить что-то окромя булевского выражения

Фантазии нету?
Например выполнить какие-нить предворительные действия перед зависанием, которые в режиме Release не будут даже компилироваться, а в Debug будут стоять. Как частный случай этого - оператор if

[Dima_G]

Фантазии нету?
Например выполнить какие-нить предворительные действия перед зависанием, которые в режиме Release не будут даже компилироваться, а в Debug будут стоять. Как частный случай этого - оператор if

Мда?
А можно пример?

[GetSmart]

Мда?
А можно пример?

Код

    DebugLoop(ShowDebugInfo(); if (I2CMasterState != I2C_SUCCESS));

Маленький минус - при какой-либо команде (без завершающего оператора if) внутри скобок придётся ставить точку с запятой.

Сообщение отредактировал GetSmart - Sep 28 2009, 10:09

[Сергей Борщ]

Код

    DebugLoop(ShowDebugInfo(); if (I2CMasterState != I2C_SUCCESS));

Код

assert(ShowDebugInfo(), I2CMasterState == I2C_SUCCESS);

[Палыч]

Код

assert(ShowDebugInfo(), I2CMasterState == I2C_SUCCESS);

Ну, и кому такое надо?
Процедура ShowDebugInfo() будет вызываться вне зависимости от выполнения условия I2CMasterState == I2C_SUCCESS.

Если уж автору исходного топика так хочется что-то (вызов неких действий) добавлять к его исходному действию по условию, то это делается элементарно. Несколько поправил исходный код:

Код

#define DebugLoop(a)    if(a) { DisableIRQ_FIQ(); while (1); }

Если мы желаем при выполнении условия ещё и процедуру ShowDebugInfo() вызвать, то следует записать

Код

DebugLoop(  (I2CMasterState != I2C_SUCCESS) ? (ShowDebugInfo(), TRUE) : FALSE )

где FALSE и TRUE соответственно ноль и не ноль.
Вот Вам и - пожалуйста: перед заходом в бесконечный цикл получите отладочную информацию.
Нечто подобное можно записать и в assert, однако аргумент его используется не только для проверки условия, но и для формирования строки вывода (не смертельно, конечно, но - некрасиво вывод будет смотреться).

P.S. В assert , конечно, вызов дополнительной процедуры нужно перенести к FALSE.

[Сергей Борщ]

Ну, и кому такое надо?
Процедура ShowDebugInfo() будет вызываться вне зависимости от выполнения условия I2CMasterState == I2C_SUCCESS.

GetSmartу. Его макрос делал то же самое. А в остальном вы все правильно расписали.

[GetSmart]

GetSmartу. Его макрос делал то же самое. А в остальном вы все правильно расписали.

Ну не знал я про assert. Вот, изобрёл велосипед, а люди вокруг запинали творческую личность Средневековье какое-то.

Знаю, что есть недостаток моего дефайна в том, что он не "ATOMIC", то есть нельзя перед ним написать if (...), хотя условие должно быть внутри (так и было задумано), но иногда из-за этого можно ненароком встать на грабли, если например мой дефайн с процедурой внутри написать сразу после else. Пришлось бы писать примерно так:

Код

if (...)  ++I2CMasterState;
else { DebugLoop(ShowDebugInfo(); if  (I2CMasterState != I2C_SUCCESS)); }