常連

## テーマ、知りたいこと

プログラムの作り方なんて、個人の自由でいいですよね。
他所の環境では、コードレビューではどんな突っ込みが入るか聞いてみたいです。

## 背景、状況

このようなif文を作りました。

```
if Dir.exist?("/hoge")
else
	return "ディレクトリが存在しません"
end
```

すると、このように直せと言ってくる人がいるかと思えば、
```
if !Dir.exist?("/hoge")
	return "ディレクトリが存在しません"
end
```

このように直せと言ってくる人もいます。
```
unless Dir.exist?("/hoge")
	return "ディレクトリが存在しません"
end
```

どうでもいいじゃんと思っているのですが、なぜに、そんな細かいことをいちいち突っ込んでこられなきゃならないのかと疑問に思っています。
皆さんはどのように思われますか。


コーディングの仕方は個人の自由でいいと思いませんか

エンジニアならこれは読んでおけ みたいな本を教えてください。
ほかにも読んですごく良かった本などでも知りたいです。

アルゴリズムの本や正規表現、TCP/IPなどなど
お願いします

読んでおいたほうがいい本

### 実現したいこと
起動したコンテナのpublic直下に404.htmlと500.htmlを反映する

### 発生している問題・分からないこと
エラーページ配信用にRailsのpublicに404.htmlと500.htmlを作成済みです
しかし、no cache状態でビルド後、コンテナ起動しても上記二つのファイルがコンテナに存在しません

以下、詳細です
- ホストのpublic/には確実に404.htmlと500.htmlが存在する
- .dockerignoreで**public**は指定されていない
- .dockerignoreで!/public/404.htmlと!/public/500.htmlを明示しても変わらず
- デフォルトのpublic/robots.txtはコンテナに反映されている
- リバプロとして設置しているNginxコンテナからエラーページを返すのが目的

### エラーメッセージ
```error
なし
```

### 該当のソースコード

```Dockerfile
# --- ビルドステージ ---
# 最新軽量版Rubyのイメージ（2025年1月時点）
FROM ruby:3.3.7-alpine AS builder

ENV LANG=C.UTF-8 \
    TZ=Asia/Tokyo

RUN apk update && apk add --no-cache \
    build-base \
    yaml-dev \
    postgresql-dev \
    postgresql-client \
    git \
    bash \
    tzdata

WORKDIR /app
COPY Gemfile Gemfile.lock /app/
RUN bundle install
COPY . /app/

# --- 実行ステージ ---
FROM ruby:3.3.7-alpine

ENV LANG=C.UTF-8 \
    TZ=Asia/Tokyo

RUN apk update && apk add --no-cache \
    postgresql-client \
    bash \
    tzdata

WORKDIR /app
COPY --from=builder /app/ /app/
COPY --from=builder /usr/local/bundle /usr/local/bundle

COPY entrypoint.sh /usr/bin/
RUN chmod +x /usr/bin/entrypoint.sh
ENTRYPOINT [ "entrypoint.sh" ]

EXPOSE 3000

CMD [ "bundle", "exec", "puma", "-C", "config/puma.rb" ]
```

```nginx.conf
## Nginxの設定

# Pumaとの通信に Unixドメインソケット を利用
upstream api {
  server unix:///app/tmp/sockets/puma.sock;
}

server {
  listen 80;
  server_name localhost;

  # ログ出力先
  access_log /var/log/nginx/access.log;
  error_log  /var/log/nginx/error.log;

  # 静的ファイルのルート（Railsの /public を参照）
  root /app/public;

  # クライアントがアップロード可能な最大リクエストサイズ(m = MB)
  client_max_body_size 100m;

  # エラーページ指定
  error_page 404             /404.html;
  error_page 502 503 504 505 /500.html;

  # エラーページへの直接アクセスをブロック
  location = /404.html {
    internal;
  }

  location = /500.html {
    internal;
  }

  # リクエストファイル($uri)が存在すれば返し、なければRailsへルーティング
  try_files $uri @api;

  # リバースプロキシ設定
  location @api {
    proxy_pass       http://api;
    proxy_set_header Host $host;
    proxy_set_header X-Real-IP $remote_addr;
    proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
  }

  keepalive_timeout 5;
}
```

### 試したこと・調べたこと
- [x] teratailやGoogle等で検索した
- [x] ソースコードを自分なりに変更した
- [ ] 知人に聞いた
- [ ] その他

##### 上記の詳細・結果
マルチステージビルド導入済み。実行ステージをコメントアウトしても変わらないので、ビルド段階で404.htmlと500.htmlが何かの理由で含まれていない気がします。

### 補足
Ruby: 3.3.7　Rails: 7.2.2.1
RailsはAPIモード

これが原因でECS on Fargateへのデプロイ（タスク起動）が失敗して困っています

Railsのpublicディレクトリ直下のファイルがコンテナに反映されない

### 前提・実現したいこと

該当箇所とされるファイルを一通り確認しても問題箇所が見つからず、困っています。
rails初心者です。

### 発生している問題・エラーメッセージ

[Railsチュートリアル10章](https://railstutorial.jp/chapters/updating_and_deleting_users?version=5.1#cha-updating_showing_and_deleting_users)のリスト10.26より、
rails testを走らせた時に10errorが発生しました。

（直近テスト　リスト10.22はエラー無し）


```Ruby
  1) Error:
UsersEditTest#test_successful_edit:
NoMethodError: undefined method `logged_in_user' for #<UsersController:0x007fa114dbe2e8>
Did you mean?  logged_in?
    test/integration/users_edit_test.rb:25:in `block in <class:UsersEditTest>'

  2) Error:
UsersEditTest#test_unsuccessful_edit:
NoMethodError: undefined method `logged_in_user' for #<UsersController:0x007fa114d6cfd8>
Did you mean?  logged_in?
    test/integration/users_edit_test.rb:12:in `block in <class:UsersEditTest>'

  3) Error:
UsersControllerTest#test_should_redirect_edit_when_logged_in_as_wrong_user:
NoMethodError: undefined method `logged_in_user' for #<UsersController:0x00000006b5e658>
Did you mean?  logged_in?
    test/controllers/users_controller_test.rb:33:in `block in <class:UsersControllerTest>'

  4) Error:
UsersControllerTest#test_should_redirect_update_when_logged_in_as_wrong_user:
AbstractController::ActionNotFound: The action 'update' could not be found for UsersController
    test/controllers/users_controller_test.rb:40:in `block in <class:UsersControllerTest>'

  5) Error:
UsersControllerTest#test_should_get_new:
ActionView::Template::Error: First argument in form cannot contain nil or be empty
    app/views/users/new.html.erb:5:in `_app_views_users_new_html_erb__4200017072968446905_56780960'
    test/controllers/users_controller_test.rb:11:in `block in <class:UsersControllerTest>'

  8) Error:
UsersLoginTest#test_login_with_valid_information_followed_by_:
ActionView::Template::Error: undefined method `name' for nil:NilClass
    app/views/users/show.html.erb:1:in `_app_views_users_show_html_erb__1377065752919381560_58665040'
    test/integration/users_login_test.rb:24:in `block in <class:UsersLoginTest>'

  9) Error:
UsersSignupTest#test_valid_signup_information:
ActionView::Template::Error: First argument in form cannot contain nil or be empty
    app/views/users/new.html.erb:5:in `_app_views_users_new_html_erb__4200017072968446905_56780960'
    test/integration/users_signup_test.rb:16:in `block in <class:UsersSignupTest>'

32 runs, 47 assertions, 0 failures, 10 errors, 0 skips
```

（字数制限により同じファイルからのエラーは一部省略してます）

### 該当のソースコード
#### テスト（10章で実装したもの）
①UsersEditTest
```Ruby
require 'test_helper'

class UsersEditTest < ActionDispatch::IntegrationTest
  
  def setup
    @user = users(:michael)
  end

  # 失敗時
  test "unsuccessful edit" do
    log_in_as(@user)
    get edit_user_path(@user)
    assert_template 'users/edit'
    patch user_path(@user), params: { user: { name:  "",
                                              email: "foo@invalid",
                                              password:              "foo",
                                              password_confirmation: "bar" } }

    assert_template 'users/edit'
  end
  
  # 成功時
  test "successful edit" do
    log_in_as(@user)
    get edit_user_path(@user)
    assert_template 'users/edit'
    name  = "Foo Bar"
    email = "foo@bar.com"
    patch user_path(@user), params: { user: { name:  name,
                                              email: email,
                                              password:              "",
                                              password_confirmation: "" } }
    assert_not flash.empty?
    assert_redirected_to @user
    @user.reload
    assert_equal name,  @user.name
    assert_equal email, @user.email
  end
end
```
②UsersControllerTest
```Ruby
require 'test_helper'

class UsersControllerTest < ActionDispatch::IntegrationTest
  
  def setup
    @user       = users(:michael)
    @other_user = users(:archer)
  end
  
  test "should get new" do
    get signup_path
    assert_response :success
  end

  test "should redirect edit when not logged in" do
    get edit_user_path(@user)
    assert_not flash.empty?
    assert_redirected_to login_url
  end

  test "should redirect update when not logged in" do
    patch user_path(@user), params: { user: { name: @user.name,
                                              email: @user.email } }
    assert_not flash.empty?
    assert_redirected_to login_url
  end


  # 他人が入った場合の失敗テスト
  
  test "should redirect edit when logged in as wrong user" do
    log_in_as(@other_user) # 例　archerさんが入った場合
    get edit_user_path(@user)
    assert flash.empty?
    assert_redirected_to root_url
  end

  test "should redirect update when logged in as wrong user" do
    log_in_as(@other_user) # 例　archerさんが入った場合
    patch user_path(@user), params: { user: { name: @user.name,
                                              email: @user.email } }
    assert flash.empty?
    assert_redirected_to root_url
  end

end
```

（③test_helper.rb）
```Ruby
ENV['RAILS_ENV'] ||= 'test'
require File.expand_path('../../config/environment', __FILE__)
require 'rails/test_help'

class ActiveSupport::TestCase
  # Setup all fixtures in test/fixtures/*.yml for all tests in alphabetical order.
  fixtures :all

  # テストユーザーがログイン中の場合にtrueを返す
  def is_logged_in?
    !session[:user_id].nil?
  end
  
  # テストユーザーとしてログインする
  def log_in_as(user)
    session[:user_id] = user.id
  end
end

  class ActionDispatch::IntegrationTest

  # テストユーザーとしてログインする
  def log_in_as(user, password: 'password', remember_me: '1')
    post login_path, params: { session: { email: user.email,
                                          password: password,
                                          remember_me: remember_me } }
  end
end

```

###ファイル（10章で実装したもの）

①UsersController
```Ruby
class UsersController < ApplicationController
  before_action :logged_in_user, only: [:edit, :update]
  before_action :correct_user,   only: [:edit, :update]
　# 上から順に行う
  
  def show
    @user = User.find(params[:id])
  end
  
  def new
    @user = User.new
  end
  
  def create
    @user = User.new(user_params)
    if @user.save
      log_in @user
      flash[:success] = "Welcome to the Sample App!"
      redirect_to @user
      # GET => redirect_to "/user/#{@user.id}" => show
    else
      render "new"
    end
  end
  
  def edit
    @user = User.find(params[:id])
  end
  
  def update
    @user = User.find(params[:id])
    if @user.update_attributes(user_params) #下のバリデーション
      # Success
      flash[:success] = "Profile updated"
      redirect_to @user
    else
      # Failure
      # => @user.errors.full_messages()
      render 'edit'
    end
  end
  
  private
  
    def user_params
      params.require(:user).permit(
         :name, :email, :password,
         :password_confirmation)
    end
  
    # beforeアクション
    
    # ログイン済みユーザーかどうか確認
    def logged_in_user
      unless logged_in?
        flash[:danger] = "Please log in."
        redirect_to login_url
      end
    end
  
    # 正しいユーザーかどうか確認
    def correct_user
      # GET   edit
      # PATCH update
      @user = User.find(params[:id])
      redirect_to(root_url) unless @user == current_user
    end
  end

```
②edit.html.erb
```Ruby
<% provide(:title, "Edit user") %>
<h1>Update your profile</h1>

<div class="row">
  <div class="col-md-6 col-md-offset-3">
    <%= form_for(@user) do |f| %>
      <%= render 'shared/error_messages' %>

      <%= f.label :name %>
      <%= f.text_field :name, class: 'form-control' %>

      <%= f.label :email %>
      <%= f.email_field :email, class: 'form-control' %>

      <%= f.label :password %>
      <%= f.password_field :password, class: 'form-control' %>

      <%= f.label :password_confirmation, "Confirmation" %>
      <%= f.password_field :password_confirmation, class: 'form-control' %>

      <%= f.submit "Save changes", class: "btn btn-primary" %>
    <% end %>

    <div class="gravatar_edit">
      <%= gravatar_for @user %>
      <a href="http://gravatar.com/emails" target="_blank">change</a>
    </div>
  </div>
</div>
```

### 試したこと
####①ルーティング
実際にプレビューでブラウザに入り、

Homeから新規登録ページに行くと以下のエラーメッセージが出ました。

![イメージ説明](efd1391a535b0eefe7aa215e176f65b7.png)

エラーメッセージに「Routing Error」とあったので、

routesを確認しました。
```Ruby
Rails.application.routes.draw do

  get 'sessions/new'

  root 'static_pages#home'
  get  '/help',    to: 'static_pages#help'
  get  '/about',   to: 'static_pages#about'
  get  '/contact', to: 'static_pages#contact'
  get  '/signup',  to: 'users#new'
  
  get    '/login',   to: 'sessions#new'
  post   '/login',   to: 'sessions#create'
  delete '/logout',  to: 'sessions#destroy'
  
  resources :users

end

```
signup_pathとnew_user_pathがあることで、

これらのnewアクションへの重複によるものかと思いました。
```Ruby
signup GET    /signup(.:format)         users#new

new_user GET    /users/new(.:format)      users#new
```

ただ、home.html.erbを見るとsignup_pathへ送っていて、

またRailsチュートリアルでも特に問題ないとのことだったので、

違うと判断しました。

####②誤字

リスト10.22〜10.26間でエラーが発生したため、

実装したチュートリアル内のコードを再度コピペし直しタイプミス等確認しましたが、エラーのままでした。

念のため10章の内容全てをコピペし直しましたが、変わりませんでした。



### 補足情報（FW/ツールのバージョンなど）

長文すみません。

1日調べましたが解決できず、先に進めなくて困っています。

ご協力お願いします。

Railsで発生した「undefined local variable or method」エラーを直したいです。

###前提・実現したいこと
WEBサービスのユーザー登録ページにアクセスすると発生する下記のエラーを解消したい。
viewやcontrollerのファイルを確認しても問題箇所が特定できず困っています。


Rails 4.2.2
Ruby 2.0.0
で開発をしています。

###発生している問題・エラーメッセージ
ユーザーの新規登録ページにアクセスすると「undefined local variable or method」が発生し
chromeブラウザでは下記のエラーが表示されます。

![イメージ説明](8c1d32d1432226f625cab33c2ed1fcc7.png)


以下がchromeで表示されているエラーです。
```
undefined local variable or method `object' for #<#<Class:0x007fca9c2187c0>:0x007fca9dafdba0>
```
同じくchromeで表示されるエラー箇所です。
```
app/views/shared/_error_messages.html.erb:1:in `_app_views_shared__error_messages_html_erb___4354769760733946946_70254143801240'
app/views/users/new.html.erb:8:in `block in _app_views_users_new_html_erb___3118286864806176088_70254102854900'
app/views/users/new.html.erb:6:in `_app_views_users_new_html_erb___3118286864806176088_70254102854900'
```



###該当のソースコード
まず、ユーザー登録機能に関係するusers_controller.rbファイルのユーザー作成箇所です。
```ruby
  def create
    @user = User.new(user_params)
    if @user.save
      @user.send_activation_email
      flash[:info] = "Please check your email to activate your account."
      redirect_to root_url
    else
      render 'new'
    end
  end
```
次に、chromeで指摘されているapp/views/shared/_error_messages.html.erbファイルです。
```ruby
<% if object.errors.any? %>
  <div id="error_explanation">
    <div class="alert alert-danger">
      The form contains <%= pluralize(object.errors.count, "error") %>.
    </div>
    <ul>
    <% object.errors.full_messages.each do |msg| %>
      <li><%= msg %></li>
    <% end %>
    </ul>
  </div>
<% end %>
```
同じくchromeで指摘されているnew.html.erbファイルです。
```ruby
<% provide(:title, 'Sign up') %>
<h1>Sign up</h1>

<div class="row">
  <div class="col-md-6 col-md-offset-3">
    <%= form_for(@user) do |f| %>
      <%= render 'shared/error_messages', object: f.object %>
      <%= render 'shared/error_messages' %>

      <%= f.label :name %>
      <%= f.text_field :name, class: 'form-control' %>

      <%= f.label :email %>
      <%= f.email_field :email, class: 'form-control' %>

      <%= f.label :password %>
      <%= f.password_field :password, class: 'form-control' %>

      <%= f.label :password_confirmation, "Confirmation" %>
      <%= f.password_field :password_confirmation, class: 'form-control' %>

      <%= f.submit "Create my account", class: "btn btn-primary btn-form" %>
    <% end %>
  </div>
</div>

```



###試したこと
[参考にしたサイト](http://stackoverflow.com/questions/29659475/undefined-local-variable-or-method-object-for-error-messages)

上記のサイトを参考に、new.html.erbファイルの
```
<%= render 'shared/error_messages' %>
```
をコメントアウトしたりしたのですが、エラー内容も変わらずで効果なしでした、、



###追記
エラー箇所の参考としてchromeでエラー画面出力された内容を記載しておきます。
（HTMLかJavascriptの出力結果があれば回答しやすいのでは、とのコメントをいただいたのですが、HTMLなどの出力結果をどうやって得るか分からず、choromeでの出力を記載させていただきます）

まずapp/views/shared/_error_messages.html.erb:1のエラー該当箇所が下記です。

```
<% if object.errors.any? %>
  <div id="error_explanation">
    <div class="alert alert-danger">
      The form contains <%= pluralize(object.errors.count, "error") %>.
    </div>
    <ul>
```

次に、app/views/users/new.html.erb:8の該当箇所が下記です。

```
      instrument("!render_template") do
        compile!(view)
        view.send(method_name, locals, buffer, &block)
      end
    rescue => e
      handle_render_error(view, e)
```

最後に、app/views/users/new.html.erb:6の該当箇所が下記です。
```
      instrument("!render_template") do
        compile!(view)
        view.send(method_name, locals, buffer, &block)
      end
    rescue => e
      handle_render_error(view, e)
```
よろしくお願いします。

Railsの「undefined local variable or method」エラーを解消したい

### 実現したいこと
戻り値や戻り値として返すの意味がわからないため理解したい。

### 発生している問題・分からないこと

①戻り値とはなんのどこにある値がどこに戻るのでしょうか？

②ソースコードの　return total_price　はなんのためのコードでしょうか。
　また、total_priceはどこに戻るのでしょうか。
　total_price　の結果はtotal_price -= 100で出てるのになぜ
return total_priceが必要なのでしょうか

③return total_priceがあってもなくても同じ結果になるのは何故でしょうか



### 該当のソースコード

```
class Menu
  attr_accessor :name
  attr_accessor :price
  
  def info
    return "#{self.name} #{self.price}円"
  end
  
  def get_total_price(count)
    total_price = self.price * count
    if count >= 3
      total_price -= 100
    end
    return total_price
  end
end

menu1 = Menu.new
menu1.name = "ピザ"
menu1.price = 800

puts menu1.get_total_price(3)
```

### 試したこと・調べたこと
- [ ] teratailやGoogle等で検索した
- [ ] ソースコードを自分なりに変更した
- [ ] 知人に聞いた
- [x] その他

##### 上記の詳細・結果
以下見ましたがわからずでした
https://terakoya.sejuku.net/question/detail/22397

### 補足
特になし

戻り値や戻り値として返すの意味がよくわかりません

### 実現したいこと
Ruby On Rails 8 において
ActiveStorage::Previewer を承継した CustomPreviewer を使って previewer として機能させたい。


### 発生している問題・分からないこと
起動時（bin/dev）にエラーが発生する。
CustomPreviewer(app/previewers/custom_previewer.rb)が読み込まれてない？か読み込む順番を間違えてしまっているか？

### エラーメッセージ
```error
`config.active_storage.previewers.unshift CustomPreviewer`の場合
11:20:59 web.1  | /home/config/application.rb:28:in '<class:Application>': uninitialized constant Home::Application::CustomPreviewer (NameError)
11:20:59 web.1  |
11:20:59 web.1  |     config.active_storage.previewers.unshift CustomPreviewer
11:20:59 web.1  |                                              ^^^^^^^^^^^^^^^

`config.active_storage.previewers.unshift ActiveStorage::Previewer::CustomPreviewer`の場合
11:22:13 web.1  | /home/config/application.rb:28:in '<class:Application>': uninitialized constant ActiveStorage::Previewer::CustomPreviewer (NameError)
11:22:13 web.1  |
11:22:13 web.1  |     config.active_storage.previewers.unshift ActiveStorage::Previewer::CustomPreviewer

```

### 該当のソースコード

```ruby:app/previewers/custom_previewer.rb
class CustomPreviewer < ActiveStorage::Previewer
...
end
```
または
```ruby:app/previewers/custom_previewer.rb
module ActiveStorage
    module Previewer
        class CustomePreviewer < Previewer
        ....
        end
    end
end
```
```ruby:config/application.rb
module Home
  class Application < Rails::Application
    config.load_defaults 8.0
    ...
    config.active_storage.previewers.unshift ActiveStorage::Previewer::CustomPreviewer
または
    config.active_storage.previewers.unshift CustomPreviewer
    ...
  end
end
```
または
```ruby:config/initializers/active_storage.rbに
Rails.application.config.active_storage.previewers.unshift CustomPreviewer
Rails.application.config.active_storage.previewers.unshift ActiveStorage::Previewer::CustomePreviewer
```

### 試したこと・調べたこと
- [x] teratailやGoogle等で検索した
- [x] ソースコードを自分なりに変更した
- [ ] 知人に聞いた
- [x] その他(ChatGPTに聞いてみた)

##### 上記の詳細・結果


### 補足
特になし

ActiveStorage: カスタム previewer を使いたい

Progateでプログラムの勉強を始めたものです。

Ruby学習レッスン１（詳細のコードはスクショ参照）

答えは同じなのですが、コードを書き直せと言われました。
答えのコードを見て理解もしましたが、ここで疑問が生じました。
「自分の書いたコードだと悪い点があるか？」です。

答えは同じですが、
・他の人が見ると見にくい
・コードが無駄に長くなってしまう
という２つのデメリットを考えましたが、あっていますか？
また、逆にメリットがあれば知りたいです。






![イメージ説明](https://ddjkaamml8q8x.cloudfront.net/questions/2024-05-13/8fccdccd-3e5b-435f-8e47-b94b80243f2e.png)

![イメージ説明](https://ddjkaamml8q8x.cloudfront.net/questions/2024-05-13/d58ba761-60e2-49ec-9e1f-9838f5c6af9e.png)

![イメージ説明](https://ddjkaamml8q8x.cloudfront.net/questions/2024-05-13/ef3ec645-2e4a-4625-a804-ec92b3716fd0.png)

![イメージ説明](https://ddjkaamml8q8x.cloudfront.net/questions/2024-05-13/3b42502a-6494-4f5a-a4e8-3b078557c4b3.png)