Слой Абстракции Для Spi

[COKPOWEHEU]

Написал слой абстракции для мастера SPI, как программного так и аппаратного. Кому не лень посмотрите что можно исправить или доработать.

Единственное, что мне не нравится, что avr-gcc ругается warning'ом на неинициализированную переменную. Я-то знаю, что после 9 сдвигов в ней не останется ни одного родного бита, но как это объяснить компилятору?

char SPI_send(char data){
 char res,i=7;

spi.zip

[Реклама]

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

[Vascom]

Так может инициализировать переменную?

[Реклама]

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

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

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

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

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

[Alex]

А почему бы её не инициализировать ?

ЗЫ: Опоздал

Изменено 25 июля, 2016 пользователем Alex

[Реклама]

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

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

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

[COKPOWEHEU]

А зачем, если это значение все равно сотрется? Если не получится вразумить компилятор, так и сделаю, но неужели нет другого способа?

[Vascom]

Затем, что это вырабатывает правильные навыки программирования. Компилятор лишь предупреждает, что так делать нехорошо, именно поэтому всего лишь Warning, а не Error.

Если тебе не важно начальное значение - инициализируй нулём. Предупреждение пропадёт и всё будет работать как надо.

Именно таков верный способ вразумить компилятор!

[Alex]

Вообще, он правильно ругается. Есть вероятность, что из функции вернётся неинициализированное значение.

В SPI_IN(x) , в условии проверки линии, поставь else. Установка бита в "x" стоит, а сброса нет. Это даже ошибка

Или инициализируй переменную нулём.

[Стальной]

В *.h (заголовочных) файлах реализации функций не пишут! Там только прототипы, а в отдельном *.c файле реализации.

[Alex]

Ерунда. С таким успехом, можно сказать, что .c файлы не подключаются includ'ом.

Если .h файл подключается всего в одном месте, то почему бы там не реализовывать код и не объявлять переменные ?

[Стальной]

Потому, что в случае подключения в разных местах будет ошибка.

Как говорилось выше, хороший стиль программирования

[Alex]

В разных местах - да, будет. И то, не всегда. Возможны просто создания одинаковых экземпляров.

Но, как я уже выше сказал :

Если .h файл подключается всего в одном месте, то почему бы там не реализовывать код и не объявлять переменные ?

[Vascom]

Самый простой и точный ответ - потому, что это создает возможность появления проблем и прививает неправильные навыки.

Ты сюда за помощью пришел или спорить? Тебе говорят как улучшить код, избавиться от ворнинга. Зачем делать неправильно?

[Alex]

Ты сюда за помощью пришел или спорить? Тебе говорят как улучшить код, избавиться от ворнинга. Зачем делать неправильно?

Это кому адресовано ?

[Vascom]

А, не посмотрел на имена, с телефона писал, сорри.

Тогда можете считать это размышлениями вслух.

[COKPOWEHEU]

Установка бита в "x" стоит, а сброса нет. Это даже ошибка

Вроде как Си гарантирует заполнение сдвинутого бита нулем, зачем его еще раз присваивать?

В *.h (заголовочных) файлах реализации функций не пишут! Там только прототипы, а в отдельном *.c файле реализации.

Специально для этого предусмотрены макроопределения MULTISOURCE_PROJECT и HEADER_IMPLEMENTATION, позволяющие при надобности отключить реализацию. Собственно, единственное почему не рекомендуется в *.h файл включать реализацию - дублирование функций с одинаковыми именами в разных *.o файлах, а линкер потом с ума сходит не зная какой использовать. С другой стороны, для проверки какой-нибудь мелочи подключать к проекту лишний *.c файл, переконфигурировать его и т.п. не оправдано. Поэтому и пошел на такой компромисс - если этот модуль используется только в одном файле, реализация подключена, если в нескольких - в настройках указывается MULTISOURCE_PROJECT, а в одном любом *.c файле (проще всего, наверное, main.c) HEADER_IMPLEMENTATION. На тестовых примерах это работает, будет ли удобно в реальности - пока не знаю.

Затем, что это вырабатывает правильные навыки программирования. Компилятор лишь предупреждает, что так делать нехорошо

Нет, компилятор не знает, что начальное значение в данном случае ни на что не повлияет. Так что, нет никакого атрибута или еще какого указания компилятору "я знаю про эту особенность, это не баг а фича"?

[Alex]

Вроде как Си гарантирует заполнение сдвинутого бита нулем, зачем его еще раз присваивать?

Это писателю известно, а компилятор считает, что переменная может не измениться.

Всё-таки, ради интереса, попробуй else добавить.

[COKPOWEHEU]

Ничего не изменилось. На счет неинициализированной переменной наверное и правда не стоит извращаться и инициализировать, пусть компилятор спит спокойно.