Пишем свое API - Страница 9

cogen · 17.05.2016, 23:37

Вообще редко встречал реально полезного обработчика кодов возврата, вот признайтесь, что вы делаете с BME280_Error дальше? if(BME280_Error), угадал?

Easyrider83 · 17.05.2016, 23:43

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

cogen · 18.05.2016, 00:25

Фактически функция инициализации отвечает на уровне хост-протокола или устанавливает глобальные переменные, не используемые в остальных функциях драйвера, так? То есть логический уровень абстракции этой функции выше драйвера. Я бы в таком случае её вообще исключил вместе с её кодами возврата, развернув прямо в месте вызова.

Ar-Gen-Tum · 18.05.2016, 02:03

Сообщение от cogen

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

Да лан , гоните конкретно

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

cogen · 18.05.2016, 02:42

ага, могут быть, иногда. Только обычно разнообразным нештатным ситуациям назначают свой код возврата, в каждом модуле своя простынь с дефайнами или enum, типа на верхнем уровне обработаем ошибку, а по факту при вызове проверяется OK или неокей и по ситуации неокей что-то делается, а что за неокей - неважно.
Ну а раз так обычно происходит, может ну их эти коды возврата? Стараться писать так, чтобы в них не было нужды?

akegor · 18.05.2016, 06:33

Сообщение от cogen

ага, могут быть, иногда. Только обычно

А когда "необычно", то переписывать с самого "откуда ноги растут". Или, может, лучше дописать кусочек в нужном месте? Ну, тот самый обработчик? Не затрагивая идеологию?

omercury · 18.05.2016, 08:48

Глядишь - и с Фобосами дело наладится...

cogen · 18.05.2016, 08:56

Не уверен, что вы, Олег (akegor), поняли мою мысль. По сути - делать рефакторинг означает переписывать всё, пользуясь работающим кодом как спецификацией поведения. Делается это с целью упростить обслуживание, обеспечить дальнейшее развитие, повысить надежность, быстродействие, энергосбережение. Достигается это, в частности, путём анализа существующего кода и отбрасывания лишнего. Я утверждаю, что коды ошибок, возвращаемые многими функциями - это лишнее, поскольку они де-факто не анализируются. Я предлагаю оптимизацию путем отказа от возврата функциями кода ошибки и путем выбрасывания функций, общающихся с системой посредством кодов ошибок.

Easyrider83 · 18.05.2016, 09:26

У меня сделана некая структура подсчета ошибок. В ней есть группы, отвечающие за разные ошибки в разных блоках. Каждая группа содержит переменную типа u16, которая инкременируется с каждым появлением ошибки. Эту структуру получает хост в виде нулевого отчета, когда нет данных для передачи. И она же выводится на SD карту в виде лога. Все довольно топорно, но работает.
И еще я сделал вывод, что I2C мастер надо делать софтварно. Использовать переферию на STM в данном случае смысла никакого нет. Вот такой повод для холивара.

STM32F0 · 18.05.2016, 09:57

Сообщение от Easyrider83

И еще я сделал вывод, что I2C мастер надо делать софтварно. Использовать переферию на STM в данном случае смысла никакого нет. Вот такой повод для холивара.

"Не прошло и полжизни!"(с).
Об этом говорили ещё со времён расцвета ПИКов. Убогий протокол, какой уж тут может быть холивар?