Size: a a a

Чат по руби

2020 July 30

VA

Valentin Andreev in Чат по руби
WANYA FORTY
Он повесился наверное?!
К таким вещам лучше с юмором относится. Всегда будет что-то из разряда "знал бы прикуп".
источник

AP

Anton Plugin in Чат по руби
Pineapple Thief
Зачем cucumber то, никогда не понимал в чем фишка лишний слой с регулярками добавлять в спеки
Идея была в том что любой человек не знакомый с программированием сможет писать спецификацию бизнес-логики и тесты одновременно.
источник

AP

Anton Plugin in Чат по руби
Работает ли он так как задумывалось- другой вопрос.
источник

PT

Pineapple Thief in Чат по руби
Вспоминается известная шутка про проблему и регулярки
источник

R

Roux in Чат по руби
Итак, есть вот такая задача:

Написать метод/функцию, которая принимет на вход текст (учитывать пунктуацию и специальные символы), и возвращает массив из 3х наиболее часто встречаемых слов в тексте в порядке убывания.
- словом является строка букв (A to Z), опционально содержащая один или более апострофов (')
- совпадения не должны быть чувствительны к регистру и слова в возвращаемом массиве необходимо привести к нижнему регистру
- Если текст содержит меньше трёх уникальных слов, вернуть пустой массив.

Я все сделал следующим образом:  https://pastebin.com/cU58JwYc

Буду очень признателен за критику от опытных товарищей в духе "то плохо, то лишнее, тут лучше так" и т.п. :)
источник

AP

Anton Plugin in Чат по руби
Roux
Итак, есть вот такая задача:

Написать метод/функцию, которая принимет на вход текст (учитывать пунктуацию и специальные символы), и возвращает массив из 3х наиболее часто встречаемых слов в тексте в порядке убывания.
- словом является строка букв (A to Z), опционально содержащая один или более апострофов (')
- совпадения не должны быть чувствительны к регистру и слова в возвращаемом массиве необходимо привести к нижнему регистру
- Если текст содержит меньше трёх уникальных слов, вернуть пустой массив.

Я все сделал следующим образом:  https://pastebin.com/cU58JwYc

Буду очень признателен за критику от опытных товарищей в духе "то плохо, то лишнее, тут лучше так" и т.п. :)
Функция не соответствует требованиям, она ничего внятного не возвращает.
источник

AP

Anton Plugin in Чат по руби
Ну и не принимает кстати тоже
источник

R

Roux in Чат по руби
Anton Plugin
Функция не соответствует требованиям, она ничего внятного не возвращает.
Да, этот момент уже понял) Спасибо большое) Благодаря вам такие косяки и находятся, сам бы даже не думал об этом
источник

R

Roux in Чат по руби
источник

PT

Pineapple Thief in Чат по руби
frequency.sort_by { |key, value| -value }
лучше неиспользуемые аргументы обозначать через _

frequency.sort_by { |_key, value| -value }
источник

R

Roux in Чат по руби
Pineapple Thief
frequency.sort_by { |key, value| -value }
лучше неиспользуемые аргументы обозначать через _

frequency.sort_by { |_key, value| -value }
Спасибо большое. Не совсем понимал эту подсказку рубокопа)
источник

P

Paul in Чат по руби
по-моему, нужно  просто _ использовать, _key и key по сути эквивалентны
источник

P

Paul in Чат по руби
а откуда  tally, что за метод?
источник

PT

Pineapple Thief in Чат по руби
Paul
по-моему, нужно  просто _ использовать, _key и key по сути эквивалентны
имхо _key читаемей
источник

P

Paul in Чат по руби
хм. я чё-то думал, _ это магия какая-то
источник

R

Roux in Чат по руби
Paul
а откуда  tally, что за метод?
Тоже удивился, в декабре был введен) https://ruby-doc.org/core-2.7.1/Enumerable.html#method-i-tally
источник

P

Paul in Чат по руби
а по сути просто имя переменной
источник

P

Paul in Чат по руби
о, не знал, спасибо. Enumerable,  значит.... будем почитать
источник

P

Paul in Чат по руби
по коду такие замечания, поехали:
источник

P

Paul in Чат по руби
строки 6-11-15 — там if такой ветвистый не нужен.
источник