ИП
Жесть какая
Size: a a a
2020 December 08
PU
Наверное, да. Можно попробовать схитрить и все сделать шаблонно. То есть вызвать NextMethod() , а потом восстанавливать атрибут класса
кажется, необязательно, можно просто сделать класс, "отнаследованный" от tibble
впрочем, это почти то же NextMethod и есть
впрочем, это почти то же NextMethod и есть
ГД
Да, если наследоваться, то скорее всего писать придется меньше. Правда, сохранение атрибута класса можно ломаться, если какой-нибудь filter об этом не заботится
ИП
кажется, необязательно, можно просто сделать класс, "отнаследованный" от tibble
впрочем, это почти то же NextMethod и есть
впрочем, это почти то же NextMethod и есть
а что ты имеешь в виду под наследованием здесь? навешивание нового аттрибута класса спереди или что-то более сложное?
PU
Иван Поздняков
а что ты имеешь в виду под наследованием здесь? навешивание нового аттрибута класса спереди или что-то более сложное?
да, добавить в атрибут класса новое значение, чтобы была цепочка new_class > tibble > data.frame
PU
а ты не хочешь просто переопределять summary для tibble в своей сессии?
PU
кажется, это будет проще, чем свой класс пилить
PU
ну или вообще создать свою my_summary и ее вызывать
ИП
вот с этого вопрос и начался. Очень многие функции, например,
as_tibble() и as.data.table() обрезают этот рожденный в муках класс, оставляя только data.table и data.frame, напримерИП
ну, это для пакета: чтоб качаешь
get_corpus(), а summary() на полученные данные выдавал бы красивое описание результатов внутри, учитывая специфику датасета
ГД
Иван Поздняков
вот с этого вопрос и начался. Очень многие функции, например,
as_tibble() и as.data.table() обрезают этот рожденный в муках класс, оставляя только data.table и data.frame, напримерНу вообще, по с
смыслу эти функции должны строго возвращать tibble и data.table иначе что-то уж совсем не так, если конвертация в класс возвращает не тот класс, который обещает
смыслу эти функции должны строго возвращать tibble и data.table иначе что-то уж совсем не так, если конвертация в класс возвращает не тот класс, который обещает
PU
ну тогда да, свой подкласс и для него summary
я бы наверное так делал
я бы наверное так делал
PU
Ну вообще, по с
смыслу эти функции должны строго возвращать tibble и data.table иначе что-то уж совсем не так, если конвертация в класс возвращает не тот класс, который обещает
смыслу эти функции должны строго возвращать tibble и data.table иначе что-то уж совсем не так, если конвертация в класс возвращает не тот класс, который обещает
+1
ИП
там я намудрил четыре s3 класса внутри пакета (это вообще мой первый, мол, не судите строго). Теперь вот вырезаю
ИП
Ну вообще, по с
смыслу эти функции должны строго возвращать tibble и data.table иначе что-то уж совсем не так, если конвертация в класс возвращает не тот класс, который обещает
смыслу эти функции должны строго возвращать tibble и data.table иначе что-то уж совсем не так, если конвертация в класс возвращает не тот класс, который обещает
ну, это верно, но ведь сами эти классы таки наследуются от датафрейма. Я бы вообще хотел сделать так, чтобы мой класс был первом в череде диспатчеризации, принимая и простые датафреймы, и тибблы, и дататейблы
ИП
и, например,
as.data.table() и as_tibble() просто навешивал свой класс в серединку тогда. Тогда это был бы просто обычный датафрейм/дататейбл/тиббл, но с кастомным саммари и плотом, прописанными как методы для соотвествующих дженериков.PU
Иван Поздняков
ну, это верно, но ведь сами эти классы таки наследуются от датафрейма. Я бы вообще хотел сделать так, чтобы мой класс был первом в череде диспатчеризации, принимая и простые датафреймы, и тибблы, и дататейблы
не совсем так. к ним применяются те методы, которые не определены для этого класса, но определены для следующего в цепочке классов. это эрзац наследования, все же.
и по твоей логике наоборот, твой класс должен быть последним
и по твоей логике наоборот, твой класс должен быть последним
ИП
последним где? наоборот же, первым на очереди: сначала проверяем, есть ли, например, method_name.corpus(), если не находится, то импользуется method_name.data.table()/method_name.tbl(), если не находится - method_name.data.frame(). Тогда класс должен быть c(«corpus», «data.table», «data.frame»). Я думал, это так работает
PU
да, ты прав
плюс для датафрейма ты все равно суперкласс не сделаешь, это очень странный кадавр будет :)
плюс для датафрейма ты все равно суперкласс не сделаешь, это очень странный кадавр будет :)