CERT C: Rec. PRE10-C

Перенесите составные макросы в цикл с условием продолжения

Описание

Управляйте определением

Перенесите составные макросы в цикл с условием продолжения. [1]

Реализация Polyspace

Это средство проверки проверяет на Макрос с несколькими операторами.

Примеры

развернуть все

Проблема

Macro with multiple statements происходит, когда макрос содержит несколько отключенных точкой с запятой операторов, независимо от того, заключены ли операторы в фигурные скобки.

Риск

Макрорасширение, в определенных контекстах, таких как if условие или цикл, может привести к непреднамеренной логике программы.

Например, рассмотрите макрос:

#define RESET(x,y) \
   x=0; \
   y=0;
В if оператор, такой как:
if(checkSomeCondition)
   RESET(x,y);
макрос расширяется до:
if(checkSomething)
   x=0;
y=0;
который может быть неожиданным, если вы хотите, чтобы оба оператора были выполнены в if блок.

Фиксация

В макроопределении перенесите несколько операторов в do...while(0) цикл.

Например, в предыдущем примере, используйте определение:

#define RESET(x,y) \
   do { \
     x=0; \
     y=0; \
   } while(0)
Этот макрос является соответствующим, чтобы расшириться во всех контекстах. while(0) гарантирует, что операторы выполняются только однажды.

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

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

Пример – макрос с несколькими операторами

#define RESET(x,y) \
   x=0; \
   y=0;

void func(int *x, int *y, int resetFlag){
    if(resetFlag)
        RESET(x,y);    
}

В этом примере дефект происходит потому что макро-RESET состоит из нескольких операторов.

Коррекция – переносит несколько операторов макроса в do-while Цикл

Перенесите операторы макроса в do..while(0) цикл в макроопределении.

#define RESET(x,y) \
   do { \
     x=0; \
     y=0; \
   } while(0)

void func(int *x, int *y, int resetFlag){
    if(resetFlag)
        RESET(x,y);    
}

Проверяйте информацию

Группа: Rec. 01. Препроцессор (PRE)

Введенный в R2020a


[1]  Это программное обеспечение было создано MathWorks, включающим фрагменты: “Веб-сайт SEI CERT-C”, © 2017 Carnegie Mellon University, веб-сайт SEI CERT-C © 2017 Carnegie Mellon University”, CERT SEI C Кодирование Стандарта – Правил для Разработки безопасных, Надежных и Защищенных систем – 2 016 Выпусков”, © 2016 Carnegie Mellon University, and “CERT SEI Стандарт Кодирования C++ – Правил для Разработки безопасных, Надежных и Защищенных систем на C++ – 2 016 Выпусков” © 2016 Carnegie Mellon University, со специальным разрешением от его Института программной инженерии.

ЛЮБОЙ МАТЕРИАЛ УНИВЕРСИТЕТА КАРНЕГИ-МЕЛЛОН И/ИЛИ ЕГО ИНСТИТУТА ПРОГРАММНОЙ ИНЖЕНЕРИИ СОДЕРЖАЛ, ЗДЕСЬ ПРЕДОСТАВЛЯЕТСЯ НА ОСНОВЕ "ASIS". УНИВЕРСИТЕТ КАРНЕГИ-МЕЛЛОН НЕ ДАЕТ ГАРАНТИЙ НИКАКОГО ВИДА, ИЛИ ВЫРАЗИЛ ИЛИ ПОДРАЗУМЕВАЛ, ОТНОСИТЕЛЬНО ЛЮБОГО ВОПРОСА ВКЛЮЧАЯ, НО НЕ ОГРАНИЧИЛ, ГАРАНТИЯ ПРИГОДНОСТИ ДЛЯ ЦЕЛИ ИЛИ ВЫСОКОГО СПРОСА, ИСКЛЮЧИТЕЛЬНОСТИ, ИЛИ ЗАКАНЧИВАЕТСЯ ПОЛУЧЕННЫЙ ИЗ ИСПОЛЬЗОВАНИЯ МАТЕРИАЛА. УНИВЕРСИТЕТ КАРНЕГИ-МЕЛЛОН НЕ ДАЕТ ГАРАНТИИ НИКАКОГО ВИДА ОТНОСИТЕЛЬНО СВОБОДЫ ОТ ПАТЕНТА, ТОВАРНОГО ЗНАКА ИЛИ НАРУШЕНИЯ АВТОРСКОГО ПРАВА.

Это программное обеспечение и сопоставленная документация не были рассмотрены, ни являются подтвержденным Университетом Карнеги-Меллон или его Институтом программной инженерии.