Структура программы, Вырабатываю хороший стиль программирования Опции

[UniBomb]

Добрый день. Возник у меня вопрос, который терзает моё эго и не даёт спокойно работать. Вопрос заключается в организации программы. В общем вот:

Допустим есть ТЗ: сделать прибор, который должен:

- считывать температуру с датчика по шине TWI
- считывать значение напряжения с АЦП с датчика влажности
- расчитывать значение влажности в общепринятых единицах
- выдать предупреждающие сигналы в случае превышения норм
- показать значения на дисплее

Зада просто и понятна (хоть и высосана из пальца). Реализовал бы всё это дело я так:

Код

#include <io.h>
volatile int counter;
int main()
{
//инициализирую всё и вся

for(;;) //пустой бесконечный цикл
;
}

ISR(SIG_OUTPUT_COMPARE1A)
{
switch(counter)
  {
   case 100: BeginTempConvers(); break; //посылаю запрос для начала считывания температуры
   case 200: GetADCValue(); break; //считываю значение с АЦП
   case 300: ProcessADCValue(); break; //раситываю необходимые значения
   case 400: GetTempValue(); break; //считываю температуру
   case 500: SetAlarm(); break; //проверяю значения и вывожу аварийные сигналы если надо
   case 600: DisplayValues(); break; //вывожу все данные на экран
  };

counter++;
if(counter == 601)
  counter = 0;
}

Так вот вопрос - хорошо ли я делаю? Какие есть ещё способы реализации поставленной задачи. Просто мне почему то кажется, что всю логику работы прибора нехорошо запихивать в таймер...

[UniBomb]

Очень правильно кажется!

Кстате, а почему плохо? Спрашиваю для общего развития, а не для спора)))

Желательно как-то так: в таймере выставлять флаги операций, а в бесконечном цикле (тот, что в main)

Код

#include <io.h>
volatile int counter;
int main()
{
//инициализирую всё и вся

for(;;) //уже не пустой бесконечный цикл
  {
   switch(counter)
   {
    case 100: BeginTempConvers(); counter++; break; //посылаю запрос для начала считывания температуры
    case 200: GetADCValue(); counter++; break; //считываю значение с АЦП
    case 300: ProcessADCValue(); counter++; break; //раситываю необходимые значения
    case 400: GetTempValue(); counter++; break; //считываю температуру
    case 500: SetAlarm(); counter++; break; //проверяю значения и вывожу аварийные сигналы если надо
    case 600: DisplayValues(); counter++; break; //вывожу все данные на экран
   } //counter инкрементируется для того, что бы функция невызывалась повторно в случае когда время итерации цикла меньше перода таймера
  }
}

ISR(SIG_OUTPUT_COMPARE1A)
{
counter++;
if(counter == 601)
  counter = 0;
}

ммм.... так?

Все в таймер явно нехорошо Можно сделать так - в прерываниях обрабатывается взаимодействие с датчиками (самое простое, без дальнейшей обработки). Вся основная логика реализуется в процедуре, которая циклически вызывается из процедуры UI (какой бы он не был - кнопки и индикатор или RS232 к PC).
Если логика не тривиальная - делается один или более взаимодействующих конечных автомата.

Да логика то в моих реальных заданиях такая же простая как в примере))) А начсёт процедур - что за процедура UI?

[XVR]

А начсёт процедур - что за процедура UI?

User Interface - взаимодействие с пользователем