Есть очень длинный метод вида:

В нем переменная bits используется для диагностики.
Я бы хотел отрефакторить этот код и разбить длинный метод на несколько, например такого вида:

Но вот вопрос: как поступить с переменной bits?
Ведь в методы one и second её передавать плохо, так как они должны точно знать какие биты проставлять, но это странно, так как методы могут вызываться из разных мест.
Можно было бы передавать некий адаптер bits с параметром смещения битов, начиная с которого следует проставлять биты в методах, но это как-то слишком сложно.
Каким тут может быть наилучшее решение?

Это сообщение отредактировал(а) Woo - 9.3.2018, 14:43

А закрытым членом класса ее сделать не вариант?  

Дело в том, что эта переменная относится только к методу, а не ко всему классу, поэтому так делать некорректно.
Мне сейчас кажется лучший вариант - это использовать прием рефакторинга, который называется Замена метода объектом методов.
И уже в этом новом объекте разместить закрытый член класса. Что-то вроде такого:

Как думаете, есть вариант получше?

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


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



Это сообщение отредактировал(а) Namira - 11.3.2018, 02:03