メソッド名のつけ方（Get/Create）

Question

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

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

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

Comments

[SagaraSousuke]

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

Answer 1

こんにちは。

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

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

Comments

[sumth]

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

Answer 2

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

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

Comments

[sumth]

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

Answer 3

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

• 結果的に値を返す以外の作用も引き起こす場合は Create や Generate など

-- ファイルを作成してファイルオブジェクトを返す
-- DB に接続（セッションを作成）してDB接続オブジェクトを返す

• 提供する情報に関してそのオブジェクトが持っているデータ（そのオブジェクト全体の場合も）から、ダウンキャスト的あるいは継承関係に無いインスタンスを生成して返す場合は Create や Generate や Convert など
• そのオブジェクト（の責任範囲）の状態に変化がないあいだは返される値が同じであればプロパティ

-- グラフィック（キャンバス）オブジェクトなどにマップされる矩形領域のサイズ
-- データベース接続オブジェクトの接続状態（Status：接続状態は「データベース接続オブジェクト」の責任範囲）

• そのオブジェクト（の責任範囲）の状態に変化がなくても外的要因の変化で返される値が変わる場合は Get、とくに時間がかかったり、計算していることを強調する場合は Calculate や Compute など

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

Comments

[sumth]

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

[yuba]

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

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

[hsk]

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

Answer 4

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

Comments

[sumth]

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

Answer 5

私なら、

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

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

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

ですかね。

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

Comments

[sumth]

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

Answer 6

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})");

Comments

[sumth]

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

Answer 7

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

Comments

[sumth]

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

Answer 8

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

Comments

[sumth]

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

Answer 9

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

Comments

[sumth]

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