Användning av struct

[thepirateboy]

Nej, Tagged Initializing Format verkar han inte köpa alls tillsammans med union. Det enda som funkar är nedanstående (då får jag varning "../main.c:78: warning: missing braces around initializer")

Kod: Markera allt

union
{
	T_TX_data_struct tx_data2;
  	unsigned char Buffer[23];

} Alla_Bytes_I_Samma_Klump = {STX,"000",';',"00",';',"000",';'," 00,0",';',"0,00",ETX};

Däremot fungerar Tagged Initializing Format bra vid "normal struct-deklarering", så det var ett bra tips. Ger ett tydligare skrivsätt.

Kod: Markera allt

	
	T_TX_data_struct tx_data = {
				.start_TX = STX,
				.counter = "000",
				.sep_sign_1 = ';',
				.serial = "00",
				.sep_sign_2 = ';',
				.ad_value = "000",
				.sep_sign_3 = ';',
				.temperature = " 00,0",
				.sep_sign_4 = ';',
				.batt_volt = "0,00",
				.stop_TX = ETX
				};

[stekern]

thepirateboy:

Kod: Markera allt

	unsigned char *ptr_tx_data = (unsigned char *)&tx_data;
	nrf24l01_write_tx_payload(ptr_tx_data, RF_PAYLOAD, true);

kan du skriva om till:

Kod: Markera allt

	nrf24l01_write_tx_payload((unsigned char *)&tx_data, RF_PAYLOAD, true);

[speakman]

Nej, Tagged Initializing Format verkar han inte köpa alls tillsammans med union. Det enda som funkar är nedanstående (då får jag varning "../main.c:78: warning: missing braces around initializer")

Kan du sätta in den kompletta källkoden som genererar det där felet?

Kod: Markera allt

union
{
	T_TX_data_struct tx_data2;
  	unsigned char Buffer[23];

} Alla_Bytes_I_Samma_Klump = {STX,"000",';',"00",';',"000",';'," 00,0",';',"0,00",ETX};

Här kan man ju tycka att kompilatorn har svårt att veta var den ska stoppa in datat. Fungerar t.ex. Alla_Bytes_I_Samma_Klump.tx_data2 = {...} ?

[thepirateboy]

Det går ju att ta ett mindre exempel, som nedan, som genererar "missing braces..."
Jag har försökt att googla och lista ut vad det är för "braces" den saknar utan att lyckas hitta något svar som funkar. Notera att det är en varning och inget error.

Kod: Markera allt

typedef struct
{
	unsigned char SenderID;
	unsigned char ReciverID;

}T_RF_Data_Struct;


union
{
	T_RF_Data_Struct RF_Data;
  	unsigned char Buffer[2];

}U_RF_Data = {0x00,0x00};

Ändrar jag till nedanstående blir det error
"../main.c:72: error: expected '=', ',', ';', 'asm' or '__attribute__' before '.' token"

Kod: Markera allt

typedef struct
{
	unsigned char SenderID;
	unsigned char ReciverID;

}T_RF_Data_Struct;


union
{
	T_RF_Data_Struct RF_Data;
  	unsigned char Buffer[2];

}U_RF_Data.RF_Data = {0x00,0x00};

[speakman]

Prova:

Kod: Markera allt

typedef struct
{
        unsigned char SenderID;
        unsigned char ReciverID;

}T_RF_Data_Struct;


union
{
        T_RF_Data_Struct RF_Data;
        unsigned char Buffer[2];

}U_RF_Data = {{0x00,0x00}};

[thepirateboy]

Kan nästan svära på att jag testat det förut, men fick bara error. Men nu när jag testar funkar det, hehe

Kod: Markera allt

typedef struct
{
	unsigned char SenderID;
	unsigned char ReciverID;

}T_RF_Data_Struct;


union
{
	T_RF_Data_Struct RF_Data;
  	unsigned char Buffer[2];

}volatile U_RF_Data = {{'x','u'}};   

Ovanstående kod genererar i AVR:en:

[TomasL]

Varför krångla med initieringen samtidigt som defineringen. finns ju inget att hämta.
Blir väl sannolikt mer svårläsligt, då man ofta har defineringen i .h filen eller någon annastans.

[Icecap]

Precis! Samtidig måste kompilern kasta in lite kod som kopierar från ROM till RAM, det kan fungera helt OK men jag gillar inte att ha kod med i mina projekt som inte jag har fullt koll på.

Men det är en fråga om smak och tycke.

[speakman]

Det var väl ändå för demonstrering initiering och definering var i samma kod. Annars kör man naturligtvis bara initiering per instansiering. Att defineringen alltid hamnar i en headerfil gäller ju bara om den ska vara nåbar från fler moduler än bara en.

[TomasL]

För läsbarhetens skull, även vid privata structar, även anonyma sådana samt tillfälliga är det dumt att ha initiering och definering på samma ställe.

Kod: Markera allt

void func ()
{
   
  #typedef a
  { 
      ....
  }
  kod 
  a=...;
}

Tycker personligen detta är bättre.
Ännu hellre att ha dem i separat .h fil, då har man allt på samma ställe, och slipper leta efter fanskapet.

var strucken hamnar har ingen betydelse, även om det är en global .h fil så spelar det ingen roll, eftersom ingenting händer föränn den defineras.
Personligen har jag alla #typedef i samma .h fil så jag enkelt kan hitta dem, sparar mycket letande.

[sodjan]

Nu är jag inte så hemma på AVR, men är av någon speciell anledning
som "U_RF_Data = {{'x','u'}}" ger "t" och "u" i Watch fönstret ?

[Icecap]

TomasL: "#typedef" ???? Menar du "typedef"???

Eller är "#typedef" en speciell kommando jag inte har uppmärksammat? *nyfiken*

[thepirateboy]

Nu är jag inte så hemma på AVR, men är av någon speciell anledning
som "U_RF_Data = {{'x','u'}}" ger "t" och "u" i Watch fönstret ?

Det är inget AVR-specifikt utan ett helt vanligt klipp o klistra fel.

Angående initiering så är förstås smaken som baken, det var mest intressant att se *hur* man kunde göra.