Resolve Plutakhin tests#42
Conversation
| /spec/reports/ | ||
| /tmp/ | ||
| /log/ | ||
| /.idea No newline at end of file |
There was a problem hiding this comment.
нужно настроить редактор так, чтобы в нем по сохранению в конце файла оставалась одна пустая строка
There was a problem hiding this comment.
так же .idea - это файл, который создается твоей средой разработки. такие настройки выносятся в твой глобальный .gitignore, а не в .gitignore проекта
| class << self | ||
| def replace(array) | ||
| max = array.max | ||
| array.map! { |a| a >= 0 ? max : a } |
There was a problem hiding this comment.
map! мутирует исходный массив, тебе тут это совершенно не нужно, можно использовать map без !
| value = array[mid] | ||
| return mid if value == query | ||
|
|
||
| if query < value |
There was a problem hiding this comment.
тут можно использовать тернарный оператор
| def rating(films) | ||
| ratios = films.map do |film| | ||
| film['rating_kinopoisk'].to_f if !film['rating_kinopoisk'].to_f.zero? && !film['country'].nil? && film['country'].split(',').count > 1 | ||
| end . compact |
There was a problem hiding this comment.
какие-то странные пробелы с точками
There was a problem hiding this comment.
если какой-то блок заканчивается end, после него уже цепочку методов не строим.
There was a problem hiding this comment.
условие в if очень большое, строка плохо читается, можно вынести его в переменную или вспомогательный метод
| end | ||
|
|
||
| def chars_count(films, threshold) | ||
| total = 0 |
There was a problem hiding this comment.
вместо глобального счетчика можно использовать reduce
| # Посчитать средний рейтинг фильмов по версии кинопоиска у которых две или больше стран | ||
| # Фильмы у которых рейтиг не задан или равен 0 не учитывать в расчете среднего. | ||
| def test_rating | ||
| skip |
There was a problem hiding this comment.
тесты заскипаны, они не запускаются
| # Написать свою функцию my_compact | ||
| def my_compact | ||
| array = MyArray.new | ||
| for elm in self |
There was a problem hiding this comment.
тут можно использовать my_each вместо for. в обычной жизни вместо for всегда используется each
| ignore_first = true | ||
| acc = first | ||
| end | ||
| index = 0 |
There was a problem hiding this comment.
можно порефакторить и не использовать никаких счетчиков. также предлагаю пользоваться своими функциями типа my_each
| end | ||
|
|
||
| def test_my_each | ||
| skip |
There was a problem hiding this comment.
тесты скипнуты, читай ридми
|
Обновил реализацию. |
| array = CSV.readlines('./test/fixtures/films.csv', headers: true) | ||
|
|
||
| result = Plutakhin::Fp.chars_count(array, 5) | ||
| assert result == 891 |
There was a problem hiding this comment.
да, значения в тесте неправильные, нужны 3966 и 42
2 participants
No description provided.