Rails 投稿 ダミー画像の表示

Question

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枚で投稿すると投稿できます。
プログラミング初心者のため、分かりにくい部分もあると思いますが、よろしくお願い致します。

Answer 1

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

[: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が入ってるということもありそう、、、

Comments

[退会済みユーザー]

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

Answer 2

###回答

画像1枚のみで投稿しようとすると、

NoMethodError in ProductsController#create
undefined method `original_filename' for nil:NilClass
と出てきます。

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

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

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

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

    #画像１枚目
    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

    #画像２枚目
    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
    
    #画像３枚目
    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

以上

Comments

[退会済みユーザー]

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