3

Есть консерн:

module Omniauthable
  extend ActiveSupport::Concern
  included do
    def self.find_for_oauth(auth)
      authorization = Authorization.where(provider: auth.provider, uid: auth.uid.to_s).first
      return authorization.user if authorization

      if auth.info.try(:email)
        email = auth.info[:email]
      else
        return false
      end

      user = User.where(email: email).first
      if user
        user.create_authorization(auth)
      else
        password = Devise.friendly_token[0, 20]
        user = User.new(email: email,
                        password: password,
                        password_confirmation: password)
        if user.valid?
          user.save!
          user.create_authorization(auth)
        else
          return false
        end
      end
      user
    end

    def create_authorization(auth)
      authorizations.create(provider: auth.provider, uid: auth.uid)
    end
  end
end

Нужно разделить self.find_for_oauth(auth), чтобы количество строк в методе было хотя бы 15.

2
  • На вас rubocop ругается что ли? :)
    – user181100
    16 дек 2015 в 9:40
  • Да, ругается, я в конфиге увеличил Metrics/MethodLength, но это ведь не решение проблемы, хотелось бы разделить как-то сам метод что бы он был не такой большой 16 дек 2015 в 9:49

2 ответа 2

2

Я бы переписал так:

module Omniauthable
  extend ActiveSupport::Concern
  included do
    def self.find_for_oauth(auth)
      authorization = Authorization.where(provider: auth.provider, uid: auth.uid.to_s).first
      if authorization
        authorization.user
      else
        @email = auth.info.try(:email)
        find_user # ищем пользователя
        create_user unless @user # если не находим, то создаем
        auth_user(auth) # пытаемся авторизовать и возвращаем либо nil либо пользователя
      end
    end

    def create_authorization(auth)
      authorizations.create(provider: auth.provider, uid: auth.uid)
    end
    private # скрываем служебные методы

    def self.find_user
      @user = User.where(email: @email).first if @email
    end

    def self.create_user
      password = Devise.friendly_token[0, 20]
      udata = {
        email: @email,
        password: password,
        password_confirmation: password
      }
      user = User.create(udata) # здесь происходит валидация и сохранение
      @user = user if user.errors.count == 0 # если после создания нет ошибок, то объявляем переменную экземпляра
    end

    def self.auth_user(auth)
      if @user # если в предыдущих методах был найден/создан пользователь, если @user = nil(не существует, то nil вернется из метода)
        @user.create_authorization(auth)
        @user
      end
    end
  end
end
5
  • Я поправил. В этот метод просто нужно передать переменную auth
    – MAXOPKA
    16 дек 2015 в 13:37
  • ааа, точно. Он же явно вызывается, значит не privare. Сейчас поправлю.
    – MAXOPKA
    16 дек 2015 в 13:45
  • password тоже поправил.
    – MAXOPKA
    16 дек 2015 в 13:48
  • а как быть с Failure/Error: authorizations.create(provider: auth.provider, uid: auth.uid) ActiveRecord::RecordNotSaved: You cannot call create unless the parent is saved 16 дек 2015 в 13:52
  • Попробуйте последнюю версию моего кода. Суда по тексту ошибки - авторизуемый пользователь не сохранен в БД
    – MAXOPKA
    16 дек 2015 в 17:19
1

Не злоупотребляйте культом Карго.

Если какой-то код работает без разбивки, то пусть работает дальше.

4
  • Не является ответом на вопрос. Чтобы оставить свои замечания или попросить у автора внести уточнения, оставьте комментарий к соответствующему сообщению. - Из очереди проверок 16 дек 2015 в 17:51
  • @Nakilon если в него не требуется добавлять функционал, то это так.
    – MAXOPKA
    16 дек 2015 в 18:19
  • @MAXOPKA, а добавляя функционал, при куче методов начнутся песнопляски с передачей аргументов и областями видимости.
    – Nakilon
    16 дек 2015 в 22:58
  • @Suvitruf: мне кажется, это вполне ответ. Просто перенос акцента от задачи к цели.
    – Nick Volynkin
    30 дек 2015 в 18:38

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.