Страница 1 из 1
Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 21:11:53
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.
Объесните мне кто-нибудь, где я ошибаюсь?
Re: Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 21:15:50
Mishany
Синтаксис неправельный, скобок не хватает
Вот форма правильная:
If ((a>b)&&(c<d))
{}
Re: Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 21:22:12
de1m
Mishany писал(а):Синтаксис неправельный, скобок не хватает
Вот форма правильная:
If ((a>b)&&(c<d))
{}
Поменял на
Код: Выделить всё
if ((dcfL>860)&&(dcfL<950)) //0
{
PORTB|=(1<<PB0);
}
Но без изменений
Re: Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 21:32:41
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 пишеться, но именно такое не нашел.
Re: Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 21:41:18
Mishany
Dcfl +=20; это как понять?
Может dcfl=dcfl+20;
Я синтаксис выдергивал и складывал в тетрадку из найденых в интернете рабочих программ с коментами, плюс есть отличия при использовании разных сред разработки
Re: Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 21:44:47
de1m
Mishany писал(а):Dcfl +=20; это как понять?
Может dcfl=dcfl+20;
это то же самое и обозначает.
Re: Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 21:45:21
Леонид Иванович
Mishany писал(а):Синтаксис неправельный, скобок не хватает. Вот форма правильная: If ((a>b)&&(c<d)){}
Не нужны тут скобки. Приоритет операции && ниже, чем > и <.
Приоритет операций:
http://ru.cppreference.com/w/cpp/langua ... precedence
Re: Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 21:47:47
Mishany
У меня на аврстудии на такие записи ругался компилятор
Re: Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 21:48:11
de1m
Единственный момонт не понятный, это что иногда пропуск идёт
Re: Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 21:51:04
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);
}
Re: Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 21:57:39
Mishany
Если dcfl в диапозоне 1000 - 1800 это норм?
Re: Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 22:01:01
de1m
Mishany писал(а):Если dcfl в диапозоне 1000 - 1800 это норм?
Это бывает, когда новая минута начинается, то есть при с 58-60 сигнал не понижается. Так что это нормально
Re: Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 22:01:39
ploop
Mishany писал(а):Я синтаксис выдергивал и складывал в тетрадку из найденых в интернете рабочих программ с коментами, плюс есть отличия при использовании разных сред разработки
Есть стандарты языка, есть компиляторы, которые эти стандарты поддерживают. При чём тут среды разработки - непонятно.
Синтаксис Си прост как валенок. Может вам всё-таки книжку почитать? И вопросов
+=20; это как понять? не будет.
de1m, где-то был косяк. Возможно тупая опечатка, копипаст, что угодно. Учитесь уже пользоваться отладкой. Даже если специальных инструментов нет - думайте логически, отключайте куски кода и методом исключения найдёте косяк. В этом коде на это потребуется пара минут, пост вы писали гораздо дольше.
Re: Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 22:02:31
HHIMERA
Леонид Иванович писал(а):Mishany писал(а):Синтаксис неправельный, скобок не хватает. Вот форма правильная: If ((a>b)&&(c<d)){}
Не нужны тут скобки. Приоритет операции && ниже, чем > и <.
Дело ещё и не только в приоритете... а ещё и в наглядности... в лучшем, однозначном, восприятии текста...
if ((a > b) && (c < d)){}
Re: Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 22:05:49
ploop
HHIMERA, это уже стиль написания кода. Сложные выражения действительно лучше визуально делить. Но ведь претензии были к логике, а не к стилю.
Re: Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 22:06:48
de1m
В этом коде на это потребуется пара минут, пост вы писали гораздо дольше.
Ошибка была не в коде.
Вот такое
не работает, а вот так работает
if (dcfL>750&&dcfL<840)
{
PORTB|=(1<<PB2);
}
то есть я так понимаю, что надо было тупо с обоих сторон ограничеть.
Re: Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 22:08:03
ploop
de1m писал(а):шибка была не в коде.
... звёзды не так сошлись?
Re: Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 22:11:32
de1m
ploop писал(а):
... звёзды не так сошлись?
В первом случае я сказал, что если dcfL не больше 840. А во втором, ещё добавил, что и не меньше 760. Это помогло
Re: Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 22:13:22
ploop
Только поправьте где-нибудь на больше или равно. А то в случае граничного значения ни одно условие не сработает.
Re: Не работает как надо If() с DCF77
Добавлено: Сб авг 30, 2014 22:16:11
HHIMERA
ploop писал(а):Сложные выражения действительно лучше визуально делить. Но ведь претензии были к логике, а не к стилю.
Я как бы и не спорю... Но понятно написанный код облегчает понимание логики... а каша без пробелов, скобок и отступов просто замыливает глаз...
