GyverLibs / microDS18B20

Легкая и удобная в обращении библиотека для работы с 1-Wire термометрами DS18B20
MIT License
46 stars 11 forks source link

Не Проблема, но вопрос ) #23

Closed vadlis closed 1 year ago

vadlis commented 1 year ago

Просто интересно - что мешает добавить флаг в readTemp и по этому флагу вызывать requestTemp в концу функции? тогда код примеров будет покороче )

GyverLibs commented 1 year ago

Ничего не понял

vadlis commented 1 year ago

float getTemp(uint8_t idx = 0, bool request = false) { ... if (request) { requestTemp(idx); } }

тогда в примерах можно убрать код с requestTemp.

ЗЫ: только начал все это смотреть - не пинайте ежели чего )

GyverLibs commented 1 year ago

Это нарушит логику работы с датчиком. Температуру нужно запросить, а ответ будет готов спустя некоторое время вплоть до 1 секунды

vadlis commented 1 year ago

понятно

vadlis commented 1 year ago

хотя не очень )). код из примера:

https://github.com/GyverLibs/microDS18B20/blob/main/examples/async_read_many/async_read_many.ino#L35-L45

в этом случае requestTemp просто переместится из скетча в библиотеку. Никакого нарушения не вижу. Прошу ткнуть носом )

GyverLibs commented 1 year ago

Сделай форк с более короткой записью. В данной библиотеке request это отдельная операция на усмотрение юзера

vadlis commented 1 year ago

Ну собсно флаг тоже был бы опциональным. Ну да ладно - на нет и суда нет ))