Замечания по программе WeatherStation

[alexey-malov]

https://github.com/liebeSonne/ood-labs/blob/aa83afa6fa88ec5478dd671c6ef49e993a6e367b/labs/02/src/Observable/Observable.php#L7-L11

• [ ] Сделать поля приватными

[alexey-malov]

https://github.com/liebeSonne/ood-labs/blob/aa83afa6fa88ec5478dd671c6ef49e993a6e367b/labs/02/src/Observable/Observable.php#L20-L23

• [ ] Метод можно спокойно сделать приватным

[alexey-malov]

• [ ] Пометить методы registerObserver, notifyObservers, removeObserver ключевым словом final

[alexey-malov]

• [ ] Интерфейс ObserverInterface лучше поместить в тот же каталог, что и Observable. Они тесно связаны (Observervable не может использоваться без Observer, а без Observable Observer смысла не имеет

[alexey-malov]

https://github.com/liebeSonne/ood-labs/blob/aa83afa6fa88ec5478dd671c6ef49e993a6e367b/labs/02/src/Model/Weather/WeatherData.php#L7-L14

• [ ] Реализовать возможность определения того, с внешнего или внутреннего датчика пришло уведомление, без введения поля наподобие type в WeatherData. В лекции это описано в разделе "Наблюдение за более чем одним субъектом"

[alexey-malov]

https://github.com/liebeSonne/ood-labs/blob/aa83afa6fa88ec5478dd671c6ef49e993a6e367b/labs/02/src/Model/Display/StatsDisplay.php#L30-L37

• [ ] Сделать возможность отображения одних и тех же статистических данных в разных форматах, не меняя реализацию класс AvgStats:

• Вывод температуры либо в градусах цельсия, либо в градусах фаренгейта. При том, что с WeatherData температура всегда отдаётся в градусах цельсия

• Давление отображать в кПА и в мм.рт.столба

• Влажность пусть выводится со знаком % в конце

• С лёгкостью должна иметься возможность графического вывода

Класс AvgStats берёт на себя 2 ответственности: вычисление минимальной, максимальной и средней температуры и вывод.