質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

89.04%

Rails 投稿 ダミー画像の表示

解決済

回答 2

投稿

  • 評価
  • クリップ 1
  • VIEW 474

14j126

score 9

Ruby on Railsで、メルカリのようなフリマサイトを作っています。
3枚まで画像を投稿することができ、画像を投稿しなかった場合、ダミー画像を表示できるようにしたいです。
1枚のみで投稿する場合、残りの画像をダミー画像を表示したいのですが、投稿すること自体できません。

new.html.erb(投稿画面)

  <div class="products">
    <ul class="new_products_form">
    <%= form_for @product do |f| %>
      <div class="field">
        <li><%= f.label :name, "商品名" %></li>
        <%= f.text_area :name %>
      </div>
      <div class="textarea_wrap">
        <li><%= f.label :description, "説明" %></li> 
        <%= f.text_area :description %>
      </div>
      <div class="field">
       <li><%= f.label :category, "カテゴリー" %></li>
       <%= f.collection_select :category, Category.all ,:id ,:name, include_blank: false %>
      </div>
      <div class="field">
        <li><%= f.label :price, "価格" %></li>
        <%= f.text_area :price %>
      </div>
      <div class="file_wrap">
        <li><%= f.label :image1, "画像1" %></li>
        <%= f.file_field :image1 %>
        <li><%= f.label :image2, "画像2" %></li>
        <%= f.file_field :image2 %>
        <li><%= f.label :image3, "画像3" %></li>
        <%= f.file_field :image3 %>
      </div>
      <div class="btn_wrap">
        <%= f.submit "送信する" %>
      </div>
    <% end %>
  </div>


products_controller.rb

def new
  @product= Product.new
end

def create
    @product = Product.new(product_params)
    @product.status = 0
    upload_file1 = params[:product][:image1]
    upload_file_name1 = upload_file1.original_filename
    output_dir = Rails.root.join('public', 'images')
    output_path = output_dir + upload_file_name1

    File.open(output_path, 'w+b') do |f|
     f.write(upload_file1.read)
    end
    @product.image1 = upload_file_name1

    upload_file2 = params[:product][:image2]
    upload_file_name2 = upload_file2.original_filename
    output_dir = Rails.root.join('public', 'images')
    output_path = output_dir + upload_file_name2

    File.open(output_path, 'w+b') do |f|
     f.write(upload_file2.read)
    end
    @product.image2 = upload_file_name2

    upload_file3 = params[:product][:image3]
    upload_file_name3 = upload_file3.original_filename
    output_dir = Rails.root.join('public', 'images')
    output_path = output_dir + upload_file_name3

    File.open(output_path, 'w+b') do |f|
     f.write(upload_file3.read)
    end
    @product.image3 = upload_file_name3

    if @product.save
      flash[:success] = "出品しました。"
      redirect_to ("/users/profiles") and return
    else
      flash[:danger] = "出品に失敗しました。"
      redirect_to ("/users/products/new")
    end
  end


routes.rb

  get '/users/products/new', to:'products#new', as: :new
  resources :products
  post '/users/products/new', to: 'products#create', as: :create

products_helper.rb

module ProductsHelper

  def image1(product)
    if product.image1.blank?
      "https://dummyimage.com/200x200/000/fff"
    else
      "/images/#{@product.image1}"
    end
  end

  def image2(product) 
    if product.image2.blank?
      "https://dummyimage.com/200x200/000/fff"
    else
      "/images/#{@product.image2}"
    end
  end

  def image3(product)
    if product.image3.blank?
      "https://dummyimage.com/200x200/000/fff"
    else
      "/images/#{@product.image3}"
    end
  end
end

show.html.erb(自分が出品した商品一覧、抜粋)

  <div class="products">
    <ul>
      <% @products.each do |product| %>
        <li>
         <div class="image">
          <%= link_to details_path(product.id) do%>
            <%= image_tag "/images/#{product.image1}" %>
          <% end %>
         </div>
          <p><%= product.name %></p>
          <p><%= product.price %></p>
        </li>
      <% end %>
    </ul>
  </div>
  <% else %>


画像1枚のみで投稿しようとすると、
NoMethodError in ProductsController#create
undefined method `original_filename' for nil:NilClass
と出てきます。

画像3枚で投稿すると投稿できます。
プログラミング初心者のため、分かりにくい部分もあると思いますが、よろしくお願い致します。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+1

回答

画像1枚のみで投稿しようとすると、
NoMethodError in ProductsController#create
undefined method `original_filename' for nil:NilClass と出てきます。

1枚だけ投稿の場合、2枚目のupload_file2 はnilなので、 upload_file2.original_filenameの行でエラーになってると思います。

image1~3の中で、パラメータが設定されているものだけ、画像保存処理をしてみてはいかがでしょうか。

以下、コードになります。

def create
    @product = Product.new(product_params)
    @product.status = 0

    #画像1枚目
    if params[:product][:image1]
      upload_file1 = params[:product][:image1]
      upload_file_name1 = upload_file1.original_filename
      output_dir = Rails.root.join('public', 'images')
      output_path = output_dir + upload_file_name1

      File.open(output_path, 'w+b') do |f|
       f.write(upload_file1.read)
      end
      @product.image1 = upload_file_name1
    end

    #画像2枚目
    if params[:product][:image2]
      upload_file2 = params[:product][:image2]
      upload_file_name2 = upload_file2.original_filename
      output_dir = Rails.root.join('public', 'images')
      output_path = output_dir + upload_file_name2

      File.open(output_path, 'w+b') do |f|
       f.write(upload_file2.read)
      end
      @product.image2 = upload_file_name2
    end

    #画像3枚目
    if params[:product][:image3]
      upload_file3 = params[:product][:image3]
      upload_file_name3 = upload_file3.original_filename
      output_dir = Rails.root.join('public', 'images')
      output_path = output_dir + upload_file_name3

      File.open(output_path, 'w+b') do |f|
       f.write(upload_file3.read)
      end
      @product.image3 = upload_file_name3
    end

    if @product.save
      flash[:success] = "出品しました。"
      redirect_to ("/users/profiles") and return
    else
      flash[:danger] = "出品に失敗しました。"
      redirect_to ("/users/products/new")
    end
  end

以上

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/01/01 23:31

    ありがとうございます。画像1枚のみでも投稿できるようになりました!

    キャンセル

+1

原因は KazuSakaさんの言われているとおりです。
で、
3枚とも同じ処理ですね。
こういう場合は

[:image1,:image2,:image3].each do |image|
  if params[:product][image]
    upload_file = params[:product][image]
    upload_file_name = upload_file.original_filename
    output_dir = Rails.root.join('public', 'images')
    output_path = output_dir + upload_file_name

    File.open(output_path, 'w+b') do |f|
     f.write(upload_file.read)
    end
    @product[image] = upload_file_name
  else
    @product[image] = ダミーimage_file_path
  end
end


とすると良いです。
@product[image] = upload_file_name のところは工夫の余地ありですが
というのは、お客さん必ず1,2,3の順で画像いれてくれるとは限らないから、image1が空でimage2が入ってるということもありそう、、、

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/01/01 23:31

    ありがとうございます。画像1枚のみでも投稿できるようになりました!様々なパターンを考えて作らないといけないですね。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 89.04%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る