Не работает как надо If() с DCF77

[de1m]

Всем привет!
Работаю, на часами на DCF77.
Если коротко, DCF77 шлёт каждую секунду 0 или 1. Если 0, то в начале секунды сигнал понижается до 25% на 100мс, а если 1 то понижение длится 200мс. Для определения начала минуты сигнал на 59-й секунде не понижается.

Собственно вот код и я ещё вложил проект в проеусе и текс для имитации сигнала DCF77(DCF77.txt)

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

#define F_CPU 8000000UL
#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>

uint8_t langeDCF;
uint16_t dcfL; //


int main(void)
{
	/*lcd_init();
	set_cursor(0,1);
	lcd_string("------");*/
	langeDCF=0;
	dcfL = 0;
	
	// Timer 0 konfigurieren
	TCCR0B |= (1<<CS02) | (1<<CS00);; // Prescaler 1024
	
	// Overflow Interrupt erlauben
	TIMSK |= (1<<TOIE0);
//	TIFR |= (1<<TOV0);
	// Global Interrupts aktivieren
	sei();
	
	DDRB = 0xFF;
	DDRA = 0x00;
	PORTA = 0x00;
	PORTB = 0x00;
    while(1)
    {
		
    }
}

ISR (TIMER0_OVF_vect)
{
	
	if ((PINA&(1<<PA0)) == 0)
	{
		if (dcfL<860)
		{
			PORTB|=(1<<PB2);
		}

		if (dcfL>860 && dcfL<950)
		{
			PORTB|=(1<<PB0);
			
		}
                
		if (dcfL>1800)
		{
			PORTB|=(1<<PB3);
		}                 

		
		PORTB&=~(1<<PB0);
		PORTB&=~(1<<PB1);
		PORTB&=~(1<<PB2);
		PORTB&=~(1<<PB3);
		dcfL=0;
		
	} 
	if ((PINA&(1<<PA0)) == 1)
	{
		PORTB|=(1<<PB1);
		dcfL +=20;
		
		
	}
	
	
	
	TCNT0 = 96; //Timer startet bei 96 (200-96=160 Ticker) 1:(8.000.000:160:1024)=0,02048s~20,4ms
}

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

Проверка происходит тут:

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

      if (dcfL<860)
      {
         PORTB|=(1<<PB2);
      }

      if (dcfL>860 && dcfL<950)
      {
         PORTB|=(1<<PB0);
         
      }
                
      if (dcfL>1800)
      {
         PORTB|=(1<<PB3);
      }  

Но у меня паралельно к PB0 загорается ещё и PB2.
Объесните мне кто-нибудь, где я ошибаюсь?

[Mishany]

Синтаксис неправельный, скобок не хватает
Вот форма правильная:
If ((a>b)&&(c<d))
{}

[de1m]

Синтаксис неправельный, скобок не хватает
Вот форма правильная:
If ((a>b)&&(c<d))
{}

Поменял на

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

if ((dcfL>860)&&(dcfL<950)) //0
		{
			PORTB|=(1<<PB0);
		}

Но без изменений

[de1m]

Вроде как заработало. Поменял везде на

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

if ((dcfL>750)&&(dcfL<840))
		{
			PORTB|=(1<<PB2);
		}

		if ((dcfL>840)&&(dcfL<950)) //0
		{
			PORTB|=(1<<PB0);
		}
		
		if ((dcfL>1800)&&(dcfL<2300)) //Min.
		{
			PORTB|=(1<<PB3);
		}

Может кстати кто хорошую книгу по С посоветовать, где такие нюансы написаны? Я до этого искал, как правильно If пишеться, но именно такое не нашел.

[Mishany]

Dcfl +=20; это как понять?
Может dcfl=dcfl+20;
Я синтаксис выдергивал и складывал в тетрадку из найденых в интернете рабочих программ с коментами, плюс есть отличия при использовании разных сред разработки

[de1m]

Dcfl +=20; это как понять?
Может dcfl=dcfl+20;

это то же самое и обозначает.

[Леонид Иванович]

Синтаксис неправельный, скобок не хватает. Вот форма правильная: If ((a>b)&&(c<d)){}

Не нужны тут скобки. Приоритет операции && ниже, чем > и <.

Приоритет операций: http://ru.cppreference.com/w/cpp/langua ... precedence

[Mishany]

У меня на аврстудии на такие записи ругался компилятор

[de1m]

Единственный момонт не понятный, это что иногда пропуск идёт

[de1m]

Не нужны тут скобки. Приоритет операции && ниже, чем > и <.

Да еслит так сделать, то тоже работает.

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

if (dcfL>750&&dcfL<840)
		{
			PORTB|=(1<<PB2);
		}

		if (dcfL>840&&dcfL<1000) //0
		{
			PORTB|=(1<<PB0);
		}
		
		if (dcfL>1800&&dcfL<2300) //Min.
		{
			PORTB|=(1<<PB3);
		}

[Mishany]

Если dcfl в диапозоне 1000 - 1800 это норм?

[de1m]

Если dcfl в диапозоне 1000 - 1800 это норм?

Это бывает, когда новая минута начинается, то есть при с 58-60 сигнал не понижается. Так что это нормально

[ploop]

Я синтаксис выдергивал и складывал в тетрадку из найденых в интернете рабочих программ с коментами, плюс есть отличия при использовании разных сред разработки

Есть стандарты языка, есть компиляторы, которые эти стандарты поддерживают. При чём тут среды разработки - непонятно.
Синтаксис Си прост как валенок. Может вам всё-таки книжку почитать? И вопросов +=20; это как понять? не будет.

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

[HHIMERA]

Синтаксис неправельный, скобок не хватает. Вот форма правильная: If ((a>b)&&(c<d)){}

Не нужны тут скобки. Приоритет операции && ниже, чем > и <.

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

if ((a > b) && (c < d)){}

[ploop]

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

[de1m]

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

Ошибка была не в коде.

Вот такое

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

if (dcfL<860)
      {
         PORTB|=(1<<PB2);
      }

не работает, а вот так работает

if (dcfL>750&&dcfL<840)
{
PORTB|=(1<<PB2);
}

то есть я так понимаю, что надо было тупо с обоих сторон ограничеть.

[ploop]

шибка была не в коде.

... звёзды не так сошлись?

[de1m]

... звёзды не так сошлись?

В первом случае я сказал, что если dcfL не больше 840. А во втором, ещё добавил, что и не меньше 760. Это помогло

[ploop]

Только поправьте где-нибудь на больше или равно. А то в случае граничного значения ни одно условие не сработает.

[HHIMERA]

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

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