функций типа get_sound_duration() распилить на разные: которая работает с одним файлом, и которая работает с папкой, забирая из нее все файлы.
Я вот теперь думаю над тем, как такие функции называть, потому что get_sound_duration_from_folder() мне кажется каким-то монстром...
Вдруг у кого есть идеи?
Я смотрю сейчас вот такой запрос на гитхабе
UPD: написал одну функцию для считывания всего из папки, read_from_folder <- function(path, FUN){...}, которая принимает в качестве аргумента функцию, так что отпала проблема называния... Но все равно интересно, что вы думаете, насколько это юзерфрендли?
Имхо, конечно, но from в названии функции - явный перебор. В твоём коде не вижу ничего про обработку вектора с названиями файлов.
В функции сейчас у тебя имитация поведения S3-дженерика. Не предлагали через S3-методы раскидать? get_sound_duration.Wave, get_sound_duration.character.
Про read_from_folder. Не уверен, что для пользователей будет удобно передавать в одну функцию другую функцию, т.к. это весьма не тривиальное действие для людей, не знакомых с функциональным программированием.
А я пока убрал это и оставил одну эту функцию.
Нет, спасибо, я возможно почитаю, что это и сделаю. Надо же когда-то разобраться с классами
Меня это тоже останавливает. Но с другой стороны я пишу для людей, которые делают все по мануалам или сами могут посмотреть в хэлп, так что может это и не страшно. Но спасибо за мнение, мне оно важно
Мне кажется для таких случаев вполне нормальна ситуация с монструозными названиями функций. Это функция, которая будет присутствовать в скрипте, скорее всего, один или немного раз, по ней сразу можно понять, что происходит, так что почему бы и нет? Единственное что, мне кажется более привычным read_ вместо get_ для чтения файлов, т.к. я встречал обычно get_ в обертках над API (твиттер, вк, genius и т.д.)
внял совету, Артем, спасибо!
Всегда пожалуйста. Обращайся, постараюсь помочь по мере возможности. Тебе, возможно ещё предстоят муки Хэдли, по компоновке кода в разные пакеты и сборки их обратно😁
Обсуждают сегодня