if文にまとめてのワンライナーでの記述方法について

Question

前提

下記、コードのワンライナーでの記述方法につきまして、
疑問に思ったことがありますので、教えていただければと思います。

私の中でのワンライナーの定義は、容量軽減のためにコードを1行でまとめて書く認識でありますが、unless文の中にif文がある場合のワンライナーでの記述方法はどのようなものになるのでしょうか？

実現したいこと

• 下記コードをワンライナーで記述方法
• 記述内容の解説をいただきたいです

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

<% unless current_user == user %>
  <% if current_user.following?(user) %>
    <%= form_with model: relationship, url: relationship_path, method: :delete, local: true do |f| %>
　　　　　　　　　　　<%= hidden_field_tag :follow_id, user.id %>
      <%= f.submit 'Unfollow', class: 'btn btn-danger btn-block' %>
    <% end %>
  <% else %>
    <%= form_with model: @set_relationship, url: relationships_path, local: true do |f| %>
　　　　　　　　　　　<%= hidden_field_tag :follow_id, user.id %>
      <%= f.submit 'Follow', class: 'btn btn-primary btn-block' %>
    <% end %>
  <% end %>
<% end %>

試したこと

ここに問題に対して試したことを記載してください。

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

ここにより詳細な情報を記載してください。

Answer 1

私の中でのワンライナーの定義は、容量軽減のためにコードを1行でまとめて書く認識でありますが

なぜこの場面でそれが必要なのでしょうか？

よほど特殊な事情がない限り、ビューは（常識的な範囲内で構築する分には）動作速度やコード量が問われる場面ではありませんので、読みやすさを優先した現状のような形で書くのが妥当です。

Comments

[maisumakun]

ここでのform_withのような、中身にいくつものメソッドが入るブロック付きメソッドをワンライナーにしても、ただ読みづらくなるだけで、ほぼメリットはないと考えます。

Answer 2

「ワンライナーの定義は、容量軽減のためにコードを1行でまとめて書く」
これが違いますね。
1行で書くと容量が軽減されるんですか？そんなことはないでしょう。
「ワンライナーの定義はコードを1行で完結させる」
です。
質問のunless if else end end だけではなく、それを含む全体を１行で 完結させるのがワンライナーです。
unless if else end end だけ１行にしたとしたらそれは単なる、、、え〜〜なんと言ったか忘れた、複数の式を１行に書いただけ

コマンドラインにプログラム書いて実行する というようなとき、複数行に渡ると書きにくいので工夫して１行で書いてしまう というのが ワンライナーのよくある使い方。

読みにくくなります。ということは Bug入りやすい。
ですから インデントきちんとつけて分けて書きましょう。

Answer 3

おすすめはしませんが繋げて書くことは出来ます

puts "OK" if true unless false # OK

puts "OK" unless false if true # OK

ただ無理せず読みやすい書き方を選びましょう