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

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

新規登録して質問してみよう
ただいま回答率
85.48%
Ruby

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

Q&A

解決済

3回答

797閲覧

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

Giants

総合スコア11

Ruby

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

0グッド

0クリップ

投稿2023/01/19 14:24

前提

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

私の中でのワンライナーの定義は、容量軽減のためにコードを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/ツールのバージョンなど)

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

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

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

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

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

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

guest

回答3

0

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

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

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

投稿2023/01/19 14:31

編集2023/01/19 14:34
maisumakun

総合スコア145183

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

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

maisumakun

2023/01/20 00:26 編集

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

0

ベストアンサー

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

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

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

投稿2023/01/20 13:48

winterboum

総合スコア23329

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

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

0

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

rb

1puts "OK" if true unless false # OK

rb

1puts "OK" unless false if true # OK

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

投稿2023/01/19 14:52

yuma.inaura

総合スコア1453

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問