почему он должен его лочить?

пока ты не сделаешь, что он просит, ничего не выйдет.

ты его поставил по дефолту?

https://rvm.io/integration/zsh ищи тут свою конфигурацию.

версию руби?

@ro31337 а ты можешь посмотреть что у тебя в .zshenv находится?

https://rvm.io/integration/zsh ищи тут свою конфигурацию.

там нет)

только напутствие

чтобы использовать bundler

zsh + oh my zsh If you want to use oh my zsh be sure not to use the bundler package. If rvm should take care of everything this would do for you anyways.

и всё

А zsh + iTerm не то ?

А zsh + iTerm не то ?

zsh + oh my zsh

так

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

поджди удалять. глянь тут ответы https://stackoverflow.com/questions/4755538/rvm-is-not-working-in-zsh

поджди удалять. глянь тут ответы https://stackoverflow.com/questions/4755538/rvm-is-not-working-in-zsh

это смотрел

не сработало

я сдаюсь :)

спроси в большой конфе, может помогут.

to Ro

спроси в большой конфе, может помогут.

в какой большой конфе?

впринципе понятно что делать

https://t.me/rubylang

я уже спрашивал этот вопрос, но чёт меня проигнорили

надо как-то заставить zsh логиниться автоматом

to RO: система win 7, начинаю постигать 50 извращений руби на винде, 2ой системой стоит mint, но т.к . я пока полный профан в рубях, смотрю уроки именно в виндах

to RO: система win 7, начинаю постигать 50 извращений руби на винде, 2ой системой стоит mint, но т.к . я пока полный профан в рубях, смотрю уроки именно в виндах

ну ты не спеши особо, тебе задача пока - настроить текстовый редактор в mint (напр.Atom), ну и разобраться с терминалом, основные команды

поверь, в линуксе тебе проще будет, винда дает попроще вход, но намного сложнее потом

Всем привет

Мне тут тестовое задание скинули

Код для рефакторинга

Показываю чисто поржать

Я бы в самом сильном помешательстве не создал этот ад

Я бы в самом сильном помешательстве не создал этот ад

может это любимое слово)

)) ппц

Алсо, мне надо будет сегодня ЭТО рефакторить

Может кто глянет незамыленным взглядом, какие тут косяки имеются?

Может кто глянет незамыленным взглядом, какие тут косяки имеются?

слишком много шакалов

Я, естественно, вижу тут кучу всего. Но исхожу из мнения, что один ум хорошо, а много - лучше

я вижу только INZERT INTO, все остальное отлично, респект таким парням

?

Я только сейчас это увидел

у меня почему-то глаз всегда опечатки находит первым делом

Они кажется решили стебануться над php

прямые переводы с русского на английский тоже всегда радуют. basket.

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

я вижу только INZERT INTO, все остальное отлично, респект таким парням

а вдруг это метод изменённый)

и там всё норм

class BasketsController < ApplicationController

добавление товаров лучше делать через ORM

Тут рефакторить нечего, все сжечь )

ой, он там еще глобальными переменными пользуется. точно сжечь

мамдорогая

Ну это же спецом все

Вообще, тут нужно не столько рефакторить, сколько контекст выстраивать вокруг этого

ну 9 строка, 15 точно можно

Ну это же спецом все

угу, на эмоции давят

Сделай как это положено в рельсах

и потом, если это активрекорд, то нахера там сиквел вообще?

причем еще и антисанитарный

Сделай как это положено в рельсах

Здравая мысль

ретурн в конце - как вишенка на торте =)

Мне почему-то нравится задание, видимо с юмором товарищи

ERROR: S client not available

Мне почему-то нравится задание, видимо с юмором товарищи

с юмором это когда вместо баскет - 111929

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

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

Напиши два варианта

Хотя, может не надо делать то, что не просят )

Господа

Есть у кого-нибудь хорошие идеи, как лаконично осуществить

a ||= b

когда у нас не 2, а 3 варианта с определенной последовательностью приоритетов

когда у нас не 2, а 3 варианта с определенной последовательностью приоритетов

def method a ||= begin b = home_address if prefers_home_address? b = work_address unless b b = addresses.first unless b end end

господа, вопрос

1. Контроллер не RESTful. add_to_basket может быть заменено на update. DHH говорит, когда возникает соблазн добавить не restful метод, он добавляет контроллер. Ничего плохого в существующем названии метода нет на мой взгляд, но я бы рассмотрел возможность заменить его на один из методов REST. 2. find_by_title - говеная функция, не надо искать по title обычно никогда. Т.к. title может меняться в процессе. Нужно юзать id. Вопрос в том, что это будет, sequential id или какой-то хеш или guid. В первом случае существует риск раскрытия инфы о том, какое количество item'ов существует в магазине. 3. params[:item_title] или params[:id] делать не надо, надо явно юзать whitelisting через permit. 4. $basket - хуйня, не должно этого быть. Корзина должна быть 1) одна 2) привязаной к текущему пользователю или сессии (зависит от того, хотим ли мы анонимным юзерам разрешить добавлять в корзину). В любом случае, метод basket можно вынести в Application Controller, и обращаться к нему во всех контроллерах. Например, в header'е возможно потребуется узнать сколько item'ов в баскете. Т.е. строку 5 вообще я бы убрал 5. Строки 9-11, rails way - сделать это через валидацию модели. Ну а это заменить методом is_valid?. Плюс пропущен return - если слишком много товаров, то они все равно добавляются почему-то. 6. 15-19, как я уже объяснил в (4) basket лучше вынести. И сделать это объектом. В объекте будет метод total, поэтому не надо будет считать каждый раз в разных местах, метод total будет доступен. Например вызываешь basket.total и все - всегда актуальная инфа. 7. Гадить в сессию на строке 19 я бы не стал. В сессии можно держать basket_id, для доступа к basket как я описал в (4). Но опять же - инкапсуляция, а значит этих деталей в BasketController не видно. 8. 21 - тут классический sql injection + неверный SQL (INZERT - не такого, есть INSERT). Еcли basket это будет модель, то достаточно будет вызова на строке 7. Ну может .save надо будет добавить. 9. 24 - тоже не нужно по причинам из (8). 10. 26 похоже на code smell. Непонятно почему доставка добавляется таким непонятным способом в basket. Если мы хотим добавлять доставку, то надо ее вынести в lib, чтобы было легче протестировать логику, ну и вообще это бизнес-логика, желательно не держать ее в контроллере. 11. Возвращается html, но title не проверяется на HTML injection. Возвращать html вообще не надо почти никогда. Результат должен быть или render view, или json. 12. Строковые константы (строка 10, 28) надо вынести в locales и обращаться по ключу. 13. Код не отформатирован. Строки 21, 24.

1. Контроллер не RESTful. add_to_basket может быть заменено на update. DHH говорит, когда возникает соблазн добавить не restful метод, он добавляет контроллер. Ничего плохого в существующем названии метода нет на мой взгляд, но я бы рассмотрел возможность заменить его на один из методов REST. 2. find_by_title - говеная функция, не надо искать по title обычно никогда. Т.к. title может меняться в процессе. Нужно юзать id. Вопрос в том, что это будет, sequential id или какой-то хеш или guid. В первом случае существует риск раскрытия инфы о том, какое количество item'ов существует в магазине. 3. params[:item_title] или params[:id] делать не надо, надо явно юзать whitelisting через permit. 4. $basket - хуйня, не должно этого быть. Корзина должна быть 1) одна 2) привязаной к текущему пользователю или сессии (зависит от того, хотим ли мы анонимным юзерам разрешить добавлять в корзину). В любом случае, метод basket можно вынести в Application Controller, и обращаться к нему во всех контроллерах. Например, в header'е возможно потребуется узнать сколько item'ов в баскете. Т.е. строку 5 вообще я бы убрал 5. Строки 9-11, rails way - сделать это через валидацию модели. Ну а это заменить методом is_valid?. Плюс пропущен return - если слишком много товаров, то они все равно добавляются почему-то. 6. 15-19, как я уже объяснил в (4) basket лучше вынести. И сделать это объектом. В объекте будет метод total, поэтому не надо будет считать каждый раз в разных местах, метод total будет доступен. Например вызываешь basket.total и все - всегда актуальная инфа. 7. Гадить в сессию на строке 19 я бы не стал. В сессии можно держать basket_id, для доступа к basket как я описал в (4). Но опять же - инкапсуляция, а значит этих деталей в BasketController не видно. 8. 21 - тут классический sql injection + неверный SQL (INZERT - не такого, есть INSERT). Еcли basket это будет модель, то достаточно будет вызова на строке 7. Ну может .save надо будет добавить. 9. 24 - тоже не нужно по причинам из (8). 10. 26 похоже на code smell. Непонятно почему доставка добавляется таким непонятным способом в basket. Если мы хотим добавлять доставку, то надо ее вынести в lib, чтобы было легче протестировать логику, ну и вообще это бизнес-логика, желательно не держать ее в контроллере. 11. Возвращается html, но title не проверяется на HTML injection. Возвращать html вообще не надо почти никогда. Результат должен быть или render view, или json. 12. Строковые константы (строка 10, 28) надо вынести в locales и обращаться по ключу. 13. Код не отформатирован. Строки 21, 24.

Роман, огромное спасибо

В закладки

домашку списал считай

ну я всё в этой жизни списываю обычно)

хочешь жить - умей вертеться))

в общем, стоило из резюме на hh убрать слово Junior, и предложения пошли валом просто)

и цену поднять с 45к до 60

Интересно )

обычный закон рынка

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

Но молодая на вид и вкус другая

кстати, диаграмма - как я представляю отношения между сущностями https://www.planttext.com/plantuml/svg/SoWkIImgAStDuO80fyJYxDGI1IKj9GKjtT82NGKb0pqzGoM5qEBK8YxFajHSP2cWE4Gz5vT3QbuAo1G0

в этом и соль

Не в первый раз встречаю ссылки на DHH, а где его читать?

алсо, я написал, что в нынешнем виде то, что есть это нифига не контроллер Rails

это просто руби-класс, который по воле прогера назван контроллер

Не в первый раз встречаю ссылки на DHH, а где его читать?

много в твитере пишет) Прям неумолкая)

и типа чтобы он стал контроллером Rails, то должен быть унаследован от ApplicaionController, или ActionController::Base, хотя последнее я бы крайне не рекомендовал

я ведь верно всё изложил?