Немного стандарта написания программ на С, применительно к контроллерам Опции

[torik]

Всем привет.

Разрабатываю программу для PIC24F. Необходимо более-менее придерживаться стандарта. У меня вопрос следующий...

Обычно все переменные я объединяю в структуры, группирую по назначению

Код

typedef struct {
  alt_buttons* buttons;                                             //ссылка на массив кнопок
  unsigned char max_key_num;                                //число кнопок в клавиатуре
  .......
  unsigned char alpha;                                             //прозрачность клавиатуры
} alt_keyboard;

Это удобно, т.к. список аргументов функций будет представлять собой не кучу всяких переменных, а лишь одну структуру (или несколько).

Правильно, вроде как, считается создавать локальные переменные и передавать их затем в качестве аргумента. Например, в main:

void main (void) {
alt_keyboard keyboard;

system_init (&keyboard);

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

Можно основные переменные (например, клавиатура, дисплей и т.п.) сделать глобальными и работать с ними из любой функции. Не?

[Dima_G]

Я следую такому правилу - делать переменную глобальную только в том случае, когда без этого - никак (например, по другому никак не передать данные в/из обработчик прерываний и тд).

Где-то вычитал правило и согласен с ним. Звучит как-то так:
"Предпочитайте локальные переменные - глобальным, глобальные статические - просто глобальным"

А чем плохо?
Переменная висит статически, в заранее известном адресе, существует всё время работы.

1. именно этим и плохо. Разово нужная переменная постоянно занимает память.
2. программы с глобальными переменными очень плохо читаются (ну это мое субъективное мнение)
3. глобальная - доступна всем желающим (во всей программе, либо в единице трансляции, если это - статик). Можно посадить трудноуловимые баги .
4, Трудность переноса кода - если функция использует только локальные переменные, ее в общем случае можно просто "взять и выдернуть". Использующая глобальные переменные функция, потянет за собой хвост этих переменных...

[MrYuran]

1. Разово нужная переменная постоянно занимает память.

Если разово, то согласен - локальные. Но бывают флаги/состояния, которые нужно хранить от вызова к вызову.
Не забываем также, что речь идёт о программировании контроллеров, а не ПиСи

2. программы с глобальными переменными очень плохо читаются (ну это мое субъективное мнение)
3. глобальная - доступна всем желающим (во всей программе, либо в единице трансляции, если это - статик). Можно посадить трудноуловимые баги .
4, Трудность переноса кода - если функция использует только локальные переменные, ее в общем случае можно просто "взять и выдернуть". Использующая глобальные переменные функция, потянет за собой хвост этих переменных...

И что, предлагаете в мэйне организовать глобальную свалку переменных со всего проекта, а в каждую функцию передавать десяток параметров?
Может, это и правильно с т.з. высоких материй (опять же применительно к PC), но в практической работе очень неудобно.

Я в процессе эволюции пришёл к созданию максимально независимых модулей со своими глобальными и статическими переменными.
Те переменные, которые надо видеть "снаружи" модуля, выношу extern-ом в h-файл модуля.
А main в таком случае состоит из подключения соответствующих модулей через h-файлы, инициализации и быстрого суперцикла.
"Взять и выдернуть" в таком случае можно не отдельную функцию, а модуль. Да и не надо его дёргать, просто подключить и использовать.