Помощь - Поиск - Пользователи - Календарь
Полная версия этой страницы: Функция вывода времени на жк.
Форум разработчиков электроники ELECTRONIX.ru > Микроконтроллеры (MCs) > AVR
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
Цитата(beer_warrior @ Apr 8 2006, 20:33) *
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);
zltigo
Цитата(beer_warrior @ Apr 8 2006, 21:04) *
Цитата
Ну хоть сообщения читайте!

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

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


Цитата(ps1x @ Apr 8 2006, 21:10) *
Чесно говоря, компилятору все равно символ это или строка, кроме того он даже не заикнулся что где нибудь может быть ошибка...

ЭТО ПРАКТИЧЕСКИ НЕВЕРОЯТНО, если конечно все 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
Цитата(Focus @ Apr 8 2006, 23:15) *
Для вывода одного символа лучше пользоваться оператором 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
Цитата(rezident @ Apr 9 2006, 02:28) *
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
Цитата(WHALE @ Apr 9 2006, 10:58) *
Что за камень?Если 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,хотя компилер ни на что не ругался,а стека не хватало,портил рабочую область,т.к. вложенность там приличная.
beer_warrior
Я бы все-таки не рекомендовал бы пользоваться sprintf
дел на копейку, а памяти отожрет не по детски. В начале ветки приводился код который можно довести до ума. Да и понимания больше будет.
ps1x
Вот в том то и дело, что код, вроде бы должен работать, но не работает, а все попытки довести его до ума привели к "никчему". А sprintf глючит. С тем и сидим.

А размер стека если вы не заметили у меня 512 байт, думаю этого должно хватать....
WHALE
ну не знаю,если ресурсы проца позволяли,я неоднократно пользовался в CV sprintf,что позволялореша
ть 1 строкой кода сразу несколько задач,все работало.Какой-то глюк в компиляторе,связанный с выде
лением стека под нее есть,проявился на tiny2313,отловил в симуляторе AVRStudio и вылечил увеличе-
нием стека(совсем ненамного).Если вы уверены,что программа виснет именно на ней,прогоните код в
студии,если зависнет,увеличте размер стека.Что происходит,прекрасно видно при прогоне в дизассем-
блере.
jorikdima
Цитата(ps1x @ Apr 9 2006, 19:14) *
Вот в том то и дело, что код, вроде бы должен работать, но не работает, а все попытки довести его до ума привели к "никчему". А 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
Цитата(beer_warrior @ Apr 9 2006, 21:09) *
Цитата

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); что-то не то со сдигом или буфер маловат. Попробуйте посимвольно. И внимательно smile.gif
Готов поставить литр пива, что там просто банальный недосмотр.
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 он несложен
ps1x
Цитата(vet @ Apr 9 2006, 22:25) *
Да конечно.
lcd_putsf('0'); - что должно быть аргументом? а что есть?


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


И вот он - ПРАВИЛЬНЫЙ ОТВЕТ!!!!
Цитата
а так если
define lcd_buffer_size 16 //тут указывается длина строки вашего жки
char lcd_buf[lcd_buffer_size]; // обявляем массив (это строка вывода)
sprintf(lcd_buf,"%02u:%02u:%02u", hour, min, sec);
как вам уже советовали. далее собственно сам вывод-должно работать
ЗЫ.а ассмблер освоить очень желательно,у avr он несложен

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

З.ы. Помоему ассемблер для х86 гораздо проще (ИМХО), я его смутно помню правда.
zltigo
Цитата(ps1x @ Apr 10 2006, 00:31) *
И вот он - ПРАВИЛЬНЫЙ ОТВЕТ!!!!


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


Цитата
Надо писать FAQ по этому вопросу - каждую неделю кто-то накалываться. char *temp - это просто указатель в никуда - ручка от чашки - а самой чашки нет.
Надо писать:
char temp[размер строки], а потом передавать в функцию указатель -
т.е. адрес первого байта массива.
ps1x
Да нет, не правы Вы, я пробовал эти варианты, в частности
char lcd_buf[2];
и выводил в него каждое преобразование отдельно часов минут и секунд, но, увы, не сработало...
Я не игнорирую никаких высказываний по теме, и вдумываюсь в каждый пост, все равно, проблема то в конечном счете моя wink.gif
zltigo
Цитата(ps1x @ Apr 10 2006, 00:57) *
Да нет, не правы Вы, я пробовал эти варианты, в частности
char lcd_buf[2];
и выводил в него каждое преобразование отдельно часов минут и секунд, но, увы, не сработало...
Я не игнорирую никаких высказываний по теме, и вдумываюсь в каждый пост, все равно, проблема то в конечном счете моя wink.gif

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

Потом раз не "помогло" - НИКОГДА не будем писать какое-то непонятное число в
квадратных скобочках. Железная логика :-(
ps1x
Цитата(zltigo @ Apr 10 2006, 02:08) *
Цитата(ps1x @ Apr 10 2006, 00:57) *

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

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

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

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

Преобразование itoa(); преобразует int в *сhar. wink.gif
vet
Цитата(ps1x @ Apr 10 2006, 01:31) *
Цитата(vet @ Apr 9 2006, 22:25) *

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


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


Поясняю.
lcd_putsf принимает аргументом строку. Вы передавали в неё символ. Всё.
ps1x
Цитата(vet @ Apr 10 2006, 02:25) *
Цитата(ps1x @ Apr 10 2006, 01:31) *

Цитата(vet @ Apr 9 2006, 22:25) *

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


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


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


Как я уже говорил выше, я пробовал и такой вариант: lcd_putsf("0");
vet
В таком случае вопросов не имею.
Вообще, такие вещи успешно отлаживаются загрузкой программы в AVR Studio и внимательной трассировкой. Тему, в общем-то, и не стоило заводить. Имхо.
defunct
> ps1x
Цитата
Как я уже говорил выше, я пробовал и такой вариант: lcd_putsf("0");

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

ps: будьте внимательны к сообщениям компилятора, Warning'и игнорировать не стоит.
WHALE
Вы только не обижайтесь,но вам действительно надо прислушаться к советам местных гуру,если вы серьезно собираетесь заниматься разработками на AVR,т.к проблема у вас была очень простая и имея
под рукой книгу по С и владя ассемблером,все решилось бы очень быстро.
З.Ы. а без ассемблера я вообще слабо представляю,как можно работать-та-же отладка,написание кусков,критичных ко времени и т.д.
ps1x
Всем спасибо!
Для просмотра полной версии этой страницы, пожалуйста, пройдите по ссылке.
Invision Power Board © 2001-2025 Invision Power Services, Inc.