New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

indexのインスタンス変数は`@users`のように複数形にすべき #68

Open

JunichiIto opened this issue Jun 4, 2016 · 0 comments

Labels

パフォーマンス改善

Collaborator

JunichiIto commented Jun 4, 2016

class AdminController < ApplicationController
  before_action :authenticate_user!
  before_action :admin_user, except: [:show]

  PER = 5
  def index
    @user = User.page(params[:page]).per(PER).all
  end

  def show
    if current_user.admin?
      @user = User.find(params[:id])
      @result = @user.results
    else
      @user = current_user
    end
  end
end

keisukefunatsu added the パフォーマンス改善 label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment