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

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

ただいまの
回答率

88.78%

Ruby on Rails nilへの対応方法を教えてください

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 2,456

riamk

score 45

前提・実現したいこと

Ruby on Rails4で開発中。

現在勉強のため、twitterのようなアプリケーションを作成しました。
deviseでログイン機能を実装し、ログインしているユーザーしか投稿できないようにしています。
投稿された記事はどのユーザーが投稿したのかを関連づけています。

今まではユーザーが退会した場合、そのユーザーが投稿した記事も削除されるようにしていたのですが、今回ユーザーが退会した場合に、その退会したユーザーが投稿した記事は削除されず残しておくようにしたいと思い取り組んでいます。

退会したユーザーのユーザーnameは「退会したユーザー」として表示して記事を残そうと思っているのですが、その方法が分かりません。

ユーザーが退会した場合、投稿したユーザーは当たり前ですがnilになってしまいます。
そのユーザーnilへの対応の仕方が分からないのでご教授お願いします。

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

ユーザーが退会した場合、userがnilになるので
・user/showへのリンクでエラー
・userのprofile_imgでエラー
・user.nameでエラー
と上記のようなエラーが発生してしまいます。

どのように対応したらいいのでしょうか?

該当のソースコード

timeline/index.html.erb

<!-- 投稿一覧 -->
  <% @timelines.each do |timeline| %>
    <%= link_to user_path(timeline.user_id) do %>
      <%= profile_img(timeline.user) %>
    <% end %>
    <%= timeline.user.name %>
    <%= link_to timeline_path(timeline) do %>
      <%= image_tag (timeline.photo_url) if timeline.photo.present? %>
      <%= timeline.content %></p>
    <% end %>
  <% end %>

applicationHelper.rb

module ApplicationHelper
  # プロフィール画像の設定
  def profile_img(user)
    return image_tag(user.avatar, alt: user.name) if user.avatar?

    unless user.provider.blank?
      img_url = user.image_url
    else
      img_url = 'no_image.png'
    end
    image_tag(img_url, alt: user.name)
  end
end

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

他に必要なコードなどあれば記載しますので、何卒よろしくお願いいたします。

nilに対して対応して試してみたこと
timeline/index.html.erb

<!-- 投稿一覧 -->
<% @timelines.each do |timeline| %>
  <% if timeline&.user_id.present? %>
    <%= link_to user_path(timeline.user_id) do %>
      <%= profile_img(timeline.user)%>
    <% end %>
  <% else %>
    # ユーザーIDがnilの場合のプロフィール画像の表示を記述
  <% end %>

  <% if timeline&.user_id.present? %>
    <%= timeline.user.name %>
    <%= link_to timeline_path(timeline) do %>
      <%= image_tag (timeline.photo_url) if timeline.photo.present? %>
      <%= timeline.content %>
    <% end %>
  <% else %>
    # ユーザーIDがnilの場合の処理
  <% end %>
<% end %>

ユーザーが存在する場合と、存在しない(nil)場合の処理をif文で条件分岐して表示すればいいのかと思い、このコードを書きました。
ですが、nilの場合の判定ができていないのか、ユーザーが存在しない場合も<%= timeline.user.name %>などが呼び出されエラーになってしまいます。

*試したことへの補足
なぜnilの判定がうまくできないのか?
ユーザーがいなくなった(退会した)記事の中身を見てみたらユーザーIDは残っていた。
これが原因でユーザーIDのnil判定ができていませんでした。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 3

checkベストアンサー

+2

helperファイルがあると思います。

使い方は簡単で、
def sample
endってヘルパーに書いたあと

ビューに<%= sample %>で呼べると思います。

if timeline&.user_id ってところを

if has_user?(timeline)に書き換えてください

ヘルパーに
def has_user?(timeline)
uid=timeline.user_id
User.where(id: uid).first ? true : false
end
と書けばいけると思います。
user_idが必ず入ってるわけじゃないならif文で入ってないときに return false すればいけると思います!

我ながらスマホで頑張った…笑

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/06/17 00:29

    ご回答ありがとうございます。
    教えて頂いたようにヘルパーメソッドを使用して試してみたところ、無事にユーザーが存在する場合としない場合で表示できました!
    スマホからの回答は大変だったかと思いますが、ほんと助かりました!
    ありがとうございます!

    キャンセル

  • 2017/06/17 04:29

    設計としてはあまり良くなくて、timelineてことはおそらく結構な量になりますよね。

    その量×ぶんのクエリを出し続けるのパフォーマンスとしてはあまり良くないのでユーザーIDが消せるならそれが良いと思います!

    アソシエーションとか調べればできそうなのと、

    強引にユーザーが退会するときにユーザーのtimelineのユーザーIDを全部nilか何かに書き換えるとかですかね

    キャンセル

  • 2017/06/17 09:33

    設計としては良くないんですね。
    はい。タイムラインで投稿量は多くなります。

    アソシエーションのdependentオプションで:nullifyに設定してユーザーが退会した場合に、そのユーザーに関連する投稿のユーザーIDをnilに書き換えることができました。
    なので、最初に教えて頂いた if timeline&.user_id.pressent? でnil判定して処理を分けて対応してみようと思います。

    分かりやすい説明と的確なアドバイス、ありがとうございます!

    キャンセル

+1

indexでは

@users = 存在するusers

showでは

if userが存在しない then
      redirecct_to :users, notice: '存在しないuserです'
end


とするのはどうですか?

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/06/16 19:25

    mbkさんに教えて頂いたものとは違う形になってしまったのですが、if文を使用してユーザーが存在しない場合の処理を分岐して表示させようとコードを書いてみました。

    ですが、nilの場合の判定がうまくいかないのかエラーが出てしまいます。
    コードとエラー内容は質問の補足情報に追記しました。
    とりあえずindexのコードのみになりますが、アドバイス頂けると嬉しいです。

    キャンセル

  • 2017/06/16 20:29

    ぼっち演算子は知らないのですが、これはどうですか?

    if timeline.user_id.present? && User.exists?(id: timeline.user_id)

    キャンセル

  • 2017/06/17 00:26

    ご教授ありがとうございます。
    if timeline.user_id.present? && User.exists?(id: timeline.user_id)
    教えて頂いたこちらのコードでnilの判定がうまくでき、if文でユーザーがいない場合の条件分岐の表示ができました!
    ありがとうございました!

    キャンセル

0

ユーザーがいない投稿を消すかいなかみたいなのはモデルの定義でなんとか変えられると思います(もし、悩んでいたら)

そこまでができてるのを前提としてちょっとコード書きます!

<% @timelines.each do |timeline| %>
   <%= if timeline&.user_id.pressent? %> # ← ユーザーIDがあったらっていう処理です 「&.」って書いてあるのはぼっち演算子って言って無くてもエラー返さないやつです
      <%= link_to user_path(timeline.user_id) do %>
        <%= profile_img(timeline.user) %>
      <% end %>
      <%= timeline.user.name %>
      <%= link_to timeline_path(timeline) do %>
        <%= image_tag (timeline.photo_url) if timeline.photo.present? %>
        <%= timeline.content %></p>
   <%= else%>
        #ここにユーザーIDがnilだったときの処理をかきます!

  <% end %>
<% end %>

ユーザーIDがnilだったときどういうふうに表示したいのか分からない(書いてたらすいません)のでここらへんは適当に書いてください

↓みたいなのでも良いのかもしれないですね

 <%= link_to リンク先は「ユーザーは削除されました」とか表示されるページに飛ばすとか do %>
        <%= #写真はTwitterの卵写真みたいなのを差し替える %>
      <% end %>
      ユーザーネームは引けないから ユーザーは削除されました とか書いておく
      <%= link_to timeline_path(timeline) do %>
        <%= image_tag (timeline.photo_url) if timeline.photo.present? %>
        <%= timeline.content %></p>

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/06/16 17:43

    ご回答いただきありがとうございます。
    ユーザーがいない投稿を消すかどうかなどは、モデルのほうで定義できてます。

    drizzing20さんに書いて頂いたコードを参考に試行錯誤しながら試してみようと思います。

    キャンセル

  • 2017/06/16 19:29

    drizzing20さんのコードを参考にif文を使用しユーザーが存在しない場合の処理を書いてみました。

    ですが、<%= if timeline&.user_id.pressent? %> でnilの判定がうまくできないのか、エラーが出てしまいます。
    書いてみたコードやエラーの内容は質問の補足情報に追記しました。
    修正点や誤った箇所などあれば、ご指摘いただけると嬉しいです。

    キャンセル

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

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

関連した質問

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