У меня организован такой цикл case:

switch(State) {
...
case IDLE: // этот код выполняется если снято с охраны
off(LED);

if( one_wire_main () ) { //если что-то приложили (ключ)

if(USER_KEYS()) {
State = IDLE_HAS_KEY;
}
}

break;

case IDLE_HAS_KEY:
...

Вопрос- вот иду я по проге, приложили USER_KEYS, я сразу прыгаю на метку IDLE_HAS_KEY? Или дохожу до оператора break, потом компилятор анализирует что там было раньше и после break прыгает уже на метку?

Что значит цикл case???

на case IDLE_HAS_KEY: вы попадете только когда оператор switch(State) выполнится заново и при этом State будет = IDLE_HAS_KEY

switch - это всего лишь переключатель в зависимости от State делается JUMP в нужную точку

если хотите в этой же итерации попасть туда используйте goto

например так:


Сообщение отредактировал KRS - Sep 6 2006, 06:28

Совет правильный, но вообще-то использование команды goto является плохим стилем программирования и очень затрудняет отладку. Лично я построил бы диспетчер следующим образом:

Ну и все это, разумеется, крутится в бесконечном цикле.

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

if( MasterKey () ) { //здесь- что, если приложен мастер
//здесь опишем процесс, записи новых ключей, только из IDLE
//если прижали мастер на 5 сек

CntT0= 0;
while (CntT0 !=95) //если в теч. 5 сек. приложили мастер, вызываю запись USERa
...
WRITE_KEY();
...
State = IDLE_HAS_KEY;
...

State = IDLE_HAS_KEY; это состояние, куда я должен попать если кратковременно приложил- отпучтил ключ, а если 5 сек. удерживаю, то в ф-ю WRITE_KEY();

как бы это завернуть покрасивше?

Сообщение отредактировал Метценгерштейн - Sep 6 2006, 08:32

так корректно будет?

CntT0= 0;
if( !one_wire_main () ) { //если отпустили ключ
if (CntT0 !=95) //если не досчитал до 5 сек., то меняю сост. на IDLE_HAS_KEY
State = IDLE_HAS_KEY;
else
WRITE_KEY();
}

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

В многих подобных системах используется след. алгоритм:
1. При кратковременном замыкании контактов считывателя переходим в режим ожидания мастер-ключа.
2. Если в течение заданного интервала обнаружили мастер - переходим в режим записи USER-ключа, иначе - возвращаемся в режим ожидания.
3. Ну, и если в режиме программирования в течение разумного промежутка не был обнаружен новый ключ или же при повтороном замыкании контактов считывателя - также возвращаемся в дежурный режим.

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

if( MasterKey () ) { //здесь- что, если приложен мастер
//здесь опишем процесс, записи новых ключей, только из IDLE
//если прижали мастер на 5 сек
//**************************** ***************** *******************************
/* CntT0= 0;
if( !one_wire_main () ) { //если отпустили ключ
if (CntT0 < 95) //если не досчитал до 5 сек., то меняю сост. на IDLE_HAS_KEY
State = IDLE_HAS_KEY;
else
WRITE_KEY();
} */
//************************************* **************** ************************
State = IDLE_HAS_KEY;
}
}

здесь закомментировано, так оно просто переключается между состояниями, ф-я записи ключей WRITE_KEY();
попав в неё, я хочу помигать лампочкой.
Как лучше тут описать? Чтобы я держал ключ, как только время достигло 4 сек, оно прыгал в ту ф-ю, а не ждало пока я ключ отпущу, и потом только прыгало?

По логике нужно использовать 1-й вариант - а то как пользователь узнает, что мастер-ключ уже можно отпускать.

while ( MasterKey () ) { //пока ещё приложен ключ
CntT0= 0;
if (CntT0 == 95) { //если не досчитал до 5 сек., то меняю сост. на IDLE_HAS_KEY
WRITE_KEY ();
}
}

такая конструкция была бы правильной? В ф-ии WRITE_KEY (); я просто мигаю лампочкой.
Но вот что- то оно не мигает....

Не так это делается
После проверки ключа на валидность надо секунды на три блокировать ввод.
Иначе можно нарваться на двойное переключение состояния, люди по разному
ключ прикладывают.
После ввода ключа, блокируешь ввод,
очищаешь счетчик в таймерном прерывании,
отсчитываешь 3 - 4 секунды
и разрешаешь ввод ключа.
Если ключ все еще приложен, он тут же считается и вот тут то и надо принимать решение
о переходе в режим ввода новых ключей.
Успехов

Пример с сетью 1-wire с "таблеткой" и другими DSxxxx есть в симуляторе PROTEUS - вы можете отладить код на ПК.
Пример находится в папке:
SAMPLES\One-Wire\NETWORK

ещё вопрос по Си (простой)
в таком виде

case ARMED: // этот код выполняется под охраной

if(signal(SENS)) {

on(SIRENA);

CntT0= 0;
if (CntT0 == 700) { // продолжительность воя сирены - подобрать
off(SIRENA);
}
}

это фрагмент, попал я в режим под охраной, как только поступил сигнал с датчика, включаю сирену, теперь надо, чтобы она либо сама отключилась через время, либо я ключом её выключил. Считает счетчик, если равно количесву прерываний счетчика 700, сирена выключается. Не понятно по скобкам, этот таймер у меня внутри ф-ии, я туда всегда буду попадать по прерыванию счетчика? или я там буду, пока SENS (датчик) сработал? Вот здесь непонятность небольшая

Таймера внутри функции лично я не вижу. По прерыванию от него вы будете попадать в соответствующую ISR (программу обслуживания прерываний). А вот, что вы будуте в этой ISR делать (просто отключать сирену, флаг какой-то устанавливать, что пора бы её выключить) - это уже ваше дело.

всё правильно, я немного не так выразился, по прерыванию от таймера я попадаю сюда

#pragma vector = TIMER0_OVF0_vect
__interrupt void overflow_timer1(void)
{
CntT0++;
}

Здесь увеличиваю переменную
А что у меня получилось? Сначала сработал датчик, я попал в эту ф-ю, включил сирену, обнулил
CntT0=0;
теперь надо, чтобы при проверке при количеству подсчетов =700, выполнялось отключить сирену. Уже вижу, что запутался. Я хотел вот что спросить, здесь
if(signal(SENS)) {
я буду считать до 700 если это условие постоянно истина? Т.е. все время идет сигнал от датчика? как только нет, я вываливаюсь из цикла и уже не проверяю на 700?

Может невпопад, за веткой не слежу. Я бы сделал так: