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

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

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

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

Ruby on Rails

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

命名規則

命名規則は、プログラミングする際に識別子の名称である文字列を決める表記法のことです。ネーミング規則・ネーミング規約・命名規約とも呼びます。

Q&A

2回答

925閲覧

rubyのメソッド名で悩んでいます

MH00214

総合スコア53

Ruby

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

Ruby on Rails

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

命名規則

命名規則は、プログラミングする際に識別子の名称である文字列を決める表記法のことです。ネーミング規則・ネーミング規約・命名規約とも呼びます。

0グッド

0クリップ

投稿2018/06/25 11:31

前提・実現したいこと

Rails初心者です。
「画像をPCとSPで出し分ける」というコードを書いていて、それ自体は動いているのですが、メソッド名があんまりしっくり来ておらず、幾つか調べてみたものの、今ひとつ分からなかったので教えていただきたく投稿しました。

書いたコード

ruby

1def official_image_tag_separate_pc_sp(name, image_name_prefix, options: {}) 2 content_tag :div do 3 %w(sp pc).each do |device| 4 options[:class] = "display_#{device}" 5 image_name = "#{image_name_prefix}_#{device}.jpg" 6 7 concat image_tag("https://s3のURL/official/#{name}/#{image_name}", options) 8 end 9 end 10 end

official_image_tag_separate_pc_spという名前にしましたが、先輩からこれだとあんまり処理の内容を示していないので、いい感じの英語名にしよう、ということで指摘をもらっています。

命名規則に詳しい方がいましたら教えていただけますと幸いです。
宜しくお願いいたします。

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

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

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

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

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

guest

回答2

0

そもそもどういうことをそのメソッドやっているのですか?

render_imageだけでいいような気がしなくはないのですが。

その中でデバイスごとにrender_image_for_pcとかrender_image_for_spとかをやっていることを期待しています。


あまりうまい名前をつけることができない場合、同じメソッドの中で複数のことをしていることがよくあります。
その場合は命名よりも設計を再考すべきです。

例えば、リサイズしてかつURLも作って返すなどは一言では言い表せないことをしている可能性があります。
それだけで例えばrenderと言えるのであれば、完結しているので良いのですが、例えば実は別のところで白黒に変換もしていたりすると、メソッドの切り分けが悪いということになります。

それともrenderというよりprepareなんでしょうか?
上記の情報だけでは全体像が見えないので、なんとも言えないです。

投稿2018/06/25 12:23

編集2018/06/25 12:29
mkgrei

総合スコア8560

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

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

0

プロフィールを見ると会社としてアプリを作られているんですよね。

会社として規約はありませんか?
それとも先輩は”まず自分で考えてみろ”という意味で指摘されたんですかね?

規約は検索すれば出てきますが、”会社として合うのか”は社外の人には分かりません。
また、会社の資産のコードをみて規約(≒法則)に気づくのもいいと思います。

その上で先輩に”xxxっていう風にします”と話されてはいかがですか?

投稿2018/06/25 11:52

oikashinoa

総合スコア2826

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問