DuMAM napisał(a):
Sprawdzanie flag jest jak najbardziej poprawne (nawet w książce tak było robione)
A w którym miejscu książki jest tak robione? Bo szukam i nie mogę znaleźć.
Jak na moje to masz błędne sprawdzanie flag. Jeśli będziesz miał ustawioną flagę minut, godzin i dni czyli clock_flag będzie miała wartość 0b00000111. I teraz w pierwszym warunku if robisz coś takiego
[syntax=c]clock_flag &=(DT_MIN | DT_HOUR)[/syntax] czyli [syntax=c]clock_flag = clock_flag & (DT_MIN | DT_HOUR)[/syntax]
clock_flag - 0b00000111
(DT_MIN | DT_HOUR) - 0b00000011
Sprawdź co będziesz miał w zmiennej clock_flag po takiej operacji 0b00000111 & 0b00000011
Flaga dni niechcący zostanie wyzerowana
Aby to potwierdzić napisałem króciutki programik, który na lcd wyświetla sekundy, minuty, godziny i dni. Jeśli sprawdzałem flagi ifami w zaremowanych liniach to wyświetlało mi tylko sekundy, minuty i godziny. Po zmianie warunków sprawdzania flag zaczęło wyświetlać mi wszystkie zmienne.
[syntax=c]#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>
#include "LCD/lcd44780.h"
#define DT_MIN 1
#define DT_HOUR 2
#define DT_DAY 4
#define DT_MONT 8
#define DT_YEAR 16
#define DT_DOT 32
typedef struct {
volatile uint8_t godz;
volatile uint8_t min;
volatile uint8_t sek;
volatile uint8_t dzien;
volatile uint8_t miesiac;
volatile uint8_t rok;
} t_dt;
typedef union {
volatile uint8_t idx[6];
volatile t_dt data_czas;
} t_datetime;
t_datetime zegarek;
volatile uint8_t clock_flag=0;
int main(void)
{
lcd_init();
DDRA |=(1<<PA7);
PORTA |=(1<<PA7);
lcd_str("Start");
TCCR1B |=(1<<WGM12);
TCCR1B |= (1 << CS12) | (1 << CS10);
OCR1A = 15624;
TIMSK |= (1 << OCIE1A);
sei();
zegarek.data_czas.godz=23;
while(1)
{
lcd_locate(1,0);
lcd_int(zegarek.data_czas.sek);
lcd_str(" ");
// if(clock_flag &=(DT_MIN | DT_HOUR))
if( (clock_flag & DT_MIN) || (clock_flag & DT_HOUR) )
{
clock_flag &=~(DT_MIN | DT_HOUR);
lcd_locate(1,3);
lcd_int(zegarek.data_czas.min);
lcd_str(" ");
lcd_locate(1,6);
lcd_int(zegarek.data_czas.godz);
lcd_str(" ");
}
// if(clock_flag &=(DT_DAY | DT_MONT | DT_YEAR)){
if( (clock_flag & DT_DAY) | (clock_flag & DT_MONT) | (clock_flag & DT_YEAR)){
clock_flag &=~(DT_DAY | DT_MONT | DT_YEAR);
lcd_locate(1,9);
lcd_int(zegarek.data_czas.dzien);
lcd_str(" ");
}
}
}
ISR(TIMER1_COMPA_vect){
zegarek.data_czas.sek++;
//if(zegarek.data_czas.sek%5==0)
// clock_set_flag(DT_DOT);
if(zegarek.data_czas.sek==60){ // ODLICZENIE 60 SEK I USTAWIENIE FLAGI
zegarek.data_czas.sek=0; // INFORMUJACEJ O KONIECZNOSCI AKTUALIZACJI
clock_flag |= DT_MIN; // WYSWIETLACZA
zegarek.data_czas.min++;
}
if(zegarek.data_czas.min==60){ // ODLICZENIE 60 MIN I USTAWIENIE FLAGI
zegarek.data_czas.min=0; // INFORMUJACEJ O KONIECZNOSCI AKTUALIZACJI
clock_flag |= DT_HOUR; // WYSWIETLACZA
zegarek.data_czas.godz++;
}
if(zegarek.data_czas.godz==24){
clock_flag |= DT_DAY;
zegarek.data_czas.godz=0;
zegarek.data_czas.dzien++;
}
}[/syntax]Statystyki: Napisane przez jacekk232 — 24 kwi 2014, o 16:56
]]>