Не общая переменная #4
Comments
alexnv4
added a commit
to alexnv4/choicemarket
that referenced
this issue
Dec 7, 2023
В подклассах Shop: - убран field shadowing - удалён неиспользуемый или повторяющийся код - setMinOrder вынесен в суперкласс - поля теперь константы Рефактор ChoiceMarketManager: - поля из класса теперь внутри методов, fixes AlexMaxpower#4 - разбиение логики метода run на методы - убрано дублирование кода, заполнение списка магазинов вынесено в другой метод, список создаётся используя reflection В enum ShopName добавлены методы для сопоставления с классом
Open
alexnv4
added a commit
to alexnv4/choicemarket
that referenced
this issue
Dec 13, 2023
В подклассах Shop: - убран field shadowing - удалён неиспользуемый или повторяющийся код - setMinOrder вынесен в суперкласс - поля теперь константы Рефактор ChoiceMarketManager: - поля из класса теперь внутри методов, fixes AlexMaxpower#4 - разбиение логики метода run на методы - убрано дублирование кода, заполнение списка магазинов вынесено в другой метод, список создаётся используя reflection В enum ShopName добавлены методы для сопоставления с классом
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
https://github.com/AlexMaxpower/ChoiceMarket/blob/2eabadc0f0ca452225082998ba1d2e288b423f9f/src/managers/ChoiceMarketManager.java#L16
Такие переменные лучше не полями делать общими, а наоборот внутрь метода заносить. Нет смысла сохранять такую переменную для разных вызовов метода, а должна быть 1 на 1 вызов
The text was updated successfully, but these errors were encountered: