ATNEL tech-forum

Re: Problem z dużym kodem

2013-04-23T10:26:21+01:00

Witam .
Algorytmy tak na prawdę nie są trudne ( czasem nieskończone diagramy odstraszają ) najważniejsze to odpowiedzieć na pytania :
co program ma robić i kiedy ?
Odpowiedzią na "co ma robić ?" są funkcje programu a odpowiedzią na "kiedy ?" to warunki wykonania tych funkcji . Prawidłowo formułując warunki można z powodzeniem napisać program bez goto czyli skoku bezwarunkowego .

Statystyki: Napisane przez pagosa747 — 23 kwi 2013, o 10:26

Re: Problem z dużym kodem

2013-04-23T10:09:28+01:00

Sam czytam do poduszki Algorytmy w C++ , no i podstawa Algorytmy + struktury danych = Programy i jeszcze Algorytmy , taka przepastna książka .
No miało się 5.0 z algorytmów dzięki tym cegłom.

Statystyki: Napisane przez Gość — 23 kwi 2013, o 10:09

Re: Problem z dużym kodem

2013-04-23T10:06:41+01:00

rezasurmar napisał(a):

(...) Napisz gdzieś na kartce sobie algorytm po kolei co robi i staraj się zmienić tak by nie było goto...... (...)

Na moich studiach dobrze narysowany algorytm to była podstawa przed napisaniem kodu... tylko kiedy to było Są jakieś kursy albo ćwiczenia do rysowania algorytmów? Warto odświeżyć sobie pamięć.

Statystyki: Napisane przez Markus — 23 kwi 2013, o 10:06

Re: Problem z dużym kodem

2013-04-23T09:38:21+01:00

klonyyy napisał(a):

Co do książki= czytałem ją 10 miesięcy temu, jak widać nie dokładnie, ale pewnie niedługo przeczytam ją ponownie.

ooo jej, 10 mies temu to może i nawet dokładnie czytałeś, ale jak widzisz nasza pamięć RAM ludzka jest zawodna i ulotna, a nie mamy wbudowanego EEPROM'a hahahaah Ja też bym pozapominał wiele rzeczy, a z książką tego typu techniczną - gdy np uczę się ostatnio JAVA to po prostu wciąż siedzę i ją wertuję gdy jestem w kodzie. Takich książek nie da rady raz przeczytać jak powieść - bo ta nasza pamięć .... eeeeh pamięć - tym bardziej u mnie ze sklerozą

klonyyy napisał(a):

W jaki sposób mogę pozbyć się instrukcji goto ?

dość proste , WUKASUJ każde goto, i postaraj się zrobić porządny ERASE mózgu żeby całkowicie zapomnieć, wtedy twoje neurony same będą szukać nowych rozwiązań i je znajdą - zapewniam cię zresztą koledzy wyżej słusznie podpowiadają.

Statystyki: Napisane przez mirekk36 — 23 kwi 2013, o 09:38

Re: Problem z dużym kodem

2013-04-23T09:33:34+01:00

A chociaż by wykorzystując break, w while, case itp.
Musiał byś sobie jeszcze raz przemyśleć ten fragment kodu.
Napisz gdzieś na kartce sobie algorytm po kolei co robi i staraj się zmienić tak by nie było goto......

No i tak to jest jak się w pracy, piszę , Ledes był pierwszy ..

Statystyki: Napisane przez Gość — 23 kwi 2013, o 09:33

Re: Problem z dużym kodem

2013-04-23T09:31:35+01:00

Po prostu ją wywal. To samo uzyskasz dobierając odpowiednio instrukcje break, continue, ify

Statystyki: Napisane przez Ledes — 23 kwi 2013, o 09:31

Re: Problem z dużym kodem

2013-04-23T09:32:17+01:00

Code:

Program:   10892 bytes (66.5% Full)
(.text + .data + .bootloader)

Data:        507 bytes (49.5% Full)
(.data + .bss + .noinit)

EEPROM:        7 bytes (1.4% Full)
(.eeprom)

Finished building: sizedummy

Chyba nie wymaga komentarza Dzięki Wam wszystkim !

Co do książki= czytałem ją 10 miesięcy temu, jak widać nie dokładnie, ale pewnie niedługo przeczytam ją ponownie.

W jaki sposób mogę pozbyć się instrukcji goto ?

Statystyki: Napisane przez klonyyy — 23 kwi 2013, o 09:26

Re: Problem z dużym kodem

2013-04-23T09:23:32+01:00

No fakt wywaliłem odrobinę funkcji (odkomentowałem) ale wszystkie napisy poszły w Flash i proszę taki wynik

Device: atmega16

Program: 2852 bytes (17.4% Full)
(.text + .data + .bootloader)

Data: 84 bytes (8.2% Full)
(.data + .bss + .noinit)

EEPROM: 7 bytes (1.4% Full)
(.eeprom)

[syntax=c]/*
* main.c
*
* Created on: 23-04-2013
*/

#include
#include
#include
#include
#include

#include "LCD/lcd44780.h"

#include
// #include "kbd4x4.h"
#include
#include
#include
#include
#include

#include "LCD/lcd44780.h"
#include "MK_USART/mkuart.h"
// #include "funkcje.h"

#define EEMEM __attribute__((section(".eeprom")))
#define WIB PORTA|=(1< #define STPWIB PORTA&=~(1< #define KEY_PIN (1< #define KEY_DOWN !(PIND & KEY_PIN)
#define CZAS_WIB 30

char const PROGMEM tab1[] = {"FLASH"};
char EEMEM tab2[] = {"EEPROM"};

volatile uint8_t pwm1, pwm2, pwm3;

uint8_t lat;
uint8_t b=-1;
unsigned char l;
uint8_t tab[4];
uint8_t ta[4]={0,1,2,15};

//int q[35]={0,1,0,1,1,0,1,/**/1,0,1,0,1,0,1,/**/0,1,0,1,0,0,1,/**/1,0,1,1,0,1,0,/**/1,0,0,1,0,1,0};
//uint8_t g[7];
//uint8_t j;
//uint8_t s,q0,q1,q2,q3,q4,q5,q6,q7;
//uint8_t lvl,m0,m1,m2,m3,m4,m5,m6,m7;
//uint8_t x=1;
//uint8_t l1=13;
//uint8_t cz=200;

uint8_t backslash[]=
{
0b00000000,//back slash
0b00010000,
0b00001000,
0b00000100,
0b00000010,
0b00000001,
0b00000000,
0b00000000
};

uint8_t latarka[]=
{
0b00011111,//back slash
0b00011111,
0b00001110,
0b00001110,
0b00001110,
0b00001110,
0b00001110,
0b00000000
};

uint8_t pdkr[]=
{
0b00000000,//back slash
0b00000000,
0b00000000,
0b00000000,
0b00000000,
0b00000000,
0b00011111,
0b00000000
};

unsigned char uart_rx( void )
{
while ( !(UCSRA & (1< return UDR; // zwroc dane z bufora
}

void uart_tx( unsigned char data )
{

while ( !( UCSRA & (1< UDR = data; //wyslij dane
}

uint8_t sw(void)
{
if( KEY_DOWN ) // klawisz wciśnięty ?
{
_delay_ms(80); // czas drgań styków
if( KEY_DOWN ) return 1; // jeśli wciśnięty? zakończ funkcję - rezultat = 1
}

return 0; // jeśli nie wciśnięty klawisz, zakończ funkcję, rezultat = 0
}

void wib(void)
{
WIB;
_delay_ms(CZAS_WIB);
STPWIB;
}

void piotrek(void)
{
WIB;
lcd_locate(0,0);
lcd_str_P(PSTR("Czesc Piotrek ;D"));
_delay_ms(1000);
lcd_cls();

if(lat==1)
{
lcd_defchar(0x80, latarka);
lcd_locate(1,15);
lcd_str_P(PSTR("\x80"));
}
STPWIB;

}

//void jak_masz_na_imie(void)
//{
// lcd_locate(0,0);
// lcd_str_P(PSTR("jak masz na imie");
// lcd_locate(1,0);
// lcd_str_P(PSTR("wpisz swoje imie");
// _delay_ms(1000);
// lcd_cls();
//
//}

void f_start(void)
{
y:
lcd_cls();
lcd_locate(0,0);
lcd_str_P(PSTR("podaj haslo:"));
uint8_t i=0;

while(i<4)
{
// b=getkbd();

if(b>=0 && b<255)
{
tab[i]=b;
lcd_locate(1,i);
lcd_str_P(PSTR("*"));
i++;
_delay_ms(300);
}
}
for(uint8_t a=0;a<4;a++)
{
if(tab[a]==ta[a])
{
if(a==3)piotrek();
}
else
{

lcd_locate(1,2);
lcd_str_P(PSTR("*"));
lcd_cls();
_delay_ms(50);
lcd_locate(0,0);
lcd_str_P(PSTR("zle haslo"));
_delay_ms(1000);
lcd_cls();
lcd_locate(0,0);
lcd_str_P(PSTR("sproboj ponownie"));
_delay_ms(1000);
goto y;
}
}

}

void haslo(void)
{
y:
lcd_cls();
lcd_locate(0,0);
lcd_str_P(PSTR("haslo cwaniaku:"));
uint8_t i=0;

while(i<4)
{
// b=getkbd();

if(b>=0 && b<255)
{
tab[i]=b;
lcd_locate(1,i);
lcd_str_P(PSTR("*"));
i++;
_delay_ms(300);
}
}
for(uint8_t a=0;a<4;a++)
{
if(tab[a]==ta[a])
{
if(a==3);//jdarka(lat ,latarka, backslash );
}
else
{

lcd_locate(1,2);
lcd_str_P(PSTR("*"));
lcd_cls();
_delay_ms(50);
lcd_locate(0,0);
lcd_str_P(PSTR("zle haslo"));
_delay_ms(1000);
lcd_cls();
lcd_locate(0,0);
lcd_str_P(PSTR("sproboj ponownie"));
_delay_ms(1000);
goto y;
}
}

}
void uspienie(void)
{
lcd_cls();
lcd_locate(0,0);
lcd_str_P(PSTR("zablokowano"));
lcd_locate(1,0);
lcd_str_P(PSTR("unlock - SHIFT+0"));
_delay_ms(700);
lcd_cls();

uint8_t l=0;
while(l<1)
{
//b=getkbd();

if(b==0 && KEY_DOWN )
{
lcd_locate(0,0);
lcd_str_P(PSTR("Odblokowano"));
_delay_ms(700);
f_start();
l++;
}
else if(b>=0 && b<255)
{
lcd_locate(1,0);
lcd_str_P(PSTR("unlock - SHIFT+0"));
_delay_ms(700);
lcd_cls();

}

}

}

int main()
{

DDRA|=(1< PORTA&=~(1<
DDRD &= ~KEY_PIN; // kierunek pinu PC6 - wejściowy
PORTD = 0xff; // podciągnięcie pinu do VCC

lcd_init();

sei();

USART_Init( __UBRR );

f_start();
uint8_t a=0;

while(1)
{

//uint8_t a=getkbd();

if(a==0 && KEY_DOWN)
{
uart_puts("s");
lcd_locate(0,0);
lcd_str_P(PSTR("LATARKA OFF"));
lcd_locate(0,16);
lcd_str_P(PSTR(" "));
lat=0;
wib();

}

else if(a==1 && KEY_DOWN)
{
uart_puts("t");
lcd_locate(0,0);
lcd_str_P(PSTR("LATARKA ON"));
lcd_locate(0,0);
lcd_defchar(0x80, latarka);
lcd_locate(1,15);
lcd_str_P(PSTR("\x80"));
lat=1;
wib();

}
else if(a==2 && KEY_DOWN)
{
uart_puts("u");
lcd_locate(0,0);
lcd_str_P(PSTR("kut"));
lcd_locate(0,0);
//lcd_defchar(0x80, kut);
lcd_locate(1,0);
lcd_str_P(PSTR("\x80"));
wib();

}

else if(a==4 && KEY_DOWN)
{
uart_puts("x");

l=uart_getc();

if(l=='3')
{
lcd_cls();
lcd_locate(1,0);
lcd_str_P(PSTR("reka pusta"));
_delay_ms(500);

}
else if(l=='4')
{
lcd_locate(1,0);
lcd_str_P(PSTR("trzymam"));
_delay_ms(500);
}

wib();
}
else if(a==5 && KEY_DOWN)
{
uart_puts("y");
lcd_locate(0,0);
lcd_str_P(PSTR("GRA"));
//gra();
wib();
}
else if(a==15 && KEY_DOWN)
{

lcd_locate(0,0);
lcd_str_P(PSTR("Zablokowac ?"));
lcd_locate(1,0);
lcd_str_P(PSTR("NIE-0"));
lcd_locate(1,7);
lcd_str_P(PSTR("TAK-1"));
uint8_t k=0;

while(k<1)
{
// b=getkbd();

if(b==0)
{

lcd_defchar(0x80, pdkr);
lcd_locate(1,0);
lcd_str_P(PSTR("\x80"));
lcd_defchar(0x80, pdkr);
lcd_locate(1,1);
lcd_str_P(PSTR("\x80"));
lcd_defchar(0x80, pdkr);
lcd_locate(1,2);
lcd_str_P(PSTR("\x80"));
_delay_ms(700);
if(lat==1)
{
lcd_defchar(0x80, latarka);
lcd_locate(1,15);
lcd_str_P(PSTR("\x80"));
}
k++;

}
else if(b==1)
{

lcd_defchar(0x80, pdkr);
lcd_locate(1,7);
lcd_str_P(PSTR("\x80"));
lcd_defchar(0x80, pdkr);
lcd_locate(1,8);
lcd_str_P(PSTR("\x80"));
lcd_defchar(0x80, pdkr);
lcd_locate(1,9);
lcd_str_P(PSTR("\x80"));
_delay_ms(700);
uspienie();
k++;

}

}

}

else if(a==0)
{

uart_puts("a");
lcd_locate(0,0);
lcd_str_P(PSTR("Zamykam chwytak"));
wib();
}
else if(a==1)
{
uart_puts("b");
lcd_locate(0,0);
lcd_str_P(PSTR("Otwieram chwytak"));
wib();
}

else if(a==2)
{
uart_puts("c");
lcd_locate(0,0);
lcd_str_P(PSTR("krece chwytakiem"));
lcd_locate(1,0);
lcd_str_P(PSTR("w lewo"));
wib();
}

else if(a==3)
{
uart_puts("d");
lcd_locate(0,0);
lcd_str_P(PSTR("krece chwytakiem"));
lcd_locate(1,0);
lcd_str_P(PSTR("w prawo"));
wib();
}

else if(a==4)
{
uart_puts("e");
lcd_locate(0,0);
lcd_str_P(PSTR("Chwytak gora"));
wib();
}
else if(a==5)
{
uart_puts("f");
lcd_locate(0,0);
lcd_str_P(PSTR("Chwytak dol"));
wib();
}
else if(a==6)
{
uart_puts("g");
lcd_locate(0,0);
lcd_str_P(PSTR("Ramie gora"));
wib();
}
else if(a==7)
{
uart_puts("h");
lcd_locate(0,0);
lcd_str_P(PSTR("Ramie dol"));
wib();
}
else if(a==8)
{
uart_puts("i");
lcd_locate(0,0);
lcd_str_P(PSTR("Zginam lokiec"));
wib();
}
else if(a==9)
{
uart_puts("j");
lcd_locate(0,0);
lcd_str_P(PSTR("Prostuje lokiec"));
wib();
}
else if(a==10)
{
uart_puts("k");
lcd_locate(0,0);
lcd_str_P(PSTR("Obracam ramie w "));
lcd_locate(1,0);
lcd_str_P(PSTR("lewo"));
wib();
}
else if(a==11)
{
uart_puts("l");
lcd_locate(0,0);
lcd_str_P(PSTR("Obracam ramie w "));
lcd_locate(1,0);
lcd_str_P(PSTR("prawo"));
wib();
}
else if(a==12)
{
uart_puts("m");
lcd_locate(0,0);
lcd_str_P(PSTR("Jade do przodu "));
wib();
}
else if(a==13)
{
uart_puts("n");
lcd_locate(0,0);
lcd_str_P(PSTR("Jade do tylu"));
wib();
}
else if(a==14)
{
uart_puts("o");
lcd_locate(0,0);
lcd_str_P(PSTR("Skrecam w lewo"));
wib();
}
else if(a==15)
{
uart_puts("p");
lcd_locate(0,0);
lcd_str_P(PSTR("Skrecam w prawo"));
wib();
}
else if(a==16)
{
uart_puts("r");
wib();
}

_delay_ms(20);
if(lat==1)
{
lcd_locate(0,0);
lcd_str_P(PSTR(" "));
lcd_locate(1,0);
lcd_str_P(PSTR(" "));
}
else if(lat==0)
{
lcd_cls();
}
STPWIB;

}

}[/syntax]

Statystyki: Napisane przez Gość — 23 kwi 2013, o 09:23

Re: Problem z dużym kodem

2013-04-23T08:54:49+01:00

rezasurmar napisał(a):

No i Mirek mnie ubiegł, a już przygotowywałem fajne porównanie rozmiaru kodu w zależności od użytych funkcji do wyświetlania na ekranie LCD . Bo sam całkiem nie dawno z tym walczyłem.

No ale takie porównanie tego konkretnego kodu to byłaby fajna rzecz nawet sam autor jeśli to zrobi to niech tu opisze ile RAMu ma zajęte przed (chociaż to już widzimy w zasadzie) a ile po Więc jeśli to skompilowałeś rezasurmar to podpowiedz nam już za wczasu bo aż sam jestem ciekaw

Statystyki: Napisane przez mirekk36 — 23 kwi 2013, o 08:54

Re: Problem z dużym kodem

2013-04-23T08:53:01+01:00

No i Mirek mnie ubiegł, a już przygotowywałem fajne porównanie rozmiaru kodu w zależności od użytych funkcji do wyświetlania na ekranie LCD . Bo sam całkiem nie dawno z tym walczyłem.

Statystyki: Napisane przez Gość — 23 kwi 2013, o 08:53

Re: Problem z dużym kodem

2013-04-23T08:46:52+01:00

Zobacz pan jak straszliwie DUŻO pamięci RAM po prostu marnujesz - bezpowrotnie marnujesz a wspominam o tym w książce, że taki zapis:

[syntax=c]lcd_str( "Czesc Piotrek ;D" );[/syntax]

jest robiony przeze mnie tak na szybko ... i nie ma nic wspólnego z optymalizacją kodu, po prostu krócej się go pisze. Nie pamiętasz ? albo nie doczytałeś skąd funkcja lcd_str() bierze argumenty ? to przypomnę z pamięci RAM. Zaczynasz już rozumieć zgrozę w twoim kodzie ??? każdy taki napis to wyrzucone w błoto bajty RAM ? a dlaczego wyrzucone w błoto ? no bo i tak chociaż byś nie chciał, muszą one przez kompilator być umieszczone w pamięci FLASH !!! żeby skądś się wzięły w tejże pamięci RAM prawda ? samo nic się w ramie nie pojawi. Więc bidny kompilator nie dość że i tak musi te twoje napisy umieścić we FLASH to jeszcze co gorsze musi napisać kawałek kodu, który po starcie procka ale jeszcze przed twoją funkcją main() napełni te zmienne w pamięci RAM tymi tekstami .... więc pomyśl ile tracisz na takich zapisach gdy robisz nie mały testowy programik a już gotowy

jak myślisz po co jest w książce opisana funkcja lcd_str_P() ???? (nie obraź się proszę ale wydaje mi się że jakoś ją przeczytałeś bardzo hmmm że tak powiem wyrywkowo i wybiórczo .... i stąd twoje problemy, co widać także po tym że nie budzi twojego zdziwienia fakt że zajętość pamięci RAM po kompilacji u ciebie jest ponad 100% !!!!!!

ale ok w książce jest jedno i drugie dokładnie opisane - ja przypomnę ci jednak tutaj - i może to skłoni cię do kolejnego czytania książki albo jej fragmentów żeby zawsze mieć pod ręką .... że zmień swoje zapisy tak:

[syntax=c]lcd_str_P( PSTR( "Czesc Piotrek ;D") );[/syntax]

ale już sam doczytaj w książce KONIECZNIE:

1. co to takiego jest to PSTR()
2. na temat stosu, jak nad nim panować i ile pamięci RAM warto utrzymywać po kompilacji

ok?

------------------------ [ Dodano po: 1 minucie ]

O! kolega wyżej dobrze zauważył - WYWAL KAŻDY rozkaz goto ! i zapomnij o nim na całe życie programując w C - bo nawet jeśli poprawisz kod i będziesz miał więcej RAM'u to goto załatwi cię tak, że nawet 100% wolnego RAMU ci nie pomoże.

------------------------ [ Dodano po: 5 minutach ]

Ledes napisał(a):

Zajętość RAM-u nie powinna być większa niż 80% (podczas kompilacji). Inaczej spodziewaj się problemów ze stosem w trakcie działania programu.

To jest taka moja podpowiedź dla kogoś kto początkuje i jeszcze nie czuje się dobrze z tym jak panować nad stosem, gdy nie zna takich narzędzi do badania stosu jakie opisałem w Zielonej książce. Ale w trakcie praktyki programowania każdy dojdzie sam do tego, że ta zajętość spokojnie może być większa (OCZYWIŚCIE w zależności od własnego indywidualnego kodu) i może np wynosić nawet 90% albo i 95% ale mówię - to już jak się chociaż troszkę dłużej popraktykuje.

A czasem gdy się mocno i świadomie korzysta ze stosu (gdy np ja bawię się warstwami na LCD), podczas tworzenia ala obiektów, albo przy korzystaniu z dynamicznego alkokowania pamięci RAM - może się okazać że np zajętość tej pamięci nie może przekroczyć nawet 70% .... Jak mówię - troszkę praktyki i każdy sam poczuje stos na własnej skórze

no ale nigdy 100% albo jak w tym przypadku więcej niż 100% !

Statystyki: Napisane przez mirekk36 — 23 kwi 2013, o 08:46

Re: Problem z dużym kodem

2013-04-23T08:42:17+01:00

[syntax=c]lcd_str_P(PSTR("napis"));[/syntax]

------------------------ [ Dodano po: 1 minucie ]

Panie drogi, a cóż robią goto w kodzie?
Toż to najłatwiejsza droga do tego aby mieć spaghetti code, jak to mawiał jeden z profesorów na moich studiach.

------------------------ [ Dodano po: 4 minutach ]

Zajętość RAM-u nie powinna być większa niż 80% (podczas kompilacji). Inaczej spodziewaj się problemów ze stosem w trakcie działania programu.

Statystyki: Napisane przez Ledes — 23 kwi 2013, o 08:42

Re: Problem z dużym kodem

2013-04-23T08:39:49+01:00

Zamień wszystkie funkcje lcd_str
lcd_str("LATARKA ON");
na lcd_Pstr Czy jakoś tak. Nie pamiętam tak na szybko. To zmniejszy ci o wiele wykorzystanie pamięci/

Statystyki: Napisane przez xamrex — 23 kwi 2013, o 08:39

Re: Problem z dużym kodem

2013-04-23T08:18:29+01:00

A jakie w tym przypadku są drogi optymalizacji:
[syntax=c]#include
#include "kbd4x4.h"
#include
#include
#include
#include
#include

#include "LCD/lcd44780.h"
#include "MKUART/mkuart.h"
#include "funkcje.h"

#define EEMEM __attribute__((section(".eeprom")))
#define WIB PORTA|=(1<#define STPWIB PORTA&=~(1<#define KEY_PIN (1<#define KEY_DOWN !(PIND & KEY_PIN)
#define CZAS_WIB 30

char PROGMEM tab1[] = {"FLASH"};
char EEMEM tab2[] = {"EEPROM"};

volatile uint8_t pwm1, pwm2, pwm3;

uint8_t lat;
uint8_t b=-1;
unsigned char l;
uint8_t tab[4];
uint8_t ta[4]={0,1,2,15};

//int q[35]={0,1,0,1,1,0,1,/**/1,0,1,0,1,0,1,/**/0,1,0,1,0,0,1,/**/1,0,1,1,0,1,0,/**/1,0,0,1,0,1,0};
//uint8_t g[7];
//uint8_t j;
//uint8_t s,q0,q1,q2,q3,q4,q5,q6,q7;
//uint8_t lvl,m0,m1,m2,m3,m4,m5,m6,m7;
//uint8_t x=1;
//uint8_t l1=13;
//uint8_t cz=200;

uint8_t backslash[]=
{
0b00000000,//back slash
0b00010000,
0b00001000,
0b00000100,
0b00000010,
0b00000001,
0b00000000,
0b00000000
};

uint8_t latarka[]=
{
0b00011111,//back slash
0b00011111,
0b00001110,
0b00001110,
0b00001110,
0b00001110,
0b00001110,
0b00000000
};

uint8_t pdkr[]=
{
0b00000000,//back slash
0b00000000,
0b00000000,
0b00000000,
0b00000000,
0b00000000,
0b00011111,
0b00000000
};

unsigned char uart_rx( void )
{
while ( !(UCSRA & (1<return UDR;// zwroc dane z bufora
}

void uart_tx( unsigned char data )
{

while ( !( UCSRA & (1<UDR = data; //wyslij dane
}

uint8_t sw(void)
{
if( KEY_DOWN )// klawisz wciśnięty ?
{
_delay_ms(80);// czas drgań styków
if( KEY_DOWN ) return 1; // jeśli wciśnięty? zakończ funkcję - rezultat = 1
}

return 0;// jeśli nie wciśnięty klawisz, zakończ funkcję, rezultat = 0
}

void wib(void)
{
WIB;
_delay_ms(CZAS_WIB);
STPWIB;
}

void piotrek(void)
{
WIB;
lcd_locate(0,0);
lcd_str("Czesc Piotrek ;D");
_delay_ms(1000);
lcd_cls();

if(lat==1)
{
lcd_defchar(0x80, latarka);
lcd_locate(1,15);
lcd_str("\x80");
}
STPWIB;

}

//void jak_masz_na_imie(void)
//{
//lcd_locate(0,0);
//lcd_str("jak masz na imie");
//lcd_locate(1,0);
//lcd_str("wpisz swoje imie");
//_delay_ms(1000);
//lcd_cls();
//
//}

void f_start(void)
{
y:
lcd_cls();
lcd_locate(0,0);
lcd_str("podaj haslo:");
uint8_t i=0;

while(i<4)
{
b=getkbd();

if(b>=0 && b<255)
{
tab[i]=b;
lcd_locate(1,i);
lcd_str("*");
i++;
_delay_ms(300);
}
}
for(uint8_t a=0;a<4;a++)
{
if(tab[a]==ta[a])
{
if(a==3)piotrek();
}
else
{

lcd_locate(1,2);
lcd_str("*");
lcd_cls();
_delay_ms(50);
lcd_locate(0,0);
lcd_str("zle haslo");
_delay_ms(1000);
lcd_cls();
lcd_locate(0,0);
lcd_str("sproboj ponownie");
_delay_ms(1000);
goto y;
}
}

}

void haslo(void)
{
y:
lcd_cls();
lcd_locate(0,0);
lcd_str("haslo cwaniaku:");
uint8_t i=0;

while(i<4)
{
b=getkbd();

if(b>=0 && b<255)
{
tab[i]=b;
lcd_locate(1,i);
lcd_str("*");
i++;
_delay_ms(300);
}
}
for(uint8_t a=0;a<4;a++)
{
if(tab[a]==ta[a])
{
if(a==3)jdarka(lat ,latarka, backslash );
}
else
{

lcd_locate(1,2);
lcd_str("*");
lcd_cls();
_delay_ms(50);
lcd_locate(0,0);
lcd_str("zle haslo");
_delay_ms(1000);
lcd_cls();
lcd_locate(0,0);
lcd_str("sproboj ponownie");
_delay_ms(1000);
goto y;
}
}

}
void uspienie(void)
{
lcd_cls();
lcd_locate(0,0);
lcd_str("zablokowano");
lcd_locate(1,0);
lcd_str("unlock - SHIFT+0");
_delay_ms(700);
lcd_cls();

uint8_t l=0;
while(l<1)
{
b=getkbd();

if(b==0 && KEY_DOWN )
{
lcd_locate(0,0);
lcd_str("Odblokowano");
_delay_ms(700);
f_start();
l++;
}
else if(b>=0 && b<255)
{
lcd_locate(1,0);
lcd_str("unlock - SHIFT+0");
_delay_ms(700);
lcd_cls();

}

}

}

int main()
{

DDRA|=(1< PORTA&=~(1<
DDRD &= ~KEY_PIN;// kierunek pinu PC6 - wejściowy
PORTD = 0xff;// podciągnięcie pinu do VCC

lcd_init();

sei();

USART_Init( __UBRR );

f_start();

while(1)
{

uint8_t a=getkbd();

if(a==0 && KEY_DOWN)
{
uart_puts("s");
lcd_locate(0,0);
lcd_str("LATARKA OFF");
lcd_locate(0,16);
lcd_str(" ");
lat=0;
wib();

}

else if(a==1 && KEY_DOWN)
{
uart_puts("t");
lcd_locate(0,0);
lcd_str("LATARKA ON");
lcd_locate(0,0);
lcd_defchar(0x80, latarka);
lcd_locate(1,15);
lcd_str("\x80");
lat=1;
wib();

}
else if(a==2 && KEY_DOWN)
{
uart_puts("u");
lcd_locate(0,0);
lcd_str("kut");
lcd_locate(0,0);
lcd_defchar(0x80, kut);
lcd_locate(1,0);
lcd_str("\x80");
wib();

}

else if(a==4 && KEY_DOWN)
{
uart_puts("x");

l=uart_getc();

if(l=='3')
{
lcd_cls();
lcd_locate(1,0);
lcd_str("reka pusta");
_delay_ms(500);

}
else if(l=='4')
{
lcd_locate(1,0);
lcd_str("trzymam");
_delay_ms(500);
}

wib();
}
else if(a==5 && KEY_DOWN)
{
uart_puts("y");
lcd_locate(0,0);
lcd_str("GRA");
gra();
wib();
}
else if(a==15 && KEY_DOWN)
{

lcd_locate(0,0);
lcd_str("Zablokowac ?");
lcd_locate(1,0);
lcd_str("NIE-0");
lcd_locate(1,7);
lcd_str("TAK-1");
uint8_t k=0;

while(k<1)
{
b=getkbd();

if(b==0)
{

lcd_defchar(0x80, pdkr);
lcd_locate(1,0);
lcd_str("\x80");
lcd_defchar(0x80, pdkr);
lcd_locate(1,1);
lcd_str("\x80");
lcd_defchar(0x80, pdkr);
lcd_locate(1,2);
lcd_str("\x80");
_delay_ms(700);
if(lat==1)
{
lcd_defchar(0x80, latarka);
lcd_locate(1,15);
lcd_str("\x80");
}
k++;

}
else if(b==1)
{

lcd_defchar(0x80, pdkr);
lcd_locate(1,7);
lcd_str("\x80");
lcd_defchar(0x80, pdkr);
lcd_locate(1,8);
lcd_str("\x80");
lcd_defchar(0x80, pdkr);
lcd_locate(1,9);
lcd_str("\x80");
_delay_ms(700);
uspienie();
k++;

}

}

}

else if(a==0)
{

uart_puts("a");
lcd_locate(0,0);
lcd_str("Zamykam chwytak");
wib();
}
else if(a==1)
{
uart_puts("b");
lcd_locate(0,0);
lcd_str("Otwieram chwytak");
wib();
}

else if(a==2)
{
uart_puts("c");
lcd_locate(0,0);
lcd_str("krece chwytakiem");
lcd_locate(1,0);
lcd_str("w lewo");
wib();
}

else if(a==3)
{
uart_puts("d");
lcd_locate(0,0);
lcd_str("krece chwytakiem");
lcd_locate(1,0);
lcd_str("w prawo");
wib();
}

else if(a==4)
{
uart_puts("e");
lcd_locate(0,0);
lcd_str("Chwytak gora");
wib();
}
else if(a==5)
{
uart_puts("f");
lcd_locate(0,0);
lcd_str("Chwytak dol");
wib();
}
else if(a==6)
{
uart_puts("g");
lcd_locate(0,0);
lcd_str("Ramie gora");
wib();
}
else if(a==7)
{
uart_puts("h");
lcd_locate(0,0);
lcd_str("Ramie dol");
wib();
}
else if(a==8)
{
uart_puts("i");
lcd_locate(0,0);
lcd_str("Zginam lokiec");
wib();
}
else if(a==9)
{
uart_puts("j");
lcd_locate(0,0);
lcd_str("Prostuje lokiec");
wib();
}
else if(a==10)
{
uart_puts("k");
lcd_locate(0,0);
lcd_str("Obracam ramie w ");
lcd_locate(1,0);
lcd_str("lewo");
wib();
}
else if(a==11)
{
uart_puts("l");
lcd_locate(0,0);
lcd_str("Obracam ramie w ");
lcd_locate(1,0);
lcd_str("prawo");
wib();
}
else if(a==12)
{
uart_puts("m");
lcd_locate(0,0);
lcd_str("Jade do przodu ");
wib();
}
else if(a==13)
{
uart_puts("n");
lcd_locate(0,0);
lcd_str("Jade do tylu");
wib();
}
else if(a==14)
{
uart_puts("o");
lcd_locate(0,0);
lcd_str("Skrecam w lewo");
wib();
}
else if(a==15)
{
uart_puts("p");
lcd_locate(0,0);
lcd_str("Skrecam w prawo");
wib();
}
else if(a==16)
{
uart_puts("r");
wib();
}

_delay_ms(20);
if(lat==1)
{
lcd_locate(0,0);
lcd_str(" ");
lcd_locate(1,0);
lcd_str(" ");
}
else if(lat==0)
{
lcd_cls();
}
STPWIB;

}

}[/syntax]

Statystyki: Napisane przez klonyyy — 23 kwi 2013, o 08:18

Re: Problem z dużym kodem

2013-04-23T07:37:49+01:00

Zielona, nie niebieska .

No i patrzałeś, przeca zmienne w RAM to ci zajmują 130%, jak chcesz by to działało
toż stos nawet biedny procesor nie ma gdzie trzymać .

PS. Ja jak mi zmienne po kompilacji zajmują więcej niż 70% szukam możliwych dróg optymalizacji, albo zmieniam procka na większego.

Statystyki: Napisane przez Gość — 23 kwi 2013, o 07:37

Re: Problem z dużym kodem

2013-04-23T07:33:37+01:00

Quote:

Program: 10772 bytes (65.7% Full)
(.text + .data + .bootloader)

Data: 1299 bytes (126.9% Full)
(.data + .bss + .noinit)

EEPROM: 7 bytes (1.4% Full)
(.eeprom)

Finished building: sizedummy

wynik kompilacji, a na książkę za chwilę zerknę

Ale chyba inna strona, bo na 325 jest sterowanie krokowcem

Statystyki: Napisane przez klonyyy — 23 kwi 2013, o 07:33

Re: Problem z dużym kodem

2013-04-23T07:31:02+01:00

Bez obejrzenia kodu to wróżenie z fusów .

Masz jakieś zasobnożerne biblioteki, typu printf?, obsługa float?.
W zielonej książce Mirka jest na 325stronie jak sprawdzać ile zajmuje stos na bieżąco .
Ile ramu masz zajęte po kompilacji?

Statystyki: Napisane przez Gość — 23 kwi 2013, o 07:31

Problem z dużym kodem

2013-04-23T07:14:25+01:00

Witam !

Mam pewien problem - napisałem dość duży kod bo aż na 10kb, jest to program na mege16, która steruje pilotem do platformy wielozadaniowej. Po dodaniu kilku bajerów typu jakaś prosta gra, program nie działa jak powinien - na wyświetlaczu krzaczki itp. Czy jest to wina stosu, i jak to rozpoznać (czy stos nadpisuje mi się w programie)? A moje drugie pytanie brzmi jak mogę rozwiązać problem z tak dużym programem ? I trzecie - jak używa się zmiennych globalnych w plikach nagłówkowych ?

serdecznie pozdrawiam!

Statystyki: Napisane przez klonyyy — 23 kwi 2013, o 07:14