Метценгерштейн 0 29 апреля, 2015 Опубликовано 29 апреля, 2015 · Жалоба есть некая ф-я, void sendDataToLCD (uint8_t _pos, uint8_t _data) { if (_pos == 1) { switch (_data) { case 1: ht1621_wr_bit( LCD_SEGMENT_B_POS1_SEG, LCD_SEGMENT_B_POS1_COM, 1); ht1621_wr_bit( LCD_SEGMENT_C_POS1_SEG, LCD_SEGMENT_C_POS1_COM, 1); break; потом case 2, 3 и так далее будет. как делать, чтобы вместо POS1 или просто вместо 1 подставлять _pos ? сам LCD_SEGMENT_B_POS1_SEG - это дефайн Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
CrimsonPig 0 29 апреля, 2015 Опубликовано 29 апреля, 2015 · Жалоба есть некая ф-я, потом case 2, 3 и так далее будет. как делать, чтобы вместо POS1 или просто вместо 1 подставлять _pos ? сам LCD_SEGMENT_B_POS1_SEG - это дефайн написать кусочек кода (функцию, например), которая принимает аргументом uint8 _pos и в зависимости от него возвращает значение, которое надо передать первыми параметром в ht1621_wr_bit() это в данном конкретном случае. "потом case 2, 3 и так далее будет." Таки до скольки ? Сдается мне, что все это можно написать гораздо лучше. Если в коде вдруг начали появляться case 1: case 2: итп, которые делают почти одинаковые действия, то это обычно означаетт что что-то тут не то. Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
Метценгерштейн 0 29 апреля, 2015 Опубликовано 29 апреля, 2015 · Жалоба вывод на семисегментный индикатор. case от 0 до 9. А вот позиции от 1 до 4-х. И в дефайне меняется только POS1 на POS2... Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
Edit2007 3 30 апреля, 2015 Опубликовано 30 апреля, 2015 · Жалоба через локальную переменную. void sendDataToLCD (uint8_t _pos, uint8_t _data) { uint8_t ferst_param; if (_pos == 1) { ferst_param = LCD_SEGMENT_B_POS1_SEG; } else { ferst_param = LCD_SEGMENT_B_POS2_SEG; } //---- switch (_data) { case 1: ht1621_wr_bit( ferst_param, LCD_SEGMENT_B_POS1_COM, 1); ht1621_wr_bit( ferst_param, LCD_SEGMENT_C_POS1_COM, 1); break; или через массив uint8_t FerstParams[MAX_PARAM_NUM]={LCD_SEGMENT_B_POS1_SEG, LCD_SEGMENT_B_POS2_SEG, ...... }; void sendDataToLCD (uint8_t _pos, uint8_t _data) { if (_pos >= MAX_PARAM_NUM ) { return; // выход за границу масива } ferst_param = FerstParams[pos]; //---- switch (_data) { case 1: ht1621_wr_bit( ferst_param, LCD_SEGMENT_B_POS1_COM, 1); ht1621_wr_bit( ferst_param, LCD_SEGMENT_C_POS1_COM, 1); break; Можно пойти еще дальше и сделать двумерный массив, но как я понимаю количество вызовов ht1621_wr_bit для каждой цифры разное. Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
Метценгерштейн 0 30 апреля, 2015 Опубликовано 30 апреля, 2015 · Жалоба имею ввиду как-то через макрос что -ли. void sendDataToLCD (uint8_t _pos, uint8_t _data) { switch (_data) { case 1: ht1621_wr_bit( LCD_SEGMENT_B_POS(_pos )_SEG, LCD_SEGMENT_B_POS1_COM, 1); как-то так. Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
psL 0 30 апреля, 2015 Опубликовано 30 апреля, 2015 · Жалоба имею ввиду как-то через макрос что -ли. void sendDataToLCD (uint8_t _pos, uint8_t _data) { switch (_data) { case 1: ht1621_wr_bit( LCD_SEGMENT_B_POS(_pos )_SEG, LCD_SEGMENT_B_POS1_COM, 1); как-то так. #define LCD_SEGMENT_B_POS_SEG(_pos ) { ... (1<<_pos)... } либо #define LCD_SEGMENT_B_POS_SEG(_pos ) { ... ## _pos ... } Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
Edit2007 3 30 апреля, 2015 Опубликовано 30 апреля, 2015 · Жалоба #define LCD_SEGMENT_B_POS_SEG(_pos ) { ... (1<<_pos)... } В любом случае разбор параметра #define LCD_SEGMENT_B_POS_SEG(_pos ) { ... ## _pos ... } не уверен что сработает, _pos - аргумент функции, который меняется при выполнении программы, а все дефайны определяются на этапе компиляции, поэтому подстановка имени будет некорректной (мое мнение, может и ошибочное). Не люблю много макросов - запутаться можно. Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
xvr 12 30 апреля, 2015 Опубликовано 30 апреля, 2015 · Жалоба имею ввиду как-то через макрос что -ли. ... как-то так. как то так - никак :rolleyes: Макросы раскрываются С препроцессором, который работает до С компилятора и абсолютно независимо от него. Т.е. он (препроцессор) никак не может подставить вместо _pos его значение из параметра функции, просто потому что он не знает, что такое _pos - для него это просто кусочек текста, не более. Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
adnega 11 30 апреля, 2015 Опубликовано 30 апреля, 2015 · Жалоба имею ввиду как-то через макрос что -ли. Вы путаете момент компиляции и момент исполнения кода. Та как вы хотите будет выглядеть примерно так: #define SEG_POS(x) do{\ if(_pos == (x))\ {\ switch(_data)\ {\ case 1:\ ht1621_wr_bit( LCD_SEGMENT_B_POS##x##_SEG, LCD_SEGMENT_B_POS##x##_COM, 1);\ ht1621_wr_bit( LCD_SEGMENT_C_POS##x##_SEG, LCD_SEGMENT_C_POS##x##_COM, 1);\ break;\ }\ }}while(0) Вызывать так SEG_POS(1). Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
esaulenka 7 30 апреля, 2015 Опубликовано 30 апреля, 2015 · Жалоба У меня предложение - выкинуть ВСЁ и сделать правильно. Полез в гугл посмотреть, что такое этот HT1621, и наткнулся на весьма внятное описание у... ардуинщиков. Рекомендую . ht1621.writeMem придётся самому делать, но, кажется мне, он уже готов. Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться