ATNEL tech-forum

Re: optomalizacja kodu

2016-11-19T15:53:34+01:00

mirekk36 napisał(a):

Zauważ, że sam autor wątku napisał na początku że w moim oryginalnym kodzie było właśnie tak

lcd_str_P( days[ dt->dayofwek -1 ] ); // days we FLASH

Przepraszam, tego nie zauważyłem.

Odniosłem się tylko do tych dwóch cytatów:

mirekk36 napisał(a):

ok spróbuj pan tak

[syntax=c]lcd_str_P( &days[ dt->dayofwek -1 ] );[/syntax]

oraz

mirekk36 napisał(a):

[syntax=c]lcd_str_P( (char*)&days[ dt->dayofwek -1 ] );[/syntax]

bo pewnie trzeba jeszcze jawne rzutowanie zrobić do typu wskaźnikowego o który chodzi

Dlatego pomyślałem "po co tak komplikować?" i zaproponowałem prostszy sposób.

Ja rozumiem, że to wszystko z pośpiechu. Ja też nie doczytałem dokładnie pierwszego posta

Statystyki: Napisane przez andrews — 19 lis 2016, o 15:53

Re: optomalizacja kodu

2016-11-19T15:23:23+01:00

andrews napisał(a):

A nie prościej byłoby zrobić tak:

Zauważ, że sam autor wątku napisał na początku że w moim oryginalnym kodzie było właśnie tak

[syntax=c]lcd_str_P( days[ dt->dayofwek -1 ] ); // days we FLASH[/syntax]

w jakimś poradniku i nie widział tam w poradniku warninga, natomiast u niego się pokazywał stąd zaproponowałem "na szybko" rzutowanie albo pokazałem jak zrobić to bez rzutowania .... w ostateczności ... czyli

[syntax=c]lcd_str_P( &days[ dt->dayofwek -1 ][0] ); // days we FLASH[/syntax]

a nie po to żeby mówić, że tak warto

A na zakończenie powiem, że rzeczywiście dla zapisu

[syntax=c]lcd_str_P( &days[ 3 ] ); // days we FLASH[/syntax]

pojawia się warning, proszę bardzo:

i dlatego rzutowanie powoduje że możemy się go pozbyć:

[syntax=c]lcd_str_P( (char*)&days[ 3 ] ); // days we FLASH[/syntax]

podobnie jak ten zapis

[syntax=c]lcd_str_P( &days[ 3 ][0] ); // days we FLASH[/syntax]

Statystyki: Napisane przez mirekk36 — 19 lis 2016, o 15:23

Re: optomalizacja kodu

2016-11-19T14:21:49+01:00

mirekk36 napisał(a):

gdybyś chciał uniknąć rzutowania to MUSIAŁBYŚ podawać jawnie indeks drugiego wymiaru tablicy bo dopiero wtedy kompilator ma pewność że chodzi ci o string, np

[syntax=c]lcd_str_P( &days[ dt->dayofwek -1 ][0] ); // days we FLASH[/syntax]

albo

[syntax=c]lcd_str( &days[ dt->dayofwek -1 ][0] ); // days w RAM[/syntax]

A nie prościej byłoby zrobić tak:[syntax=c]// zamiast pisać
lcd_str_P( (char*)&days[ dt->dayofwek -1 ] ); // days we FLASH
// napisać po prostu
lcd_str_P( days[ dt->dayofwek -1 ] ); // days we FLASH[/syntax]
Wtedy też nie będzie warninga

Statystyki: Napisane przez andrews — 19 lis 2016, o 14:21

Re: optomalizacja kodu

2016-11-18T12:05:44+01:00

tonygryps napisał(a):

wiec należy to rozumieć tak że w orginalnym kodzie kompilator "się domyślił o co chodziło " a po przeróbce już nie .

Nie, warning zawsze wystąpi - nie ma tak że kompilator "raz" się domyśla a "innym razem" się nie domyśla.

Statystyki: Napisane przez mirekk36 — 18 lis 2016, o 12:05

Re: optomalizacja kodu

2016-11-18T11:48:25+01:00

Dziękuje za wyłożenie mi tego ja tylko się zastanawiałem czemu tak jest bo w orginalnym kodzie nie było rzutowania
[syntax=c]lcd_str(days[dt->dayofwek -1])[/syntax]
i nie było warminga chociaż pewnie rzutowanie by nie przeszkodziło w niczym .

Quote:

więc niestety coś kręcisz bo nie ważne czy masz zapis taki

Składnia: [ Pobierz ] [ Ukryj ]
język c

lcd_str_P( (char*)&days[ dt->dayofwek -1 ] ); // days we FLASH

GeSHi

czy taki
Składnia: [ Pobierz ] [ Ukryj ]
język c

lcd_str( (char*)&days[ dt->dayofwek -1 ] ); // days w RAM

GeSHi

to w każdym wypadku MUSISZ robić rzutowanie (char*)

gdybyś chciał uniknąć rzutowania to MUSIAŁBYŚ podawać jawnie indeks drugiego wymiaru tablicy bo dopiero wtedy kompilator ma pewność że chodzi ci o string, np
Składnia: [ Pobierz ] [ Ukryj ]
język c

lcd_str_P( &days[ dt->dayofwek -1 ][0] ); // days we FLASH

wiec należy to rozumieć tak że w orginalnym kodzie kompilator "się domyślił o co chodziło " a po przeróbce już nie .
a tak naprawdę poprawnie to powinno w orginalnym kodzie to wyglądać tak?
[syntax=c]lcd_str( (char*)&days[ dt->dayofwek -1 ] );[/syntax]

Statystyki: Napisane przez tonygryps — 18 lis 2016, o 11:48

Re: optomalizacja kodu

2016-11-18T09:45:59+01:00

jeszcze raz ci powtarzam - bo już pisałem wcześniej, że jeśli chodzi o rzutowanie to nie ma to NIC a NIC wspólnego czy jakaś zmienna jest w RAM czy we Flash - zrozum to proszę.

więc niestety coś kręcisz bo nie ważne czy masz zapis taki

[syntax=c]lcd_str_P( (char*)&days[ dt->dayofwek -1 ] ); // days we FLASH[/syntax]

czy taki
[syntax=c]lcd_str( (char*)&days[ dt->dayofwek -1 ] ); // days w RAM[/syntax]

to w każdym wypadku MUSISZ robić rzutowanie (char*)

gdybyś chciał uniknąć rzutowania to MUSIAŁBYŚ podawać jawnie indeks drugiego wymiaru tablicy bo dopiero wtedy kompilator ma pewność że chodzi ci o string, np
[syntax=c]lcd_str_P( &days[ dt->dayofwek -1 ][0] ); // days we FLASH[/syntax]

albo

[syntax=c]lcd_str( &days[ dt->dayofwek -1 ][0] ); // days w RAM[/syntax]

Statystyki: Napisane przez mirekk36 — 18 lis 2016, o 09:45

Re: optomalizacja kodu

2016-11-18T09:07:25+01:00

Dziękuje wszystkim wydaje mi się że zrozumiałem o co chodzi .
Męczy mnie tylko jedna rzecz.

Quote:

Poza tym co ma wspólnego rzutowanie z tym czy coś jest w pamięci RAM czy nie ? .... podpowiedź - nie ma to NIC wspólnego. Rzutowanie służy do uzgodnień typów danych jeśli się różnią pomiędzy typem np argumentu funkcji a przekazywanej zmiennej.

Otóż chodzi mi o to że jeśli bufor był w pamięci ram
[syntax=c]char days[7][4] = {
"pon", "wto", "sro", "czw", "pia", "sob", "nie"
};[/syntax]
to nie trzeba było robić rzutowania a jak jest w pamięci flash to trzeba je robić
przecież zapisanie tego tak
[syntax=c]const char days[7][4] PROGMEM = {
"pon", "wto", "sro", "czw", "pia", "sob", "nie"
};[/syntax]
nie zmienia typu zmiennej tylko rodzaj pamięci wykorzystywanej
a funkcja lcd_str_P() jako argumentu oczekuje adresu komórki pamięci flash o typie char.
[syntax=c]void lcd_str_P(const char * str)[/syntax]
więc skoro bufor we flashu jest typu char i funkcja oczekuje takiego typu to dlaczego trzeba to rzutować?

Statystyki: Napisane przez tonygryps — 18 lis 2016, o 09:07

Re: optomalizacja kodu

2016-11-18T00:19:57+01:00

Dla niniejszego przykładu warto się zastanowić nad nieco innym sposobem rozwiązania, które sam z powodzeniem stosuję.
Chodzi o sposób przedstawiony w tym poradniku:
http://mirekk36.blogspot.com/2013/11/ta ... ogmem.html

Można zatem zrobić takie rozwiązanie:

[syntax=c]/********* Tablica stringów dni tygodnia *********************************/
const char pon[] PROGMEM = "pon";
const char wto[] PROGMEM = "wto";
const char sro[] PROGMEM = "sro";
const char czw[] PROGMEM = "czw";
const char pia[] PROGMEM = "pia";
const char sob[] PROGMEM = "sob";
const char nie[] PROGMEM = "nie";

const PROGMEM char * const days[] PROGMEM = {
pon,
wto,
sro,
czw,
pia,
sob,
nie
};
/*************************************************************************/[/syntax]

Mając tak zdefiniowane napisy można zmodyfikować funkcję wyświetlającą do postaci:
[syntax=c]static void show_date( T_DATETIME * dt ) {
lcd_gotoxy(LCD_CTRL_0, 0, 1);

lcd_puts( LCD_CTRL_0, "20" );

if ( dt->year < 10 ) {
lcd_puts( LCD_CTRL_0, "0" );
}
lcd_int ( LCD_CTRL_0, dt->year );
lcd_puts( LCD_CTRL_0, "-" );

if ( dt->month < 10 ) {
lcd_puts( LCD_CTRL_0, "0" );
}
lcd_int ( LCD_CTRL_0, dt->month );
lcd_puts( LCD_CTRL_0, "-" );

if ( dt->day < 10 ) {
lcd_puts( LCD_CTRL_0, "0" );
}
lcd_int ( LCD_CTRL_0, dt->day );

lcd_puts( LCD_CTRL_0, " " );

const char *ptr = pgm_read_byte( days + dt->dayofweek-1 );
lcd_puts_p( LCD_CTRL_0, ptr );

}[/syntax]

Uwaga! w moim przykładzie zastosowałem inną bibliotekę do LCD, zatem są nieco inne funkcje do wyświetlania, ale całkiem podobne.
Taki sposób wydaje mi się nieco bardziej elegancki i zarazem jest to dobre ćwiczenie na wskaźniki i odczyt z pamięci FLASH.
Dodatkowo łatwo modyfikować same napisy, np w jednym miejscu wpisując "poniedzialek" zamiast "pon", a nie martwiąc się o resztę kodu.

Polecam.

Statystyki: Napisane przez szymonciow — 18 lis 2016, o 00:19

Re: optomalizacja kodu

2016-11-17T22:55:59+01:00

a to jest tekst w pamięci RAM ???
[syntax=c]const char days[7][4] PROGMEM = {
"pon", "wto", "sro", "czw", "pia", "sob", "nie"
};[/syntax]
No chyba żartujesz ?

Poza tym co ma wspólnego rzutowanie z tym czy coś jest w pamięci RAM czy nie ? .... podpowiedź - nie ma to NIC wspólnego. Rzutowanie służy do uzgodnień typów danych jeśli się różnią pomiędzy typem np argumentu funkcji a przekazywanej zmiennej.

Statystyki: Napisane przez mirekk36 — 17 lis 2016, o 22:55

Re: optomalizacja kodu

2016-11-17T22:02:44+01:00

Dziękuję tak działa tylko czemu tak, przecież jak jest w pamięci ram to nie trzeba rzutowania robić.

Statystyki: Napisane przez tonygryps — 17 lis 2016, o 22:02

Re: optomalizacja kodu

2016-11-17T21:45:09+01:00

[syntax=c]lcd_str_P( (char*)&days[ dt->dayofwek -1 ] );[/syntax]

bo pewnie trzeba jeszcze jawne rzutowanie zrobić do typu wskaźnikowego o który chodzi

Statystyki: Napisane przez mirekk36 — 17 lis 2016, o 21:45

Re: optomalizacja kodu

2016-11-17T21:15:44+01:00

coś tam z wojska zostało (jakieś nawyki sapera ) a tak na poważnie próbuje to zrozumieć i najpierw staram się sam ile wiem i umiem podziałać ale nie zawsze wychodzi.
próbowałem tak jak Mirku piszesz i też jest warming
./DS3231/ds3231.c: In function 'show_date':
../DS3231/ds3231.c:75:2: warning: passing argument 1 of 'lcd_str_P' from incompatible pointer type [enabled by default]
lcd_str_P(&days[dt->dayofwek -1]);
wkleiłem warming z consoli.

działa ale jest warming,
a co do funkcji lcd_str_P to jak rozumie to wyświetla String z pamięci flash a jako argument trzeba jaj przekazać adres tej komórki to makro PSTR jest makrem pozyskiwania tego adresu czy się mylę ?

Statystyki: Napisane przez tonygryps — 17 lis 2016, o 21:15

Re: optomalizacja kodu

2016-11-17T21:01:51+01:00

tonygryps napisał(a):

Próbowałem tak lcd_str_P() i tak lcd_str_P(PSTR()) ale wtedy wywala błąd

No bo widać, że zamiast postarać się dojść jak to działa to robisz próby jak saper albo przeciąć ten kabelek albo inny i BUM....

tonygryps napisał(a):

co do źródła nie wklejałem bo to Mirku z twojego bloga ale już wklejam.

ale mi chodzi właśnie o źródło stringa a nie o całe kody

ok spróbuj pan tak

[syntax=c]lcd_str_P( &days[ dt->dayofwek -1 ] );[/syntax]

funkcja lcd_str_P() wymaga wskaźnika na string w pamięci Flash - więc dodawanie tu pgm_read_ .... jest kiepskim pomysłem

Statystyki: Napisane przez mirekk36 — 17 lis 2016, o 21:01

Re: optomalizacja kodu

2016-11-17T20:55:08+01:00

Próbowałem tak lcd_str_P() i tak lcd_str_P(PSTR()) ale wtedy wywala błąd
co do źródła nie wklejałem bo to Mirku z twojego bloga ale już wklejam.
[syntax=c]const char days[7][4] PROGMEM = {
"pon", "wto", "sro", "czw", "pia", "sob", "nie"
};[/syntax]
to po zmianie do pamięci flash
a to orginalny kod
[syntax=c]char days[7][4] = {
"pon", "wto", "sro", "czw", "pia", "sob", "nie"
};[/syntax]
dodam żę jeśli napiszę
[syntax=c]lcd_int(pgm_read_byte( &days[dt->dayofwek -1]));[/syntax]
wyświetla kod ascii pierwszej litry dnia .

Próbuje wyświetlić string tyle że z pomięci flash.

Statystyki: Napisane przez tonygryps — 17 lis 2016, o 20:55

Re: optomalizacja kodu

2016-11-17T20:34:45+01:00

eeeeh a masz Bluebooka i zajrzeć to co ?

przede wszystkim to chyba masz tam opisane do czego są funkcje z suffixem _P ? czy nie masz ? a więc

lcd_str_P() a nie lcd_str()

w takim przypadku.

A po drugie to ja nie wiem i pewnie ty chyba też sam nie wiesz - z tego co piszesz - co ty chcesz wyświetlić ... więc sam się zastanów i napisz.

1. czy chcesz wyświetlić string ?
2. czy chcesz wyświetlić liczbę ?

a jeśli chcesz wyświetlić string - no to pokaż jego źródło

Statystyki: Napisane przez mirekk36 — 17 lis 2016, o 20:34

optomalizacja kodu

2016-11-17T20:12:17+01:00

witam
Otóż próbowałem optymalizować kod Mirka z poradnika o ds3231 w sensie wyświetlać z pamięci flash zamiast ram i mam problem z jedną funkcją
[syntax=c]void show_date( TDATETIME * dt ) {
//char a = 0;
lcd_str_P(PSTR("20"));
if( dt->year < 10 ) lcd_str_P(PSTR("0"));
lcd_int(dt->year);
lcd_str_P(PSTR("-"));
if( dt->month < 10 ) lcd_str_P(PSTR("0"));
lcd_int(dt->month);
lcd_str_P(PSTR("-"));
if( dt->day < 10 ) lcd_str_P(PSTR("0"));
lcd_int(dt->day);
lcd_str_P(PSTR(" "));
// lcd_str(days[dt->dayofwek -1]) // orginalny zapis w funkcji
lcd_str(pgm_read_byte( &days[dt->dayofwek -1]));
}[/syntax]

w tej linii mam warminga [syntax=c]lcd_str(pgm_read_byte( &days[dt->dayofwek -1]));[/syntax] który dotyczy tego że powinienem użyć funkcji lcd_int
tyle że ja nie chcę wyświetlać liczby tylko string .
jak to zrobić żeby wyświetlać zawartość bufora chat w pamięci flash.

Statystyki: Napisane przez tonygryps — 17 lis 2016, o 20:12