k, v читабельнее потому что короче.

Не соглашусь

Может они именно с k v игнорят, ок

Они? инструменты? слепое следование правилам хуже их отсутствия.

Не соглашусь

Ну это твое личное имхо

Они? инструменты? слепое следование правилам хуже их отсутствия.

Я уже писал про слепоту выше

Ну это твое личное имхо

С 3 раза понял?)

Твое имхо базируется на выводе твоего кодстайл инструмента, разве нет?

Эстетика ещё для меня. Как отсутствие скобок. Это просто лучше выглядит

Твое имхо базируется на выводе твоего кодстайл инструмента, разве нет?

Как видишь, ещё немного дальше уходит. Я черпаю, но не слепо следую

бесмысленные расшифровки только снижают читабельность

бесмысленные расшифровки только снижают читабельность

Расшифровка — это пояснение, следующее за сокращением, не?

Помнишь упртй стиль? Это короче и понятно. Почему мы не перешли на такое общение?

потому что k, v любому нормальному программисту очевидно

если ты k, v разворачиваешь в длинную форму не имея на то особого смысла то ты в типичный шаблон в моей голове не вписываешься, следовательно снижается т.н. читабельность

если ты k, v разворачиваешь в длинную форму не имея на то особого смысла то ты в типичный шаблон в моей голове не вписываешься, следовательно снижается т.н. читабельность

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

более полноценные названия - key, val, key, value делают однострочник длиннее без всякого на то смысла

более полноценные названия - key, val, key, value делают однострочник длиннее без всякого на то смысла

Для меня +10 к эстетике и +5 к читаемости и очевидности. Давай уже сойдёмся на том, что это субъективно, и закончим это. Я лишь подкинул своё мнение, для разнообразия вариантов, а не хочу разжигать срачи

+5 к читаемости там где k, v имеют такую же читаемость?

блин серьезно, в чем суть key, value если они не объясняют какие именно ключи/значения лежат?

+5 к читаемости там где k, v имеют такую же читаемость?

Для меня они имеют хуже читаемость

вот если я делаю reject{ |domain, _| domain.end_with?(".ru") } это добавляет читабельности

А вот если я делаю key, value это бесмысленно

блин серьезно, в чем суть key, value если они не объясняют какие именно ключи/значения лежат?

Да, кстати, пояснять по возможности — ещё лучше на мой взгляд

Потому что это не объясняет что там лежит внутри них

так в чем суть?

Так вот именно в пояснении смысл есть, а вот разворачивать k, v в длинные слова и удлинять однострочник без причины - следование правилам без применения здравого смысла.

Не буду дальше спорить.

так в чем суть?

Сталкивался вроде с кастомными классами, где v не означало value, а интерфейс похож

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

#работа Ruby on Rails dev, фултайм/парттайм, оплата почасовая - 500 руб./ч (в месяц при фултайме будет выходить 40-80 т.р.) Ищем человека в команду - создаем облачный сервис для поиска частных исполнителей (аналоги - youdo.com, thumbtack.com). Летом 2015 запустили MVP - чисто рельсовое приложение - получили отдачу от первых пользователей, и ушли в оптимизацию. Сейчас у нас сервер с API, iOS-приложение, новый дизайн и верстка. Готово все на 90%. Человека ищем для админки и веб-версии - взять верстку (slim, sass, производный от bootstrap фреймворк + верстка по БЭМ), и сделать традиционное фулстек RoR приложение, rails engine. Я - сооснователь, занимаюсь версткой, над рельсовым приложением будем работать совместно. Пишите в личку.

А ключи то у хеша почему-то получаются строки

https://gist.github.com/lbvf50mobile/d4e24c0392d4da528b06493796e26a29#file-select_and_cut_key-rb-L26

Интреесно вообще ключь он по определеню всега символ, или может быть то символом то строкой?

k.to_s сперва делают - а потом жалуются...

потому что ты в .map делаешь .to_s

потому что ты в .map делаешь .to_s

понял

Интреесно вообще ключь он по определеню всега символ, или может быть то символом то строкой?

Хоть объектом

Хоть объектом

понял

и вообще.

Боролся с простынями, получилась лыжа: https://gist.github.com/lbvf50mobile/b15d8f931f9d5f0eef1a0c0e5b5bad78#file-dry_select_cut_strip-rb-L26

и вообще.

кстати map{&:strip} ругается потому что оно разворачивается в map{|p| p.strip} а p === [k,v]

{&:strip}?

hash.select{|k,v| k.to_s.start_with?(prefix)}.inject({}){|x,(k,v)| x[k.to_s.sub(Regexp.new("/{prefix}/"),"").to_sym] = v; x}.map{|k,v| [k,v.strip]}

Ого

перл атакует

более трешовой строчкия еще не видел.

lambda тут зачем?

что если я скажу что однострочники можно разбивать по строкам?

более трешовой строчкия еще не видел.

а как красиво сделать

я не против, я только за

я за красоту DRY и все прочее

для начала скажи что здесь происходит

используйте рубокоп

с DRY можно об стенку расшибиться если юзать его налево и направо.

используйте рубокоп

+

и вообще, в inject можно делать .strip

и тогда одним перебором меньше

@lbvf50 скажи что этот кусок кода вообще делает?

@lbvf50 скажи что этот кусок кода вообще делает?

этот кусок кода разрезает массив на два подмассива для использования в api

@lbvf50 скажи что этот кусок кода вообще делает?

входной массив имеет ключи с префиксами to и префиксами from

Подожди

чего ты хочешь добиться?

кстати, этот select тоже вполне себе можно выполнить в inject

вот я увижу у тебя хэшик сверху

что ты хочешь получить из него?

а как красиво сделать

Типичный Ruby-программист.

а не проще на вход нормально разделенный хэш давать?

Однажды мне на собеседовании сказали, что я сделал НЕКРАСИВО.

ERROR: S client not available

Типичный Ruby-программист.

ТАКАЯ портянка никуда не годится.

ТАКАЯ портянка никуда не годится.

Ну вообще да, но задача изначально поставлена немного странно.

Почему бы руками не задать эти два хэша, если один уже задан.

а не проще на вход нормально разделенный хэш давать?

Я про это.

Regexp.new(...) тут не нужен же , просто .sub(prefix,"")

Вообще на собеседовании максимум что могут требовать по красоте это чтобы дефолтный рубокоп проходил например

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

prefix.to_r

почему, что хотят то и требуют, хозяин - барин

Во, у тебя с детства привили нелюбовь к руби с этими красивостями

Смотрие, речь о рефакторинге данной функции мы о ней говорили дня 2 назад: https://gist.github.com/lbvf50mobile/2fd55a4a331a3ca9015ca95c02fcb91a

и все мы пришли к выводу что так жить нельзя. Начинаем спервых строк формирования двух хешей

строчки 12-20 21-28

prefix.to_r

тогда и в select надо k.match(...)

с DRY можно об стенку расшибиться если юзать его налево и направо.

Есть пример?

Есть пример?

вобщем мой пример. и есть

Есть пример?

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

хотя...

т.е схожий код но не один и тотже.

погодите

сразу с фронта должны отправить хэш вида {to: {}, from: {}}

короче какие задаи 1) высленить for и to 2) сотвественно обзать 3) strip

сразу с фронта должны отправить хэш вида {to: {}, from: {}}

ну да... это точно. Тем не менее на на данном этапе как сделать.

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

Не понял, можно более внятно? Ну или пример кода

это рубокоп: https://github.com/bbatsov/rubocop ?

да

еще советую сразу .rubocop.yml найти

а то он просто возьмет и обосрет каждую строчку... и отчасти будет прав

а то он просто возьмет и обосрет каждую строчку... и отчасти будет прав

Почему отчасти?

некоторые правила далеко не всегда применимы. К примеру, правила с документированием кода практически всегда я стараюсь отключить.

@lbvf50 сделай пожалуйста p params и скинь сюды

@lbvf50 сделай пожалуйста p params и скинь сюды

там хеш есть в шапке