2013-12-10 4 views
1

Я работаю над созданием библиотеки MODBUS. Библиотека отлично работает для тестирования. Я использую измеритель мощности на базе Modbus. У меня очень странная проблема. Попытка напечатать значение на консоли, используя printf. Выбрав первое значение бок о бок, я печатаю его, а второе значение выборки поступает в массив правильно, но при использовании кода pritnf get crashed. :( я пытался отладить код также и узнал об использовании Printf для значения второго временного кода ГЭТ разбилсяprintf вызывает сбой кода

Вот часть кода:.

int wmain(void) 
{ 
    FLOAT dataFloat[5] = {0}; 
    CHAR dataBuffer[200] = {0}; 
    // Initializing Structure declared in library header 
    psModbusRTU pModbus =(psModbusRTU)malloc(sizeof(psModbusRTU)); 

    // Array pointer to store data 
    BYTE *readData = (BYTE *) malloc(NUMBER_OF_READ_REGISTERS * sizeof(BYTE)); 
    memset(readData, 0, NUMBER_OF_READ_REGISTERS * sizeof(BYTE)); 

    // Function to read the voltage 
    if (!ModbusRTUReadRegister(pModbus, FC_READ_HOLDING_REGISTERS, VOLTAGE_REGISTER_ADDRESS, NUMBER_OF_READ_REGISTERS, readData)) 
    { 
     printf("\nERROR Read register error %d", GetLastError()); 
     getchar(); 
     return 0; 
    } 
    dataFloat[0] = hexToFloatPointConversion(readData); 
    printf("\nvoltage : %f", dataFloat[0]); //(THIS IS FIRST PRINTF TILL HERE CODE IS FINE) 

    /// Function reads the frequnecy 
    if (!ModbusRTUReadRegister(pModbus, FC_READ_HOLDING_REGISTERS, FREQUENCY_REGISTER_ADDRESS, NUMBER_OF_READ_REGISTERS, readData)) 
    { 
     printf("\nERROR Read register error %d", GetLastError()); 
     getchar(); 
     return 0; 
    } 
    dataFloat[1] = hexToFloatPointConversion(readData); // Saving frequency value in array 
    printf("\nFrequency : %f", dataFloat[1]); // (THIS IS WHERE PROBLEM ARISES CODE CRASH WITH NO REASON) 

hexToFloatPointConversion Функция:

FLOAT hexToFloatPointConversion(BYTE *readData) 
{ 
    DWORD dataHex = 0; 
    DWORD exponent = 0; 
    DWORD fraction = 0; 
    DWORD signBit = 0; 
    FLOAT dataFloat = 0; 

    /// Saving BYTE array data into one DWORD 
    dataHex = readData[0]; 
    dataHex = dataHex << 8; 
    dataHex = dataHex + readData[1]; 
    dataHex = dataHex << 8; 
    dataHex = dataHex + readData[2]; 
    dataHex = dataHex << 8; 
    dataHex = dataHex + readData[3];  

    /// Getting sign bit, exponent and fraction 
    signBit = dataHex >> 31; 
    exponent = (dataHex >> 23 & 0xFF); 
    fraction = dataHex & 0x7FFFFF; 

    /// Calculation for converting data into float. 
    dataFloat = (pow(2, (exponent - SINGLE_PRECISION_CONSTANT)) * (1 + (fraction * pow(2, -23)))); 

    return dataFloat; 
} 

psModbusRTU структура:

typedef struct sModbusRTU 
{ 
    HANDLE hModbus; 
    BYTE slaveNumber; 
    BYTE functionCode;   
    BOOL fModbusInitialize; 
    BOOL fSlaveAddress; 
    BOOL fModbusOpen; 
    BOOL fException; 
    WCHAR portName[16]; 
    DWORD exceptionData; 
    DCB  dcb; 
}sModbusRTU, *psModbusRTU; 

Пробовал отлаживать во время выполнения и посмотрел на точку разрыва сборки, но не понял, почему это происходит. Вот Дис сборки:

40050658 str   r3, [sp, #0x14] 
4005065C str   r5, [sp, #8] 
40050660 str   r2, [sp] 
40050664 str   r4, [sp, #4] 
40050668 bl   40031078 
4005066C __debugbreak_ce   //Break point 
40050670 mvn   r3, #0x37, 24 
40050674 eor   r3, r3, #0xAB 
40050678 ldr   r2, [r3] 
4005067C movw  r3, #0xF7F8 
40050680 movt  r3, #0xF101 

Пожалуйста, помогите мне, мой проект застрял в середине только из-за этой маленькой проблемы. Пожалуйста, ответьте обратно в ближайшее время

+0

Вы проверяете свой «readData» после второго вызова, если он содержит действительные данные или нет. И что бы вы вернули, если «hexToFloatPointConversion» терпит неудачу? – Abhineet

+0

Проблема может быть вокруг 'hexToFloatPointConversion (readData)', можно ли увидеть эту функцию? – Gabson

+0

Вы уверены, что «malloc» достаточно памяти? Единый регистр - 2 байта в Modbus. Возможно, вы должны сделать 'BYTE * readData = malloc (NUMBER_OF_READ_REGISTERS * 2);' предполагается, что 'sizeof (BYTE) == sizeof (char)' (== 1). – user694733

ответ

0

% е спецификатор ожидает форматирование double (8 байт), а не float (4 байта) (см this MSDN page for details. Попробуйте заливку значения double в ваших printf заявлениях, например printf("\nvoltage : %f", (double) dataFloat[0]);

+1

«Поплавок», переданный в функцию [Variadic] (http://en.wikipedia.org/wiki/Variadic_function#Example_in_C), автоматически повышается до 'double '. Это поощрение по умолчанию для параметров функции с ... или без прототипа C11dr §6.5.2.2 6. Использование 'printf()', '% f' спецификатор работает для' double', а также 'float'. – chux

+0

David LaPorte: Спасибо за ответ,% f не проблема, потому что, даже если я использую printf во второй раз, просто для того, чтобы напечатать простое целое число, используя код ошибки% d get crashed – Rajat

+0

chux: Yes ur right – Rajat

1

Неправильный размер передается malloc().

// was psModbusRTU pModbus =(psModbusRTU)malloc(sizeof(psModbusRTU)); 
// -----------------------------------------------v-------------- 
psModbusRTU pModbus = (psModbusRTU) malloc(sizeof(*psModbusRTU)); 

Предложить альтернативный malloc() идиома. Используйте sizeof(*target).
Кроме того, если вы не компилятор требует его (который не должен), рекомендуем уронить бросание

// example 
psModbusRTU *pModbus = malloc(sizeof(*pModbus)); 
+0

psModbusRTU является указателем на структуру Я также бросил кастинг, но не успел – Rajat

+0

@ Rajat вы добавили '*' в 'psModbusRTU pModbus = (psModbusRTU) malloc (sizeof (* psModbusRTU));'? – chux

+0

Да, я попробовал, получив ошибку '' malloc ': слишком мало аргументов для вызова' Я думаю, что это означает, что нужно лить компилятора – Rajat

Смежные вопросы