Форум разработчиков электроники ELECTRONIX.ru > как правильно обработать ф-ю с дефайном?

Помощь - Поиск - Пользователи - Календарь

Полная версия этой страницы: как правильно обработать ф-ю с дефайном?

Форум разработчиков электроники ELECTRONIX.ru > Сайт и форум > В помощь начинающему > Программирование

Метценгерштейн

Apr 29 2015, 14:50

есть некая ф-я,

Код

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:01

Цитата(Метценгерштейн @ 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: итп, которые делают почти одинаковые действия, то это обычно означаетт что что-то тут не то.

Метценгерштейн

Apr 29 2015, 18:14

вывод на семисегментный индикатор. case от 0 до 9. А вот позиции от 1 до 4-х. И в дефайне меняется только POS1 на POS2...

редактор

Apr 30 2015, 05:35

через локальную переменную.

Код

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 для каждой цифры разное.

Метценгерштейн

Apr 30 2015, 06: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);

как-то так.

psL

Apr 30 2015, 06:19

Цитата(Метценгерштейн @ Apr 30 2015, 09:06)

имею ввиду как-то через макрос что -ли.

Код

как-то так.

Код

#define LCD_SEGMENT_B_POS_SEG(_pos ) { ... (1<<_pos)... }

либо

Код

#define LCD_SEGMENT_B_POS_SEG(_pos ) { ... ## _pos ... }

редактор

Apr 30 2015, 09:17

Цитата

#define LCD_SEGMENT_B_POS_SEG(_pos ) { ... (1<<_pos)... }

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

Цитата

#define LCD_SEGMENT_B_POS_SEG(_pos ) { ... ## _pos ... }

не уверен что сработает, _pos - аргумент функции, который меняется при выполнении программы, а все дефайны определяются на этапе компиляции, поэтому подстановка имени будет некорректной (мое мнение, может и ошибочное).
Не люблю много макросов - запутаться можно.

XVR

Apr 30 2015, 10:02

Цитата(Метценгерштейн @ Apr 30 2015, 09:06)

имею ввиду как-то через макрос что -ли.
...
как-то так.

как то так - никак

Макросы раскрываются С препроцессором, который работает до С компилятора и абсолютно независимо от него. Т.е. он (препроцессор) никак не может подставить вместо _pos его значение из параметра функции, просто потому что он не знает, что такое _pos - для него это просто кусочек текста, не более.

adnega

Apr 30 2015, 10:05

Цитата(Метценгерштейн @ 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

Apr 30 2015, 10:23

У меня предложение - выкинуть ВСЁ и сделать правильно.

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

ht1621.writeMem придётся самому делать, но, кажется мне, он уже готов.

Для просмотра полной версии этой страницы, пожалуйста, пройдите по ссылке.