program in mikro ..current part has some problem.can u check?

DIDNT GET "The documentation tells that if you use floattostr(0 then the string should be 17 to 23 charcaters i.e., val[17] and val1[17]. The other thing is you don't have to write val[5] = '\0'. "

Click to expand...

Code C - [expand]
1
val[5] = '\0'

Code C - [expand]
1
val1[5] = '\0'

jayanth.devarayanadurga said:

I don't remember, but it is in some mikroC documentation... It is mentioned that if you use floattostr() function the string variable used must be declared with a minimum of 17 bytes. You have declared strings val[5] and val1[5]. You have to change it to val[17] and val1[17] in the variable declaration. Another thing is val[5] and val1[5] has 5 elements from 0 to 4, So, val[5] and val1[5] doesn't exist.

So, you can't use

---

Code C - [expand]
1
val[5] = '\0'

---

and

---

Code C - [expand]
1
val1[5] = '\0'

---

Are you using mikroC or mikroC Pro? I think you are using the old version of mikroC v8 or v8.1 or v8.2. Am I right? If yes, then you are also using 8 bit mode for LCD and So, you have to use lcd8 library. You have used PORTD for LCD. How are the lcd pins connected to portd? I think your have used lcd_init() function incorrectly. See the mikroC help for lcd_init() and lcd8_init() functions.

Click to expand...

jayanth.devarayanadurga said:

I don't remember, but it is in some mikroC documentation... It is mentioned that if you use floattostr() function the string variable used must be declared with a minimum of 17 bytes. You have declared strings val[5] and val1[5]. You have to change it to val[17] and val1[17] in the variable declaration. Another thing is val[5] and val1[5] has 5 elements from 0 to 4, So, val[5] and val1[5] doesn't exist.

So, you can't use

---

Code C - [expand]
1
val[5] = '\0'

---

and

---

Code C - [expand]
1
val1[5] = '\0'

---

Are you using mikroC or mikroC Pro? I think you are using the old version of mikroC v8 or v8.1 or v8.2. Am I right? If yes, then you are also using 8 bit mode for LCD and So, you have to use lcd8 library. You have used PORTD for LCD. How are the lcd pins connected to portd? I think your have used lcd_init() function incorrectly. See the mikroC help for lcd_init() and lcd8_init() functions.

Click to expand...

unsigned float adcval,adcval1,V1;
unsigned float Is,I,I1,Is1,w1,e;
char val[7];char val1[7];
char val2[7],val3[7];
void main()
{
e=0;
w1=0;
TRISA=0XFF;
TRISD=0X00;
ADCON1=0X80;
lcd_init(&PORTD);
lcd_cmd(lcd_clear);
lcd_cmd(lcd_cursor_off);
 while(1)
 {
adcval=Adc_Read(0)*.0048;
Is=adcval/100;
I=Is/2.5;
V1=I*5000;
floattostr(V1,val);
val[5]='\0';
adcval1=Adc_Read(1)*.0048;
Is1=adcval1/100;
I1=Is1*1000;
floattostr(I1,val1);
val1[5]='\0';
lcd_out(1,1,"V =");
lcd_out(1,4,val);
lcd_chr(1,10,'V');
lcd_out(1,12,"I=");
lcd_out(1,15,val1);
lcd_chr(1,16,'A');
w1=I1*V1;
floattostr(w1,val2);
val2[5]='\0';
lcd_out(2,1,"P =");
lcd_out(2,4,val2);
lcd_chr(2,10,'W');
e=e+(w1*1000);
floattostr(e,val3);
val3[5]='\0';
lcd_out(2,12,"E =");
lcd_out(2,15,val3);
lcd_chr(2,16,'W');
delay_ms(1000);
}
}

jayanth.devarayanadurga said:

Ok. You are right. I thought you had declared val[5] and val1[5] but you have declared it as val[7] and val1[7]. So, val[5] = '\0' and val1[5] = '\0' is right. Are you getting values of V and I correctly? Can you zip and post your proteus .dsn file and also the mikroc files with both hex and cof files?

Edit: You have to use CMCON = 0x07; to disable the Comparators on PORTA. See datasheet CMCON should be 0b00000111 to disable both Comparators.

Click to expand...

unsigned float adcval,adcval1,V1,Is,I,I1,Is1,power,energy;
char val[7];
void main()
{
energy  = 0;
TRISA=0XFF;
TRISD=0X00;
ADCON1=0X80;
lcd_init(&PORTD);
lcd_cmd(lcd_clear);
lcd_cmd(lcd_cursor_off);

lcd_out(1,1,"V=");

lcd_out(1,9,"I=");

lcd_out(2,1,"E=");
 while(1)
{
adcval=Adc_Read(0)*.0048;
adcval1=Adc_Read(1)*.0048;
Is=adcval/100;
I=Is/2.5;
V1=I*5000;
floattostr(V1,val);
val[5]='\0';
lcd_out(1,3,val);
Is1=adcval1/100;
I1=Is1*1000;
floattostr(I1,val);
val[5]='\0';
lcd_out(1,11,val);
power = V1 * I1;
//floattostr(power,p_val);
//p_val[5]='\0';
//lcd_out(2,3,p_val);
energy =(energy + power)*(5/18);
floattostr(energy,val);
//val[5]='\0';
lcd_out(2,3,val);
delay_ms(1000);
}
}