Функция вывода времени на жк. Опции

[ps1x]

Код

lcd_clear();
itoa(hour,temp);                
if (hour<10){lcd_putsf('0');lcd_puts(temp);}else{lcd_puts(temp);}
lcd_putsf(":");
itoa(min, temp);      
if (min<10){lcd_putsf('0');lcd_puts(temp);}else{lcd_puts(temp);}
lcd_putsf(":");
itoa(sec, temp);      
if (sec<10){lcd_putsf('0');lcd_puts(temp);}else{lcd_puts(temp);}

- по идее этот код должен выводить на экран время в формате ЧЧ:ММ:СС причем если число часов, минут или секунд меньше 10 то перед ним добавляется 0. Однако этого почему-то не происходит ((
Вместо этого если например реальное время - 08:23:09 то получается - 88:23:99

Где-же ошибка?

з.ы. Среда разработки - CVAVR

[jorikdima]

А чем lcdputsf от lcdputs отличается. Я думаю дело в последовательном использовании этих команд, го сам CVAVR не знаю.

[beer_warrior]

lcd_putsf('0')
lcd_putsf(":");
Так что же выводится символ или строка?

[zltigo]

lcd_putsf('0')
Так что же выводится символ или строка?

Символ конца строки :-) :-)

И что? Скажите компилятор не выругался когда ему в функции вместо указателя байт подсунули!
Ну хоть сообщения читайте!

[beer_warrior]

Ну хоть сообщения читайте!

Читаю, только вот по такому коду дельный ответ дать трудно.
Скорее всего напутано с буферами, а где фиг поймешь.

[ps1x]

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

Вот что написано в lcd.h

// write the string str located in SRAM to the LCD
void lcd_puts(char *str);
// write the string str located in FLASH to the LCD
void lcd_putsf(char flash *str);

Вот так объявленны переменные.
char *temp;
int year,month,day,hour,min,sec;

з.ы.
если написать lcd_putsf("Вот такие вот дела"); то строка выводится.

[e-moe]

void lcd_putsf(char flash *str) - вывод символов из FLASH памяти
void lcd_puts(char *str) - вывод из памяти данных

[ps1x]

Ну, я собственно это и написал, впрочем спасибо за перевод )

[beer_warrior]

Вот так объявленны переменные.
char *temp;
int year,month,day,hour,min,sec;

Надо писать FAQ по этому вопросу - каждую неделю кто-то накалываться. char *temp - это просто указатель в никуда - ручка от чашки - а самой чашки нет.
Надо писать:
char temp[размер строки], а потом передавать в функцию указатель -
т.е. адрес первого байта массива.
lcd_puts(temp); что эквивалентно lcd_puts( &temp[0]).

В качестве эксперимента можно попробовать :
lcd_puts( &temp[1]);что эквивалентно
temp++;
lcd_puts(temp);

Сообщение отредактировал beer_warrior - Apr 8 2006, 19:01

[zltigo]

Ну хоть сообщения читайте!

Читаю, только вот по такому коду дельный ответ дать трудно.
Скорее всего напутано с буферами, а где фиг поймешь.

Это не к Вам относилось :-( Я имел ввиду автора который НЕ ПРОЧИТАЛ предупреждающие сообщения
компилятора.
Извините за невнятно изложенную мысль.

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

ЭТО ПРАКТИЧЕСКИ НЕВЕРОЯТНО, если конечно все warnings не задешены на корню в
командной строке/проекте.

[Focus]

Для вывода одного символа лучше пользоваться оператором lcd_putchar
и вообще надо для преобразования числа в сторку использовать оператор sprintf.
например

char lcd_buuff[2]; // обявляем массив (это строка)

time=8;
sprintf(lcd_buff,"%-i",time);
if (time<10) lcd_buff[0]='0';
lcd_dotoxy(x,y);
lcd_puts(lcd_buff);

[ps1x]

Для вывода одного символа лучше пользоваться оператором lcd_putchar
и вообще надо для преобразования числа в сторку использовать оператор sprintf.
например

char lcd_buuff[2]; // обявляем массив (это строка)

time=8;
sprintf(lcd_buff,"%-i",time);
if (time<10) lcd_buff[0]='0';
lcd_dotoxy(x,y);
lcd_puts(lcd_buff);

Такой код приводит к заполнению всего дисплея (2х16) черными квадратами.

Проблема не решена.

[rezident]

sprintf не совсем хороша для вывода непосредственно на LCD, т.к. в конце преобразованной строки символов '\0' добавляет. Однако если сначала формируется буфер и только потом целиком выводится на LCD, то можно и sprintf использовать. Тогда и ведущие нули не нужно самому добавлять.

Код

pos+=sprintf(&lcd_buf[pos],"%02u:%02u:%02u", hour, min, sec);

преобразует переменные времени в строку с ведущими нулями, разделителем ':' и кроме того вернет количество символов, добавав их к текущему значению pos которое можно трактовать как позицию для смещения курсора или для добавления другой строки, что иногда тоже полезно.
Если формировать строку не нужно, то pos можно и не использовать.

Код

sprintf(lcd_buf,"%02u:%02u:%02u", hour, min, sec);

[ps1x]

sprintf не совсем хороша для вывода непосредственно на LCD, т.к. в конце преобразованной строки символов '\0' добавляет. Однако если сначала формируется буфер и только потом целиком выводится на LCD, то можно и sprintf использовать. Тогда и ведущие нули не нужно самому добавлять.

Код

pos+=sprintf(&lcd_buf[pos],"%02u:%02u:%02u", hour, min, sec);

преобразует переменные времени в строку с ведущими нулями, разделителем ':' и кроме того вернет количество символов, добавав их к текущему значению pos которое можно трактовать как позицию для смещения курсора или для добавления другой строки, что иногда тоже полезно.
Если формировать строку не нужно, то pos можно и не использовать.

Код

sprintf(lcd_buf,"%02u:%02u:%02u", hour, min, sec);

Тоже не работает!!! (виснет)
Может дело в том что я использую протеус?

[WHALE]

Что за камень?Если Tiny,то с функцией sprintf возникают проблемы.Ест много флэш и стек надо увеличи
чить по сравнению с выделяемым CV по умолчанию.