あるメソッドを繰り返すだけのメソッドの名前はどうしていますか？

Question

例えば、

void set(int index, int num){
a[index] = num;
}

というメソッドがあるとして、「標準入力を受け付けて、setする」を繰り返すだけのメソッドを作りたいとします。

void repeatSet(int repeat){
for(int j = 0; j < repeat; j++){
int i,n;
//標準入力からiとnに入力
set(i,n);
}
}

メソッド名は単語の最初の文字を大文字にして、repeatSetかなと思ったのですが、これだと元のsetとrepeat「Set」で形が変わってしまってなんかカッコ悪い気がします(個人の感想です)。
同じ理由でrepeatSettingも違う気がします。。。

みなさんはどうしているんでしょうか？

Answer 1

setとaの時点でダメだと思いますが、
aに相当するのが何かわからないので名付けようがないですね。

一般にスクリプト言語などでコマンドラインで入力と実行を繰り返すプログラムをrepl( read - eval - print - loop )と呼びます。
それを参考に、readXXXsFromConsoleとかどうでしょうか。
ユーザーからの入力を受け付ける事もわかった方が良いので。

入力結果は戻り値になってくれた方が扱いやすい気がしますね。

int[] readXXXsFromConsole(int times){

Answer 2

Mr_Robotoさんも言われているように、メソッド名命名の基本は「動詞+名詞」です。

で、今回の質問ですが、そもそも最終的に格納しようとしている変数名がaだというのが気に入りません。
ここをしっかり意味のある名前を定義しないと後のメソッド名もしっくりきません。

私なら、とりあえず変数aをarrayとしたら、
setメソッドは

void setArray(int index, int num){
array[index] = num;
}

とします。
そして、それを繰り返すメソッドは

void setArrayWithRepeat(int repeat){
   for(int j = 0; j < repeat; j++){
      int i,n;
      //標準入力からiとnに入力
      setArray(i,n);
   }
}

とします。

Answer 3

C++タグが付いてたのでC++で:

#include <iostream>

template<typename Function>
void times_do(int repeat, Function f) {
  for ( int i = 0; i < repeat; ++i ) {
    f(i);
  }
}

int main() {
  times_do(5, [](int n) { std::cout << n << "回目\n"; } );
}

Answer 4

配列aが何なのか判らないので名前を付けにくいですね。aはもうちょっと意味のある名前にした方が良いです。また、repeatというのは誤解を生じやすい単語なのでcountとかtimesのような名前の方が良いでしょう。1回繰り返すというと2回実行することを意味しますから。

私なら、aに何かをセットするという意味で単にsetではなくsetAとし、複数のaを入力するという意味でinputAsとかinputMultiAのような名前にします。

Answer 5

VisualStudioは日本語のメソッドも定義できますよ！
無理しないで日本語で定義してみはどうですか？

Comments

[Ryo]

VSだから日本語メソッド定義できる訳ではないですね。言語の仕様次第です。

Answer 6

基本は、動詞＋名詞です。

なので、 setInput とかじゃないですか。
そもそも提示されているコードにリピートはないですよね。

Comments

[akamakku]

誤解させてしまったかもしれません。
提示したsetメソッドを繰り返すメソッドの名前のことです。

[akamakku]

ただsetInputはしっくりくるので候補に入れたいと思います。

[Mr_Roboto]

では、誤解のないように質問を修正してくださいね ^^

[退会済みユーザー]

複数だから

setInputs

ではといってみる