0
Artem Kovalev 2 years ago 0
Пишем Aftorization failed
0
Дима Фомин 1 year ago 0

"Улучшает читабельность кода. Намного проще понять, что делает метод getTax() чем строка orderPrice() * -2." Мне кажется это здесь лишнее, т.к. и метод basePrice() и переменная basePrice одинаково удобочитаемы

0
Under review
Madan kumar nath 2 years ago • updated by Alexander Shvets 2 years ago 3
An "else" eliminates evaluation of nested if conditions. So factoring this way will make code slow
0
Completed
Mihhail Sidorin 3 years ago • updated by Alexander Shvets 3 years ago 1
Всё достаточно просто и с примерами. Выложите примеры в github, чтобы люди могли добавлять примеры на своих языках, я например добавил бы для ruby. Так же могла бы происходить и локализация =)
Answer
Alexander Shvets 3 years ago
Спасибо за ваш отзыв, Михаил. Милости просим на гитхаб: https://github.com/shvetsgroup/refactoring.guru-examples
0
Fixed
zeggel 3 years ago • updated by Alexander Shvets 3 years ago 1
В примере в классе "Employee" метод "getType" объявляется абстрактным, следовательно и сам класс надо объявить абстрактным, тогда в статическом методе "create" в ветке default оператора switch будет ошибка, т.к. будет попытка создания экземпляра абстрактного класса.
Answer
Alexander Shvets 3 years ago
Спасибо за сообщение, ошибку исправил.
0
Viktor Fursov 2 years ago 0
И очень хорошо организованную инфоррмацию!
0
Thanks
Ян Иванов 3 years ago • updated by Alexander Shvets 3 years ago 1
Большое спасибо вам за отличный материал и изумительную его подачу!
Answer
Alexander Shvets 3 years ago
Спасибо за ваш отзыв!
0
Thanks
Alexandr Zhdanov 2 years ago • updated by Alexander Shvets 2 years ago 1
Answer
Alexander Shvets 2 years ago
Спасибо за отзыв!
0
Fixed
Marcos Atán 2 years ago • updated by Alexander Shvets 2 years ago 1
The code for the solution is incomplete:

String foundPerson(String[] people){  List candidates =
    Arrays.asList(new String[] {"Don", "John", "Kent"});
  for (int i=0; i
Answer
Alexander Shvets 2 years ago
Fixed, thanks!
0
Fixed
Marcos Atán 2 years ago • updated by Alexander Shvets 2 years ago 1
In the solution part, I read "Move the algorithm structure and identical steps to a subclass". I think it's "... to a superclass".
Answer
Alexander Shvets 2 years ago
Thanks for reporting this bug. I've corrected the text on the website.