My code is about interfacing a temperature sensor with PIC18F4550 microcontroller and then it is to be sent using Xbee module ZB series 2 to a gateway ConnectportX2.

Can anyone please tell me if there is something wrong with my code regarding the data is there anything else i should add...i wasnt the xbee to read the values from port RA2 after its conversion to ASCII and then transmit it..anything to fix with the delays too!!??.

I am using MikroC pro for C coding

// LCD module connections
sbit LCD_RS at RB0_bit;
sbit LCD_EN at RB1_bit;
sbit LCD_D4 at RB4_bit;
sbit LCD_D5 at RB5_bit;
sbit LCD_D6 at RB6_bit;
sbit LCD_D7 at RB7_bit;

sbit LCD_RS_Direction at TRISB0_bit;
sbit LCD_EN_Direction at TRISB1_bit;
sbit LCD_D4_Direction at TRISB4_bit;
sbit LCD_D5_Direction at TRISB5_bit;
sbit LCD_D6_Direction at TRISB6_bit;
sbit LCD_D7_Direction at TRISB7_bit;
// End LCD module connections

// Define Messages
char message0[] = "LCD Initialized";
char message1[] = "Body Temperature";
// String array to store temperature value to display
char *tempC = "000.0";
// Variables to store temperature values
int ANSEL;
unsigned int tempinC;
unsigned long temp_value;

void Display_Temperature() {
// convert Temp to characters
if (tempinC/10000)
// 48 is the decimal character code value for displaying 0 on LCD
tempC[0] = tempinC/10000 + 48;
else tempC[0] = ' ';
tempC[1] = (tempinC/1000)%10 + 48; // Extract tens digit
tempC[2] = (tempinC/100)%10 + 48; // Extract ones digit
// convert temp_fraction to characters
tempC[4] = (tempinC/10)%10 + 48; // Extract tens digit
// print temperature on LCD
Lcd_Out(2, 1, tempC);

void main() {
ANSEL = 0b00000100;
ADCON0 = 0b01001100; // Connect AN2 to S/H, select Vref=1.19V
TRISB = 0b00000000; // PORTb All Outputs
TRISA = 0b00001110; // PORTA All Outputs, Except RA3 and RA2

UART1_Init(9600); // Initialize UART module at 9600 bps
Delay_ms(100); // Wait for UART module to stabilize

Lcd_Init(); // Initialize LCD
Lcd_Cmd(_LCD_CLEAR); // CLEAR display
Lcd_Cmd(_LCD_CURSOR_OFF); // Cursor off

Lcd_Out(1,1,message1); // Write message1 in 1st row

// Print degree character

// Different LCD displays have different char code for degree symbol
// if you see greek alpha letter try typing 178 instead of 223

do {
temp_value = ADC_Read(2);
temp_value = temp_value*48876;
tempinC = temp_value/10000;
tempinC = tempinC*10;


Delay_ms(1000); // Temperature sampling at 1 sec interval

if (UART1_Tx_Idle() == 1) {
} while(1);