Для чего присваивать результат вызова метода переменной?

Question

[fomiash]

Недавно в тз наткнулся в требованиях к коду. Вкратце, если в пределах одной видимости вызываются одинаковые статические методы одного класса, то рекомендовано присвоить вывод метода переменной и использовать переменную. Для чего это нужно может быть? Ведь это можно решить проще внутри самого класса присвоением разового расчета внутренней переменной и возвратом этого значения.

Comments

[Дмитрий]

Ну типо чтоб два раза метод не дергать, а так можно и внутри разок посчитать и сделать кэш

Answer 1

[gradmir]

По сути тут речь о том, что можно закэшировать результат расчёта либо внутри метода, либо снаружи.
Снаружи лучше, потому что:
1) каждый вызов метода - это в любом случае накладные расходы, если метод вызывается часто (например, в цикле), это может стать заметным
2) результат расчёта может зависеть от внешних данных, тогда нужно как-то инвалидировать кэш, снаружи это сделать проще

Comments

[fomiash]

Тут, в общем то, и возразить нечего, так как если брать создание новой переменной против многократного вызова метода... Читаемость, оптимизация и прочее зависит от конкретного кода, а не упомянутого принципа в тз.
Вообще бы добавил кеширование и в метод и в код обязательным, но установил бы число повторов в тз, с которого начинается использование дополнительной переменной.

[Алексей Сундуков]

gradmir

тогда нужно как-то инвалидировать кэш, снаружи это сделать проще

Это логика не совсем правильная. В этом случае вы отдаете ответственность инвалидации наружу и значит снаружи нужно полностью понимать логику данного метода, какими данными он оперирует и какова логика построения этого кэша. Т.е. полное игнорирование инкапсуляции.
Второй минус указанного подхода: внешнего кода можно быть много. И потенциально получаем бесконечно количество мест которые занимаются инвалидацией. И тут нарушаем принцип единственности ответственности. Правильно иметь внутри ровно одно место (метод) которое отвечает за логику кэширования и когда при смене этой логики в коде потребуется отрефакторить ровно одно место.

[Василий Банников]

Алексей Сундуков, с другой стороны - всё-таки пусть кэшированием занимается другой объект, который не имеет непосредственного отношения ни к вызываемому методу, ни к вызывающему. В C# я бы для этого использовал Lazy, например, если бы заранее знал что результат вычисляется 1 раз и никогда не устаревает. Но чаще проще и понятнее складывать результат в переменную (ибо инвалидация всё равно происходит в момент выхода из блока)