сигнализация на mege8

[igumich]

доброго времени суток. решил собрать сигнализацию на меге8-ой.
не могу понять как сделать чтобы при любой открытой двери PORTC.2 один раз включился, выключился, и не зацикливался.


/*****************************************************
This program was produced by the
CodeWizardAVR V1.25.8 Professional
Automatic Program Generator
© Copyright 1998-2007 Pavel Haiduc, HP InfoTech s.r.l.
http://www.hpinfotech.com

Project :
Version :
Date : 08.06.2012
Author : 123
Company : F4CG
Comments:


Chip type : ATmega8
Program type : Application
Clock frequency : 1,000000 MHz
Memory model : Small
External SRAM size : 0
Data Stack size : 256
*****************************************************/

#include <mega8.h>
#include <delay.h>
// Declare your global variables here

void main(void)
{

int a=0;
int b=0;

// Declare your local variables here

// Input/Output Ports initialization
// Port B initialization
// Func7=In Func6=In Func5=In Func4=In Func3=In Func2=In Func1=Out Func0=Out
// State7=T State6=T State5=T State4=T State3=P State2=P State1=0 State0=0
PORTB=0x0C;
DDRB=0x03;

// Port C initialization
// Func6=In Func5=In Func4=In Func3=Out Func2=Out Func1=Out Func0=Out
// State6=P State5=P State4=P State3=0 State2=0 State1=0 State0=0
PORTC=0x70;
DDRC=0x0F;

// Port D initialization
// Func7=In Func6=In Func5=In Func4=In Func3=In Func2=In Func1=In Func0=In
// State7=P State6=P State5=P State4=P State3=P State2=P State1=P State0=P
PORTD=0xFF;
DDRD=0x00;

// Timer/Counter 0 initialization
// Clock source: System Clock
// Clock value: Timer 0 Stopped
TCCR0=0x00;
TCNT0=0x00;

// Timer/Counter 1 initialization
// Clock source: System Clock
// Clock value: Timer 1 Stopped
// Mode: Normal top=FFFFh
// OC1A output: Discon.
// OC1B output: Discon.
// Noise Canceler: Off
// Input Capture on Falling Edge
// Timer 1 Overflow Interrupt: Off
// Input Capture Interrupt: Off
// Compare A Match Interrupt: Off
// Compare B Match Interrupt: Off
TCCR1A=0x00;
TCCR1B=0x00;
TCNT1H=0x00;
TCNT1L=0x00;
ICR1H=0x00;
ICR1L=0x00;
OCR1AH=0x00;
OCR1AL=0x00;
OCR1BH=0x00;
OCR1BL=0x00;

// Timer/Counter 2 initialization
// Clock source: System Clock
// Clock value: Timer 2 Stopped
// Mode: Normal top=FFh
// OC2 output: Disconnected
ASSR=0x00;
TCCR2=0x00;
TCNT2=0x00;
OCR2=0x00;

// External Interrupt(s) initialization
// INT0: Off
// INT1: Off
MCUCR=0x00;

// Timer(s)/Counter(s) Interrupt(s) initialization
TIMSK=0x00;

// Analog Comparator initialization
// Analog Comparator: Off
// Analog Comparator Input Capture by Timer/Counter 1: Off
ACSR=0x80;
SFIOR=0x00;

while (1)
{
if (PIND.6==0)// Если нажата то постановка на охрану

{
if(PIND.4==0)// слабый удар
{
PORTC.0=1;
delay_ms(700);
PORTC.0=0;
delay_ms(700);
PORTC.0=1;
delay_ms(700);
PORTC.0=0;
delay_ms(700);
PORTC.0=1;
delay_ms(700);
PORTC.0=0;
delay_ms(700);
};
if(PIND.5==0)
{
b=1;
}
if(PIND.7==0) // Нажат сброс
{
b=0;
}
if(b==1)// Сильный удар. Мигаем до тех пор пока не нажмем сброс
{
PORTC.0=1;
delay_ms(200);
PORTC.0=0;
delay_ms(200);

};


if((PIND.0==0)|(PIND.1==0)|(PIND.2==0)|(PIND.3==0))// любая из дверей открыта
{
a=1;
}

if(PIND.7==0) // Нажат сброс
{
a=0;
}


if(a==1) //Если переменная равна 1 включить цикл мигания пока не нажмём сброс
{

PORTC.1=1;
delay_ms(200);
PORTC.1=0;
delay_ms(100);
};

if((PIND.0==0)|(PIND.1==0)|(PIND.2==0)|(PIND.3==0))
{
PORTC.2==1;
delay_ms(200);
PORTC.2==0;
}
}
};
}

ругается expression with possibly no effect на

PORTC.2==1;
delay_ms(200);
PORTC.2==0;

[igor-x]

ругается expression with possibly no effect на

PORTC.2==1;
delay_ms(200);
PORTC.2==0;

после

Код:

while(1) {...}


в программе не выполнится ни одна срока при правильном функционировании схемы, но не ругается а просто предупреждает

[uk8amk]

Код:

PORTC.2==1;
delay_ms(200);
PORTC.2==0;

Действительно, данное выражение не представляет собой никакого смысла.
Было бы логичнее написать так:

Код:

PORTC.2=1;
delay_ms(200);
PORTC.2=0;

[igor-x]

хорошо что hybroid этого не видел..

подозреваю что это было бы

[user@radiokot.ru] cat "сигнализация на mege8" | grep говнокод >forum.radiokot.ru

[radteh]

Естественно на PORTC.2 постоянно мигание, ведь нет ограничивающего условия, например такого:

Код:

if((PIND.0==0)|(PIND.1==0)|(PIND.2==0)|(PIND.3==0))// любая из дверей открыта
{
a=1;
c=1;
}

if(PIND.7==0) // Нажат сброс
{
a=0;
c=0;
}

if((a==1)&&(c==1))
{
PORTC.2=1;
delay_ms(200);
PORTC.2=0;
c=0;
}