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

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

ただいまの
回答率

88.61%

カラムに入っている[]を外す方法を教えてください

解決済

回答 1

投稿

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

rui-watanabe

score 18

前提・実現したいこと

Rails初心者です。
複数画像選択の実装をしていて、保存には成功したのですが、カラムに[""]が入ってしまいます。
createメソッドが実行する前にこのカッコを外したいのですが、やり方がわからず質問させていただきました。

どなたかわかる方がいらっしゃいましたら、ご回答いただけると非常に助かります。

発生している問題・エラーメッセージ

複数画像選択で、カラム保存の時["couple-260899_1920.jpg"]などとはいってしまうので余計なものを外したい。

products_controller

省略

def new
    @product = Product.new
    @product.product_images.build
  end

  def create
    @product = current_user.products.build(product_params)
    unless @product.valid?
      render "/products/new"
    else
      @product.save
      redirect_to root_path
    end
  end

省略

def product_params
    params.require(:product).permit(
      :name,
      :description,
      :price,
      :quality,
      :delivery_origin,
      :delivery_status,
      :delivery_cost,
      :prep_days,
      :category_id,
      product_images_attributes: [:id, :image_url[]]
    )
end

products/new.html.haml

= form_for @product do |f|
  .sell-product__container__form
    .preview-content#sell-content
      .preview-content__main
        .preview-content__main__box
          %label
            商品画像
            %span.require 必須
          %p
            最大3枚までアップロードできます(ドラッグ&ドロップまたはクリックしてファイルをアップロード)
          .preview-content__main__box__container
            .preview-content__main__box__dropbox
              %pre
              %ul
                - 3.times do
                  %li
                    .view_box
                      = f.fields_for :product_images do |i|
                        = i.file_field :image_url, class: 'file',multiple: true

model/products.rb

省略

 has_many :product_images, dependent: :destroy
  accepts_nested_attributes_for :product_images, allow_destroy: true

省略

model/product_images.rb

class ProductImage < ApplicationRecord
  # Association
  belongs_to :product

  mount_uploaders :image_url, ImageUploader
end

補足情報(FW/ツールのバージョンなど)

Ruby version 2.5.1
Rails version 5.2.3

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

0

文字列の配列なので["str"]となっているだけのように見えますが、
本当に[" "]を取り除いてもいいのですか?
例では要素が1つなのでいいのかもしれませんが、複数の場合の動作はどうなるでしょうか?
もっと別のアプローチが必要なのでは?例えば以下のような...

attribute :image_url, :string, array: true
@product = current_user.products.build(product_params) do |product|
  product.product_images.build(*image_params)
end

def product_params
  params.require(:product).permit(%i[name description price quality delivery_origin delivery_status delivery_cost prep_days category_id])
end

def image_params
  params.require(:product).permit(product_images_attributes: [:id, :image_url[]])
end


雰囲気で書いたんですけどこんな感じですかね...?

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/08/20 18:43

    迅速なご回答誠にありがとうございます。
    カラムに入れたものを整形する時は、例えばproductsのindexページで、
    = image_tag("#{@product.product_images.last.eval('["image_url"]')}", class:"main__image__size")とするとまた別のエラーが出てしまいました。
    https://gyazo.com/1eeedab8365e9059fe63682c72d10274
    度々で申し訳ございませんが。上記のエラーの対応などをご教示いただけると非常に助かります。
    指定の仕方などが間違っているのでしょうか。

    キャンセル

  • 2019/08/21 09:59

    eval の使い方が誤っています。あとeval を使うのはあまりお勧めできないです。
    モデルに serialize を指定する方法もあるんですが、これもお勧めできないです。
    複数のレコードに分割させるのが一番いいと思います。

    キャンセル

  • 2019/08/23 21:15

    記載通り記述したら解決しました。
    ご丁寧に教えてくださりありがとうございます!
    非常に助かりました。

    キャンセル

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

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

関連した質問

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