Conversation
test/exercise/fp/solution.rb
Outdated
| def film_multicountry?(film) | ||
| return false if film['country'].nil? | ||
|
|
||
| film['country'].include?(',') |
There was a problem hiding this comment.
запятая конечно на тоненького, по хорошему должно быть упоминание про две страны и больше
There was a problem hiding this comment.
я что-то не могу придумать как проверить количество стран без запятой, там же просто строка возвращается при обращении, а не коллекция у которой можно узнать размер
можно конечно засплитить строку в массив, по запятой, но это разве не то же самое по сути?
There was a problem hiding this comment.
не тоже самое, include просто запятую проверяет (как будто нам только запятая важно), а сплит хотя бы семантику количества добавит
test/exercise/fp/solution.rb
Outdated
|
|
||
| def rating(array) | ||
| multicountry_rated_films = [] | ||
| array.map { |film| multicountry_rated_films << film if film_rated?(film) && film_multicountry?(film) } |
There was a problem hiding this comment.
используешь мап и мутации, мап новый массив возвращает. И т.к. у тебя здесь по сути фильтрация, то функцию фильтрации и нужно использовать
There was a problem hiding this comment.
функция фильтрации это select? просто в задании сказано использовать map и reduce
или ты имел ввиду что-то такое?
multicountry_rated_films = array.filter_map { |film| film if film_rated?(film) && film_multicountry?(film) }
test/exercise/fp/solution.rb
Outdated
| 0 | ||
| def chars_count(films, threshold) | ||
| top_rated_films = [] | ||
| films.map { |film| top_rated_films << film if raiting_of_film(film) > threshold } |
There was a problem hiding this comment.
здесь тоже самое про фильтрацию и мутацию
test/exercise/fp/solution.rb
Outdated
| 0 | ||
| def chars_count(films, threshold) | ||
| top_rated_films = [] | ||
| films.map { |film| top_rated_films << film if raiting_of_film(film) > threshold } |
There was a problem hiding this comment.
и еще здесь абстрация raiting_of_film протекла, она должна полностью и ответить подходит фильм для посчета или нет
There was a problem hiding this comment.
а когда переделаю новый PR делать или сделать новый коммит и написать тут об этом?
2 participants
No description provided.