ATNEL tech-forum

Re: Rozmiar tablicy struktur

2016-07-28T19:08:04+01:00

mpo napisał(a):

A jaka jest zaleta wyciągania danych ze struktury w zewnętrznej funkcji, bo nie dostrzegam

Nie wiem do czego używasz funkcji write_registers() (do tej funkcji przekazywałeś elementy struktury w swoim przykładzie), więc trudno mi się w tej kwestii wypowiadać. Jeśli jest Ci tak wygodniej, to możesz rozbić strukturę we funkcji regs_set() i przekazać do funkcji write_registers() jej poszczególne składniki (tak, jak to miałeś zrobione):
[syntax=c]static void regs_set(void)
{
uint8_t reg;
uint16_t val;

for (uint8_t i = 0; i < ( sizeof(init_vals)/sizeof(init_vals[0]) ); i++ )
{
reg = init_vals[i].reg;
val = init_vals[i].val;
write_register(reg, val);
}
}[/syntax]
Wielkiej różnicy nie będzie, gdzie to będziesz rozbijał, gdzieś i tak musisz.

Zrobiłem tak, bo według mnie jest czytelniejsze przekazanie do funkcji wskaźnika na strukturę, niż poszczególnych elementów. Być może jest i szybsze (nie sprawdzałem), bo przekazujesz 2 bajty (wskaźnik) zamiast 3 bajtów (rozmiar struktury). W tym przypadku to nie ma większego znaczenia, ale zastanów się, co by było, gdyby struktura zawierała więcej elementów (może nawet jakąś tablicę) i chciałbyś je wszystkie osobno przekazywać do funkcji write_register()... Odpowiedz sobie sam

Statystyki: Napisane przez andrews — 28 lip 2016, o 19:08

Re: Rozmiar tablicy struktur

2016-07-28T17:17:56+01:00

Ja bym proponował coś takiego:
[syntax=c]typedef struct
{
uint8_t reg;
uint16_t val;
} regval_t;

// deklaracja tablicy we flash
static const __flash regval_t init_vals[] =
{
{CTRL1, 0x12B0},
{CTRL2, 0x0007},
{CTRL3, 0x008C}
};

// funkcja regs_set()
static void regs_set(void)
{
for (uint8_t i = 0; i < (sizeof(init_vals)/sizeof(init_vals[0])); i++)
{
write_register(&init_vals[i]);
}
}

// dostęp do elementów struktury we funkcji write_register() wygląda tak
static void write_register(const __flash regval_t *regv)
{
uint8_t r = regv->reg;
uint16_t v = regv->val;
}[/syntax]

Statystyki: Napisane przez andrews — 28 lip 2016, o 17:17

Re: Rozmiar tablicy struktur

2016-07-28T14:40:49+01:00

mpo napisał(a):

A jak podmieniłem swoją funkcję na tą elegancką

Jaką tam zaraz elegancką ... ot jakaś tam wersja

a jak już używasz debugera to załóż pułapkę i sprawdź czy ci do write_register(reg, val); są przekazywane poprawne wartości

------------------------ [ Dodano po: 2 minutach ]

tylko czy ty nie powinieneś zrobić tam

[syntax=c]sizeof(init_vals)/sizeof(regval_t)[/syntax]

?? w pętli for (albo nawet na razie wstaw ręcznie ilość elementów dla testu)

------------------------ [ Dodano po: 2 minutach ]

sorki, że jak tak z doskoku bez dokładnego sprawdzenia i na szybko odpowiadam ale po prostu odrywam się co chwilę od innego już nużącego mnie zajęcia ...

Statystyki: Napisane przez mirekk36 — 28 lip 2016, o 14:40

Re: Rozmiar tablicy struktur

2016-07-28T13:37:30+01:00

a próbowałeś tak?

[syntax=c]static void regs_set(void)
{
for (uint8_t i = 0; i < sizeof(init_vals); i++ )
{
uint8_t reg = pgm_read_byte( &init_vals[i].reg) );
uint16_t val = pgm_read_word( &init_vals[i].val) );

write_register(reg, val);
}
}[/syntax]

bo jakoś tak troszkę zakręcone było to co robisz - tak mi się wydaje no ale jak działa to działa

Statystyki: Napisane przez mirekk36 — 28 lip 2016, o 13:37

Re: Rozmiar tablicy struktur

2016-07-28T12:45:08+01:00

mpo napisał(a):

Właśnie sizeof(pixel_t) da w wyniku 2.

Taki szczegół dla ścisłości, sizeof(pixel_t*) da w wyniku 2, podobnie jak sizeof(node). Z kolei sizeof(*node) będzie równoznaczne z sizeof(pixel_t).

mpo napisał(a):

Czy jest sposób aby to jakoś wyłuskać bez przekazywania funkcji "na chama" tej informacji?

Obawam się, że nie ma. Funkcja void draw_polygon(pixel_t *node) na etapie kompilacji nie wie nawet, że node to wskaźnik do tablicy (jak już chyba sam zauważyłeś, sizeof(*node) zwraca wartość rozmiaru tylko jednego jej elementu).

Jak Cię nuży pisanie: "polygon1, sizeof(polygon1)/sizeof(pixel_t)" bo za długie, zawsze możesz sobie stworzyć makro preprocesora, które będzie robić to za Ciebie.

Statystyki: Napisane przez andrews — 28 lip 2016, o 12:45

Re: Rozmiar tablicy struktur

2016-07-28T12:38:03+01:00

mpo napisał(a):

Właśnie sizeof(pixel_t) da w wyniku 2.

Nigdy w życiu jak podasz nazwę typu do sizeof() to dostaniesz rozmiar struktury w bajtach ... AAAAAAAAAA sorry dopiero teraz zobaczyłem że chodzi jeszcze o tablicę struktur to przepraszam ... wtedy pozostaje ręczne przekazanie rozmiaru jako dodatkowy argument funkcji. A wyżej kolega andrews pokazał nawet jak to zrealizować również z sizeof.

jeśli masz oczywiście własną konwencję nazw to ok - nie było mojej uwagi

Statystyki: Napisane przez mirekk36 — 28 lip 2016, o 12:38

Re: Rozmiar tablicy struktur

2016-07-28T12:16:13+01:00

sizeof() to operator rozmiaru. Obliczenie rozmiaru dokonywane jest na etapie kompilacji, więc nie da się określić rozmiaru na podstawie wskaźnika przekazywanego do funkcji, który przecież może wskazywać na obiekty o różnym rozmiarze, więc na etapie kompilacji nie da się ustalić rozmiaru tablicy przekazanej do funkcji.

Jeśli tablica ma zawsze ten sam rozmiar można zadeklarować funkcję tak:[syntax=c]void draw_polygon(pixel_t node[3]) {}[/syntax]a jeśli rozmiar wskazywanej tablicy będzie zmienny, można użyć dodatkowego parametru z rozmiarem:[syntax=c]void draw_polygon(pixel_t *node, size_t size) {} // gdzie size będzie oznaczać ilość struktur w tablicy
// i później wywoływać funkcję poprzez
draw_polygon( polygon1, sizeof(polygon1)/sizeof(pixel_t) );[/syntax]

Statystyki: Napisane przez andrews — 28 lip 2016, o 12:16

Re: Rozmiar tablicy struktur

2016-07-28T11:58:07+01:00

mpo napisał(a):

Co robię źle?

Robisz źle to, że przecież sam zobacz, upraszczam twój kod:

[syntax=c]void draw_polygon(pixel_t *node) {

volatile uint8_t x=sizeof(*node);[/syntax]

Przecież ty nie sprawdzasz tu operatorem sizeof rozmiaru struktury tylko ? no jak myślisz ? rozmiar panie kochany wskaźnika i pewnie się dziwisz dlaczego wychodzi ci 2. Dwa bo wskaźniki są dwubajtowe .... sizeof nie obliczy rozmiaru struktury na podstawie wskaźnika więc albo przekaż rozmiar ręcznie albo użyj sizeof( pixel_t )

------------------------ [ Dodano po: 3 minutach ]

Przy okazji zastanów się na przyszłość dla lepszej przejrzystości kodu nad nadawaniem porządnych nazw typów gdy definiujesz strukturę, bo teraz to można powiedzieć, że jest bajzelek w kodzie ... Ja bym dał nazwę typu

[syntax=c]TPIXEL a nie pixel_t[/syntax]

później gdy definiuje się zmienną albo podaje jako argument to ładnie widać

[syntax=c]TPIXEL polygon1;[/syntax]

nieprawdaż ?

Statystyki: Napisane przez mirekk36 — 28 lip 2016, o 11:58