Encontro de Líderes Técnicos - STI 2013

Transcript

1. I Encontro de Líderes Técnicos 2013 segunda-feira, 17 de junho de 13
2. Sumário • Fat model, skinny controller • before_filter • ActiveRecord • ActiveModel::Validations • REST • SQL Injection • Protecting Attributes • Default Values • to_s • Comentários segunda-feira, 17 de junho de 13
3. segunda-feira, 17 de junho de 13
4. Fat model, skinny controller class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end segunda-feira, 17 de junho de 13
5. Fat model, skinny controller Tá bom? Dá pra refatorar? segunda-feira, 17 de junho de 13
6. Fat model, skinny controller Tá bom? Dá pra refatorar? Qual o problema dessa action? segunda-feira, 17 de junho de 13
7. Fat model, skinny controller class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end segunda-feira, 17 de junho de 13
8. Fat model, skinny controller class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end Quantas condições? class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end segunda-feira, 17 de junho de 13
9. Fat model, skinny controller class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end Quantas condições? class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end segunda-feira, 17 de junho de 13
10. Fat model, skinny controller class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end Quantas condições? class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end segunda-feira, 17 de junho de 13
11. Fat model, skinny controller class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end Quantas condições? class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end segunda-feira, 17 de junho de 13
12. Fat model, skinny controller class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end Quantas condições? Na action? class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end segunda-feira, 17 de junho de 13
13. Fat model, skinny controller Como testa? segunda-feira, 17 de junho de 13
14. Fat model, skinny controller segunda-feira, 17 de junho de 13
15. Fat model, skinny controller class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) if tweet.user == current_user flash[:notice] = "Sorry, you can't retweet your own tweets" elsif tweet.retweets.where(:user_id => current_user.id).present? flash[:notice] = "You already retweeted!" else t = Tweet.new t.status = "RT #{tweet.user.name}: #{tweet.status}" t.original_tweet = tweet t.user = current_user t.save flash[:notice] = "Succesfully retweeted" end redirect_to tweet end end segunda-feira, 17 de junho de 13
16. Fat model, skinny controller segunda-feira, 17 de junho de 13
17. Refatorando... segunda-feira, 17 de junho de 13
18. Refatorando... class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) flash[:notice] = tweet.retweet_by(current_user) redirect_to tweet end end segunda-feira, 17 de junho de 13
19. Refatorando... class TweetsController < ApplicationController def retweet tweet = Tweet.find(params[:id]) flash[:notice] = tweet.retweet_by(current_user) redirect_to tweet end end class Tweet < ActiveRecord::Base def retweet_by(retweeter) if self.user == retweeter "Sorry, you can't retweet your own tweets" elsif self.retweets.where(:user_id => retweeter.id).present? "You already retweeted!" else ... "Succesfully retweeted" end end end segunda-feira, 17 de junho de 13
20. Como testa? • Agora temos um método de instância chamado retweet_by • Testamos facilmente pelo RSpec • O número de testes necessários no Cucumber também foi reduzido... segunda-feira, 17 de junho de 13
21. before_filter class TweetsController < ApplicationController def edit @tweet = Tweet.find(params[:id]) ... end def update @tweet = Tweet.find(params[:id]) ... end def destroy @tweet = Tweet.find(params[:id]) end end segunda-feira, 17 de junho de 13
22. before_filter segunda-feira, 17 de junho de 13
23. before_filter Todas as minhas actions usam o mesmo find... segunda-feira, 17 de junho de 13
24. before_filter Todas as minhas actions usam o mesmo find... Então...eu crio um before_filter? segunda-feira, 17 de junho de 13
25. before_filter class TweetsController < ApplicationController before_filter :get_tweet, :only => [:edit, :update, :destroy] def get_tweet @tweet = Tweet.find(params[:id]) end def edit ... end def update ... end def destroy end end segunda-feira, 17 de junho de 13
26. before_filter class TweetsController < ApplicationController before_filter :get_tweet, :only => [:edit, :update, :destroy] def get_tweet @tweet = Tweet.find(params[:id]) end def edit ... end def update ... end def destroy end end Ficou melhor? segunda-feira, 17 de junho de 13
27. before_filter class TweetsController < ApplicationController before_filter :get_tweet, :only => [:edit, :update, :destroy] def get_tweet @tweet = Tweet.find(params[:id]) end def edit ... end def update ... end def destroy end end Ficou melhor? segunda-feira, 17 de junho de 13
28. before_filter O que há de errado? segunda-feira, 17 de junho de 13
29. before_filter O que há de errado? Essa forma de fazer não é sustentável... segunda-feira, 17 de junho de 13
30. before_filter segunda-feira, 17 de junho de 13
31. before_filter #comofaz? segunda-feira, 17 de junho de 13
32. class TweetsController < ApplicationController def edit @tweet = get_tweet(params[:id]) end def update @tweet = get_tweet(params[:id]) end def destroy @tweet = get_tweet(params[:id]) end private def get_tweet(tweet_id) Tweet.find(tweet_id) end end segunda-feira, 17 de junho de 13
33. before_filter Quando eu uso então? segunda-feira, 17 de junho de 13
34. before_filter Quando eu uso então? - Autorização - Log segunda-feira, 17 de junho de 13
35. Models e o ActiveRecord • Os models não precisam necessariamente herdar de ActiveRecord! • Existem classes que não precisam saber persistir no banco de dados. • Um exemplo: • Um formulário de envio de e-mail... segunda-feira, 17 de junho de 13
36. <h1>Contact Us</h1> <%= form_for :contact, :url => send_email_path do |f| %> <div class="field"> <%= f.label :name %><br /> <%= f.text_field :name %> </div> <div class="field"> <%= f.label :email %><br /> <%= f.text_field :email %> </div> <div class="field"> <%= f.label :body %><br /> <%= f.text_area :body %> </div> <div class="actions"> <%= f.submit %> </div> <% end %> segunda-feira, 17 de junho de 13
37. class ContactUsController < ApplicationController def new end def send_email name = params[:contact][:name] email = params[:contact][:email] body = params[:contact][:body] if name.blank? || email.blank? || body.blank? flash.now[:notice] = "Please fill out all fields" render :action => 'new' else Notifications.contact_us(name, email, body).deliver flash[:notice] = "Email sent, we'll get back to you" redirect_to root_path end end end segunda-feira, 17 de junho de 13
38. class ContactUsController < ApplicationController def new @contact_form = ContactForm.new end def send_email @contact_form = ContactForm.new(params[:contact_form]) if !@contact_form.valid? render :action => 'new' else Notifications.contact_us(@contact_form).deliver flash[:notice] = "Email sent, we'll get back to you" redirect_to root_path end end end segunda-feira, 17 de junho de 13
39. class ContactUsController < ApplicationController def new @contact_form = ContactForm.new end def send_email @contact_form = ContactForm.new(params[:contact_form]) if @contact_form.valid? Notifications.contact_us(@contact_form).deliver flash[:notice] = "Email sent, we'll get back to you" redirect_to root_path else render :action => 'new' end end end segunda-feira, 17 de junho de 13
40. class ContactUsController < ApplicationController def new @contact_form = ContactForm.new end def send_email @contact_form = ContactForm.new(params[:contact_form]) if @contact_form.valid? Notifications.contact_us(@contact_form).deliver redirect_to root_path, :notice => "Email sent, we'll get back to you" else render :action => 'new' end end end segunda-feira, 17 de junho de 13
41. class ContactUsController < ApplicationController def new @contact_form = ContactForm.new end def send_email @contact_form = ContactForm.new(params[:contact_form]) if @contact_form.valid? Notifications.contact_us(@contact_form).deliver redirect_to root_path, :notice => "Email sent, we'll get back to you" else render :new end end end segunda-feira, 17 de junho de 13
42. class ContactForm include ActiveModel::Validations include ActiveModel::Conversion attr_accessor :name, :email, :body validates_presence_of :name, :email, :body def initialize(attributes = {}) attributes.each do |name, value| send("#{name}=", value) end end def persisted? false end end segunda-feira, 17 de junho de 13
43. Um outro exemplo... segunda-feira, 17 de junho de 13
44. Antes... segunda-feira, 17 de junho de 13
45. Depois! segunda-feira, 17 de junho de 13
46. REST class UsersController < ApplicationController def subscribe_mailing_list current_user.subscribe(params[:id]) redirect_to current_user, :notice => "You've been subscribed" end def unsubscribe_mailing_list current_user.unsubscribe(params[:id]) redirect_to current_user, :notice => "You have been unsubscribed" end end segunda-feira, 17 de junho de 13
47. REST class SubscriptionsController < ApplicationController def create current_user.subscribe(params[:id]) redirect_to current_user, :notice => "You've been subscribed" end def destroy current_user.unsubscribe(params[:id]) redirect_to current_user, :notice => "You have been unsubscribed" end end segunda-feira, 17 de junho de 13
48. REST class SubscriptionsController < ApplicationController def create current_user.subscribe(params[:id]) redirect_to current_user, :notice => "You've been subscribed" end def destroy current_user.unsubscribe(params[:id]) redirect_to current_user, :notice => "You have been unsubscribed" end end ActiveResource::Base segunda-feira, 17 de junho de 13
49. SQL Injection User.where("name = #{params[:name]}") segunda-feira, 17 de junho de 13
50. SQL Injection User.where("name = ?", params[:name]) segunda-feira, 17 de junho de 13
51. SQL Injection User.where(:name => params[:name]) segunda-feira, 17 de junho de 13
52. SQL Injection Tweet.where("created_at >= :start_date AND created_at <= :end_date", {:start_date => params[:start_date], :end_date => params[:end_date]}) segunda-feira, 17 de junho de 13
53. SQL Injection Tweet.where(:created_at => (params[:start_date].to_date)..(params[:end_date].to_date)) segunda-feira, 17 de junho de 13
54. Protecting Attributes attr_protected vs attr_accessible segunda-feira, 17 de junho de 13
55. Protecting Attributes attr_protected vs attr_accessible segunda-feira, 17 de junho de 13
56. Protecting Attributes attr_protected vs attr_accessible Prefira whitelists! segunda-feira, 17 de junho de 13
57. Default Values class AccountSetting < ActiveRecord::Base belongs_to :user before_create :set_default_timezone def set_default_timezone self.time_zone = "EST" end end segunda-feira, 17 de junho de 13
58. Default Values class AccountSetting < ActiveRecord::Base belongs_to :user end class AddDefaultTimeZoneToAccountSettings < ActiveRecord::Migration def self.up change_column_default :account_settings, :time_zone, 'EST' end def self.down change_column :account_settings, :time_zone, :string, nil end end segunda-feira, 17 de junho de 13
59. to_s class User < ActiveRecord::Base def display_name "#{first_name} #{last_name}" end end <%= @user.display_name %> segunda-feira, 17 de junho de 13
60. to_s class User < ActiveRecord::Base def to_s "#{first_name} #{last_name}" end end <%= @user %> segunda-feira, 17 de junho de 13
61. Comentários segunda-feira, 17 de junho de 13
62. Comentários Cuidado com eles! segunda-feira, 17 de junho de 13
63. Comentários Se o seu método precisa de comentários, ele merece uma atenção especial! segunda-feira, 17 de junho de 13
64. Comentários Seus métodos devem ser especialistas. segunda-feira, 17 de junho de 13
65. Comentários Seus métodos devem ser especialistas. Eles devem ter um motivo óbvio e explícito para existir... segunda-feira, 17 de junho de 13
66. Comentários Seus métodos devem ser especialistas. Se isso não acontece... Eles devem ter um motivo óbvio e explícito para existir... segunda-feira, 17 de junho de 13
67. Comentários Seus métodos devem ser especialistas. Se isso não acontece... Refatore! Eles devem ter um motivo óbvio e explícito para existir... segunda-feira, 17 de junho de 13
68. Acontece na STI... segunda-feira, 17 de junho de 13
69. Acontece na STI... segunda-feira, 17 de junho de 13
70. Livros segunda-feira, 17 de junho de 13
71. Livros segunda-feira, 17 de junho de 13
72. Livros segunda-feira, 17 de junho de 13
73. Livros segunda-feira, 17 de junho de 13
74. Obrigado! segunda-feira, 17 de junho de 13