before_validate refactor question

32 views
Skip to first unread message

Dave Castellano

unread,
Sep 3, 2013, 8:55:04 AM9/3/13
to rubyonra...@googlegroups.com
Had this working until I refactored by putting into a hash and checking
for blank. Now I get the error: undefined method `key?' for
nil:NilClass.

I'm still kind of new at this and would appreciate suggestions on what I
am missing...


before_validation :capitalize_first_words

def capitalize_first_words
[:question, :correct_ans_1, :correct_ans_2, :correct_ans_3].each
do |key|
self.key = self.key.gsub(/^(\W*?[a-z])/) { |m| m.upcase } if
self.key != blank
end
end


Thanks,

Dave

--
Posted via http://www.ruby-forum.com/.

Michael Lutsiuk

unread,
Sep 3, 2013, 12:36:49 PM9/3/13
to rubyonra...@googlegroups.com
Maybe you mean this?

before_validation :capitalize_first_words

def capitalize_first_words
      [:question, :correct_ans_1, :correct_ans_2, :correct_ans_3].each do |key| 
            self.key = key.gsub(/^(\W*?[a-z])/) { |m| m.upcase } if !key.blank?
      end 
end 

вторник, 3 сентября 2013 г., 15:55:04 UTC+3 пользователь Ruby-Forum.com User написал:

Dave Castellano

unread,
Sep 3, 2013, 3:10:36 PM9/3/13
to rubyonra...@googlegroups.com
Michael Aleksandrovich wrote in post #1120515:
> Maybe you mean this?
>
> before_validation :capitalize_first_words
>
> def capitalize_first_words
> [:question, :correct_ans_1, :correct_ans_2, :correct_ans_3].each
> do
> |key|
> self.key = key.gsub(/^(\W*?[a-z])/) { |m| m.upcase }
> if !key.blank?
> end
> end
>
> , 3 2013 ., 15:55:04 UTC+3 Ruby-Forum.com
> User :


Thanks!

Dave Castellano

unread,
Sep 3, 2013, 6:03:10 PM9/3/13
to rubyonra...@googlegroups.com
Now giving error:
undefined method `key' for #<Question:0x007fdff26b7798>

In Question model:
before_validation :capitalize_first_words

def capitalize_first_words
[:question, :correct_ans_1, :correct_ans_2, :correct_ans_3].each
do |key|

self.key = self.key.gsub(/^(\W*?[a-z])/) { |m| m.upcase } if
!self.key.blank?
end
end

Am I missing something really quite simple?

Thanks

Michael Lutsiuk

unread,
Sep 4, 2013, 2:18:18 AM9/4/13
to rubyonra...@googlegroups.com
 before_validation :capitalize_first_words 

    def capitalize_first_words 
      [:question, :correct_ans_1, :correct_ans_2, :correct_ans_3].each do |key| 

          self.key = key.to_s.gsub(/^(\W*?[a-z])/) { |m| m.upcase } if !key.to_s.blank? 
      end 
    end 

Note the difference betweet self.key and key.
self.key calls method key on self.
key - is block local variable, which is instance of Symbol ( :question, :correct_ans_1 ...).
Symbol has not methods '.gsub' and '.blank?', so it must be converted to String to call '.gsub' and '.blank?'

среда, 4 сентября 2013 г., 1:03:10 UTC+3 пользователь Ruby-Forum.com User написал:

Dave Castellano

unread,
Sep 4, 2013, 8:17:47 AM9/4/13
to rubyonra...@googlegroups.com
But, even the following causes error: undefined method `key=' for
#<Question:0x007fcba5972f20>

before_validation :capitalize_first_words

def capitalize_first_words
[:question, :correct_ans_1, :correct_ans_2].each do|key|
self.key = "x"
end
end

key is still not being interpreted as "question" or "correct_ans_1",
ect...
Could it be a scope issue? I am in the Question model.

Walter Lee Davis

unread,
Sep 4, 2013, 8:27:03 AM9/4/13
to rubyonra...@googlegroups.com

On Sep 4, 2013, at 8:17 AM, Dave Castellano wrote:

> But, even the following causes error: undefined method `key=' for
> #<Question:0x007fcba5972f20>
>
> before_validation :capitalize_first_words
>
> def capitalize_first_words
> [:question, :correct_ans_1, :correct_ans_2].each do|key|
> self.key = "x"

maybe

self.send(":#{key}=", "x")

or:

self[key] = "x"

Not sure of either. But your instinct is correct, key isn't being evaluated before it is passed to the self reference, so you're still looking for a key method that isn't there.


> end
> end
>
> key is still not being interpreted as "question" or "correct_ans_1",
> ect...
> Could it be a scope issue? I am in the Question model.
>

Walter

> --
> Posted via http://www.ruby-forum.com/.
>
> --
> You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to rubyonrails-ta...@googlegroups.com.
> To post to this group, send email to rubyonra...@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/rubyonrails-talk/1e2a081f66832bf47930d5c31699819f%40ruby-forum.com.
> For more options, visit https://groups.google.com/groups/opt_out.

Michael Lutsiuk

unread,
Sep 4, 2013, 9:45:17 AM9/4/13
to rubyonra...@googlegroups.com
Instance of Question doesn't have attribute "key".
If this is ActiveRecord, you maybe should make migration "add_column :questions, :key, :string", OR within class scope write "attr_accessor :key"

среда, 4 сентября 2013 г., 15:17:47 UTC+3 пользователь Ruby-Forum.com User написал:

Dave Castellano

unread,
Sep 4, 2013, 3:25:32 PM9/4/13
to rubyonra...@googlegroups.com
Michael Aleksandrovich wrote in post #1120644:
> Instance of Question doesn't have attribute "key".
> If this is ActiveRecord, you maybe should make migration "add_column
> :questions, :key, :string", OR within class scope write "attr_accessor
> :key"
>
> , 4 2013 ., 15:17:47 UTC+3 Ruby-Forum.com User
> :

I thought key was a variable containing the attribute. The instance of
Question should have the attribute contained in the variable... It just
doesn't seem to be evaluating key.

I'm sure i'm missing something simple...

Any other possibilities?

Dave
Reply all
Reply to author
Forward
0 new messages