0
Answered
Roman Revin 3 years ago • updated 3 years ago 5
Возможно ли дополнить статью Replace Temp with Query разделом с подводными комнями? Навскидку: при замене на метод теряется возможность модифицировать коэффициент, который находится в переменной. 
0
Ilya Filisteev 2 years ago 0
Вы избавляетесь от ненужного дублирование кода, и, таким образом, уменьшаете его размер.
опечатка в слове "дублирование"
0
Napat Suriyapun 2 years ago 0

i think i don't understand this. temporary field is trying to move useless field in class to other class but it's not occur with null object. is there a example for us? thank you

0
Женя Васильев 2 years ago 0
Хочется ткнуть на звездочку и увидеть дифф исходного и итогового кода без прохождения всех шагов туториала. Во многих случаях этого более чем достаточно.
0
Vitaly Gridasov 2 years ago 0
В описании идет речь об извдечении кода из циклов и условий. Но в примере этого нет
https://refactoring.guru/home/slides/long-method/14
0
Maxim Boyarskiy 2 years ago 0
В описании есть пример, как здорово и понятно иметь getTax(), а не что-то * какой-то коэффициент, но этого нет в коде. Читая только код - прием кажется надуманным.
0
Under review
Romanov V 3 years ago • updated by Alexander Shvets 3 years ago 1
0
Fixed
Георгий Туревич 2 years ago • updated by Alexander Shvets 2 years ago 1
Выводит сообщение об ошибке ""Whoops, looks like something went wrong.""
Answer
Alexander Shvets 2 years ago
Исправил ошибку, извините за неудобство.
Answer
Alexander Shvets 3 years ago
Спасибо, сделал этот тултип выскакивающим сверху, это должно решить проблему.
Answer
Alexander Shvets 3 years ago
Очень конструктивный отзыв)