Преобразование массива в код (Си)

[-=FISHER=-]

Всех приветствую!

Есть задача, в зависимости от значений элементов массива (всего 8 элементов), заполнить переменную типа char ноликами и единицами. Если точнее, перебираем весь массив, если в элементе trig будет 1, значит в переменную code на место бита под номером i записываем 1, если 0 то ничего не делаем. Я написал это так:

for (i=0; i=7; i++)
{
  if(trig[i]==1){code|=(1<<i);}
}

У меня есть подозрения что это неверная запись. Подправьте меня пожалуйста если это так. 

[Реклама]

Реклама: ООО ТД Промэлектроника, ИНН: 6659197470, Тел: 8 (800) 1000-321

[kt118]

-

Изменено 11 декабря, 2017 пользователем kt118

[Реклама]

20% скидка на весь каталог электронных компонентов в ТМ Электроникс!

Акция "Лето ближе - цены ниже", успей сделать выгодные покупки!

Плюс весь апрель действует скидка 10% по промокоду APREL24 + 15% кэшбэк и бесплатная доставка!

Перейти на страницу акции

Реклама: ООО ТМ ЭЛЕКТРОНИКС, ИНН: 7806548420, info@tmelectronics.ru, +7(812)4094849

[Yurkin2015]

8 минут назад, -=FISHER=- сказал:

Подправьте меня

code = 0;
for (i = 0;  i < 8; i++)
{
  if(trig[i] == 1){ code |= (1 << i); }
}

 

[Реклама]

Выбираем схему BMS для корректной работы литий-железофосфатных (LiFePO4) аккумуляторов

 Обязательным условием долгой и стабильной работы Li-FePO4-аккумуляторов, в том числе и производства EVE Energy, является применение специализированных BMS-микросхем. Литий-железофосфатные АКБ отличаются такими характеристиками, как высокая многократность циклов заряда-разряда, безопасность, возможность быстрой зарядки, устойчивость к буферному режиму работы и приемлемая стоимость. Но для этих АКБ, также как и для других, очень важен контроль процесса заряда и разряда, а специализированных микросхем для этого вида аккумуляторов не так много. Инженеры КОМПЭЛ подготовили список имеющихся микросхем и возможных решений от разных производителей. Подробнее>>

Реклама: АО КОМПЭЛ, ИНН: 7713005406, ОГРН: 1027700032161

[ARV]

Только вот это (1 <<  i) работает достаточно медленно и порождает достаточно большой код. Я бы дополнил в целях оптимизации так:

code = 0;
uint8_t mask = 1;
for (i = 0;  i < 8; i++)
{
  if(trig[i] == 1){ code |= mask; }
  mask <<= 1;
}

 

[Yurkin2015]

14 минуты назад, ARV сказал:

в целях оптимизации

Ну, если полируем код, тогда можно обойтись и без вспомогательной маски

code = 0;
for(i = 0; i < 8; i++)
{
if(trig[i] == 1) code |= 0x80;
code >>= 1;
}

 

[-=FISHER=-]

1 час назад, Yurkin2015 сказал:

code = 0;
for (i = 0;  i < 8; i++)
{
  if(trig[i] == 1){ code |= (1 << i); }
}

 

Это опечатка..то что =7, должно быть ==7. А в остальном всё правильно?

[Yurkin2015]

5 минут назад, -=FISHER=- сказал:

должно быть ==7

не понял, что это?

[-=FISHER=-]

Только что, Yurkin2015 сказал:

не понял, что это?

У меня написано так for (i=0; i=7; i++) , а должно было быть вот так for (i=0; i==7; i++) 

[Yurkin2015]

Не, должно быть вот так:

for (i = 0;  i < 8; i++) 

[-=FISHER=-]

ОК

7 минут назад, Yurkin2015 сказал:

Не, должно быть вот так:

for (i = 0;  i < 8; i++) 

ОК

[Yurkin2015]

Ну, значит, договорились! 

И, это, зануляйте перед циклом обязательно

code = 0;

[-=FISHER=-]

1 час назад, Yurkin2015 сказал:

И, это, зануляйте перед циклом обязательно

code = 0;

Договорились

[ARV]

2 часа назад, Yurkin2015 сказал:

Ну, если полируем код, тогда можно обойтись и без вспомогательной маски

code = 0;
for(i = 0; i < 8; i++)
{
if(trig[i] == 1) code |= 0x80;
code >>= 1;
}

 

Я, конечно, дико извиняюсь, но если полировать слишком сильно, можно сточить все под ноль, как вы и сделали. Что там у вас на выходе в code будет?

Изменено 11 декабря, 2017 пользователем ARV

[Yurkin2015]

А, точно, маху дал, полируем дальше 

code = 0;
for(i = 0; i < 8; i++)
{
    code >>= 1;
    if(trig[i] == 1) code |= 0x80;
}

 

[Alex]

17 часов назад, ARV сказал:

Я бы дополнил в целях оптимизации так:

Смотря что оптимизировать. Если по скорости исполнения, то лучше будет так :

code=0;
if(trig[0] == 1) code |= (1<<0);
if(trig[1] == 1) code |= (1<<1);
if(trig[2] == 1) code |= (1<<2);
if(trig[3] == 1) code |= (1<<3);
...........

Или табличкой - будет как ракета. Не, не получится тут никак таблицу применять. Поторопился с ответом, с просони...

[ruhi]

 

Now this looks like a job for me 
So everybody just follow me 
Cuz we need a little controversy, 
Cuz it feels so empty without me 

                                        Eminem

code = 0;
for(int i = 0; i < 8; i++)
{
code <<= 1;
code |= (trig == 1);
}

[ARV]

Только что, ruhi сказал:

code |= (trig == 1);

Ох уж мне эти сишные изыски... сам я не большой любитель подобного, но раз пошла такая пьянка, тогда так ваще красивенько:

code |= !!trig[i];

 

[ruhi]

2 минуты назад, ARV сказал:

Ох уж мне эти сишные изыски... сам я не большой любитель подобного, но раз пошла такая пьянка, тогда так ваще красивенько:

code |= !!trig[i];

 

!!trig и

trig == 1; не одно и тоже,

если надо только единицу фильтровать не подойдет.

[Alex]

Кстати, в некоторых случаях (как в этом), при работе с массивами, можно увеличить скорость выполнения, применив указатель.

unsigned char *pTrig=trig;
code = 0;
for(unsigned char i = 0; i < 8; i++)
{
code <<= 1;
code |= (*trig++ == 1);
}

Такой код должен выполниться быстрее, чем такой :

code = 0;
for(unsigned char i = 0; i < 8; i++)
{
code <<= 1;
code |= (trig[i] == 1);
}

Естественно, речь идёт о всяческих 8-ми битниках, ущемлённых нормальным набором инструкций.

[ARV]

7 минут назад, ruhi сказал:

если надо только единицу фильтровать не подойдет

Да я вангую: топикстартеру надо строковое бинарное представление числа в байт превратить

[Alex]

9 минут назад, ARV сказал:

Да я вангую:

ИМХО, ванговать действия человека, который пишет :

20 часов назад, -=FISHER=- сказал:

У меня есть подозрения что это неверная запись.

бессмысленно. Там может оказаться всё, что угодно

[-=FISHER=-]

2 минуты назад, Alex сказал:

бессмысленно. Там может оказаться всё, что угодно

12 минуты назад, ARV сказал:

топикстартеру надо строковое бинарное представление числа в байт превратить

Верно.

 

 

[Alex]

2 минуты назад, -=FISHER=- сказал:

Верно.

Ну значит ARV'у +1 в карму, за развитую эстрасенсорику

[ARV]

1 час назад, -=FISHER=- сказал:

Верно

Если сие верно, то тогда сие не верно в принципе:

4 часа назад, Alex сказал:

trig[i] == 1

потому как верно сие:

trig[i] == '1';

А все из-за того, что вопрос задан изначально не так, как стоит задача...

Изменено 12 декабря, 2017 пользователем Alex

[-=FISHER=-]

6 минут назад, ARV сказал:

Если сие верно, то тогда сие не верно в принципе:

Есть массив из 8 элементов типа char, каждый элемент равен либо 1 либо 0. Так же есть переменная code типа char. Нужно сделать так чтобы из этого:

trig[8] = {1,0,0,1,1,0,1,1};

Получилось вот это:

code = 0b10011011;