Мне кажется, что первый пример с явной передачей в конструктор зависимостей на много приятнее читать, понятнее и не подразумевает под собой лишних вопросов в духе: «ой, а как это у нас эта переменная появилась с объектом».
По хорошему вам нужен code review, чтобыразбить все на отдельные сервис классы хотя бы:
— RestClient вынести в отдельный класс, посмотреть может быть хорошо бы подошел Her.
— STATUSES в модели это или state machine или хотя бы enum
— create_bill станет намного короче, если все эти валидации, создание платежей раскидаются по логическим классам, а не такая портянка
— константы лучше в ENV закинуть
— Код в духе
серверный код просто ужасен, нет тестов, не свежий Rails 4.1.1 и ужасно пахнет (константы в моделях, хотя можно enum, as_jsonn с двумя nn) все от моделей до контроллеров, не надо так!
Еще раз, сиволы это очень полезный инструмент из мира функционального программирования, то что вам кажется глупостью, помогает всем остальным работать с языком и получать нужные результаты. Если вы не видите разницы между строкой и символом поизучайте вопрос его практического приминения, например в сети — www.randomhacks.net/2007/01/20/13-ways-of-looking-at-a-ruby-symbol
Полностью согласен, причем почти 100% шансы если клиент хочет встретиться по каждому поводу и обсуждать, то это будет вода и бесполезная трата времени, он не понимает, что хочет, кроме того, чтобы я ему донес его же желание. Потому письмом выразить все проще и лаконичнее.
Основная проблема диалога голосом и голосовых сообщений, что потом не найти концы. Кто и что сказал, почему было сделано именно так.
А текст вот он, бери, доказывай.
Я конечно понимаю это ваш блог, но возьмите из статьи слово «продукт» и «улучшать жизнь людей», у вас документация дичайшим образом оформлена и неудобна. А еще бы не помешало иметь сразу готовые коннекторы на разных языках к API.
Я жил на севере Таиланда (в Чиангмае) 1.5 года, даже в офис ездил каждый день. Да жить можно, но везде и свои плюсы и минусы, кому-то в речке купаться претит, если рядом есть море.
Паттаю я упомянул в контексте «самые популярные направления», на Пхукет я думаю тоже едут «отдыхать», тут вообще можно везде или отдыхать или работать, тут уж выбор человека самостоятельный.
например выдаст true, так что в этом случае лучше все же проверить значение через .to be_truthy
вместо
можно писать
2. should уже давно не рекомендуют использовать, тем более не красиво смешивать вместе с expect
github.com/rspec/rspec-expectations/blob/master/Should.md
3. Стоит использовать встроенные проверки
вместо
можно
4. К полезным мелочам я бы еще добавил в .rspec
очень приятная мелочь при выводе сообщений
По хорошему вам нужен code review, чтобыразбить все на отдельные сервис классы хотя бы:
— RestClient вынести в отдельный класс, посмотреть может быть хорошо бы подошел Her.
— STATUSES в модели это или state machine или хотя бы enum
— create_bill станет намного короче, если все эти валидации, создание платежей раскидаются по логическим классам, а не такая портянка
— константы лучше в ENV закинуть
— Код в духе
можо в одну строку написать
А текст вот он, бери, доказывай.
А занятия точно такие же как и везде, спорт, друзья, работа, кино, шопинг.