Watchman assignment submission

[codeblahblah]

@jwo Assignment is mostly complete apart from the "What would you like to learn more about?" feedback scenario.

[codeblahblah]

@jwo Show.by_day(input) is a class method call. It works. Is this the right approach?

[codeblahblah]

@jwo Show.by_day(input) is a class method. It works. Is this the correct approach?

[jwo]

Yes, the class method is the correct approach.

Great stuff, and I was greatly surprised at the use of lorem ipsum for your own beers :)

[codeblahblah]

@jwo Awesome! Had another Ah-ha moment. I no longer see Sinatra code. Or Rails code. It's all Ruby!!! The question I ask is: "What would Ruby do?". Or rather, "How would it look in Ruby?".
By knowing the Ruby-isms, you can find your way to the solution. Rails itself is based on Ruby's own conventions...

P.S: I should create a craft beer named Lorem Ipsum which would be targeted specifically to Web Devs who refuse to use Ruby/Rails.

[jwo]

AWESOME

[jwo]

Great submission.

Had another Ah-ha moment. I no longer see Sinatra code. Or Rails code. It's all Ruby!!! The question I ask is: "What would Ruby do?". Or rather, "How would it look in Ruby?".
By knowing the Ruby-isms, you can find your way to the solution. Rails itself is based on Ruby's own conventions...

That's SO COOL.

[codeblahblah]

@jwo Thanks a lot. After I wrote that comment, I came to the realisation that I can change my custom Show#by_day to Show#find_all_by_day_of_week which ActiveRecord should make available to me. More convention! I literally 'guessed' this. Show.methods.sort doesn't list any 'find' methods, are they hidden in the _find_callback method? Metaprogramming, maybe?

[jwo]

Yes, the “find_by” is metaprogramming, using method_missing. If you call a method that doesn’t exist, it will see if the method you sent starts with “find_by” — if it does, then it does it’s magic

Note that the “find_by_email” is deprecated in Rails 4. Instead, it has a “find_by” method.

So, instead of

User.find_by_email "[email protected]"

You’d

User.find_by email: "[email protected]"

————————

Secondly, to query for many things, you’ll want to use “where”

Instead of

Show.find_all_by_day_of_week "Tuesday"

You should

Show.where(day_of_week: "Tuesday")

[jwo]

More info on the ActiveRecord 3.0 query interface:

• http://railscasts.com/episodes/202-active-record-queries-in-rails-3
• http://railscasts.com/episodes/215-advanced-queries-in-rails-3

And on the 4.0 changes:

• http://blog.remarkablelabs.com/2012/12/what-s-new-in-active-record-rails-4-countdown-to-2013

[codeblahblah]

@jwo Many thanks. Will go read/watch.
Does using Show#where and the other changes stop SQL injections (i.e. better sanitization)?

[jwo]

As long as you use the AR interface, you will be ok.

That is, don’t hand craft SQL…

But, find_by_email(“jesse”) is safe from attacked like find_by_email(“jesse ‘; DROP TABLE USERS”)

Jesse Wolgamott @ Comal Productions, Inc.
Web. Mobile. Design. Training.
@jwo
http://comalproductions.com

On Tuesday, February 18, 2014 at 10:15 AM, drammopo wrote:

@jwo (https://github.com/jwo) Many thanks. Will go read/watch.
Does using Show#where and the other changes stop SQL injections (i.e. better sanitization)?

—
Reply to this email directly or view it on GitHub (#30 (comment)).