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

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

ただいまの
回答率

90.53%

  • C#

    8836questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

  • 命名規則

    37questions

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

メソッド名のつけ方(Get/Create)

解決済

回答 9

投稿

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

sumth

score 16

値を取得するようなメソッド名をつけるとき、いつもGet~とするかCreate~とするか迷います。

・Get~は値を取得する場合
・Create~はオブジェクトを生成する場合
のような感覚で使っているのですが、
例えば、pointなどの構造体を計算して取得する場合はどちらを使うべきか?などよく迷います。
そもそも、Get~というメソッド名自体があいまいなので別の単語を使ったほうがいいような気もします。

このあたりのメソッド名のつけ方として何らかの指針となる考え方等ありましたらご教示お願いいたします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • SagaraSousuke

    2017/01/16 18:35

    オブジェクト指向言語の場合は、動詞 + 名詞が多いと思います。get は何かを取得する場合。createは値は取得せず、内部でデータ等を作る時に使います

    キャンセル

回答 9

checkベストアンサー

+3

私はメソッド名の先頭には処理を表す動詞(create,calculate)を付けますね。
変数をそのまま返すようなメソッドは極力作らないようにします。

C#はプロパティにアクセサを指定できるのでgetメソッドを作る必要もないのではと思います。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/01/18 01:04 編集

    ご回答いただきありがとうございます。とても参考になりました。
    値を取得するようなものはGetは使わずプロパティにし、その他メソッドは処理を表す動詞を付けるというのがそれぞれの意味を明確に表現でき、自分的にとてもしっくりきました。ありがとうございました。

    キャンセル

+3

こんにちは。

createはnewする時、getはnewしない時はどうでしょう?

C#の場合はどちらの場合でも結果は大差ないですが、それだけに「単独のオブジェクト」なのか「他のオブジェクト」の共有なのか少しでも区別できるようにしておくと、トラブルを多少なりと低減しやすくなる筈です。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/01/18 01:04

    ご回答いただきありがとうございます。とても参考になりました。

    キャンセル

+1

私は、リソースをあまり使わない処理のばあいには、
引数が無いとき→プロパティにしてます。
引数がある場合→引数をどの様にしたいかで頭の動詞を決めてます。
ですからGet~とかCalc~とかはあまり使いません。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/01/18 01:03

    ご回答いただきありがとうございます。とても参考になりました。

    キャンセル

+1

こんにちは。
使用するフレームワーク(.NET Framework)やライブラリのそれと合わせることを基本としながら、もし「いちから自分で選べ」となったときは

  • 結果的に値を返す以外の作用も引き起こす場合は Create や Generate など
    -- ファイルを作成してファイルオブジェクトを返す
    -- DB に接続(セッションを作成)してDB接続オブジェクトを返す
  • 提供する情報に関してそのオブジェクトが持っているデータ(そのオブジェクト全体の場合も)から、ダウンキャスト的あるいは継承関係に無いインスタンスを生成して返す場合は Create や Generate や Convert など
  • そのオブジェクト(の責任範囲)の状態に変化がないあいだは返される値が同じであればプロパティ
    -- グラフィック(キャンバス)オブジェクトなどにマップされる矩形領域のサイズ
    -- データベース接続オブジェクトの接続状態(Status:接続状態は「データベース接続オブジェクト」の責任範囲)
  • そのオブジェクト(の責任範囲)の状態に変化がなくても外的要因の変化で返される値が変わる場合は Get、とくに時間がかかったり、計算していることを強調する場合は Calculate や Compute など

と、私は選択しているようです。。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/01/18 01:03

    ご回答いただきありがとうございます。とても参考になりました。

    キャンセル

  • 2017/01/19 12:41

    get系でも適切な動詞を選べという考え方には賛成です。

    外部サービスから取得する場合(fetch)、外部機器(というかディスク)から取得する場合(load)なんていうのをよく使っています。
    fetchでも、DBからとWebからというのを区別できるとより良いのですが。

    キャンセル

  • 2017/01/20 10:08 編集

    他のプログラマがその名前を見て想起する内容が、コードを書いたプログラマの意図とより近くなることが(しかも正確に具体的であれば尚良)、自明ですが命名の基本なのだろうと思います(英語圏一般人の語義にすり寄るためではなく、同業者・あるいは将来の自分向けに...)
    Load はファイルなどから読み込む意図が私にも分かり易いです。
    私個人がWeb系に暗いので、fetchとみるとどうしてもDBのレコードセットから1レコード読み込みことを思い浮かんでしまいますが、webでの fetch は語義が議論されていたりなかなか手が込んでいる動作なのですね...
    http://jxck.hatenablog.com/entry/whatwg-fetch/

    キャンセル

0

私の場合、
計算しての場合、Calcを付けることもありますが、
Pointなどを計算して移動させるとなるとMoveとか。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/01/18 01:04

    ご回答いただきありがとうございます。とても参考になりました。

    キャンセル

0

統一されていれば良いのではないかと思います。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/01/18 01:04

    ご回答いただきありがとうございます。とても参考になりました。

    キャンセル

0

僕なら、calculateとgetのメソッドを分けます。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/01/18 01:04

    ご回答いただきありがとうございます。とても参考になりました。

    キャンセル

0

Create …ファクトリやビルダ等で新しいクラスインスタンスを生成して返す時
Get …リポジトリ等から既に存在する値を取得する時

Point構造体は構造体のためにほとんどの場合GetもCreateも使いません

クラス名が何をするか表しているならCalculateにするし
でなければPointの変更を表す関数名(移動だったらMove等)にします

var hero = CharactersRepository.Get(heroId);
var enemy = CharactersBuilder.Default
    .Location(Map.FindNearestEmptyLocation(hero.Location))
    .Create(Enemy.Goblin);

//...

enemy.MoveAwayFrom(hero);
Console.WriteLine($"{enemy.Name}は{hero.Name}から離れた({enemy.Location})");

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/01/18 01:03

    ご回答いただきありがとうございます。とても参考になりました。

    キャンセル

0

私なら、

■ もともとあるデータを編集して返す場合 -> 別の名前

■ (引数等を元とにするなりして) 新しく生成オブジェクトを生成して返す -> create...

■ メンバ変数 ( フィールド ) の値を返す -> get...

ですかね。

別の名前は、その処理によって変わります。calc だったり、makeだったり。popだったり。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/01/18 01:02

    ご回答いただきありがとうございます。とても参考になりました。

    キャンセル

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

  • C#

    8836questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

  • 命名規則

    37questions

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