search

Biryukov239 / matrix-multiplication-kotlin

0 stars 0 forks source link

Не идемпотентные геттеры — антипаттерн #12

Closed lounres closed 4 months ago

lounres commented 4 months ago

У Вас GPUMultiplier.Companion.defaultDevice определён с геттером, который выполняет сложную работу. Если он не идемпотентный, то его стоить заменить на функцию getDefaultDevice. Иначе лучше использовать by lazy { ... }.

Biryukov239 commented 4 months ago

Заменил.

lounres commented 4 months ago

Ага. Лучше, честно говоря, делать через lazy, что по сто раз не вычислять одно и то же.