Größe eines Strukturfeldes über Pointer ermitteln

Stephan Plaschke wrote:
> Das gibt mir natürlich nur den Wert 6 da es ja nur auf die erste
> Struktur zeigt und nicht auf das gesammte Element.
> Finde keine andere Art das zu machen, daher meine Frage ob das überhaupt
> so machbar ist oder ob es nicht machbar ist. Für jede Hilfe bin ich echt
> dankbar.

Ist nicht machbar.
Wenn du nur einen Pointer hast, der auf ein Array zeigt, ist es
nicht möglich, die Größe des Arrays zu bestimmen.

Genau das ist zb. die Krux bei allen String-verarbeitenden
Funktionen. Ein strcpy kann sich per Definition nicht davor
schützen, das Zielarray zu überschreiben, weil es aus dem erhaltenen
Pointer nicht die Größe des Arrays ermitteln kann.

-> Größe mit übergeben oder ein Dummy-Element ins Array einfügen,
welches die Rolle von 'ich bin das letzte' spielt.

> In der wird die Länge des Befehls und der Befehl selber
> definiert.

Diese Länge brauchst du wiederrum nicht. Diese Länge ist per
strlen aus dem Kommando bestimmbar.


In deinem Fall kannst du die Sache mit einer weiteren
Struktur retten, in der du einen Zeiger zu einem Menü
mit der Größe des Menüs verknüpfst.

1

static UART_MEN UART_menu_C[]={

2

    {3, "lr", UART_menu_flash_clr},

3

    {5, "lear", UART_menu_flash_clr}

4

};

5

6

static UART_MEN UART_menu_D[]={

7

    {3, "ar", UART_menu_flash_clr},

8

    {5, "beat", UART_menu_flash_clr}

9

};

10

11

struct Menus {

12

  UART_MEN* UART_menu;

13

  size_t    UART_menu_size;

14

}

15

16

#define ARRAY_SIZE(x) ( sizeof(x) / sizeof(*x) )

17

18

struct Menus UART_menu[] = {

19

   { UART_menu_C, ARRAY_SIZE( UART_menu_C ) },

20

   { UART_menu_D, ARRAY_SIZE( UART_menu_D ) }

21

};

OliverSo wrote:
>>Da ich im Programm aber nur mit Pointern arbeite kann ich
> nuri=sizeof>(*pUART_menu_sub);
>>verwenden.
>
> Und wer hindert dich daran, trotzdem

1

i=sizeof(UART_menu_C);

> zu benutzen?
>

Dann setzt mal in diesem Code für die ??? richtig ein

1

static UART_MEN *pUART_menu[]={

2

    UART_menu_C,

3

    UART_menu_D,

4

    ...

5

}

6

7

void HandleMenu( UART_MENU* pMenu )

8

{

9

  // male alle Einträge des Menüs

10

11

  for( i = 0; i < sizeof( ???? ); ++i )

12

    ...

13

}

14

15

int main()

16

{

17

  int i = 0;

18

19

  ...

20

21

  HandleMenu( pUART_menu[i] );

22

}

@ Karl heinz Buchegger
Dachte ich mir. Werd das mit einer weiteren Struktur lösen. Vielen Dank!

@
OliverSo
Mein Programm! Ich habe ja nur einen Buchstaben und der zeigt über einen 
Pointer auf das erste Element des Feldes. Da ich aber nicht immer mit 
if() oder ähnlichen abfragen will welchen Buchstaben ich habe ist es 
nicht möglich immer direkt abzufragen.

Danke noch mal!!

Steph0815

Karl heinz Buchegger wrote:
>

1

> #define ARRAY_SIZE(x) ( sizeof(x) / sizeof(*x) )

2

> 

3

> struct Menus UART_menu[] = {

4

>    { UART_menu_C, ARRAY_SIZE( UART_menu_C ) },

5

>    { UART_menu_D, ARRAY_SIZE( UART_menu_D ) }

6

> };

7

>

Dies liese sich mit einem weiteren Makro noch ein bischen
einfacher und weniger fehleranfällig machen

1

#define ARRAY_SIZE(x) ( sizeof(x) / sizeof(*x) )

2

#define UART_ENTRY(x) x, ARRAY_SIZE(x)

3

4

struct Menus UART_menu[] = {

5

   { UART_ENTRY( UART_menu_C ) },

6

   { UART_ENTRY( UART_menu_D ) }

7

};

Am Prinzip ändert sich aber nichts. Ein 'Menus' Eintrag enthält
einen Pointer zum Menü-Array sowie dessen Größe.

Stephan Plaschke wrote:
> Mein Programm! Ich habe ja nur einen Buchstaben und der zeigt über einen
> Pointer auf das erste Element des Feldes. Da ich aber nicht immer mit
> if() oder ähnlichen abfragen will welchen Buchstaben ich habe ist es
> nicht möglich immer direkt abzufragen.

Üder diese Abfrage sollte man sowieso noch mal reden.
Anstatt da mit Buchstaben rumzuwurschteln, warum benutzt
du nicht einfach einen strcmp()?

http://www.mikrocontroller.net/articles/FAQ#strcmp.28_const_char.2A_str1.2C_const_char.2A_str2_.29

String wird komplett von der UART geholt und danach das Menü
Eintrag für Eintrag durchgegangen und mittels strcmp() verglichen
ob eines der Kommandos übereinstimmt. Wenn ja, wird die
zugehörige Funktion aufgerufen. Fertig!

1

int EvaluateCommand( struct Menus* pSubMenu, const char* cmd )

2

{

3

  for( size_t i = 0; i < pSubMenu->UART_menu_size; ++i ) {

4

    if( strcmp( pSubMenu->UART_menu[i].cmd, cmd ) == 0 ) {

5

      if( pSubMenu->UART_menu[i].fct != NULL )

6

        (*(pSubMenu->UART_menu[i].fct))();

7

      return 1;

8

    }

9

  }

10

11

  return 0;

12

}

Karl heinz Buchegger wrote:
> Stephan Plaschke wrote:
>> Mein Programm! Ich habe ja nur einen Buchstaben und der zeigt über einen
>> Pointer auf das erste Element des Feldes. Da ich aber nicht immer mit
>> if() oder ähnlichen abfragen will welchen Buchstaben ich habe ist es
>> nicht möglich immer direkt abzufragen.
>
> Üder diese Abfrage sollte man sowieso noch mal reden.
> Anstatt da mit Buchstaben rumzuwurschteln, warum benutzt
> du nicht einfach einen strcmp()?


Da hast du wirklch recht. Habe das jetzt auch so geändert. Ich suche 
immer noch nach den ersten Buchstaben, ist dieser üngültig exit. Danach 
nehme ich dann aber nicht strcmp() sondern strcasecmp() ist ja 
eigentlich das gleiche nur unterscheidet Groß- und Kleinbuchstaben. Ist 
dann ein gültiger Befehl erkannt ausführen des Befehls wenn nicht 
Abbruch mit Error. Ganz einfach wenn ich das so betrachte!

Vielen Dank Karl heinz Buchegger, viel Arbeit erspart!!