o615ye161 писал(а): ↑12.11.2020{, 20:35}
Добрый вечер ecoins, я исправил ваши замечания по проекту и немного его еще дополнил могли бы вы еще раз на него взглянуть? Может что то еще посоветуете как оптимизировать проект, очень полезно будет мнение профессионала!
Посмотрел Ваш проект. В целом нормально. Есть небольшие замечания:
1.Рекомендую создавать первую плату, которая через условие выполняется один для инициализации параметров. Назовем её "ИНИЦИАЛИЗАЦИЯ".
2.У Вас на второй плате к выводам структуры привязываются контакты со своими именами. Очень полезная конструкция, в том числе и потому, что в отличии от строгих требованиях к именам параметрам структур, имена контактов могут быть любыми - текст на русском, пробелы, точки и т.п.
При этом полезно понимать, что такая конструкция не вызывает выполнение каких-то процессорных команд - это всего-лишь дополнительная привязка имен контактов к уже имеющимся переменным (в описываемом примере это делается при первом объявлении структуры).
Рекомендую:
2.1. Все используемые в проекте структуры определять в плате "ИНИЦИАЛИЗАЦИЯ" и к выходам этих структур сразу привязать контакты с удобными именами.
2.2. В проекте, когда требуется присвоить значение элементам структуры, устанавливать блок структуры, который уже определен в плате "ИНИЦИАЛИЗАЦИЯ".
Такой подход улучшает читаемость проекта и защищает от целого ряда неприятностей при проектировании.
3.На плате 20,21,22,23 у Вас 4-е блока ds1820, подключенные к разным пинам. Эти платы можно свести на одну плату, времена опроса блоков сделать немного отличающимися и некратными периоду вызова плат, например 1005,1074,1185,1296. Проект получиться компактнее. Такая плата может вызываться и как свободная, оперативная и быстрая. Рекомендую как "быстрая"
4. На плате 13 идет присвоение параметров структуры по сенсору, подключенном удаленно через UART. Выделять это в отдельную плату совершенно излишне - следует объединить платой 4, где читаются данные из UART.
5. На плате 5 возможно не будет как задумано работать блок "Buzzer" (возможно не прав);
------
Успехов.