Wyświetlanie napisów. Dlaczego wypisuje krzaczki?

Question

Witam! Mam problem z odczytaniem napisów. W programie napisy wpisałem w taki sposób: Jeżeli wywołuję napisy jak poniżej, to jest wszystko OK, lecz jeżeli robię to w ten sposób to wypisuje krzaki na LCD. Czy może mi ktoś wytłumaczyć dlaczego tak jest i jak wywoływać odpowiedni napis zmienną??...

jam_es · Accepted Answer

Witam Możliwe, że ma to związek z deklaracją m. Ustaliłeś jako unsigned char. Spróbuj jako integer. Pozdrawiam

Bigfoot · Accepted Answer

Ja Kaszpirowskim nie jestem ale na 99% problem lezy w tym, ze do Twojej funkcji, ktorej prawdopodobnie argumentem powinien byc const Ty "wkladasz" zmienna, ktora nie jest const. Jesli sie myle to niech mnie ktos poprawi.

Pokaz definicje funkcji pisz_tekst, powiedz tez w jakim srodowisku piszesz.

BF

zumek · Accepted Answer

Dlaczego upierasz się , by wepchać wskaźniki do pamięci programu :?:

Mówisz kompilatorowi - umieść wskaźnik do pamięci programu , w pamięci programu ...

const prog_char* info[] PROGMEM = ...

... , a powinieneś powiedzieć - umieść wskaźnik do pamięci programu , w pamięci data(RAM)

const char* info[] = { PM_UST_ZEG,   PM_UST_CZAS,  PM_UST_DATE};

To samo masz tu ...

void pisz_tekst(prog_char* tekst)

... , więc zmień na ...

void pisz_tekst(const char* tekst)

... i po bulu

const char* można zastąpić PGM_P

Piotrek

JacekCz · Accepted Answer

Może kombinuję (nie czuje intuicyjnie tych słów kluczowych)

const prog_char* info[] PROGMEM = ...

prog_long /*prog_void */ const prog_char* info[] PROGMEM = ...

warto sprawdzić odwrocenia kolejności słów, ma to inny sens.

Manipulując również kolejnością słow ... musi być rozwiązanie lepsze niż wszystko w RAM

2.
Takie coś:
char *LINE1 = PSTR("Pierwsza linia na LCD");

3.
I jeszcze: jeśli już literki będą w xxxROM, wskazniki bolą nas że są nie takie albo kosztują RAM ... może ich nie trzmać ? w ciągłym obszarze literek, ile ich, 1k, 2k ? szukam szybką pętlą stringu nr N i używam.
Być może szybkośc nie jest krytyczna.

.... tab[]=
"Tekst 1szy\0Tekst 2gi\0"
"Tekst 3ci\0tekst 4ty\0"
....
;

(sklejenia stringów każdy ludzki kompilator akceptuje)

4. Sugeruję, że użycie operatora pytajnika? jak w twojej pętli jest przesadą i nic pozytywnego nie daje. To powinien byc normalny if.

5.
while(0 != ....
też jest hakerstwem kogoś kto nie czuje się pewnie z C. Albo to w kilku konstrukcjach jak ma byc estetycznie,
albo bez tego 0 != jak ma być hakersko
[/code]

adamusx · Accepted Answer

Zmien const char* na PGM_P jak kolega Zumek napisal i po kłopocie. Wtedy przezucisz te zmienne z RAMu do Flasha.

zumek · Accepted Answer

Chyba się nie zrozumieliśmy Dane możesz umieścić w pamięci programu ... Kod: text Rozwiń Zaznacz wszystko Kopiuj do schowka prog_char PM_UST_ZEG[]  = Ustaw zegar: ; prog_char PM_UST_CZAS[] = ; prog_char PM_UST_DATE[] = ; ... , ale wskaźniki na te tablice , kompilator umieści w RAM-ie(bo gdzież by indziej) ... Kod: text Rozwiń Zaznacz wszystko Kopiuj do schowka const char* info[] = { PM_UST_ZEG,   PM_UST_CZAS,  PM_UST_DATE}; ... więc Twój fragment powinien wyglądać tak: Kod: text Rozwiń Zaznacz wszystko Kopiuj do schowka prog_char PM_UST_ZEG[]   = Ustaw zegar: ; prog_char PM_UST_CZAS[]   = ; prog_char PM_UST_DATE[]   = ; const prog_char* info[] PROGMEM ={  PM_UST_ZEG,  PM_UST_CZAS,PM_UST_DATE} //... void pisz_tekst(const char* tekst) {    char zn;        while(0 != (zn = pgm_read_byte(tekst++)))    {          if(zn == CR) // czy znak nowej linii          {          wiersz==1?wiersz=0:++wiersz;          kolumna = 0;          lcd_xy(wiersz,kolumna);             }             else          {             pisz_dlcd(zn);            }    } } Każdy wskaźnik zajmuje 2 bajty RAM-u , więc dla 50 tablic umieszczonych w ROM-ie , zużyjesz 100B RAM-u , to chyba niewiele Jeśli już żałujesz tego RAM-u , to zastosuj np.switch] Kod: text Rozwiń Zaznacz wszystko Kopiuj do schowka const char * romptr; //.... switch (m) { case  1: romptr=PM_UST_CZAS; break; //pośrednio case  2: pisz_tekst(PM_UST_ZEG); break; //bezpośrednio i                                                           //niepotrzebna tablica wskaźników //itd. }                 //.... pisz_tekst(romptr); //... Wybierz najlepsze dla Ciebie rozwiązanie , bo nie ma powodu rozdzierać szat Piotrek

aceman · Answer

Niestety powyższa propozycja również nie działa. Deklarowałem zmienną m na wiele różnych sposobów i wciąż krzaki.

JacekCz · Answer

unsigned char jest legalny choć zaskakujący. Możesz mieć jakies efekty uboczne, cos zamazuje coś innego itd, ale nie dajesz źródel. Wrożka ze szklaną kulą ma wolne.

aceman · Answer

To co byś zaproponował Jacku?? Próbowałem deklarować zmienną również jako int i to samo.

JacekCz · Answer

Błąd jest gdzie indziej. Wynika mi to z metody Kaszpirowskiego.

aceman · Answer

Trochę cienka ta metoda, skoro nie wiesz w czym problem ;P

aceman · Answer

W końcu jakieś konkrety!
Funkcja pisz_tekst() wygląda następująco, a piszę w C pod WinAVR.

void pisz_tekst(prog_char* tekst) 
{
	char zn;
	
	while(0 != (zn = pgm_read_byte(tekst++)))
	{
			if(zn == CR) // czy znak nowej linii
			{
			wiersz==1?wiersz=0:++wiersz;
			kolumna = 0;
			lcd_xy(wiersz,kolumna);	
			}	
			else
			{	
			pisz_dlcd(zn);				}
	}
}

Spróbuję bez const.

Dodano po 8 [minuty]:

Niestety bez operatora const efekt jest taki sam

(

aceman · Answer

Witaj Piotrze!

Twoje rozwiązanie w końcu zaczęło działać.
Należało poprawić jeszcze:

char PM_UST_ZEG[]   = "Ustaw zegar: "; 
char PM_UST_CZAS[]   = "<    Ustaw czas    >";   // itd

zamiast:

prog_char PM_UST_ZEG[]   = "Ustaw zegar: "; 
prog_char PM_UST_CZAS[]   = "<    Ustaw czas    >";

I już się ucieszyłem, lecz stworzył się kolejny problem!!
Ponieważ napisów jest ok 40-50, to po powyższej zmianie zajęty obszar pamięci data(RAM) nie wynosi już 31,3%, lecz 93,4%.

I CO Z TYM DALEJ ROBIĆ :cry:

Pozdrawiam, Artur

Dodano po 1 [godziny] 22 [minuty]:

To może podejdźmy do problemu inaczej:
Jak rozpisać tabelę z wyświetlanymi napisami, żeby w prosty sposób można było je wyświetlać poprzez poniższą funkcję. Napisy chcę zapisać w osobnym pliku napisy.h oraz żeby dane napisy były umieszczone ekonomicznie w pamięci.

void pisz_tekst(prog_char* tekst) 		// pisz tekst na LCD wskazywany przez *tekst
{
	char zn;
	
	while(0 != (zn = pgm_read_byte(tekst++)))
	{
			if(zn == CR)					// czy znak nowej linii
			{
			wiersz==1?wiersz=0:++wiersz;	// przejdz do nowej linni
			kolumna = 0;
			lcd_xy(wiersz,kolumna);			// ustaw obowiazujace po zmianie wspolrzedne LCD
			}	
			else
			{	
			pisz_dlcd(zn);	// umiesc pojedynczy znak tekstu na LCD
			}
	}
}

aceman · Answer

Dziękuję za pomoc.
Wszystko już chodzi jak należy. Pewnie temat dla was jest banalny, ale ja jeszcze nie czaję pewnych spraw.
Błędy, które popełniałem to:
Po zmianie na (const char* tekst) w funkcji pisz_tekst() pisałem;

while(0 != *tekst++)

zamiast:

while(0 != (zn = pgm_read_byte(tekst++)))

No i równie pomocna okazała się zmiana const char* na PGM_P;

Pozdrawiam gorąco.
Artur

zumek · Answer

Tak mnie zaciekawił temat danych w pamięci programu , że z chęci zaoszczędzenia RAM-u (zawsze go mało) , spłodziłem takie coś


#include <avr/io.h>
#include <avr/pgmspace.h>

//tablice z danymi w ROM
prog_char ROM1[]  = "AAAAAAAAAA";
prog_char ROM2[]  = "BBBBBBBBBB";
prog_char ROM3[]  = "CCCCCCCCCC";

//tablica w ROM , z adresami tablic w ROM ;-) 
const prog_char*  rom_tab_adr[] PROGMEM = { ROM1,ROM2,ROM3};

char buf[22]={0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0};//dla symultora

void copy_to_buf(const char* rom_data)
{
   char chr,i=0;
   while(0 != (chr = pgm_read_byte(rom_data++)))
   	{ 
		buf[(int)i++]=chr;//lub do LCD,UART , czy gdzie dusza zapragnie ;) 
   	}
}

int main(void)
{
unsigned char i;

while(1)
	{
	for(i=0;i<3;i++)
		{
		copy_to_buf((const char*)pgm_read_word(&rom_tab_adr[i]));
		}
	}
return 0;
}

Piotrek

aceman · Answer

He he Temat wydaje się banalny, lecz byle nowicjusz nie napisze tego ot tak sobie. W końcu praktyka czyni mistrza .

Wyświetlanie napisów. Dlaczego wypisuje krzaczki?

Post #1

Post #2

Post #3

Post #4

Post #5

Post #6

Post #7

Post #8

Post #9

Post #10

Post #11

Post #12

Post #13

Post #14

Post #15

Post #16

Post #17

Podsumowanie tematu