не знаю ни одну девушку программера( (имею ввиду не хелло ворлд по хартлу)

Есть женщины и программеры и математики. Их мало, но они есть. В НГУ при формировании данного университета во времена СССР соотножение женщин к мужчинам было 1 к 7.

При том что в НГУ во времена СССР поподали люди имеющие победы на олимпиадах республиканского уровня. Т.е. разбирающиеся в логике и матиматике на должном уровне.

https://www.ozon.ru/context/detail/id/33253422/ каждой девушке - программистке в подарок?

вообще читал что женщины мало работают в технических сферах потому что им общения не хватает

ой, да как будто в технической сфере нельзя пообщаться)

общение серьезное видимо, логическое, а женщинам эмоции подавай? Хотя я работал раньше в проектной организации и там целый женский отдел был, все время болтали. Хорошо я в наушниках сидел.

в наушниках можно слух испортить.

может быть, сейчас дома работаю, так что без?

ты испортил слух?

https://gist.github.com/anonymous/b76d3f289e79bb92b47753a423e706b6 - в коде присутсвуют длиннющие простыни разворачивающие элементы массива params в одноименные переменные. По моему это не самый лучший код. Как решать такие вопросы подобающим образом?

а что ты с этими переменными потом делаешь?

а что ты с этими переменными потом делаешь?

формирую объект который будет использован для работы с lob API: https://lob.com/docs/ruby#intro т.е. эти переменные будущие поля нового объекта

а что ты с этими переменными потом делаешь?

https://lob.com/docs/ruby#postcards_create - конкретно вот что

в конструктор объекта нельзя просто передать хэш параметров?

в конструктор объекта нельзя просто передать хэш параметров?

нельзя, потому как как хеш параметров - "плоский", а для создания нового объекта требуется хеш с двумя подмассивами.

Короче я смотрю на код: https://gist.github.com/anonymous/1c719a7ac6bc6bd6b714fa78b745eb4e И что-то внутри меня говорит, что так делать не надо, и что тут нарушины какие-то основопологающие подходы в проектировании rails. Можно как-то изящней и красивей делать.

Ага

вытащить всё это месиво в отдельный класс

схренали в модельке простыня одного метода на пару страниц?

схренали в модельке простыня одного метода на пару страниц?

это контроллер

ТЕМ БОЛЕЕ

ТЕМ БОЛЕЕ

вот и я так думаю.

Короче я смотрю на код: https://gist.github.com/anonymous/1c719a7ac6bc6bd6b714fa78b745eb4e И что-то внутри меня говорит, что так делать не надо, и что тут нарушины какие-то основопологающие подходы в проектировании rails. Можно как-то изящней и красивей делать.

тебе достаточно отфильтровать хеш каким-нибудь .select и модифицировать ключи каким-нибудь .map

тебе достаточно отфильтровать хеш каким-нибудь .select и модифицировать ключи каким-нибудь .map

ясно будем рабзераться

вся эта простыня в начале заменяется на params.permit(:from, :blah).map(&:strip)

&:strip это как прочитать?

явно метод strip запускается

а вот &

map{|e| e.strip}

map{|e| e.strip}

спасибо

http://stackoverflow.com/questions/1217088/what-does-mapname-mean-in-ruby

http://stackoverflow.com/questions/1217088/what-does-mapname-mean-in-ruby

спасибо

Кокой длинный метод

спасибо

Попробуй dry-validation или virtus

Попробуй dry-validation или virtus

ясно, нашел гемы. спасибо

я вот думаю, чтоб не было таких больших методов. Надо по мимо инструментов всторенных в руби, и отдельных гемов, еще внутри класса создавать свои подклассы. чтоб в них переносить логику

Так же делают?

или отдельные файлы создают с классами

Делают класс который делает какую то задачу

и потом в контроллере просто дергаешь этот класс.

я вот думаю, чтоб не было таких больших методов. Надо по мимо инструментов всторенных в руби, и отдельных гемов, еще внутри класса создавать свои подклассы. чтоб в них переносить логику

Ну если у тебя есть наследуемая логика

А вот с наследованием поаккуратнее.

Ну если у тебя есть наследуемая логика

я не из-за наследования. А потому как в ruby koans встречал классы внутри класса

т.е. даже не надо заморачиваться созданием нового файла

Плохая идея.

Зачем пилить класс внутри класса?

Зачем пилить класс внутри класса?

а как надо?

заморачиваться? лол, лучше заморочиться но такие портянки не иметь

как правильно?

Как тебе удобнее. Я пилю просто специальный класс который делаешт эту задачу и все.

Нет правильного решения, есть решения лучше или хуже.

Как тебе удобнее. Я пилю просто специальный класс который делаешт эту задачу и все.

а куда его ложить?

Это называется сервисы

А какую задачу он выполняет?

Погугли руби сервисы

В данном случае сервис да.

Но в модельке бывает своя большая логика и это вроде как не сервисы.

А какую задачу он выполняет?

какую либо многосутпенчатую валиадцию пришедших данных

какую либо многосутпенчатую валиадцию пришедших данных

https://github.com/orgsync/active_interaction

Лучше вынести в сервис, модели это вообще зло

Вот я этот юзал, не знаю насколько он хорош, но мне понравился

Лучше вынести в сервис, модели это вообще зло

понял буду гуглить про сервисы

Он и валидации умеет.

https://github.com/orgsync/active_interaction

спасибо

Есть еще большой вариант: https://github.com/collectiveidea/interactor

Не знаю не юзал. Но функционала у него побольше.

ERROR: S client not available

Я бы начал с первого.

Есть еще большой вариант: https://github.com/collectiveidea/interactor

ништяк :)

Я юзаю теперь постоянно

Охуенная штука

будем разбираться и рефакторить систему

Я юзаю теперь постоянно

кого?

Охуенная штука

Да, выглядит охуенно.

сервисы

Второй полагаю.

кого?

Интеракторы

а--аа ясно

Всем советую перекатиться

И самое главное их тестировать проще.

Методы контроллеров должны быть небольшие совсем.

Иначе, скорее всего что то не так.

Можно только их и протестировать

И у тебя 90% считай

Ну да. Но все равно можно обжечься.

Я вот даже не знаю что лучше тестировать интерактор или весь контроллер в целом.

Еще я нашел что есть интеракторы на других языках

С контроллером получается полное покрытие. А интерактор банально быстрее протестировать.

Интерактор это паттерн.

Покажи

https://en.wikipedia.org/wiki/Interaction_design_pattern Думаю оно

Или нет.

http://stackoverflow.com/questions/35746546/android-mvp-what-is-an-interactor Вот еще по теме.

> Advantages of using interactor in a separate class is that it will decouple your class, thus making it cleaner and testable

Круто

не особо (

Я бы хотел получить фидбэк о лучшем варианте. Есть configure в котором есть метод default_backend. Как выглядит более идиоматично/лучше? ActiveSMS.configure do |config| c.default_backend = :my_backend_name end Или ActiveSMS.configure do |config| c.default_backend :my_backend_name end

Забыл уточнить что там есть проверка вводимого значения.

Я бы хотел получить фидбэк о лучшем варианте. Есть configure в котором есть метод default_backend. Как выглядит более идиоматично/лучше? ActiveSMS.configure do |config| c.default_backend = :my_backend_name end Или ActiveSMS.configure do |config| c.default_backend :my_backend_name end

1 вызывает меньше вопросов потому что есть =