Можно ли в keil разбить #define на несколько строк ? Опции

[MiklPolikov]

Вопрос : есть ли способ в Keil записать такую строку

#define VOLTAGE_RANGE_1 {PWR->CR &=~ PWR_CR_VOS_1; PWR->CR |= PWR_CR_VOS_0;}

в виде двух

#define VOLTAGE_RANGE_1 {PWR->CR &=~ PWR_CR_VOS_1;
PWR->CR |= PWR_CR_VOS_0;}

?

[megajohn]

Код

#define VOLTAGE_RANGE_1    {\
PWR->CR &=~ PWR_CR_VOS_1;\
PWR->CR |= PWR_CR_VOS_0; }

[MrYuran]

и обязательно в скобках do{}while(0)

[MiklPolikov]

Код

#define VOLTAGE_RANGE_1    {\
PWR->CR &=~ PWR_CR_VOS_1;\
PWR->CR |= PWR_CR_VOS_0; }

Спасибо !

и обязательно в скобках do{}while(0)

Не понимаю о чём речь. Объясните пожалуйста. Что в скобках ?

[Палыч]

Не понимаю о чём речь. Объясните пожалуйста. Что в скобках ?

Речь вот о чём. При использовании VOLTAGE_RANGE_1 в Вашей программе будет подставлено (содержимое скобок опустил): {...}
Обычно, при оформлении программ, после VOLTAGE_RANGE_1 ставит точку с запятой, чтобы было похоже на "настоящий" оператор. Например, так:

Код

if(xxx) VOLTAGE_RANGE_1;

После подстановки значения будет

Код

if(xxx) {...};

В данном примере "лишняя" точка с запятой не помешает, но - что получится при такой конструкции ?

Код

if(xxx) VOLTAGE_RANGE_1;
else ....

Вот тут точка с запятой окажется лишней. Чтобы она лишней не оказалась, то обрамляют фигурные скобки оператором do ... while(0)

Код

#define VOLTAGE_RANGE_1    do {\
PWR->CR &=~ PWR_CR_VOS_1;\
PWR->CR |= PWR_CR_VOS_0; } while(0)

[MiklPolikov]

Речь вот о чём. При использовании VOLTAGE_RANGE_1 в Вашей программе будет подставлено (содержимое скобок опустил): {...}
Обычно, при оформлении программ, после VOLTAGE_RANGE_1 ставит точку с запятой, чтобы было похоже на "настоящий" оператор. Например, так:

Код

if(xxx) VOLTAGE_RANGE_1;

После подстановки значения будет

Код

if(xxx) {...};

В данном примере "лишняя" точка с запятой не помешает, но - что получится при такой конструкции ?

Код

if(xxx) VOLTAGE_RANGE_1;
else ....

Вот тут точка с запятой окажется лишней. Чтобы она лишней не оказалась, то обрамляют фигурные скобки оператором do ... while(0)

Код

#define VOLTAGE_RANGE_1    do {\
PWR->CR &=~ PWR_CR_VOS_1;\
PWR->CR |= PWR_CR_VOS_0; } while(0)

Спасибо!

[Allregia]

А почему не делать так:

Код

__INLINE void VOLTAGE_RANGE_1(void} {
    PWR->CR &=~ PWR_CR_VOS_1;
    PWR->CR |= PWR_CR_VOS_0;
}

?

и вызывать где угодно как VOLTAGE_RANGE_1();

По моему, так понятнее, без do..while.

[ViKo]

А я иногда делаю так

Код

#define WDELAY_RESTART();        \
  TIM7->EGR = TIM_EGR_UG;        \
  TIM7->SR = 0;                \
  TIM7->CR1 |= TIM_CR1_CEN;

Не буду оппонировать "монстрам от программирования", но случаи "а если..." мне кажутся надуманными.

[aaarrr]

А я иногда делаю так

Код

#define WDELAY_RESTART();        \
  TIM7->EGR = TIM_EGR_UG;        \
  TIM7->SR = 0;                \
  TIM7->CR1 |= TIM_CR1_CEN;

Не буду оппонировать "монстрам от программирования", но случаи "а если..." мне кажутся надуманными.

Чудовищно. В этом случае if(xxx) WDELAY_RESTART(); просто молча породит неправильный код.
Вам кажется надуманным использование оператора "if"?

По моему, так понятнее, без do..while.

do {} while(0) - стандартная конструкция, она не должна быть непонятна в принципе.

[Allregia]

do {} while(0) - стандартная конструкция, она не должна быть непонятна в принципе.

Да, но не в этом случае, и чем хуже инлайн?

[aaarrr]

Да, но не в этом случае, и чем хуже инлайн?

Почему не в этом случае?

Инлайн ничем не хуже, кроме того, что собственно встраивание не гарантируется. А если, например, нужно вернуть из блока значение, то без инлайна никак.

[ViKo]

Чудовищно. В этом случае if(xxx) WDELAY_RESTART(); просто молча породит неправильный код.
Вам кажется надуманным использование оператора "if"?

В данном случае - да.
Тем более, я всегда могу написать

Код

if (xxx) {
  WDELAY_RESTART();
}

[aaarrr]

В данном случае - да.

Тогда рекомендую убрать (); в конце макроса, дабы сразу было видно, что это лажа, а не нормальная языковая конструкция.

Тем более, я всегда могу написать

Вы - возможно, если вспомните. А кто-то другой получит сюрприз.

[ViKo]

Тогда рекомендую убрать (); в конце макроса, дабы сразу было видно, что это лажа, а не нормальная языковая конструкция.
Вы - возможно, если вспомните. А кто-то другой получит сюрприз.

На то даны БОЛЬШИЕ_БУКВЫ.

[aaarrr]

На то даны БОЛЬШИЕ_БУКВЫ.

Т.е. теперь БОЛЬШИЕ_БУКВЫ всегда оборачивать в {} безопасности для?

Зачем писать нарочито плохо, если можно без каких бы то ни было усилий написать хорошо и правильно?

[ViKo]

Т.е. теперь БОЛЬШИЕ_БУКВЫ всегда оборачивать в {} безопасности для?

Зачем писать нарочито плохо, если можно без каких бы то ни было усилий написать хорошо и правильно?

На мой взгляд, макрос в виде функции выглядит логичнее, чем конструкция с довеском do ... while(0)

[aaarrr]

"Логичнее"? А что ведет себя совсем иначе - это ничего?

[ViKo]

Т.е. теперь БОЛЬШИЕ_БУКВЫ всегда оборачивать в {} безопасности для?

Не всегда. Только функции БОЛЬШИЕ_БУКВЫ();
КОНСТАНТЫ - не нужно.

[aaarrr]

Не всегда. Только функции БОЛЬШИЕ_БУКВЫ();
КОНСТАНТЫ - не нужно.

Правильно, константы я бы в таком коде превинтивно заворачивал в (). Во избежание.

[ViKo]

"Логичнее"? А что ведет себя совсем иначе - это ничего?

Ведет себя, именно, как функция. Набор выражений.

[aaarrr]

Этот "набор выражений" и ведет себя как набор не связанных друг с другом выражений, а не как функция.

[ViKo]

Этот "набор выражений" и ведет себя как набор не связанных друг с другом выражений, а не как функция.

Да ну? Отличие только в том, что ничего не возвращает (всегда). Тот пример, что я показал, запросто можно оформить в виде функции.

[aaarrr]

Да ну? Отличие только в том, что ничего не возвращает (всегда). Тот пример, что я показал, запросто можно оформить в виде функции.

Понятно, в виде функции - это "#define MACRO {}"? Немного получше, но все равно не то.

А уж этому -

Код

#define WDELAY_RESTART();        \
  TIM7->EGR = TIM_EGR_UG;        \
  TIM7->SR = 0;                \
  TIM7->CR1 |= TIM_CR1_CEN;

- точно не место в ветке для начинающих. Разве что как пример того, как делать не следует никогда.
Примерно такой же "полезности" макрос:

Код

#define  10   5 + 5

[A. Fig Lee]

и обязательно в скобках do{}while(0)

Это уже "на все случаи жизни".
И так хорошо.
ИМХО.

[Raven]

и обязательно в скобках do{}while(0)

Спасибо за интересный прием. Действительно полезно для макросов. Возьму на вооружение.

[ViKo]

Понятно, в виде функции - это "#define MACRO {}"? Немного получше, но все равно не то.

Хорошо, добавим фигурные скобки. Можете привести пример, когда такая конструкция будет работать неправильно?

Код

#define WDELAY_RESTART(); {     \
  TIM7->EGR = TIM_EGR_UG;        \
  TIM7->SR = 0;             \
  TIM7->CR1 |= TIM_CR1_CEN;     \
}

[igorle]

Хорошо, добавим фигурные скобки. Можете привести пример, когда такая конструкция будет работать неправильно?

Код

#define WDELAY_RESTART(); {     \
  TIM7->EGR = TIM_EGR_UG;        \
  TIM7->SR = 0;             \
  TIM7->CR1 |= TIM_CR1_CEN;     \
}

Элементарно. Я исхожу из предположения, что точка с запятой здесь (WDELAY_RESTART(); ) поставлена умышленно:

Код

#define aaa(); { printf("Good style saves you\n"); }

int main()
{
    int i = 0;
    if (0)
        if (1)
            aaa();

    return 0;
}

Будет напечаттано сообщение? Проверьте здесь

Теперь предположим, что точка с запятой стоят ошибочно (а это скорее всего так и есть)
Попытайтесь скомпилировать этот код:

Код

#define aaa() { printf("XXX\n"); }

int main()
{
    if (0)
        if (1)
            aaa();
        else
            printf("Yo are so wrong\n");

    return 0;
}

Сообщение отредактировал igorle - Sep 28 2013, 08:34

[ViKo]

Будет напечатано сообщение? Проверьте

Точка с запятой стоит намеренно. Но я, хоть убей не понимаю поведения codepad! Подставляю ту же строку из макро - и не печатает! (первый пример)

[Ruslan1]

Прочитал все написанное но так и не понял почему так все сложно.
И я совершенно не представляю ситуацию когда не будет работать конструкция

Код

#define MacroName(параметры_если_есть) \
{ /*комментарий0 если нужен */ \
оператор1; /*комментарий1 если нужен */ \
оператор2; /*комментарий2 если нужен */ \
}

Будьте добры, приведите мне любой пример кода, где определенный так макрос будет работать некорректно. Неужто кто-то знает такая комбинацию компилятора, параметров и операторов, когда это будет работать иначе чем задумано программистом при написании этого макроса?
Пока что я пребываю в дремучем незнании и полной уверенности что оно работает, буду рад любому кто раскроет мне глаза на подлую сущность вещей.
Как говориться, "thanks in advance".

[igorle]

Точка с запятой стоит намеренно. Но я, хоть убей не понимаю поведения codepad! Подставляю ту же строку из макро - и не печатает! (первый пример)

Такой результат получится с любым исправным компилятором. Если есть под рукой GCC - посмотрите результат препроцессора. У меня под рукой нет. Поэтому сделаем препроцессинг вручную.

Код

if (0)
    if (1)
        aaa();

превращается в

Код

if (0)
    if (1)
       ; { printf("Good style saves you\n"); };

Или, после причесывания

Код

if (0)
    if (1)
           ;
{
    printf("Good style saves you\n");
};

А вывод один - обрамляйте любую макрофункцию, состоящую более чем из вызова одной функции, do {} while(0), и будет вам счастье.

я совершенно не представляю ситуацию когда не будет работать конструкция

Код

#define MacroName(параметры_если_есть) \
{ /*комментарий0 если нужен */ \
оператор1; /*комментарий1 если нужен */ \
оператор2; /*комментарий2 если нужен */ \
}

Будьте добры, приведите мне любой пример кода, где определенный так макрос будет работать некорректно.

Ruslan1, выше был пример. Но я повторю его, так как выше смешал два случая:

Код

#define aaa() { printf("XXX\n"); }
if (1)
    aaa();
else
    printf("Yo are so wrong\n");

просто не скомпилируется.

[ViKo]

Такой результат получится с любым исправным компилятором. Если есть под рукой GCC - посмотрите результат препроцессора. У меня под рукой нет. Поэтому сделаем препроцессинг вручную.

Код

if (0)
    if (1)
      ; { printf("Good style saves you\n"); };

Насчет GCC не скажу, но в Keil C-99 точка с запятой не отрывается от имени макрофункции.

[Ruslan1]

Ruslan1, выше был пример. Но я повторю его, так как выше смешал два случая:

Код

#define aaa() { printf("XXX\n"); }
if (1)
    aaa();
else
    printf("Yo are so wrong\n");

просто не скомпилируется.

Это не интересно. Я спросил пример кода, где определенный так макрос будет работать некорректно.
То есть когда оно работает, но некорректно. То есть компилятор молча пережевал и сгенерировал код, но код работает не так как задумывал программист.
Варианты, которые детектируются любым компилятором как варнинги или ошибки не интересны, так как очевидно, что любой даже варнинг требует внимания программиста и документирования как допустимый или модификации кода для его устранения.

Вот если Вы укажете компилятор, в котором приведенный Вами пример не сгенерирует варнинг или ошибку (и, само собой, работать будет некорректно)- я с Вами соглашусь.

[Flood]

Это не интересно. Я спросил пример кода, где определенный так макрос будет работать некорректно.
То есть когда оно работает, но некорректно.

К чему такие придирки? Или если написано "чуть-чуть неправильно" - то так делать можно, а вот если "совсем неправильно" - уже нельзя?

Очевидно, что простой блок в макросах способен вызвать проблемы, знать об этом и осознанно писать проблемные макросы - как минимум странно. С точки зрения повторного использования будет не важно, код не компилируется, или компилируется, но работает некорректно. Это влияет только на скорость выявления проблем. Все равно придется лезть в чужой (или в свой старый) макрос и его исправлять, или переписывать код так, чтобы после макроса не было точки с запятой.
И зачем это нужно, если есть старый как сам C способ написания корректного кода?
http://c-faq.com/cpp/multistmt.html

О нем можно не знать, это нормально. Но знать и не использовать - в чем выгода?

[igorle]

Поддерживаю предыдущего оратора.

И еще.
"функции" без точки с запятой в конце, сбивают механизм автоматической индентации. Например, я сейчас напечатал в виме код, и он выровнял мне его так:

Код

if (1)
    aaa()
        bbb();

А это некрасиво. И это уже достаточный повод не делать этого.

[ViKo]

Например, я сейчас напечатал в виме код, и он выровнял мне его так:

Что такое "вим"?

[Flood]

Что такое "вим"?

Адский текстовый редактор. (Самый?) мощный, но по-настоящему красноглазый.
http://ru.wikipedia.org/wiki/Vim

[A. Fig Lee]

А вывод один - обрамляйте любую макрофункцию, состоящую более чем из вызова одной функции, do {} while(0), и будет вам счастье.

Не согласен.
После if всегда надо пользовать фигурные скобки.
Назвисимо от использования всяких define.
Это улучшает читаемость и предсказуемость кода.
Чем больше напихано в #define, тем сложнее их читать.

То, что компилятор не скомпилирует не проблема.
поправим. Проблема если скомпилирует не так как надо.

[aaarrr]

После if всегда надо пользовать фигурные скобки.
...
Это улучшает читаемость и предсказуемость кода.

Код

if(x) zzz();

По-моему, в подобном выражении фигурные скобки не улучшат читаемость. Скорее, наоборот.

То, что компилятор не скомпилирует не проблема.
поправим. Проблема если скомпилирует не так как надо.

Если что-то надо править, то это все равно проблема. Она, конечно, меркнет на фоне второго варианта развития событий, но все же.

[ig_z]

Не согласен.
После if всегда надо пользовать фигурные скобки.
Назвисимо от использования всяких define.
Это улучшает читаемость и предсказуемость кода.
Чем больше напихано в #define, тем сложнее их читать.

То, что компилятор не скомпилирует не проблема.
поправим. Проблема если скомпилирует не так как надо.

Маладец! Герой! Аплодирую стоя, один прет против всех! Прям как макаревич в своих нетленках.
Теперь нужно срочно наставить на путь истинный девелоперов линукс ядра, lwip, contiki и т.д. Залезть в их репозитории и поубивать ненавистные дувайлы