refactoring(コードを整理する)

※本サイトで紹介している商品・サービス等の外部リンクには、アフィリエイト広告が含まれる場合があります。

refactoringとは

プログラミングのコードを書いていくと、無駄なコードが出てくる

(当初は必要と思ったコードでも、後々不要になったりする)

定期的に不要なコードを整理して、読みやすくしておく

この作業を「refactoring」という


ブランチを作成する

ブランチ名、「refactoring」で分岐を作成する

不要なコードを整理する

現在進めているコードはこちら


意味がなくなっているコードを整理する

config/routes.rbを見る

resources :articles,
only: [:show, :new, :create, :edit, :update, :destroy]

only:以降にresourcesしたいコードを書いているが、ほぼ全部になっているのでonly:は不要である

(:index以外、全部書いてある)

only:オプションを削除してスッキリ

resources :articles

重複しているコードを整理する

プログラミングの中で、同じコードは二度と書かない(ドライと言う)

ひとつ変更があったとき、すべて変更しないといけないから効率が悪い

@article = Article.find(params[:id])が多い

    def show
        @article = Article.find(params[:id])
    end

    def edit
        @article = Article.find(params[:id])
    end

    def update
        @article = Article.find(params[:id])
           #中略
    end

before_actionを使って重複しているコードを削除する

  before_action :set_article, only:[:show, :edit, :update]

           #中略

    private
    def set_article
        @article = Article.find(params[:id])
    end

#DAY15