Email: Password: Remember Me | Create Account (Free)

Back to Subject List

Old thread has been locked -- no new posts accepted in this thread
???
08/21/05 12:21
Read: times


 
#99770 - The compiler has told you the problem
Responding to: ???'s previous message
The compiler is telling you the problem. At a quick look, I cannot locate the lines that are causing the problem.

Where is the variable 'flag' declared?

Your code has many architectural problems - having global variables is one such issue.

this line looks suspicious 0.001AH: LCD_init();


Also, Why don't you write a lcd_pstring function rather than having individual calls for each character to write to the lcd!

void lcd_pstring(char *str)
{
while(*str)
{
lcd_write(1,*str++);
}
}

Why do you keep on calling lcd_init()? Write a routine to locate the cursor. It would make your code easier to read!

Write your code a separate routines that pass information to each other rather than rely on each other explicitly! This makes your code easier to read and to debug.
to print a string: lcd_pstring("Hello");



List of 4 messages in thread
TopicAuthorDate
Digital amp-hour meter is having errors            01/01/70 00:00      
   The compiler has told you the problem            01/01/70 00:00      
   Look and ye shall find!            01/01/70 00:00      
   get used to it, C does rarely post the e            01/01/70 00:00      

Back to Subject List