Здравейте
първо да кажа че съм начинаещ в програмиране и постепенно се уча.Захванах се да си направя една уж проста автоматика,а то се оказа голяма греда с този loop.Използвам Arduino като развойна платка.Трябва да се изпълняват следните функции,като само една ми прави проблем и не ми достигат знания за да я реша.

1. натискане и задържане на Бутон UP -->RELAY_UP работи докато се задържа бутон UP
всички други релета са Off
след пускане на бутона всички релета Off

2. натискане и задържане на Бутон DOWN --> RELAY_UP работи 3 сек. и се Off
RELAY_COMPR се On след като RELAY_UP се е Off и работи докато е задържан бутона
RELAY_DOWN да се On 1 сек. след RELAY_COMPR и да работи до пускане на бутона
след пускане на бутона всички релета Off

Имам проблем с това RELAY_UP понеже влиза в цикъл и се On за кратко време когато не трябва.

Това ми е кода


може би ако по някакъв начин извадя тази част от loop ще заработи както трябва



и точно това не знам как да стане.Ще се радвам ако някой може да ми помогне.

Трябва ти или запомняне,история, флагове, стейт-машини, броячи за време....
....или по-простичко ...лупове в лупа.

Ако видиш задействан бутон DOWN , (IF DOWN=1 {1.2.3. }
- изпълняваш първата операция 1. релеUP=1 , дилей 3 секунди, изключване
- изпълняваш втората операция 2.релеCOMPR =1 , дилей 1 секунда ,изключване
- включваш реле DOWN и почваш луп 3.- стоиш в него, докато бутонът ти е натиснат.

Последният луп и дилей функциите са ... обичайни за начинаещите.
Направи го така...после мисли как изобщо да нямаш дилей и изчакващи лупове.
По време на тези неща процесорът не прави нищо,не отговаря на външния свят....става така, но за прости неща.

Този подход станал навик се изкоренява много трудно, лупове, лупчета .... това може да си го позволиш след като си разбрал как се прави със стейтове, никакви лупове, цялата програма ти е един луп, в нея си имаш проверки за определени състояния и флагове за съответните, дали са само флагове или конкретни параметри няма значение, въртиш ифовете, като има нещо което е дошъл момента да се изпълни изпълняваш, променяш съответните стейтове и продължаваш нататък, ако имаш нещо на прекъсване си го обслужваш до колкото е разумно там другото което е неразмно си го изпълняваш отново някъде в основния луп ... като схванеш идеята след това пиши както искаш, лошото е ако никога не схванеш идеята, лично мое мнение ако искаш да се занимаваш с ембедед.

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

Много ми е трудно да разбера какво е имал в предвид автора като задание. Пускам ти един пример какво бих направил аз.
Нещо се загубих в описанието но принципа е този , просто разшири стейт машината

Пооправи си малко синтаксисът - това е на C++ с FreeRTOS
Никога не съм бачкал с Ардуино, а с 'C' много отгавна
идеята е, че помниш статуса на бутоните и при промяна изпълняваш съответният state . Мисля , че това ще ти свърши работа.
потърси с кои функции можеш да смениш xTaskGetTickCount и vTaskDelay
Поздрави

Промених кода както ме посъветвахте



със всяко натискане на бутона down се променя състоянието.Но си зададох въпроса удачно ли е да е така.

виж PM
Няма много разлика от 1-вата ти версия

Не зная с тоя delay (100); какво точно имаш в предвид

видях поста ви колега,но този език ми е непознат все още.Ще се постарая да го разбера и да хвана логиката.

ок успех

Идеята е с постоянно натиснат бутон down се си извърта всички state и да остане на последния до пускането на бутона или ако бутона се пусне някъде другаде от цикъла веднага всичко да е Off

[/quote]

задържам малко заради трепването което се получава при превключването на бутона,иначе се получава малко хаус в работата и прескача state.

Ако много бързаш - направи го с лупчета.
Ако не бързаш - представи си ,че 3те релета и 2та бутона са само 1/10 от всичко в ТВОЯТА система,,,,и трябва да обслужваш
още 9/10 .
Тогава тая 1/10 подистема я разглеждаш като нещо , което си има текущо сътояние и трябва да реагира на промени в средата.
Няма значение колко променливи ще трябва да дефинираш, това е пътят.

Обучението по програмиране обикновено включва съставяне на блок-схеми....обаче система със състояния ,памет - малко трудно се представя с блок схема. Та ....не бързай ,мисли и питай. И запомни, че много трудно се чете чужд код повече от 20реда.

Учжд, аз след 2 години моя не мога да чета ти за чужд говориш. Рисуването на блок схема понякога помага. Някой път не мога да "видя" алгоритъма и ако дълго време не успявам да го видя - почвам да рисувам - някой път помага.

Абе в момента съм на съвсем друга вълна и не съм гледал кодовете, но както Стефан е писал, идеята е че имаш състояния, затова и се нарича state machine. Имаш определени състояния за отделните подсистеми. Най-просто, имаш светодиод имаш бутон. Бутона като вход може да има две състояния, 0/1 диода като изход също. В същото време обаче тези и двете може да имат и други състояния/статуси, свързани с тяхното функциониране. Примерно бутона при продължително натискане влиза в състояние задаване интервал честота на мигане, съответно при следващото дълго излиза от това състояние, а при кратки натискания вкл/изкл. светодиода или променя честотата с една стъпка. В този случай примерно дебоунсинга го правиш на база състоняието на входа, абстрахирай се от това дали е истински вход или твой флаг. Това са няколко прости стейт машини с по две състояния, в този случай вместо switch case може да използваш if. Съответно имаш флаг който казва бутона е натиснат, това е състояние в което бутона преминава от 1 в 0. Отделно имаш флаг/състояние което казва бутона е бил натиснат късо или пък дълго, което съответно определяш по това колко време е минало от натискането на бутона до неговото отпускане. Отделно знаеш кога бутона е отпуснат. Да речем ако времето е по-малко от 200 мс, приемаш че е смущения, тогава нулираш флага указващ че бутона е бил натиснат и чакаш следващо натискане. Ако времето е между 200 и 500 мс, приемаш че бутона е натиснат т.е. дигаш флаг valid_button и нямаш промяна във функционалното състояние. Ако времето е приметно 1000 мс казваш че стейта на функцията на бутона е дълго натискане.
След това когато проверяваш флага valid_button в случай че е вдигнат първо правиш switch / case на функцията на бутона, ако приемем че имаш два режима може да го направиш с if, обаче бутона може да има и 10 режима, съответно в тоя случай switch / case е по-удачния вариант. Така ако да речем стейта е 0, инвертираш флага за състояние на леда съответно той се вкл. или изключва, ако стейта е 1 отиваш на следващата стъпка на честотата за мигане, ако е 2 пък може да правиш нещо друго и т.н.
Малко трудно се описва нещо което трябва да се начертае или пък да се напише като код ... не знам дали схвана идеята.
Естествено много от тия неща, особено свързани с бутони, дебонсинг и т.н. могат да се реализират в голяма степен хардеурно, примерно вместо да проверяваш постоянно бутона дали е натиснат или отпуснат, чакаш съответното прекъсване прекъсване, вместо да проверяваш постоянно някакъв таймер може и той да е на прекъсване като така ти остава време да правиш и други неща.
Докато правиш 1-2 неща много по лесен изглежда подхода с лупове и някакви фиксирани състояния, но когато нещата започнат да стават повече това спира да работи. Не можеш да въртиш в някакъв луп чакайки някой да отпусне или натисне бутона защото ще трябва да вършиш и други неща, примерно ако диода не свети постоянно а мига ще трябва да го мигаш и т.н. ...
Та моя съвет е макар и да изглежда по-трудно опитай да пишеш код без лупове, освен main луп избягвай други лупове, особено пък такива от които нямаш надежден изход.

п.с. това че скачаш е нормално, за да не се получава трябва да правиш дебонсинг. Не знам дали нямаш някаква стандартна функция за това в Ардуиното, но винаги е по-добре да знаеш как да си го направиш. В зависимост как работиш имаш много варианти за дебонсинг, ако имаш само полиране както по горе ти писах направи си флаг, button-poressed, button_valid преди да вдигнеш втория чакаш да мине известно време, ако бутона се отпусне може да нулираш първия, т.е. да започне да тече отново дебонсинг тайма, или просто проверяваш чак след като изтече времето и ако е още натиснат вдигаш button_valid. Сега най-удачно е използването на системен таймер, когато ти станат повече неща трудно ще е да водиш отделни броячи за всяко, освен ако не си съвсем на границите на ресурсите подобре е да сравняваш тайм стамп който си взел при възникване на събитието с текущо време.

Може секвенциално(стейт машини), може комбинаторно и всякак. Ако съм разбрал питащия правилно, иска да реализира някаква такава фунция:


която би се емулирала в Ардуиното с някакъв такъв код:


с уговорката че не съм писал за Ардуино от години и сигурно има нужда от някакви дооправки. Пин 2 и 3 са бутоните. Пин 4,5 и 6 са релетата.

Здравейте
първо искам да изкажа благодарности на всички за съветите,препоръките и примерите.Кода е написан благодарение на огромната помощ на колегата panchev68 за което много му благодаря .Много хубаво обяснява и показва.Тези дни ще направя платка и ще се тества в реални работни условия.
Благодаря ви!