Вопросы по С/С++ (СИ)

[ARV]

Это так?

ну как... есть одна-единственная функция в программе, которая содержит в себе тот самый крутой автомат состояний, выполняющий ВСЁ. и эта единственная функция - обработчик прерывания, поэтому выходить из неё особо и не надо - некуда, кроме как снова в себя...

[Ivanoff-iv]

[uquote="viiv",url="/forum/viewtopic.php?p=3346939#p3346939"]Компилятор все заоптимизирует. Не будет никаких вызовов.[/uquote] точно! заоптимизировал!

Добавлено after 4 minutes 11 seconds:
где можно почитать про возможности компилятора, про то как с ним более конструктивно взаимодействовать?
--------------------------------
ПС OSA скачал - начал изучать, но свой велосипед допилю - дело принципа!

[ARV]

про то как с ним более конструктивно взаимодействовать?

не надо с ним взаимодействовать, потому как его пути неисповедимы. лучше сразу писать прилично, и радоваться.

[Ivanoff-iv]

доделаю сам, буду пользоваться сам, а с вами не поделюсь, злые вы
пока задача - изучение языка, я знаю, что, уже для всего написаны готовые библиотеки, но, хочу чтото своё сделать и посмотреть насколько "коричневым" будет этот велосипед...

Добавлено after 1 minute 55 seconds:
пока я добился ощутимого сокращения потребления тока... и, вроде, ничего не глючит

Добавлено after 2 minutes 3 seconds:
пока смирюсь с отсутствием автонумерации и попробую побольше всего втолкать, посмотрю, насколько удобно вышло, а потом всё это на РТОС повторю

Добавлено after 1 minute 39 seconds:
на этот вопрос для себя я нашел консенсус, можете больше не отвечать.

Добавлено after 33 minutes 12 seconds:
viiv, перепроверил - с упаковкой в функцию получается на одно слово тяжелей, чем без упаковки.

Добавлено after 14 minutes 49 seconds:
________________________
всё, оценил масштаб - буду переписывать на функциях, а то на дефайнах слишком жирно выходит, спасибо за идеи.

[Мурик]

[uquote="Ivanoff-iv",url="/forum/viewtopic.php?p=3346741#p3346741"]а нормальная ОС в тини влезет?[/uquote]Зачем себя так ограничивать? Можно взять МК скажем с 128 КБ флеша и 20 КБ ОЗУ. Запустить на нем ОСь и пусть крутятся на ней скажем 10 задач.
Нормальная ОСь это не только планировщик задач, но и средства для доступа к общим ресурсам, очереди, программные таймеры и т. д.

[pokk]

[uquote="Ivanoff-iv",url="/forum/viewtopic.php?p=3346671#p3346671"]сейчас код выглядит так (пример использования)[/uquote]
Это напоминает Протопотоки (protothreads),поздновато увидел вашу дискуссию, так что последний страницы пришлось прочитать по диагонали, так что извиняйте если такое уже поднималось.
Подключал несколько DS1821, в таймере, получилось как-то так.

Спойлер

Код: Выделить всё

#define Setdelay_us(Value)	TIM1->PSC=40-1;TIM1->ARR = Value-1;TIM1->CNT = 0; PT_YIELD(pt);
#define Setdelay_ms(Value)	TIM1->PSC=39800-1;TIM1->ARR = Value-1;TIM1->CNT = 0; PT_YIELD(pt);

PT_THREAD(DS1821_thread(struct pt *pt,unsigned char id)){
static unsigned char ReadData;
static unsigned char StatusDS1821=SENSOR_INIT_VALIDATION_SUCCESS;
static unsigned char Status;
unsigned char i;
unsigned char flag_exit;
//--------------------------------------------------------------------------------------------------
PT_BEGIN(pt);
//--------------------------------------------------------------------------------------------------
	PT_INIT(&Point_Ds1821_TransmitData);
	PT_INIT(&Point_Ds1821_ReadData);
	PT_INIT(&Point_Ds1821_ResetData);
	//---------------------Инициализация выводов термодатчиков------------------------------------------
	InitPinDS1821_HOT();
	//-------------------------------Предварительная инициализация--------------------------------------
	DS1821_ZT_OFF
	DS1821_OFF
	DS1821_HOT_IN
	DS1821_UL_IN
	DS1821_COLD_IN
	DS1821_COLLING_IN
	Setdelay_ms(2);
	//----------------------------------------------------------------------------------------------
	DS1821_ON
	DS1821_W
	//--------------------------------Порты на выход------------------------------------------------
	DS1821_HOT_OUT
	DS1821_COLD_OUT
	DS1821_UL_OUT
	DS1821_COLLING_OUT
	//------------------------Записать в порт логическую единицу------------------------------------
	DS1821_HOT_ON
	DS1821_COLD_ON
	DS1821_UL_ON
	DS1821_COLLING_ON
	//----------------------------------------------------------------------------------------------
	Setdelay_ms(2);
	DS1821_ZT_ON
	//-----------------------Обработчик аварий термодатчиков----------------------------------------
	if(StatusDS1821==SENSOR_INIT_VALIDATION_FAIL){
		//1) Проверяем какой термодатчик привёл к аварии

		//-------------Находим термодатчик который выдал аварию---------------------
		for(i=0;i<4;i++){
			if(TypeErrorTemp[i]>1){
				CounterErrorTemp[i]=CounterErrorTemp[i]+1;
				if(CounterErrorTemp[i]>=3){
					TypeErrorTemp[i]=TYPE_T_BLOCK;
					Send_MSG(MSG_EXIST_ERROR);
				}
			//HandlerError(CurrentTemp);
			}
		}
	}
	//------------------Задержка на включение термодатчика------------------------------------------
	Setdelay_ms(2000);
	//--------------------------Reset---------------------------------------------------------------
	PT_WAIT_THREAD(pt,ds1821_Reset_thread(&Point_Ds1821_ResetData,&StatusDS1821));
	//-------------------write 0x0C-------------------------------------------------------------
	PT_WAIT_THREAD(pt,ds1821_transmit_data_thread(&Point_Ds1821_TransmitData,0x0C));
	//-------------------write 0x00-------------------------------------------------------------
	PT_WAIT_THREAD(pt,ds1821_transmit_data_thread(&Point_Ds1821_TransmitData,0x00));
	//----------------------------------------------------------------------------------------------
	//-----------------------------Главный цикл работы термодатчика---------------------------------
	//----------------------------------------------------------------------------------------------
	while(1){
		//-------------------------------------------------------------------------------------
		PT_WAIT_THREAD(pt,ds1821_Reset_thread(&Point_Ds1821_ResetData,&StatusDS1821));
		//-------------------Отправка команды на преобразование температуры-------------------------
		PT_WAIT_THREAD(pt,ds1821_transmit_data_thread(&Point_Ds1821_TransmitData,0xEE));
		//-------------------Выдержка времени преобразования температуры----------------------------
		Setdelay_ms(DS18B2_PERIOD_POLLING_AND_CONVERSIONS_T);
		//--------------------------------RESET-----------------------------------------------------
		PT_WAIT_THREAD(pt,ds1821_Reset_thread(&Point_Ds1821_ResetData,&StatusDS1821));
		//-------------Отправка команды на чтение температуры---------------------------------------
		PT_WAIT_THREAD(pt,ds1821_transmit_data_thread(&Point_Ds1821_TransmitData,0xAA));
		//-----------------------------Чтение температуры-------------------------------------------
		PT_WAIT_THREAD(pt,ds1821_read_data_thread(&Point_Ds1821_ReadData,CurrentTemp));
		//------------------------------------------------------------------------------------------
		for(i=0;i<4;i++){
			if(TypeErrorTemp[i]==TYPE_T_BLOCK){
				//Термодатчик заблокирован
			}else{
				SetStatusTemp(CurrentTemp[i],i);// Если термодатчик не заблокирован то выставляем статус
			}
		}
		SendBroadcastMessages(TEMP_COMPLEATED);	//Для нескольких процессов в главном цикле.
		//------------------------------------------------------------------------------------------
		if(StatusDS1821==SENSOR_INIT_VALIDATION_FAIL){
				//-------------Находим термодатчик который выдал аварию---------------------
				for(i=0;i<4;i++){
					if(TypeErrorTemp[i]>=1){	//какая то авария КЗ на ноль или 1
                        CurrentStatusTemp[i]=STATUS_T_SHORT_CIRCUIT;
						if(TypeErrorTemp[i]==TYPE_T_BLOCK){
							//Термодатчик заблокирован можно продолжать работу
						}else{
							flag_exit=1;
						}
					}else{
						CounterErrorTemp[i]=0;	//Перезапустить счетчик аварий, так как чтение в норме.
					}
				}
				//-----дожидаемся установления всех статусов--------------
				if(flag_exit==1){
					PT_EXIT(pt);
				}
        }
		//------------------------------------------------------------------------------------------
		PT_END_YIELD(pt);
	}
//--------------------------------------------------------------------------------------------------
PT_END(pt);
}


PT_THREAD(ds1821_Reset_thread(struct pt *pt,unsigned char *ReturnValue)){
static unsigned short int Ack[2];
PT_BEGIN(pt);
	//---------------------------------------------
	//---------------------------------------------
	//---------------------------------------------
	DS1821_ZT_OFF
	//---------Порт на выход-----------------------
	DS1821_HOT_OUT
	DS1821_COLD_OUT
	DS1821_UL_OUT
	DS1821_COLLING_OUT
	//-----------Запись ноль в порт----------------
	DS1821_HOT_OFF;
	DS1821_COLD_OFF
	DS1821_UL_OFF
	DS1821_COLLING_OFF
	//---------- региста на запись------------
	DS1821_W
	DS1821_ZT_ON
	Setdelay_us(600);
	//---------------------------------------------
	DS1821_R
	//---------- на вход----------------------
	DS1821_HOT_IN
	DS1821_UL_IN
    DS1821_COLD_IN
    DS1821_COLLING_IN
	//---------------------------------------------
	Setdelay_us(80);
	//---------------------------------------------
	Ack[0]=DS1821_READ_ALL;
	//---------------------------------------------
	Setdelay_us(300);
	//---------------------------------------------
	Ack[1]=DS1821_READ_ALL;
	//---------------------------------------------
	//---------------------------------------------
	//-------тут надо сделать проверку всех--------
	//---------------------------------------------
	//--------Если хоть 1 не ответил то выдаём команду  на зависание
	if((Ack[0] == 0)&&(Ack[1] == (DS1821_HOT_PIN|DS1821_COLD_PIN|DS1821_COLLING_PIN|DS1821_UL_PIN))){
		*ReturnValue=SENSOR_INIT_VALIDATION_SUCCESS;
	}else{
		//------------------------------------------------------------------------------------------
		*ReturnValue=SENSOR_INIT_VALIDATION_SUCCESS; //Если не кто не изменил статус значит аварии заблокированны
		//------------------------------------------------------------------------------------------
		//-------------------- какой термодатчик в аварии---------------------------------
		//------------------------------------------------------------------------------------------
		//------------------------Термодатчик горячий-----------------------------------------------
		if((Ack[0]&DS1821_HOT_PIN)==(Ack[1]&DS1821_HOT_PIN)){
			if(TypeErrorTemp[T_HOT]!=TYPE_T_BLOCK){	//Если авария не заблокирована
				Send_MSG(MSG_EXIST_ERROR);
				*ReturnValue=SENSOR_INIT_VALIDATION_FAIL;
				if(Stm32BitIsSet(Ack[0],DS1821_HOT_PIN)){
					TypeErrorTemp[T_HOT]=TYPE_T_SHORT_CIRCUIT_1; //Замыкание 1
				}else{
					TypeErrorTemp[T_HOT]=TYPE_T_SHORT_CIRCUIT_0; //Замыкание 0
				}
			}
		}else{
            TypeErrorTemp[T_HOT]=STATUS_T_NORM; //Сбрассываем предыдуший флаг аварии
        }
		//------------------------Термодатчик Холодный-----------------------------------------------
		if((Ack[0]&DS1821_COLD_PIN)==(Ack[1]&DS1821_COLD_PIN)){
			if(TypeErrorTemp[T_COLD]!=TYPE_T_BLOCK){	//Если авария не заблокирована
				Send_MSG(MSG_EXIST_ERROR);
				*ReturnValue=SENSOR_INIT_VALIDATION_FAIL;
				if(Stm32BitIsSet(Ack[0],DS1821_COLD_PIN)){
					TypeErrorTemp[T_COLD]=TYPE_T_SHORT_CIRCUIT_1; //Замыкание 1
				}else{
					TypeErrorTemp[T_COLD]=TYPE_T_SHORT_CIRCUIT_0; //Замыкание 0
				}
			}
		}else{
            TypeErrorTemp[T_COLD]=STATUS_T_NORM; //Сбрассываем предыдуший флаг аварии
        }
		//--------------------------------------------------------
		if((Ack[0]&DS1821_COLLING_PIN)==(Ack[1]&DS1821_COLLING_PIN)){
			if(TypeErrorTemp[T_COLLING]!=TYPE_T_BLOCK){	//Если авария не заблокирована
				Send_MSG(MSG_EXIST_ERROR);
				*ReturnValue=SENSOR_INIT_VALIDATION_FAIL;
				if(Stm32BitIsSet(Ack[0],DS1821_COLLING_PIN)){
					TypeErrorTemp[T_COLLING]=TYPE_T_SHORT_CIRCUIT_1; //Замыкание 1
				}else{
					TypeErrorTemp[T_COLLING]=TYPE_T_SHORT_CIRCUIT_0; //Замыкание 0
				}
			}
		}else{
            TypeErrorTemp[T_COLLING]=STATUS_T_NORM; //Сбрассываем предыдуший флаг аварии
        }
		//--------------------------------------------------------
		if((Ack[0]&DS1821_UL_PIN)==(Ack[1]&DS1821_UL_PIN)){
			if(TypeErrorTemp[T_OUT]!=TYPE_T_BLOCK){	//Если авария не заблокирована
				Send_MSG(MSG_EXIST_ERROR);
				*ReturnValue=SENSOR_INIT_VALIDATION_FAIL;
				if(Stm32BitIsSet(Ack[0],DS1821_UL_PIN)){
					TypeErrorTemp[T_OUT]=TYPE_T_SHORT_CIRCUIT_1; //Замыкание 1
				}else{
					TypeErrorTemp[T_OUT]=TYPE_T_SHORT_CIRCUIT_0; //Замыкание 0
				}
			}
		}else{
            TypeErrorTemp[T_OUT]=STATUS_T_NORM; //Сбрассываем предыдуший флаг аварии
        }
		//--------------------------------------------------------
	}
    //---------------------------------------------
PT_END(pt);
}

[ARV]

и чем это проще традиционного подхода? вроде как при традиционном подходе писанины даже поменьше...

[pokk]

Лучше тем что нету не каких delay, а структура программы осталась "как бы линейная".
Писанины много из за того что там у меня 4 датчика подключалось, и я хотел сделать универсально, по этому много лишнего(всякие проверки на КЗ).

[ARV]

Лучше тем что нету не каких delay

чем же это лучше?
никак не пойму: все так страшатся этого делея, что просто удивляюсь... может, я чего-то не понимаю? ну так объясните.
или это догма навроде "goto открывает ворота в АД"?

[pokk]

[uquote="ARV",url="/forum/viewtopic.php?p=3351173#p3351173"]чем же это лучше?[/uquote]
Тем что процессор не сидит и не крутит while, а выполняет другие операции, которых куча, по мимо одного термодатчика.
ARV, судя по вашей логике, вы бы и на время конвертации температуры, сидели ждали бы пока термодатчик выдаст результат?

[ARV]

ARV, судя по вашей логике, вы бы и на время конвертации температуры, сидели ждали бы пока термодатчик выдаст результат?

именно так и делал, когда делал управление инкубаторами. и что? ни один заказчик не был недоволен.

[pokk]

Ну круто, я тоже так иногда делаю когда мигаю светодиодом

[ARV]

Так вот и я о том же: почему мигать можно, а другие задачи решать нельзя при помощи задержек? Вы думаете, CPU быстрее износится, если будет "напрасно" молотить в пустом цикле?

[Z_h_e]

Пустой цикл в "делэе", намного пустее чем в "вайле" .

[pokk]

[uquote="ARV",url="/forum/viewtopic.php?p=3351402#p3351402"]Так вот и я о том же: почему мигать можно, а другие задачи решать нельзя при помощи задержек? Вы думаете, CPU быстрее износится, если будет "напрасно" молотить в пустом цикле? [/uquote]

Тем что процессор не сидит и не крутит while, а выполняет другие операции, которых куча, по мимо одного термодатчика.

[Z_h_e]

А если нет кучи дел?

[ARV]

[uquote="Z_h_e",url="/forum/viewtopic.php?p=3351577#p3351577"]А если нет кучи дел?[/uquote]а правильнее вопрос сформулировать так: а если программу разрабатывать так, чтобы вся куча делалась, а delay этому не мешала?

людям ради того, чтобы решить проблему СЛОЖНО, хотя и круто, не лень изобретать самодельные велосипеды в виде подобий ОС... хотя необходимость в этом, с моей точки зрения, возникает достаточно редко. я ж не против

[Siarzhuk]

[uquote="ARV",url="/forum/viewtopic.php?p=3351706#p3351706"]а если программу разрабатывать так, чтобы вся куча делалась, а delay этому не мешала?[/uquote]
Хорошая куча - маленькая куча, особенно в перспективе её поддержки в длинючей перспективе и во всех деталях [которые забываются]. А delay - тот самый хромой бегун в эстафете, будь вся остальная команда хоть трижды быстроходные эфиопы - равняться будут по нему, и иногда это обидно. Но если никто и никуда не бежит - почему бы и нет?
[uquote="ARV",url="/forum/viewtopic.php?p=3351706#p3351706"]людям ради того, чтобы решить проблему СЛОЖНО, хотя и круто, не лень изобретать самодельные велосипеды в виде подобий ОС...[/uquote]
Всякая концепция компактна - но черти [которые в деталях] ждут автора по мере погружения в болото реальности. Вряд-ли желание сложнокрутости можно считать определяющим изначально - скорее наоборот - ибо человек по природе ленив и сделав что-то один раз хочет просто копи-вставлять это решение, не особо приходя в сознание далее. Просто по мере реализации жизнь обставляет концепцию лесом костылей. А мы видим конечный результат и думаем про человека не очень позитивно. Стремление к инкапсуляции не самый страшный из грехов.

[Мурик]

Так вот и я о том же: почему мигать можно, а другие задачи решать нельзя при помощи задержек?

Потому что если несколько задач выполняются друг за другом и паузы для них должны быть разные, то это лишнее усложнение кода, которое не улучшает чтение программы и ее дальнейшую доработку.
Статья по теме https://habrahabr.ru/post/249273/

Спойлер

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

Вот задача для примера: у нас есть 4 выхода, на которых необходимо выводить импульсы разной длительности с разными паузами. Все, что у нас есть – это системный таймер, который считает в миллисекундах.

Усложняем задачу в духе “сам себя замучаю на ардуино”. Таймеры заняты другим, PWM не подходит, ибо не на всех ножках он работает, да и не загонишь его на нужные режимы обычно. Немного подумав, садимся и пишем примерно такой код

Код: Выделить всё

// инициализация
int time1on=500; // Время, пока выход 1 должен быть включен
int time1off=250; // Время, пока выход 1 должен быть выключен
unsigned int now=millis();
....
// где-то в цикле
if(millis()<now+time1on)
{
  port1=ON;
}
else
{
port1=OFF;
if(millis()>now+time1on+time1off)
  {
  now=millis();
  }
}

И так или примерно так для всех 4 портов. Получается приличная портянка на несколько экранов, но эта портянка работает и работает довольно быстро, что для микроконтроллера важно.

Потом внезапно программист замечает, что при каждом цикле дергается порт, даже если его состояние не меняется. Правит всю портянку. Потом число портов с такими же потребностями увеличивается в два раза. Программист плюет и переписывает все в одну функцию типа PortBlink(int port num).

Почти наступило счастье, но внезапно потребовалось что бы на каком-то порту вместе с управлением “на выход” что-то предварительно считывалось и уже на основе этого считанного управлялся порт. Программист снова матерится и делает еще одну функцию, специально под порт.

Счастье? А вот фигу. Заказчик что-то этакое прицепил и это считанное может легко тормознуть процесс на секунды … Начинается стенания, программисты правят в очередной раз код, окончательно превращая его в нечитаемый треш, менеджеры выкатывают дикие прайсы заказчику за добавление функционала, заказчик матерится и решает больше никогда не связываться со встроенными решениями.

(типа реклама и восхваление) А все почему? Потому что изначально было принято неправильное решение о платформе. Если есть возможность, мы предлагаем навороченную платформу даже для примитивных задач. По опыту стоимость разработки и поддержки потом оказываются гораздо ниже. Вот и сейчас для управления 8мю выходами я возьму STM32F3, который может работать на 72МГц. (шепотом) На самом деле просто у меня под рукой демоплата с ним (смаил). Была еще с L1, но мы ее нечаянно использовали в одном из проектов.
Открываем STM32Cube, выбираем плату, включаем галочку около FreeRTOS и собираем проект как обычно. Нам ничего этакого не надо, поэтому оставляем все по умолчанию.

Что такое FreeRTOS? Это операционная система почти реального времени для микроконтроллеров. То есть все, что вы слышали про операционные системы типа многозадачности, семафоров и прочих мутексов. Почему FreeRTOS? Просто ее поддерживает STM32Cube . Есть куча других подобных систем – та же ChibiOS. По своей сути они все одинаковые, только различаются командами и их форматом. Тут я не собираюсь переписывать гору книг и инструкций по работе с операционными системами, просто пробегусь широкими мазками по наиболее интересным вещам, которые очень сильно помогают программистам в их нелегкой работе.

А если нет кучи дел?

Обычно это в простых программах, типа мегания светодиодом. В более или менее сложной, дела обычно есть. А если их нет, то МК можно отправить спать.

[Аlex]

Статья по теме https://habrahabr.ru/post/249273/

Статья ниАчом... Человек, обхаяв одну портянку кода, сделал другую, ничем не меньше, и при всём при этом, потратил на каждый мигающий светодиод по 610 байт оперативки.
Отличный пример, однако, ничего не скажешь ...

Добавлено after 2 minutes 39 seconds:
Ах, да, ещё. Пришлось ещё вместо tiny взять STM32... Супер...