ATNEL tech-forum

Re: Zerowanie się zmiennej... WHY?

2012-10-15T11:06:47+01:00

Bravo!

Statystyki: Napisane przez szopler — 15 paź 2012, o 11:06

Re: Zerowanie się zmiennej... WHY?

2012-10-15T09:58:17+01:00

Zamień:
[syntax=c]char DispLine_1[16];
char DispLine_2[16];[/syntax]
na:
[syntax=c]char DispLine_1[17];
char DispLine_2[17];[/syntax]
Problem rzeczywiście jest trudny do wykrycia. Trzeba na to uważać. Równie dobrze koniec łańcucha '/0' mógł zamazywać inną zmienną.

Statystyki: Napisane przez Krauser — 15 paź 2012, o 09:58

Re: Zerowanie się zmiennej... WHY?

2012-10-14T22:50:05+01:00

Uuu... widzę, że większy problem jest Nikt nie znalazł buga?

Statystyki: Napisane przez szopler — 14 paź 2012, o 22:50

Re: Zerowanie się zmiennej... WHY?

2012-10-09T20:00:29+01:00

Może to nie ma wpływu na powyższy problem (ja się nie znam), ale pierwszy raz widzę konstrukcję struktury w C, która zaczyna się i kończy dokładnie taką samą nazwą. W tym przypadku mamy przed nawiasem nazwę struktury _FLAGA i po kolejnym nawiasie (zamykającym) również _FLAGA. Trochę jak by masło maślane . Ciekawe jak taką sytuację traktuje kompilator? Z ciekawości przejrzałem kilka książek o C i trochę kodów na AVRFreaks(tutoriale) i żadne źródła nie podają takiej konstrukcji. Może ktoś bardziej oblatany w C może się wypowie co do moich wątpliwości.

Pozdrawiam
PP

Statystyki: Napisane przez Piotrek_P — 9 paź 2012, o 20:00

Re: Zerowanie się zmiennej... WHY?

2012-10-09T19:49:12+01:00

Wstawiłem... bez zmian

Statystyki: Napisane przez szopler — 9 paź 2012, o 19:49

Re: Zerowanie się zmiennej... WHY?

2012-10-09T18:41:20+01:00

kurczę ja teraz nie pamiętam z głowy składni sprintfa ale jeszcze raz pytam czy kłopotem jest wstawić sobie

lcd_int( Licznik_kWh );

Statystyki: Napisane przez mirekk36 — 9 paź 2012, o 18:41

Re: Zerowanie się zmiennej... WHY?

2012-10-09T18:02:41+01:00

[syntax=c] sprintf(DispLine_1," %010lu ", Licznik_kWh); break;[/syntax]

Dalej to samo

Statystyki: Napisane przez szopler — 9 paź 2012, o 18:02

Re: Zerowanie się zmiennej... WHY?

2012-10-09T12:46:28+01:00

Wiesz co ? weź ty sobie wyświetl gdzieś normalnie wartość tej zmiennej Licznik_kWh, bo jak na razie to widzisz ją tylko przez to

Code:

sprintf(DispLine_1," %07lu.%03u kWh", (uint32_t)(Licznik_kWh/1000), (uint16_t)(Licznik_kWh%1000)); 

czy nie prościej dla sprawdzenia wyświetlać sobie do testów samą zmienną bez żadnego dzielenia ?

Code:

lcd_int( Licznik_kWh );

bo coś to dziwnie wszystko wygląda

Statystyki: Napisane przez mirekk36 — 9 paź 2012, o 12:46

Re: Zerowanie się zmiennej... WHY?

2012-10-09T09:47:20+01:00

[syntax=c]/*
* main.c
*
* Created on: 02-08-2012
* Author: Michal
*/

/* Dolaczenia plikow */
#include
//#include
#include
#include
#include "LCD/lcd44780.h"

/* Definicje makr i uproszczeń */
#define sbi(byte,bit) byte |= (1<#define cbi(byte,bit) byte &= ~(1<#define bit(byte,n) (byte>>n)&1
#define FLAG_S1
#define FLAG_R0

/* Deklaracje zmiennych */
volatile struct _FLAGA {
uint8_tDisplay:1;
uint8_tAddTokWh:1;
} _FLAGA;

volatile uint16_tTimer_1s;
volatile uint16_tCounter;
volatile uint8_tpoprzedni_stan_portu_B;
volatile uint16_tMocCzynna;
volatile uint32_tLicznik_kWh;
charDispLine_1[16];
charDispLine_2[16];
volatile uint8_tDISPCNT = 0;

/* Deklaracje funkcji */
void obsluga_wyswietlacza(void);
//void zwieksz_licznik(void);

/* ...KOD... */
void obsluga_wyswietlacza(void) {
if (_FLAGA.Display == FLAG_S) {

//lcd_locate(0,0);
//lcd_str(" ");

switch (DISPCNT) {
case 0:
sprintf(DispLine_1,"Welcome!"); break;
case 1:
sprintf(DispLine_1," %4u W ", MocCzynna); break;
case 2:
sprintf(DispLine_1," %07lu.%03u kWh", (uint32_t)(Licznik_kWh/1000), (uint16_t)(Licznik_kWh%1000)); break;
}

lcd_locate(0,0);
lcd_str(DispLine_1);

// lcd_locate(1,0);
//lcd_str(DispLine_2);

_FLAGA.Display = FLAG_R;
}
}

/*void zwieksz_licznik(void) {
if (_FLAGA.AddTokWh == FLAG_S) {
Licznik_kWh++;
_FLAGA.AddTokWh = FLAG_R;
}

}*/

ISR(TIMER2_COMPA_vect) {// Wywoływane co 4ms przy OCR2A = 250
//TCNT2 = 0;// Zerowanie timera - W trybie CTC automatyczne!
Timer_1s++;

if (Timer_1s >= 1250) {
Timer_1s = 0;
MocCzynna = Counter;// Ilość impulsów zliczonych w ciągu 5 sekund daje moc w Watach
Counter = 0;
}

if (Timer_1s % 125 == 0) {
_FLAGA.Display = FLAG_S;// Odświeżaj LCD co 1/2 sekundy
}

if (Timer_1s % 500 == 0) {
DISPCNT++;
if (DISPCNT >= 3) DISPCNT = 1;
}

PORTD ^= 1<<0;

}

ISR(TIMER1_COMPA_vect) {
//_FLAGA.AddTokWh = FLAG_S;
Licznik_kWh++;
}

/*ISR(PCINT1_vect) {
uint8_t stan_portu_B = (~(PINB) & 0b00000010); // T1

if ( (stan_portu_B ^ poprzedni_stan_portu_B) & stan_portu_B ) {
Counter++;
}

poprzedni_stan_portu_B = stan_portu_B;
}*/

ISR(INT1_vect) {
Counter++;
}

/* INICJALIZACJA */
//----------------
void _INIT(void) {
/* Konfiguracja Timer2 jako Timer - odliczanie czasu na potrzeby systemu */
sbi(TCCR2B, CS22); sbi(TCCR2B, CS21); cbi(TCCR2B, CS20);// Konfiguracja na 16MHz/256 = 62.5kHz
cbi(TCCR2A,WGM22); sbi(TCCR2A,WGM21); cbi(TCCR2A,WGM20);// Włączenie trybu CTC (automatyczne zerowanie TCNT2 przy Compare)
OCR2A = 250;// Compare Int co 250/62500 = 4ms (250Hz)
sbi(TIMSK2, OCIE2A);// Włączenie przerwania typu compare od rejestru A

/* Konfiguracja Timer1 jako Counter - zliczanie impulsów z przetwornika */
sbi(TCCR1B,CS12); sbi(TCCR1B,CS11); cbi(TCCR1B,CS10);// External Clock Source @ Falling Edge
cbi(TCCR1B,WGM13); sbi(TCCR1B,WGM12); cbi(TCCR1A,WGM11); cbi(TCCR1A,WGM10);// Włączenie trybu CTC
OCR1A = 720; // Compare Interrupt co 720 impulsów = 0.001kWh [720 000imp = 1kWh]
sbi(TIMSK1, OCIE1A);// Włączenie przerwania typu compare od rejestru A

/* Wersja 1 - zmostkowanie T1 z INT1 */
sbi(EICRA,ISC11); cbi(EICRA,ISC10);// Przerwanie INT1@Falling Edge
sbi(EIMSK,INT1);// Włączenie przerwania

/* Wersja 2 - przerwanie typu PCINT z programowym wykrywaniem zbocza */
//sbi(PCMSK1,PCINT9);// Maska przerwania dla T1 [PCI9]
//sbi(PCICR,PCIE1);// Włączenie przerwania dla grupy PCINT1

cbi(DDRB,1);// T1 jako wejście
sbi(PORTB,1);// Pullup On

lcd_init(); // Inicjalizacja wyswietlacza

sbi(DDRD,0);
cbi(PORTD,0);
}

int main(void)
{
_INIT();
sei();// Globalne wlaczenie przerwan

for (;;)
{
//zwieksz_licznik();
obsluga_wyswietlacza();
}
return 0;
} // End of main
[/syntax]

Statystyki: Napisane przez szopler — 9 paź 2012, o 09:47

Re: Zerowanie się zmiennej... WHY?

2012-10-09T08:58:07+01:00

Wiesz - mi lepiej się patrzy na kod - więc pokaż go po poprawkach.

Przy okazji to UL

if (Timer_1s >= 1250UL)

jest zupełnie niepotrzebne bo przecież ta zmienna jest 16-bit i stała też domyślnie jest traktowana jak 16bit (ale to tylko uwaga kosmetyczna bo nie wpływa to na działanie programu)

poza tym _FLAG wygląda jakby była używana w przerwaniu i funkcjach więc brak volatile

Statystyki: Napisane przez mirekk36 — 9 paź 2012, o 08:58

Re: Zerowanie się zmiennej... WHY?

2012-10-09T07:39:31+01:00

Łolaboga... zapomniałem
(do pewnego momentu było OK, po minimalnym rozbudowaniu kodu się skiepściło).

PS. Dodałem 'UL' i dzieje się to samo... volatile też nic nie załatwia :/
Może jakiś błąd logiczny w kodzie machnąłem...

Statystyki: Napisane przez szopler — 9 paź 2012, o 07:39

Re: Zerowanie się zmiennej... WHY?

2012-10-08T23:53:05+01:00

(Licznik_kWh/10000), (Licznik_kWh%10000)

poczytaj koniecznie to:

http://atnel.pl/domyslna-promocja-do-typu-int.html

i sprawdź kod pod tym kątem

Statystyki: Napisane przez mirekk36 — 8 paź 2012, o 23:53

Zerowanie się zmiennej... WHY?

2012-10-08T22:56:09+01:00

Pisałem to już tutaj, potem załapałem co jest źle i post skasowałem zanim ktokolwiek odpowiedział, a teraz znów mam zamułkę...
Zmienna Licznik_kWh przeskakuje tylko z 0 na 1 i zaraz się zeruje... Wrzucam cały kod:
[syntax=c]/*
* main.c
*
* Created on: 02-08-2012
* Author: Michal
*/

/* Dolaczenia plikow */
#include
//#include
#include
#include
#include "LCD/lcd44780.h"

/* Definicje makr i uproszczeń */
#define sbi(byte,bit) byte |= (1<#define cbi(byte,bit) byte &= ~(1<#define bit(byte,n) (byte>>n)&1
#define FLAG_S1
#define FLAG_R0

/* Deklaracje zmiennych */
struct _FLAGA {
uint8_tDisplay:1;
uint8_tAddTokWh:1;
} _FLAGA;

uint16_tTimer_1s;
uint16_tCounter;
uint8_tpoprzedni_stan_portu_B;
uint16_tMocCzynna;
uint32_tLicznik_kWh;
charDispLine_1[16];
charDispLine_2[16];
uint8_tDISPCNT = 0;

/* Deklaracje funkcji */
void obsluga_wyswietlacza(void);
void zwieksz_licznik(void);

/* ...KOD... */
void obsluga_wyswietlacza(void) {
if (_FLAGA.Display == FLAG_S) {

//lcd_locate(0,0);
//lcd_str(" ");

switch (DISPCNT) {
case 0:
sprintf(DispLine_1,"Welcome!"); break;
case 1:
sprintf(DispLine_1," %4u W ", MocCzynna); break;
case 2:
sprintf(DispLine_1," %06lu.%04lu kWh", (Licznik_kWh/10000), (Licznik_kWh%10000)); break;
}

lcd_locate(0,0);
lcd_str(DispLine_1);

// lcd_locate(1,0);
//lcd_str(DispLine_2);

_FLAGA.Display = FLAG_R;
}
}

void zwieksz_licznik(void) {
if (_FLAGA.AddTokWh == FLAG_S) {
Licznik_kWh++;
_FLAGA.AddTokWh = FLAG_R;
}

}

ISR(TIMER2_COMPA_vect) {// Wywoływane co 4ms przy OCR2A = 250
//TCNT2 = 0;// Zerowanie timera - W trybie CTC automatyczne!
Timer_1s++;

if (Timer_1s >= 1250UL) {
Timer_1s = 0;
MocCzynna = Counter;// Ilość impulsów zliczonych w ciągu 5 sekund daje moc w Watach
Counter = 0;
}

if (Timer_1s % 125 == 0) {
_FLAGA.Display = FLAG_S;// Odświeżaj LCD co 1/2 sekundy
}

if (Timer_1s % 1250 == 0) {
DISPCNT++;
if (DISPCNT >= 3) DISPCNT = 1;
}

//PORTD ^= 1<<0;

}

ISR(TIMER1_COMPA_vect) {
_FLAGA.AddTokWh = FLAG_S;
}

ISR(PCINT1_vect) {
uint8_t stan_portu_B = (~(PINB) & 0b00000010); // T1

if ( (stan_portu_B ^ poprzedni_stan_portu_B) & stan_portu_B ) {
Counter++;
}

poprzedni_stan_portu_B = stan_portu_B;
}

/*ISR(INT1_vect) {
Counter++;
}*/

/* INICJALIZACJA */
//----------------
void _INIT(void) {
/* Konfiguracja Timer2 jako Timer - odliczanie czasu na potrzeby systemu */
sbi(TCCR2B, CS22); sbi(TCCR2B, CS21); cbi(TCCR2B, CS20);// Konfiguracja na 16MHz/256 = 62.5kHz
cbi(TCCR2A,WGM22); sbi(TCCR2A,WGM21); cbi(TCCR2A,WGM20);// Włączenie trybu CTC (automatyczne zerowanie TCNT2 przy Compare)
OCR2A = 250;// Compare Int co 250/62500 = 4ms (250Hz)
sbi(TIMSK2, OCIE2A);// Włączenie przerwania typu compare od rejestru A

/* Konfiguracja Timer1 jako Counter - zliczanie impulsów z przetwornika */
sbi(TCCR1B,CS12); sbi(TCCR1B,CS11); cbi(TCCR1B,CS10);// External Clock Source @ Falling Edge
cbi(TCCR1B,WGM13); sbi(TCCR1B,WGM12); cbi(TCCR1A,WGM11); cbi(TCCR1A,WGM10);// Włączenie trybu CTC
OCR1A = 72; // Compare Interrupt co 720 impulsów = 0.001kWh [720 000imp = 1kWh]
sbi(TIMSK1, OCIE1A);// Włączenie przerwania typu compare od rejestru A

/* Wersja 1 - zmostkowanie T1 z INT1 */
//sbi(EICRA,ISC11); cbi(EICRA,ISC10);// Przerwanie INT1@Falling Edge
//sbi(EIMSK,INT1);// Włączenie przerwania

/* Wersja 2 - przerwanie typu PCINT z programowym wykrywaniem zbocza */
sbi(PCMSK1,PCINT9);// Maska przerwania dla T1 [PCI9]
sbi(PCICR,PCIE1);// Włączenie przerwania dla grupy PCINT1

cbi(DDRB,1);// T1 jako wejście
sbi(PORTB,1);// Pullup On

lcd_init(); // Inicjalizacja wyswietlacza

sbi(DDRD,0);
cbi(PORTD,0);
}

int main(void)
{
_INIT();
sei();// Globalne wlaczenie przerwan

for (;;)
{
zwieksz_licznik();
obsluga_wyswietlacza();
}
return 0;
} // End of main
[/syntax]

Statystyki: Napisane przez szopler — 8 paź 2012, o 22:56