Помощь - Поиск - Пользователи - Календарь
Полная версия этой страницы: как правильно обработать ф-ю с дефайном?
Форум разработчиков электроники ELECTRONIX.ru > Сайт и форум > В помощь начинающему > Программирование
Метценгерштейн
есть некая ф-я,
Код
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
Цитата(Метценгерштейн @ Apr 29 2015, 15:50) *
есть некая ф-я,
потом case 2, 3 и так далее будет.
как делать, чтобы вместо POS1 или просто вместо 1 подставлять _pos ?
сам LCD_SEGMENT_B_POS1_SEG - это дефайн


написать кусочек кода (функцию, например), которая принимает аргументом uint8 _pos и в зависимости от него возвращает значение, которое надо передать первыми параметром в ht1621_wr_bit() это в данном конкретном случае.

"потом case 2, 3 и так далее будет."
Таки до скольки ? Сдается мне, что все это можно написать гораздо лучше. Если в коде вдруг начали появляться case 1: case 2: итп, которые делают почти одинаковые действия, то это обычно означаетт что что-то тут не то.

Метценгерштейн
вывод на семисегментный индикатор. case от 0 до 9. А вот позиции от 1 до 4-х. И в дефайне меняется только POS1 на POS2...
редактор
через локальную переменную.
Код
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 для каждой цифры разное.
Метценгерштейн
имею ввиду как-то через макрос что -ли.
Код
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
Цитата(Метценгерштейн @ Apr 30 2015, 09:06) *
имею ввиду как-то через макрос что -ли.
Код
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 ... }
редактор
Цитата
#define LCD_SEGMENT_B_POS_SEG(_pos ) { ... (1<<_pos)... }

В любом случае разбор параметра

Цитата
#define LCD_SEGMENT_B_POS_SEG(_pos ) { ... ## _pos ... }

не уверен что сработает, _pos - аргумент функции, который меняется при выполнении программы, а все дефайны определяются на этапе компиляции, поэтому подстановка имени будет некорректной (мое мнение, может и ошибочное).
Не люблю много макросов - запутаться можно.
XVR
Цитата(Метценгерштейн @ Apr 30 2015, 09:06) *
имею ввиду как-то через макрос что -ли.
...
как-то так.
как то так - никак rolleyes.gif
Макросы раскрываются С препроцессором, который работает до С компилятора и абсолютно независимо от него. Т.е. он (препроцессор) никак не может подставить вместо _pos его значение из параметра функции, просто потому что он не знает, что такое _pos - для него это просто кусочек текста, не более.
adnega
Цитата(Метценгерштейн @ Apr 30 2015, 09:06) *
имею ввиду как-то через макрос что -ли.

Вы путаете момент компиляции и момент исполнения кода.

Та как вы хотите будет выглядеть примерно так:
Код
#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
У меня предложение - выкинуть ВСЁ и сделать правильно.

Полез в гугл посмотреть, что такое этот HT1621, и наткнулся на весьма внятное описание у... ардуинщиков. Рекомендую .

ht1621.writeMem придётся самому делать, но, кажется мне, он уже готов.
Для просмотра полной версии этой страницы, пожалуйста, пройдите по ссылке.
Invision Power Board © 2001-2025 Invision Power Services, Inc.