AV
но я так понял проблема, что эти ТапТуда, ТапСюда доступны и для других "страниц", поэтому хочется их не дублировать внутри, а один раз написать. А когда начейнено - это уже сложнее.
Size: a a a
2019 October 22
AV
Если бы чейнинга не было - то рефакторнуть можно было очень быстро.
AB
1) плохо читается
2) предположим, что надо будет ещё один тап добавить для каждого файла — придётся вставлять его в несколько разных мест, потому что код тупо дублируется
2) предположим, что надо будет ещё один тап добавить для каждого файла — придётся вставлять его в несколько разных мест, потому что код тупо дублируется
AB
привет тем, кто на конференциях советует активнее использовать копи-пейст :))))
AB
чейнинг рефакторингу не мешает нисколько, усилий надо ровно одинаково
AV
Вместо Page1.upload;
Page1.tapHere
Page1.tapThere
заменили бы Page1 на CommonTappingPanel в нижних двух строчках.
Page1.tapHere
Page1.tapThere
заменили бы Page1 на CommonTappingPanel в нижних двух строчках.
AB
не с тем боретесь :)
AV
еще раз прочитал код - всё читается, и всё понятно.
AV
чейнинг рефакторингу не мешает нисколько, усилий надо ровно одинаково
ну я не вижу как быстро рефакторнуть, если у тебя методы внутри page1 возвращают this и начейнены как на фотке. Рефакторнуть просто, но оооочень много ручных исправлений, не вижну как search&replace -ом можно поменять.
AB
refactoring это не search&replace, а специальные кнопочки в среде разработки. и они отлично этот чейнинг разрулят
AV
ну я птичка, мне такое сложно. Но если что - ткни какие именно.
AV
Я так понимаю условия. Было:
class Page1И теперь оказалось, что click() лучше подходит в class
method1() return this
method2() return this
method3() return this
click() return this
class Test1
new Page1().method1()
.method2()
.click()
.method1()
.click();
class commonКак переделать? С помощью функций рефакторинга в IDE.
click() return что-то или ничего
AB
ну, я же не так предлагаю переделывать. так конечно не получится
AV
ну, я же не так предлагаю переделывать. так конечно не получится
вот, и я говорю, что не получится) А если бы не чейнилось, то просто получилось бы. Согласен?
AB
согласен. для той трансформации, которую хочешь ты, чейнинг является препятствием
AV
согласен. для той трансформации, которую хочешь ты, чейнинг является препятствием
😊
AB
а для той, которую предлагаю я — не является
AV
а для той, которую предлагаю я — не является
наверное. Покажи как ты трансформируешь.
AB
я предлагаю просто методы для начала сгруппировать, то есть сделать более крупные методы, в которые спрятать вызовы серии более мелких. тем самым устраняется дублирование. а потом эти более мелкие методы выделить в отдельный вспомогательный класс. объект этого класса можно будет использовать и в других местах
AB
то есть вот эта повторяющаяся для каждого прикрепляемого файла серия вызовов тап-сюда-тап-туда-введи-это заменяется на один метод, внутри которого все эти тапы, клики, вводы выполняются