常連

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

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

## 背景、状況

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

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

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

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

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


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

### 実現したいこと
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 を使いたい

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

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

```
require 'open-uri'
require 'json'

# 指定したURLのFacebookいいね数を取得するクラス
class FbShareCounter
# FacebookグラフAPIのURL
BASE_URL = 'http://graph.facebook.com/?id='

# target_urlにいいね数をカウントする対象URLを指定
def initialize(target_url)
@request_url = BASE_URL + target_url
end

def count
# グラフAPIにアクセス
response = open(@request_url).read

# テキストをハッシュに変換
hash = JSON.parse(response)

# いいね数を取得
hash['share']['share_count']
end
end

counter = FbShareCounter.new('http://gihyo.jp')
p counter.count
```
というソースコードで、コマンドプロンプトで実行した結果、
```
C:/samples/samples/09_03/like_counter.rb:16:in `initialize': Invalid argument @ rb_sysopen - http://graph.facebook.com/?id=http://gihyo.jp (Errno::EINVAL)
from C:/samples/samples/09_03/like_counter.rb:16:in `open'
from C:/samples/samples/09_03/like_counter.rb:16:in `count'
from C:/samples/samples/09_03/like_counter.rb:27:in `<main>'
```
という結果が帰ってくるのですが、なぜでしょうか？
それで、解決したいことは、プログラムを修正して、いいね数を取得したいのです。
教えていただけると幸いです。よろしくお願いします。


ルビー いいね数を取得したい

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

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

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

②ソースコードの　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

### 補足
特になし

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

### 前提・実現したいこと

該当箇所とされるファイルを一通り確認しても問題箇所が見つからず、困っています。
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」エラーを直したいです。

### 実現したいこと
戻り値とputsの違いがわからず苦戦しています。

色々な質問や記事を自分なりに解読し以下にまとめてみたので、
違っている点があれば教えていただけると幸いです。

### 発生している問題・分からないこと
■putsとreturnの違い
　-putsとは-
　puts は学習や確認のために使うもので、実際にウェブ上に表示させるために使用するものではなく、　contoroller内(●●.rb)に学習や確認のために一時的に使用するもの。

　-returnとは-
　returnは実際にウェブ上に表示させるためのコードで、
　returnを記述しないとputsと同じで処理結果をウェブ上に表示できない



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

```Ruby
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)
```

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

##### 上記の詳細・結果
class Menu
  attr_accessor :name
  attr_accessor :price
  
  def info
     "#{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)

### 補足
return を使用しても使用しなくても
コンソール結果は同じ結果になりました。

（現在ウェブ上で確認できる環境が整っていないため、progate という学習アプリでコンソール結果のみは確認できました。）

戻り値とputsの違いについて

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)