turnon 1 18 марта, 2016 Опубликовано 18 марта, 2016 · Жалоба В одной из ячеек хранится float. Его корректность проверяется следующим образом: flash_data=flash_read(page_address+8); // Читаем OPTIONS_VAR.VoltageMAX=UintToFloat(flash_data); //Преобразуем во float if ((OPTIONS_VAR.VoltageMAX<12) || (OPTIONS_VAR.VoltageMAX>20)) // проверяем диапазон и если там некорректные данные то... Надуманное решение. А если там в результате сбоя окажется 13, а было сохранено 15? Делю настройки на отдельные подразделы и каждый раздел сопровождается CRC. Кроме того настройки хранятся в двух секторах, чтобы не потерять настройки при совпадении момента их записи и отключения питания. Плюс то же резервирование - не удалось прочитать из первого сектора - читаем из второго и после этого "обновляем" оба сектора (перезаписыаем). Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
jcxz 224 19 марта, 2016 Опубликовано 19 марта, 2016 · Жалоба по стилю написания: 1. никогда не используйте в тексте "магические числа", определяйте все такие величины заранее . В Вашем случае- неплохо иметь файл "defaults.h" и в нем дефайны вроде #define CONFIG_V_MIN 12 ; минимально допустимая величина напряжения в конфиге Добавлю и я свои 5 копеек. :-) flash_data=flash_read(page_address+8); // Читаем Никогда не пытайтесь выполнять работу за компилятора или линкёра! Они придуманы много десятилетий назад и с тех пор не нужно распределять байты в памяти руками, а всего лишь нужно написать struct и описать в нём желаемую структуру. И правильно указать компоновщику в какой регион памяти её линковать. Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться