Frequencies again!

[mgh]

Why?

This code returns FreqMhz = 128.30 which is correct:

double FreqMHz;
.
.
.
struct AIRCRAFT_RADIO
{
double FreqCom1A;
};
.
.
.
SimConnect_AddToDataDefinition(hSimConnect,
                               DATA_AIRCRAFT_RADIO,
                               "COM ACTIVE FREQUENCY:1",
                               "MHz");
.
.
.
switch (pObjData->dwRequestID)
    {
 case REQ_AIRCRAFT_RADIO:
        {                                                           
        // Cast void pointer to point to new data structure
        AIRCRAFT_RADIO *pS = (AIRCRAFT_RADIO *)&pObjData->dwData;

        // Extract data from structure
        FreqMHz = pS->FreqCom1A;
.
.
.

Changing to this (with changes highlighted in red) returns FreqBCD = 0

[COLOR="Red"]short int FreqBCD;
.[/COLOR].
.
.
.
.struct AIRCRAFT_RADIO
{
[COLOR="red"]short int FreqCom1A;[/COLOR]
};
.
.
.
SimConnect_AddToDataDefinition(hSimConnect,
                               DATA_AIRCRAFT_RADIO,
                               "COM ACTIVE FREQUENCY:1",
                               [COLOR="red"]"Frequency BCD16");[/COLOR]
.
.
.
switch (pObjData->dwRequestID)
    {
    case REQ_AIRCRAFT_RADIO:
        {                                                           
        // Cast void pointer to point to new data structure
        AIRCRAFT_RADIO *pS = (AIRCRAFT_RADIO *)&pObjData->dwData;

        // Extract data from structure
       [COLOR="red"] FreqBCD = pS->FreqCom1A;[/COLOR]
.
.
.

 

[ddawson]

Why?

This code returns FreqMhz = 128.30 which is correct:

Changing to this (with changes highlighted in red) returns FreqBCD = 0

SimConnect_AddToDataDefinition(hSimConnect,
                               DATA_AIRCRAFT_RADIO,
                               "COM ACTIVE FREQUENCY:1",
                               [COLOR="red"]"Frequency BCD16");[/COLOR]

Try:
SimConnect_AddToDataDefinition(hSimConnect,DATA_AIRCRAFT_RADIO,"COM ACTIVE FREQUENCY:1","Frequency BCD16", SIMCONNECT_CLIENTDATATYPE_INT16);

SimConnect_AddToDataDefinition defaults to requesting information in a FLOAT64 format if you don't ask it to do otherwise.

Doug

 

[mgh]

Thanks. I tried that without success. It results in pObjData->dwRequestID always being equal to zero so it never now goes beyond the switch statement. The following snippet shows the revised statements in red adjacent to the initial, working statements in green now commented out.

[COLOR="SeaGreen"]]//double FreqMHz;[/COLOR]
[COLOR="Red"]unsigned short FreqBCD;[/COLOR]

struct AIRCRAFT_RADIO
{
[COLOR="seagreen"]//double FreqCom1A;[/COLOR]
[COLOR="Red"]unsigned short FreqCom1A;[/COLOR]
};



SimConnect_AddToDataDefinition(hSimConnect,
                               DATA_AIRCRAFT_RADIO,
                               "COM ACTIVE FREQUENCY:1",
                               [COLOR="seagreen"]//"MHz");[/COLOR]
                             [COLOR="red"]  "Frequency BCD16",
                               SIMCONNECT_CLIENTDATATYPE_INT16);[/COLOR]

SIMCONNECT_RECV_SIMOBJECT_DATA *pObjData =  (SIMCONNECT_RECV_SIMOBJECT_DATA *) pData;

// Get object ID
//DWORD ObjectID = pObjData->dwObjectID;

// Switch based on Request ID
switch (pObjData->dwRequestID)
    {
    case REQ_AIRCRAFT_RADIO:
        {                                                           
        // Cast void pointer to point to new data structure
        AIRCRAFT_RADIO *pS = (AIRCRAFT_RADIO *)&pObjData->dwData;

        // Extract data from structure
       [COLOR="seagreen"] // FreqMHz = pS->FreqCom1A;[/COLOR]
      [COLOR="red"] FreqBCD = pS->FreqCom1A;[/COLOR]

 

[ddawson]

Gerry,

This is working:

short int iFreq;

static enum DATA_DEFINE_ID {
  DATA_DEFINITION_PLANE_POSITION=128,
  DATA_DEFINITION_AIRCRAFT_RADIO
};

static enum DATA_REQUEST_ID{
  DATA_REQUEST_PLANE_POSITION=192,
  DATA_REQUEST_AIRCRAFT_RADIO,
};

void OnRecvOpen (SIMCONNECT_RECV_OPEN *pOpen)
{
SimConnect_AddToDataDefinition(hSimConnect,DATA_DEFINITION_AIRCRAFT_RADIO,"COM ACTIVE FREQUENCY:1","Frequency BCD16",SIMCONNECT_DATATYPE_INT32);
SimConnect_RequestDataOnSimObject(hSimConnect, DATA_REQUEST_AIRCRAFT_RADIO, DATA_DEFINITION_AIRCRAFT_RADIO, SIMCONNECT_OBJECT_ID_USER, SIMCONNECT_PERIOD_SIM_FRAME, SIMCONNECT_DATA_REQUEST_FLAG_CHANGED);
}

void OnRecvSimobjectData(SIMCONNECT_RECV_SIMOBJECT_DATA *pObjData, DWORD cbData)
{
switch( pObjData->dwRequestID )
  {
  case DATA_REQUEST_AIRCRAFT_RADIO:
  iFreq = *(short int*)&pObjData->dwData;
  break;
  }
}

 

[mgh]

Thanks. It has to be SIMCONNECT_DATATYPE_INT32 even though unsigned short is 16 bits.

EDIT

It's now become a bit academic because I've just realised I can extract the digits more easily from a double FreqMHz.

String s = FloatToStrF(FreqMHz, ffFixed, 6, 2);

char Digit1 = s[1];
char Digit2 = s[2];
...