The "undefined method" becomes "for nil: NilClass, where it should not

I tested the fragment today

unless resource.nil?
  resource = resource.becomes(Accounts::Admin)
end

it causes an error

undefined method becomes for nil: NilClass

if i do that

unless resource.nil?
  a = resource.becomes(Accounts::Admin)
  resource = a
end

Everything is going right.

What is the difference if the first part of the statement is executed =?

EDIT:

Something unpleasant happens, the last line in if falseis executed, but "ALOHA" is never printed.

<%
puts "AAAA #{resource.inspect}"
if false
    puts "ALOHA"

    # this line is being executed !
    # if I comment it out the BBBB output is correct
    resource = nil
end 
puts "BBBB #{resource.inspect}"
%>

he prints

AAAA User ID: nil, nome: nil, endereco_id: nil, created_at: nil, updated_at: nil, email: ""

BBBB nil

but if i do it

<%
res = resource

puts "AAAA #{res.inspect}"
if false
  puts "ALOHA"
  res = nil
end 
puts "BBBB #{res.inspect}"
%>

printed correctly

AAAA User ID: nil, nome: nil, endereco_id: nil, created_at: nil, updated_at: nil, email: "

BBBB User ID: nil, nome: nil, endereco_id: nil, created_at: nil, updated_at: nil, email: ""

. devise/registrations/new.html.erb. , resource User, RegistrationController

, , , - .

~/.rvm/gems/ruby-2.3.3@mygem/gems/devise-4.2.0/app/controllers/devise/registrations_controller.rb

class Devise::RegistrationsController < DeviseController
  prepend_before_action :require_no_authentication, only: [:new, :create, :cancel]
  prepend_before_action :authenticate_scope!, only: [:edit, :update, :destroy]
  prepend_before_action :set_minimum_password_length, only: [:new, :edit]

  # GET /resource/sign_up
  def new
    build_resource({})
    yield resource if block_given?
    respond_with resource
  end

  # POST /resource
  def create
    build_resource(sign_up_params)

    resource.save
    yield resource if block_given?
    if resource.persisted?
      if resource.active_for_authentication?
        set_flash_message! :notice, :signed_up
        sign_up(resource_name, resource)
        respond_with resource, location: after_sign_up_path_for(resource)
      else
        set_flash_message! :notice, :"signed_up_but_#{resource.inactive_message}"
        expire_data_after_sign_in!
        respond_with resource, location: after_inactive_sign_up_path_for(resource)
      end
    else
      clean_up_passwords resource
      set_minimum_password_length
      respond_with resource
    end
  end

  # GET /resource/edit
  def edit
    render :edit
  end

  # PUT /resource
  # We need to use a copy of the resource because we don't want to change
  # the current user in place.
  def update
    self.resource = resource_class.to_adapter.get!(send(:"current_#{resource_name}").to_key)
    prev_unconfirmed_email = resource.unconfirmed_email if resource.respond_to?(:unconfirmed_email)

    resource_updated = update_resource(resource, account_update_params)
    yield resource if block_given?
    if resource_updated
      if is_flashing_format?
        flash_key = update_needs_confirmation?(resource, prev_unconfirmed_email) ?
          :update_needs_confirmation : :updated
        set_flash_message :notice, flash_key
      end
      bypass_sign_in resource, scope: resource_name
      respond_with resource, location: after_update_path_for(resource)
    else
      clean_up_passwords resource
      respond_with resource
    end
  end

  # DELETE /resource
  def destroy
    resource.destroy
    Devise.sign_out_all_scopes ? sign_out : sign_out(resource_name)
    set_flash_message! :notice, :destroyed
    yield resource if block_given?
    respond_with_navigational(resource){ redirect_to after_sign_out_path_for(resource_name) }
  end

  # GET /resource/cancel
  # Forces the session data which is usually expired after sign
  # in to be expired now. This is useful if the user wants to
  # cancel oauth signing in/up in the middle of the process,
  # removing all OAuth session data.
  def cancel
    expire_data_after_sign_in!
    redirect_to new_registration_path(resource_name)
  end

  protected

  def update_needs_confirmation?(resource, previous)
    resource.respond_to?(:pending_reconfirmation?) &&
      resource.pending_reconfirmation? &&
      previous != resource.unconfirmed_email
  end

  # By default we want to require a password checks on update.
  # You can overwrite this method in your own RegistrationsController.
  def update_resource(resource, params)
    resource.update_with_password(params)
  end

  # Build a devise resource passing in the session. Useful to move
  # temporary session data to the newly created user.
  def build_resource(hash=nil)
    self.resource = resource_class.new_with_session(hash || {}, session)
  end

  # Signs in a user on sign up. You can overwrite this method in your own
  # RegistrationsController.
  def sign_up(resource_name, resource)
    sign_in(resource_name, resource)
  end

  # The path used after sign up. You need to overwrite this method
  # in your own RegistrationsController.
  def after_sign_up_path_for(resource)
    after_sign_in_path_for(resource)
  end

  # The path used after sign up for inactive accounts. You need to overwrite
  # this method in your own RegistrationsController.
  def after_inactive_sign_up_path_for(resource)
    scope = Devise::Mapping.find_scope!(resource)
    router_name = Devise.mappings[scope].router_name
    context = router_name ? send(router_name) : self
    context.respond_to?(:root_path) ? context.root_path : "/"
  end

  # The default url to be used after updating a resource. You need to overwrite
  # this method in your own RegistrationsController.
  def after_update_path_for(resource)
    signed_in_root_path(resource)
  end

  # Authenticates the current scope and gets the current resource from the session.
  def authenticate_scope!
    send(:"authenticate_#{resource_name}!", force: true)
    self.resource = send(:"current_#{resource_name}")
  end

  def sign_up_params
    devise_parameter_sanitizer.sanitize(:sign_up)
  end

  def account_update_params
    devise_parameter_sanitizer.sanitize(:account_update)
  end

  def translation_scope
    'devise.registrations'
  end
end

2.3.3 (4.2.7.1) devise (4.2.0)

+4
2

ruby:

if true
  foo = "hello"
end
puts foo

#=> hello

:

if false
  foo = "hello"
end
puts foo

#=> nil

if-statements , . , , if-, if-statement.

, , Ruby , if-statement true false. , ruby ​​ nil, .

:

unless resource.nil?
  resource = resource.becomes(Accounts::Admin)
end

- ruby, . , resource = resource, resource resource, .

:

undefined ` ' nil: NilClass

resource, . , resource nil. , , , resource= nil .

:

def blah
  "foo"
end

unless blah.nil?
  blah = blah.size
end
puts blah

, :

def blah
  "foo"
end

def blah= value
  #do nothing
end

unless blah.nil?
  self.blah = blah.size
end

puts blah

, , devise resource=(). , - , - :

unless resource.nil?
  res = resource.becomes(Accounts::Admin)
end
puts res

, ruby ​​ , , . :

if x="foo"
  puts x
end
#=> "foo"

puts y if y="foo"
#NameError: undefined variable or method 'y'

ruby, matz.

+5

IRb, :

a = 'hello'
unless a.nil?
  a = a.upcase
end
a
# => "HELLO"

, = var .

, ? , stacktrace , becomes ?

0

Source: https://habr.com/ru/post/1676381/


All Articles