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

[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 по умолчанию.

[ps1x]

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

Mega8 в протеусе. С функцией sprintf и до этого были проблемы, насколько я помню...

[WHALE]

Увеличь размер стека до 64 байт,и все должно пойти.

[ps1x]

Data Stack area: 60h to 25Fh
Data Stack size: 512 byte(s)
Estimated Data Stack usage: 15 byte(s)

-- Это вроде бы нормально....

[WHALE]

Тогда может действительно протеус глючит?Или дальше в программе вывода ошибка?sptintf при таком
раскладе виснуть не должна.Проверь на симуляторе студиию.и на всякий случай чуть-чуть увеличь еще область стека,у меня
как-то был глюк,правда на тини-тоже висла программа на sprintf,хотя компилер ни на что не ругался,а стека не хватало,портил рабочую область,т.к. вложенность там приличная.

Сообщение отредактировал WHALE - Apr 9 2006, 10:36

[beer_warrior]

Я бы все-таки не рекомендовал бы пользоваться sprintf
дел на копейку, а памяти отожрет не по детски. В начале ветки приводился код который можно довести до ума. Да и понимания больше будет.

[ps1x]

Вот в том то и дело, что код, вроде бы должен работать, но не работает, а все попытки довести его до ума привели к "никчему". А sprintf глючит. С тем и сидим.

А размер стека если вы не заметили у меня 512 байт, думаю этого должно хватать....

[WHALE]

ну не знаю,если ресурсы проца позволяли,я неоднократно пользовался в CV sprintf,что позволялореша
ть 1 строкой кода сразу несколько задач,все работало.Какой-то глюк в компиляторе,связанный с выде
лением стека под нее есть,проявился на tiny2313,отловил в симуляторе AVRStudio и вылечил увеличе-
нием стека(совсем ненамного).Если вы уверены,что программа виснет именно на ней,прогоните код в
студии,если зависнет,увеличте размер стека.Что происходит,прекрасно видно при прогоне в дизассем-
блере.

[jorikdima]

Вот в том то и дело, что код, вроде бы должен работать, но не работает, а все попытки довести его до ума привели к "никчему". А sprintf глючит. С тем и сидим.

А размер стека если вы не заметили у меня 512 байт, думаю этого должно хватать....

А debugger то использовался? Может глянуть все по шагам.

[beer_warrior]

2 ps1x
Давайте вс сначала, есть код в начале топика. Цифирь выводит, только криво.Ниже показано в чем ошибка. Исправляете и смотрите на результаты. К sprintf вернемся попозже.

[ps1x]

Дык я ж пробовал и так и сяк в том числе и так, как написанно выше, там не менее результата три либо показывает 55:34:88 вместо 05:34:08 либо заполняет экран черными квадратиками, либо не показывает ничего (в случае с sprintf). К сожалению дебаггера нет, да и асма я не знаю толком.

[beer_warrior]

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) черными квадратами.

Это с предыдущей страницы.
К itoa не относиться.
Я еще раз повторюсь, вернитесь к исходному варианту и попробуйте снова.
Судя по всему после разнородных советов наделано ошибок немеряно.
Каждый новый вариант постите, что бы было понятно о чем идет речь.
(если хотите помощи)

[ps1x]

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) черными квадратами.

Это с предыдущей страницы.
К itoa не относиться.
Я еще раз повторюсь, вернитесь к исходному варианту и попробуйте снова.
Судя по всему после разнородных советов наделано ошибок немеряно.
Каждый новый вариант постите, что бы было понятно о чем идет речь.
(если хотите помощи)

В прошлый раз видимо в чем то ошибся.

Конкретно этот код выводит "0" и все. Так что не работает.

[beer_warrior]

Тогда проблема с lcd_puts(lcd_buff); что-то не то со сдигом или буфер маловат. Попробуйте посимвольно. И внимательно
Готов поставить литр пива, что там просто банальный недосмотр.

[vet]

Да конечно.
lcd_putsf('0'); - что должно быть аргументом? а что есть?

[WHALE]

а так если
define lcd_buffer_size 16 //тут указывается длина строки вашего жки
char lcd_buf[lcd_buffer_size]; // обявляем массив (это строка вывода)
sprintf(lcd_buf,"%02u:%02u:%02u", hour, min, sec);
как вам уже советовали. далее собственно сам вывод-должно работать
ЗЫ.а ассмблер освоить очень желательно,у avr он несложен

Сообщение отредактировал WHALE - Apr 9 2006, 18:37

[ps1x]

Да конечно.
lcd_putsf('0'); - что должно быть аргументом? а что есть?

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



И вот он - ПРАВИЛЬНЫЙ ОТВЕТ!!!!

а так если
define lcd_buffer_size 16 //тут указывается длина строки вашего жки
char lcd_buf[lcd_buffer_size]; // обявляем массив (это строка вывода)
sprintf(lcd_buf,"%02u:%02u:%02u", hour, min, sec);
как вам уже советовали. далее собственно сам вывод-должно работать
ЗЫ.а ассмблер освоить очень желательно,у avr он несложен

Пиво в студию!!!

З.ы. Помоему ассемблер для х86 гораздо проще (ИМХО), я его смутно помню правда.

[zltigo]

И вот он - ПРАВИЛЬНЫЙ ОТВЕТ!!!!

Потрясающая способность начисто игнорировать ответы на заданные вопросы.
"правильный ответ" до этого давали Вам уже минимум ДВАЖДЫ.

Только вот куда указывает temp и есть-ли там куда он указывает свободное место
для размещения получившейся строки.... По тексту это неинециализированный указатель при
записи по которому чего-нибудь и порушите, или, как минимум, в пустоту запишите и потом из этой
пустоты чего-нибудь прочитаете. Как минимум компилятор должен был выдать вполне определенные
ругательства на этот счет. Не стоило их игнорировать.
char temp[MAX_SIZE_OF_STRING+1];
Ну а вообще надо заставить себя книжку дочитать!

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

[ps1x]

Да нет, не правы Вы, я пробовал эти варианты, в частности
char lcd_buf[2];
и выводил в него каждое преобразование отдельно часов минут и секунд, но, увы, не сработало...
Я не игнорирую никаких высказываний по теме, и вдумываюсь в каждый пост, все равно, проблема то в конечном счете моя

[zltigo]

Да нет, не правы Вы, я пробовал эти варианты, в частности
char lcd_buf[2];
и выводил в него каждое преобразование отдельно часов минут и секунд, но, увы, не сработало...
Я не игнорирую никаких высказываний по теме, и вдумываюсь в каждый пост, все равно, проблема то в конечном счете моя

Бездумные пробы :-( не увенчались успехом, поскольку:
1) фиг знает что Вы делали за "преобразования";
2) буфер должен быть на _ТРИ_ байта.

Потом раз не "помогло" - НИКОГДА не будем писать какое-то непонятное число в
квадратных скобочках. Железная логика :-(

[ps1x]

Да нет, не правы Вы, я пробовал эти варианты, в частности
char lcd_buf[2];
и выводил в него каждое преобразование отдельно часов минут и секунд, но, увы, не сработало...
Я не игнорирую никаких высказываний по теме, и вдумываюсь в каждый пост, все равно, проблема то в конечном счете моя

Бездумные пробы :-( не увенчались успехом, поскольку:
1) фиг знает что Вы делали за "преобразования";
2) буфер должен быть на _ТРИ_ байта.

Потом раз не "помогло" - НИКОГДА не будем писать какое-то непонятное число в
квадратных скобочках. Железная логика :-(

Почему непонятное, очень даже понятное char lcd_buf[2]; - фактически, это массив из char'ов размерностью 2, а 3 их там надо, поскольку строка в С всегда должна завершаться \0. ))

Преобразование itoa(); преобразует int в *сhar.

[vet]

Да конечно.
lcd_putsf('0'); - что должно быть аргументом? а что есть?

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

Поясняю.
lcd_putsf принимает аргументом строку. Вы передавали в неё символ. Всё.

[ps1x]

Да конечно.
lcd_putsf('0'); - что должно быть аргументом? а что есть?

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

Поясняю.
lcd_putsf принимает аргументом строку. Вы передавали в неё символ. Всё.

Как я уже говорил выше, я пробовал и такой вариант: lcd_putsf("0");

[vet]

В таком случае вопросов не имею.
Вообще, такие вещи успешно отлаживаются загрузкой программы в AVR Studio и внимательной трассировкой. Тему, в общем-то, и не стоило заводить. Имхо.

[defunct]

> ps1x

Как я уже говорил выше, я пробовал и такой вариант: lcd_putsf("0");

врать нехорошо.

ps: будьте внимательны к сообщениям компилятора, Warning'и игнорировать не стоит.

[WHALE]

Вы только не обижайтесь,но вам действительно надо прислушаться к советам местных гуру,если вы серьезно собираетесь заниматься разработками на AVR,т.к проблема у вас была очень простая и имея
под рукой книгу по С и владя ассемблером,все решилось бы очень быстро.
З.Ы. а без ассемблера я вообще слабо представляю,как можно работать-та-же отладка,написание кусков,критичных ко времени и т.д.

[ps1x]

Всем спасибо!