The Single Responsibility Principle

@Artmal · Регистрация: 14.11.2015

Студворк — интернет-сервис помощи студентам

Есть класс Weather, в котором, на данный момент, есть методы getMaxTemperature(), getAvgTemperature(), getDayWithLongestComment(). Куда выносить подобные методы, чтобы избежать нарушения SRP?

@gubarevs · 27.05.2017, 19:21

Artmal, а ты уверен что с текущим набором методов применим принцип SRP ?

@Artmal · 27.05.2017, 20:01 **[ТС]**

Правильно ли я понимаю, что тут SRP вообще не нарушается, из-за того что нет никаких модификаций объекта(только чтение)?

Как я понимаю, моя проблема называется божественный объект. Как вот ее решить? И есть ли она вообще в данной ситуации?

@gubarevs · 27.05.2017, 21:22

Artmal, ты правильно понимаешь, тут нет модификаций. Божественный обьект собирает в себе большое количество свойств и методов так, что большая часть программы взаимодействует только с ним (обьект берет на себя слишком много). Ну если рассматривать эту проблему, можно разделить методы с температурой и длинной дня в разные классы.

@korvin_ · 28.05.2017, 10:37

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

Правильно ли я понимаю, что тут SRP вообще не нарушается, из-за того что нет никаких модификаций объекта(только чтение)?

Неправильно. Чтение/изменение объекта никакого отношения к SRP не имеет.

Каково назначение класса Weather?

Навскиду, можно

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

есть методы getMaxTemperature(), getAvgTemperature()

вместо этих двух методов лучше сделать один, возвращающий массив/список/поток температур за период, типа

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
import java.time.ZonedDateTime;
import java.util.stream.DoubleStream;
 
public final class Weather {
 
    private static final String TEMPERATURE_REPORT_FORMAT = "Today's%n\tmax t = %f%n\tavg t = %f%n";
 
    public static void main(String[] args) {
        final Weather w = new Weather();
        final ZonedDateTime today = ZonedDateTime.now();
        System.out.printf(TEMPERATURE_REPORT_FORMAT,
                w.getTemperatures(today).max().orElse(Double.NaN),
                w.getTemperatures(today).average().orElse(Double.NaN)
        );
    }
 
    public DoubleStream getTemperatures(ZonedDateTime day) {
        final double[] dayTemperatures = new double[0]; // load temperatures for specified day
        return DoubleStream.of(dayTemperatures);
    }
}

Но, конечно, если эти методы выполняют SQL-запросы, например, и соответственно SQL-сервер подсчитывает max/avg, или эти методы просто проксируют запросы к REST-сервису, который предоставляет готовые функции вычисления max/avg, то всё не так очевидно.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .
Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .

@Artmal 88 / 86 / 55 Регистрация: 14.11.2015 Сообщений: 1,099

	The Single Responsibility Principle 27.05.2017, 19:03. Показов 1590. Ответов 4 Метки нет (Все метки) Есть класс Weather, в котором, на данный момент, есть методы getMaxTemperature(), getAvgTemperature(), getDayWithLongestComment(). Куда выносить подобные методы, чтобы избежать нарушения SRP? 0

@gubarevs 62 / 64 / 14 Регистрация: 25.03.2017 Сообщений: 156
	27.05.2017, 19:21
	Artmal, а ты уверен что с текущим набором методов применим принцип SRP ? 1

@Artmal 88 / 86 / 55 Регистрация: 14.11.2015 Сообщений: 1,099
	27.05.2017, 20:01 [ТС]
	Правильно ли я понимаю, что тут SRP вообще не нарушается, из-за того что нет никаких модификаций объекта(только чтение)? Как я понимаю, моя проблема называется божественный объект. Как вот ее решить? И есть ли она вообще в данной ситуации? 0

@gubarevs 62 / 64 / 14 Регистрация: 25.03.2017 Сообщений: 156
	27.05.2017, 21:22
	Artmal, ты правильно понимаешь, тут нет модификаций. Божественный обьект собирает в себе большое количество свойств и методов так, что большая часть программы взаимодействует только с ним (обьект берет на себя слишком много). Ну если рассматривать эту проблему, можно разделить методы с температурой и длинной дня в разные классы. 1

The Single Responsibility Principle

Решение