使いたいメソッド名が実際に使う場合と違う場合

Question

Web側で見積作成機能を追加しているのですが、同じようなことをバッチ処理機能で見つけ利用しようとしたのですが、
メソッド名がtestForBatch();とバッチ用になっているために、
新たにtestForWeb();メソッドを作成してtestForBatch();を呼ぶかそのままのメソッド名で呼ぶか悩んでます。

本当はtestForBatchをもっと汎用性のある名前に変えたいのですが、
すでにいくつで利用されているので、再テストやバグが怖いのでそれは避けたいと考えてます。

名前だけの問題なのですが、みなさんならどうしますでしょうか。
他にもよい方法があれば教えてください。
よろしくお願いします。

Answer 1

こんにちは。

実際のコードの状況を見ないとはっきりしたことは言えない、という前提の
もとで回答させて頂きます。

WEBからの入力で、見積り機能を呼ぶときに、

testForBatch();

をそのまま使うのは、私だったら
「ちょっと嫌だな。なんとかしたいな」
と思うだろうと思います。
たとえ、testForWeb()が、testForBatch()を、全くどこも修正しなくても
ＯＫである場合であっても

function testForWeb()
{
  return testForBatch();
}

のように、testForWeb()を作成すると思います。

ただし、このような名前へのこだわり度というのは、個人差があるので、

function testForWeb()
{
  return testForBatch();
}

のようなものをわざわざ作るのは意味がないという考えをする方も
いらっしゃると思えまして、それはそれで間違いとはいえないかと
思います。

なので、以下、あくまで私見ということになりますが、
既存の関数
testForBatch()
を（ほぼ）そのままWEB入力からも流用するという状況が発生したということは、
そのような状況が発生したその時点においては
testForWeb()
は
testForBatch()
とまるっきり同じでよいとしても、将来的にはおそらく

testForBatch() ＝ 見積り作成処理本体＋バッチ用のI/O

testForWeb() ＝ 見積り作成処理本体＋WEB用のI/O

のように分解して、見積り作成処理本体はひとつの関数（なりクラスなり）にし、
バッチとＷＥＢとで異なるI/O部分だけをそれぞれの関数に書いたほうが見通しが
よいような状況に進展するのではないか？ さらに言えば、第3のI/Oに対応した

testForXxx()

の必要性も出てくるのではないか、と私だったら推測してしまうので、
その布石として

function testForWeb()
{
  return testForBatch();
}

であったり、もしくは現状の
function testForBatch()
をリネームして
function testCommon()
としたうえで、

function testForBatch()
{
  return testCommon(); //元testForBatch
}

function testForWeb()
{
  return testCommon(); //元testForBatch
}

の２つを作成して、先の将来予測に備えるかもしれません。

それと、ひと手間かけて

function testForWeb()
{
  return testCommon(); //元testForBatch
}

を作っておいて、WEBからの入力はこちらを使うことにしたときの
（ささやかな or 場合によってはけっこう大きな）メリットとして、
このWEB入力のコードを誰かに改修をお願いするとして、その中に
出てくる見積り作成が、testForBatch() のままだと

「このtestForBatch() なのですが、ここでバッチを起動しているんですか？
バッチはcronやJenkinsで夜間に回してたりとかしないんですか？」

といった質問をされない、ということがあるかと思います。
この質問への回答として、

「バッチじゃないんだけど、(関数名は)バッチなんです。（キリッ」

といった（やや苦しい）説明を私なら避けたいと思いますので。

お役に立てる回答になっているか心もとないのですが、
私が思うところは以上です。

Comments

[matsujin_]

返答ありがとうございます。
返信が遅れて申し訳ありません。
ここまで丁寧に解説していただけるとは思ってなかったので
とても感謝しておりますm(_ _)m
ykt68さんの意見をもとに修正していきたいと思います。
ありがとうございます。

P・S こういう意見を謙虚に丁寧に言える方と仕事や勉強会をしてみたいと初めて思いました。