Телеграмм чат группы rubylang страница 10568

Вот кстати, есть ещё один непонятный момент.
В example.rb первый и второй CustomObjectTwo.
Во второй передан аргументом nil, в первый не передано ничего.
В readme к первому выводится пустой хэш, ко второму ничего.
Но в custom_object_two.rb там при инициализации присваивается пустой хэш в любом случае.
Одинаковый же вывод должен быть, исходя из этого?

источник

05:07пожаловаться #8

Nekotari in Ruby, Rails, Hanami | dry-rb

Опечатка?

источник

05:07пожаловаться #9

Nekotari in Ruby, Rails, Hanami | dry-rb

Или я чего-то не понимаю?

источник

05:07пожаловаться #10

Hans Hans in Ruby, Rails, Hanami | dry-rb

Nekotari

Вот свой основной логгер принесла
https://drive.google.com/file/d/1WVZeo7f3fpoJavCMcl3qU1pw7o0jtWhx/view?usp=sharing

сразу укажу что я спец по гавнокоду, ну так как все спят толкну пару идей

источник

06:42пожаловаться #11

Hans Hans in Ruby, Rails, Hanami | dry-rb

Time.now отдельную переменную не делай а сразу ее в @file.write("|#{time} внеси

источник

06:43пожаловаться #12

Hans Hans in Ruby, Rails, Hanami | dry-rb

и с этой же строке забыла завершаюший |\n"

источник

06:44пожаловаться #13

Hans Hans in Ruby, Rails, Hanami | dry-rb

дальше, вот эта конструкция if object.class == CustomObject немного некрасива

источник

06:45пожаловаться #14

Nekotari in Ruby, Rails, Hanami | dry-rb

Hans Hans

дальше, вот эта конструкция if object.class == CustomObject немного некрасива

Лучше is_a?

источник

06:48пожаловаться #15

Hans Hans in Ruby, Rails, Hanami | dry-rb

нужно в самих класах CustomObject и CustomObject2 добавить каконить def to_log который и будет генерировать хэш со строками и данными которые ты просто будеш вызывать из CustomLogа и выводить в лог

источник

06:49пожаловаться #16

Hans Hans in Ruby, Rails, Hanami | dry-rb

тоесть формирования строк и списка переменных перенеси с сами классы

источник

06:49пожаловаться #17

Nekotari in Ruby, Rails, Hanami | dry-rb

Hans Hans

тоесть формирования строк и списка переменных перенеси с сами классы

Да, я поняла.

источник

06:50пожаловаться #18