🎄teratailクリスマスプレゼントキャンペーン2024🎄』開催中!

\teratail特別グッズやAmazonギフトカード最大2,000円分が当たる!/

詳細はこちら
Devise

Deviseとは、Ruby-on-Railsの認証機能を追加するプラグインです。

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Bootstrap

BootstrapはウェブサイトデザインやUIのWebアプリケーションを素早く 作成する可能なCSSフレームワークです。 Twitter風のデザインを作成することができます。

Q&A

解決済

1回答

713閲覧

ユーザープロフィール画像を変更するとエラーが起きる

kousuke24

総合スコア29

Devise

Deviseとは、Ruby-on-Railsの認証機能を追加するプラグインです。

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Bootstrap

BootstrapはウェブサイトデザインやUIのWebアプリケーションを素早く 作成する可能なCSSフレームワークです。 Twitter風のデザインを作成することができます。

0グッド

0クリップ

投稿2019/09/29 14:22

Ruby on Railsでgem 'carriewave'を使い、ユーザープロフィールの画像を変更する機能を追加しました。

ビュー

<div class="container"> <h3 class="text-center text-secondary mb-4">ユーザー情報</h3> <div class="offset-sm-3 col-sm-6"> <div class="offset-sm-3 col-sm-6 text-center"> <h4 class="mb-3"><%= @user.name %></h4> <%= form_for(@user) do |f| %> <% if @user.image? %> <%= image_tag @user.image.thumb.url, class: "round-img" %> <% else %> <%= image_tag "/assets/default.jpg", class: "round-img" %> <% end %> </div> </div> <div class="offset-sm-4 col-sm-4 offset-sm-4 text-center"> <% if @user == current_user %> <button type="button" class="btn btn-outline-secondary rounded-pill mt-3"> <%= f.file_field :image, accept: 'image/jpg,image/gif,image/png' %> </button> <%= f.submit "プロフィール画像変更", class: 'btn btn-outline-dark mt-1 form-control' %> <%= link_to "登録情報編集", edit_user_registration_path, class: "btn btn-secondary mt-3 form-control" %> <%= link_to "退会", user_destroy_path, method: :delete, data: { confirm: "本当に退会しますか?" }, class: "btn btn-danger mt-3 form-control" %> <% end %> <br> <% end %> </div> <hr> </div> <div class="photos-parent text-center"> <h5><i class="fas fa-paw"></i> <%= @user.name %>の投稿画像</h5> <% @post.each do |post| %> <%= link_to(post_path(post)) do %> <div class="photos-children"> <%= image_tag post.photos.first.image.url(:medium) %> </div> <% end %> <% end %> </div> <div class="paginate-parent"> <div class="paginate-children mt-3"> <%= paginate @post %> </div> </div>

プロフィール画像変更のボタンを押すと下記のエラーが出てしまいます。
イメージ説明

エラーが出ている箇所は、ユーザーが投稿した画像を一覧表示しているコードです。
まったく関連がない部分でエラーが出てしまい解決できずに立ち往生しています。
ご教示頂ければ幸いです。


該当コード

ルーティング

Rails.application.routes.draw do devise_for :users, controllers: { registrations: 'registrations', omniauth_callbacks: 'users/omniauth_callbacks' } root to: 'tops#index' get 'tops/show', to: 'tops#show' get '/users/:id', to: 'users#show', as: :user patch '/users/:id', to: 'users#update' delete '/users/:id', to: 'users#destroy', as: :user_destroy resources :posts, only: %i(new create show destroy) do resources :photos, only: %i(create) resources :likes, only: %i(create destroy) end # For details on the DSL available within this file, see https://guides.rubyonrails.org/routing.html end

コントローラー

class UsersController < ApplicationController def show @user = User.find_by(id: params[:id]) @post = @user.posts.order(id: "DESC").page(params[:page]).per(9) end def edit @user = User.find(params[:id]) end def update @user = User.find(params[:id]) if current_user == @user if @user.update(user_params) flash[:success] = 'プロフィール画像を変更しました' render :show else flash.now[:danger] = 'プロフィール画像の変更に失敗しました' render :show end else redirect_to user_path end end def destroy @user = User.find_by(id: params[:id]) @posts = Post.find_by(user_id: params[:id]) flash[:notice] = "ユーザーを削除しました" if @posts.nil? @user.destroy redirect_to root_path else @posts.destroy @user.destroy redirect_to root_path end end private def user_params params.require(:user).permit(:image) end end

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

redirect_torenderの違いですね。

どうしても、renderの方にしたいのならば

ruby

1 if current_user == @user 2 if @user.update(user_params) 3 flash[:success] = 'プロフィール画像を変更しました' 4 else 5 flash.now[:danger] = 'プロフィール画像の変更に失敗しました' 6 end 7 @post = @user.posts.order(id: "DESC").page(params[:page]).per(9) 8 render :show 9 else

と、なります。

投稿2019/09/29 15:12

asm

総合スコア15149

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

kousuke24

2019/09/29 15:53

できました! renderだとビューを表示させようとして、他のメソッドやアクションを処理してしまいエラーが起きていた。という解釈で間違い無いでしょうか?
asm

2019/09/29 15:56

renderは、viewを違うファイルに変更するためのメソッドで redirect_toは、違うURLをブラウザに開き直させ、結果としてルーティング→コントローラーをやり直しているのです。
kousuke24

2019/09/29 16:37

なるほど、、、 わかりやすい説明ありがとうございます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.36%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問