クラス名・変数名に迷ったら書き込むスレ。Part20
■ このスレッドは過去ログ倉庫に格納されています
0001デフォルトの名無しさん
2011/07/29(金) 21:32:01.00命名規則や設計の善し悪しについて議論するのは基本的に禁止。
前スレ
クラス名・変数名に迷ったら書き込むスレ。Part19
http://hibari.2ch.net/test/read.cgi/tech/1293201229/
0058デフォルトの名無しさん
2011/08/17(水) 20:03:13.18いくら設計議論のスレでなくても、さすがにそれは根本的に考え方がおかしいといわざるを....
0059デフォルトの名無しさん
2011/08/17(水) 20:05:50.81いや、考えようによっては、名前を決めるということは設計に他ならないともいえる。
だから、設計に口出すのも似たようなもんだとも思える。
0060デフォルトの名無しさん
2011/08/17(水) 22:41:29.34具体的にはStreamクラスがあります。基本は派生クラスのFileStreamやらでストリームを
扱います。ただ、WindowsのCOMに渡すとき、COMのIStreamインターフェースを
渡す必要があるのですが、そのために、既にStreamAdpaterクラスが用意されてます。
また、COMから返るのはIStreamインターフェースなのですが、通常のライブラリ
では、Streamクラスをベースに扱うようにできてるので、逆バージョンがほしいのです。
0061デフォルトの名無しさん
2011/08/17(水) 22:46:02.420062デフォルトの名無しさん
2011/08/17(水) 23:35:44.87ではベタにComStreamAdapterとか。
まあこれでよければ質問しないかもしれんけどw
でも変に捻っても意味ないよねこういうのは。
アダプダーだと考えなければ、単にComStreamというのもあり？
0063デフォルトの名無しさん
2011/08/18(木) 00:47:55.23StreamAdapterのまんまでIStreamAdapterぐらいしか思いついていませんでした。
ありがとうございます。
0064デフォルトの名無しさん
2011/08/20(土) 09:56:05.41前とか先とか関連だと思うのですが...
0065デフォルトの名無しさん
2011/08/20(土) 09:58:10.48predecessor
かな？
0067デフォルトの名無しさん
2011/08/21(日) 22:01:19.31public:
void hoge(bool _isXXX){ // _isXXXはXXXであるか判定する処理を行うかのフラグ
if (_isXXX) isXXX();
...
}
private:
bool isXXX(){...} // XXXであるか判定する
}
ここではアンダーバーをつけて区別しましたが、isXXXの名前がかぶってしまいます。
引数の方のisXXXの命名がおかしい気はするのですが、なんて命名すればいいのでしょうか？
0068デフォルトの名無しさん
2011/08/21(日) 22:40:37.91返り値使わずに捨ててるし
0069デフォルトの名無しさん
2011/08/21(日) 22:44:25.26ああメソッドの呼び出しの部分は省略しすぎました。
XXXであるかの判定結果は利用してる物としてください。
0070デフォルトの名無しさん
2011/08/21(日) 22:50:11.76class C {
public:
void hoge(bool doCheckXXX){ // doCheckXXXはXXXであるか判定する処理を行うかのフラグ
if (doCheckXXX) isXXX();
...
}
private:
bool isXXX(){...} // XXXであるか判定する
}
0071デフォルトの名無しさん
2011/08/21(日) 23:19:25.82ストリームからデータを読み取る関数 long ReadLong(), bool ReadBytes(int size) などを持つ
シーク機能は持たない
が定義されているとして、このインターフェイスを使用する側に x バイト以上の読み取りを禁止させるアダプタークラスの名前を考えて欲しい
・Xxx クラス（またはインターフェイス）: IBinaryReader インターフェイスから継承
void SetReadableBytes(int) 関数で読み取ることができるバイト数を設定することができる
設定されたバイト数を超える読み込みが発生したのであれば、その地点で例外なんたらかんたら
0072デフォルトの名無しさん
2011/08/22(月) 00:00:28.56ニュアンスがちょっと違ってくるかもしれんが、
Blocked という単語を付けてはどうだろうか
たとえば、IBinaryReaderBlocked、IBlockedBinaryReader とか
0073デフォルトの名無しさん
2011/08/22(月) 00:06:20.46その発想だと、IBinaryReaderを実装したXxxクラスをIBinaryReaderとして扱ったとき、
予期しない（チェック不能な）例外が発生しうることになる。
ちょっとおかしいのでは？
個人的にはIBinaryReader自体にストリームがサイズという概念を持つかどうかを
表すプロパティ（HasLengthとか？）を持たせた方が自然に感じる。
0074デフォルトの名無しさん
2011/08/22(月) 00:32:26.18多分君ら勘違いしてる
0075デフォルトの名無しさん
2011/08/22(月) 04:20:11.60007671
2011/08/22(月) 04:20:40.83参考にさせて貰う
自分で考えたのは
IBinaryZoneReader、IBinaryPartReader
こんな感じ
>>73
サイズという概念を持たないものが IBinaryReader だとして、サイズを持つものはどういう名前にするべき？
アダプタークラスなんて書いたのが混乱の元になっているのかもしれない
IBinaryReader 自体はソースストリームからのバイナリ形式の読み取りを行うためのインターフェイス
Xxx も基本概念は同じだけど、読み取り可能バイト数（ReadableBytes）という概念を持つ
ReadableBytes = ソースストリームのサイズ（または残りのバイト数）とは限らない
007771
2011/08/22(月) 04:34:33.18Xxx newReader(reader, 3) // 3 = 読み取り可能バイト数
pTarget->onRead(newReader);
newReader->skip(newReader->getReadableSize()); // skip(x) = x バイトを読み捨てる
}
bool HogeTarget::onRead(Xxx reader) {
if (reader->getReadableSize() < 4) {
return false;
} else {
this.value = reader->readLong();
}
}
c++とかjavaに似せたコードで書くとこんな感じ？
0078デフォルトの名無しさん
2011/08/22(月) 05:27:51.680079デフォルトの名無しさん
2011/08/22(月) 07:12:45.69それかヘッダ(3バイト・・だと)を高速に読み出したいとか。
008171
2011/08/22(月) 12:09:47.92HogeTargetにXxxを引数として受け取るセッタを持たせたい
>>77ではonRead(Xxx)になっているが、SetValue(Xxx)に置き換えてくれ
同様に、クラス名もHogeTargetからLongTargetに置き換えてくれ
また、IBinaryReader派生物はソースストリームのシークをサポートしない
0082デフォルトの名無しさん
2011/08/22(月) 13:43:29.250084デフォルトの名無しさん
2011/08/22(月) 13:59:58.820085デフォルトの名無しさん
2011/08/27(土) 04:40:13.03はーまじはー
0086デフォルトの名無しさん
2011/09/02(金) 02:50:23.14CreateかLoadにするか迷ってます
0087デフォルトの名無しさん
2011/09/02(金) 03:15:40.650088デフォルトの名無しさん
2011/09/02(金) 03:28:18.750089デフォルトの名無しさん
2011/09/02(金) 09:46:35.380090デフォルトの名無しさん
2011/09/02(金) 23:39:04.370091デフォルトの名無しさん
2011/09/03(土) 09:13:19.780092デフォルトの名無しさん
2011/09/14(水) 10:07:17.60なんて言えばいいか、非侵入型のメンバ関数を集めたクラス？
class xxx // ←これ
{
const std::wstring & GetName(void * pTarget) const;
void Print(void * pTarget, const std::wstring & s);
int Hoge(void * pTarget, int foo, int bar);
};
int data;
xxx xxxInstance;
int dataHoge = xxxInstance(&data, 2, 3);
0093デフォルトの名無しさん
2011/09/14(水) 16:00:38.58いまいち掴めないんだが、xxxのインスタンスはターゲットを使って自己の内部状態を更新するの？
0094デフォルトの名無しさん
2011/09/14(水) 16:43:58.98誤: int dataHoge = xxxInstance(&data, 2, 3);
誤: int dataHoge = xxxInstance.Hoge(&data, 2, 3);
>>93
xxx クラスのメンバ関数の記述次第だが、
xxxInstance はメンバ関数を呼び出した際に、仮引数 pTarget が示す先を更新する可能性がある
xxxInstance はメンバ関数を呼び出した際に、xxxInstance 自身（this）を更新する可能性がある
0095デフォルトの名無しさん
2011/09/14(水) 17:03:26.48う～む、それはやはりコード中での役割で名付けるべきなんじゃないかなあと思う
集めたから、ってだけでは名付けにくいな
0096デフォルトの名無しさん
2011/09/14(水) 17:38:43.76以下2つの記述は同じ動作を意味している感じ
class Foo
{
const std::wstring & GetName() const;
void Print(const std::wstring & s);
int Hoge(int foo, int bar);
...
};
Foo foo;
foo.Print("てｓｔ");
// -----
class xxx
{
const std::wstring & GetName(const Foo * pTarget) const;
void Print(const Foo * pTarget, const std::wstring & s);
int Hoge(const Foo * pTarget, int foo, int bar);
...
};
xxx xxxInstance;
Foo foo;
xxxInstance.Print(&foo, "てｓｔ");
0097デフォルトの名無しさん
2011/09/14(水) 19:37:20.12TargetHelperとかTargetExtenderみたいな感じだろうか
009892
2011/09/15(木) 01:32:42.52上記の場合、FooExtenderとか、Foo追加する機能Extenderがいいかもな
ありがとう。とりあえずその線で命名してみる
0099デフォルトの名無しさん
2011/10/05(水) 10:49:54.29そんな名前のメソッドをお願いします。
GetUserIDs(ARecordID)
みたいな。
更新したのニュアンスの英語をどこにつければいいでしょうか。お願いします。
0100デフォルトの名無しさん
2011/10/05(水) 12:45:24.43英文の中なら UserIDs と Updated の間に which を入れる
0101デフォルトの名無しさん
2011/10/05(水) 15:06:11.850102デフォルトの名無しさん
2011/10/05(水) 21:41:35.180103デフォルトの名無しさん
2011/10/05(水) 22:33:48.570104デフォルトの名無しさん
2011/10/06(木) 01:08:19.61でいいだろ
0105デフォルトの名無しさん
2011/10/06(木) 13:24:20.550106デフォルトの名無しさん
2011/10/06(木) 15:25:44.90やっぱ、しっくりくるものなさげですね。>>100みたいな感じにします。
0107デフォルトの名無しさん
2011/10/06(木) 16:14:58.500108デフォルトの名無しさん
2011/10/06(木) 16:23:54.890109デフォルトの名無しさん
2011/10/06(木) 17:45:20.740110デフォルトの名無しさん
2011/10/06(木) 17:57:06.130111デフォルトの名無しさん
2011/10/06(木) 18:00:50.42GetUpdatedByUserIDs(RecordID)ってのも名前に使うときはありなんですかね。
ググってたらそんな感じのがありましたんで。
0112デフォルトの名無しさん
2011/10/06(木) 18:48:33.150113デフォルトの名無しさん
2011/10/06(木) 19:01:37.420114デフォルトの名無しさん
2011/10/06(木) 20:33:47.81よくわからんが、GetModifiersとかGetUpdatersみたいな感じ？
>>110
むしろウンココード量産するタイプの奴はたいてい君のように嘯く
0115デフォルトの名無しさん
2011/10/06(木) 21:20:36.65↓
プログラマ脳による翻訳: ユーザーIDによって更新された"それ"を取得する
現実: ユーザーIDで更新されてください。
0116デフォルトの名無しさん
2011/10/06(木) 21:29:01.49>>104が惜しいくらいであとはセンス無し
0117デフォルトの名無しさん
2011/10/06(木) 21:56:32.16こういう場合、どういうセンスが求められているの？
0118デフォルトの名無しさん
2011/10/09(日) 15:46:05.85○○マネージャーは駄目だっていうけどこの場合マネージャーしか思い浮かばない
0119デフォルトの名無しさん
2011/10/09(日) 16:06:26.070120デフォルトの名無しさん
2011/10/09(日) 17:36:38.55一括管理って、具体的に何をするの？
0121デフォルトの名無しさん
2011/10/09(日) 18:46:08.07実際に確保解放するのをマネージャに任せて
他のリソースが必要なクラスはこのマネージャを仲介して貰う
マネージャは参照数などを管理したり別スレッドで実際の確保解放を行ったりする
0122デフォルトの名無しさん
2011/10/09(日) 20:03:54.01実質何も解決しないけど
0123デフォルトの名無しさん
2011/10/09(日) 20:25:36.91└ Hoge
じゃなくて
Hoge
└ HogeDevice
みたいに、マネージャの方をシンプルな名前にするのは？
0124デフォルトの名無しさん
2011/10/09(日) 20:39:32.12そのマネージャを利用する側のクラスの立場に立てば、
そのマネージャはリソースを持っているかのように見えるわけだ
実際に参照数を管理してるのかどうかは、利用する側にとってはどうでもいい話
もしかしたら実際には参照数以外の別の何かを管理して上手くやっているのかも知れん
スレッドが作られるかどうかも、利用する側には関係ないことなんだろ？
つまり、リソースが必要ならがそのマネージャが持っている事が分かりさえすれば良い
というわけで、単に Resource クラスでいいような気がするが、どうだろう
Resource クラスが別にあって、それを束ねているのなら ResourcePool とか
0125デフォルトの名無しさん
2011/10/10(月) 03:52:59.070126デフォルトの名無しさん
2011/10/10(月) 08:01:33.85そのまま答えろよｗ
0127デフォルトの名無しさん
2011/10/10(月) 09:28:10.270128デフォルトの名無しさん
2011/10/10(月) 12:36:45.74GCない言語でGCみたいな働きをするクラスが欲しいのかも知れんけど、
ちょっと無理があるんじゃないかと。
参照数を管理する、なんて簡単に言うけどどうやってやるんだろう....
0129デフォルトの名無しさん
2011/10/10(月) 12:52:13.65集中管理する意味は知らないけど別にできなくはない
0130デフォルトの名無しさん
2011/10/10(月) 12:56:12.77そうやって、リリース側を呼び忘れるとｗ
不穏な設計に付き合わされる日々が待っている、と。
0131デフォルトの名無しさん
2011/10/10(月) 13:08:57.68そのまま英語にしますと
transformedInfomation
translatedInformation
convertedInformation
という辺りが妥当ですが、さすがにちょっと長いです
なので省略したいのですが、後半の Information は Info に略せるとして、
前半は
transed
conved
などとしても、海外の方たちもちゅんと意図をくんでくれるでしょうか
0132デフォルトの名無しさん
2011/10/10(月) 13:26:23.42その変数が100箇所に登場するとかでなければ、変に略す必用ないでしょ。
略すにしても暗号みたいのは却下。
ただでさえ他人の書いたコードなんて暗号なのに、そんな下らない謎解きまでさせるのは勘弁して欲しい。
略すならInformationを丸ごと削る方向で。
translatedとかmodifiedとか。
もちろんこれはローカルのみ可
0133デフォルトの名無しさん
2011/10/10(月) 14:23:26.820134デフォルトの名無しさん
2011/10/10(月) 14:25:31.09> そんな下らない謎解き
つまり、transed や conved などは一般的ではない謎解きレベルということですね
参考になりました
translated にします
0135デフォルトの名無しさん
2011/10/10(月) 14:30:49.410136デフォルトの名無しさん
2011/10/10(月) 16:16:35.52例えば圧縮変換ならcompressedにすればtranslatedやencodedより明確になる
0137デフォルトの名無しさん
2011/10/10(月) 16:51:34.46なるほど、確かにその通りです
ただ、今回の質問の趣旨は
プログラミングの世界で transed や conved が一般的かどうか、というもので、
それを訊く為に仮想的な状況を作って質問しました
素直に最初から 「transed や conved は一般的か」 とだけ質問すべきでした
すいませんでした
0138デフォルトの名無しさん
2011/10/15(土) 09:00:16.830139デフォルトの名無しさん
2011/10/15(土) 10:12:57.780140デフォルトの名無しさん
2011/10/17(月) 19:30:05.18一的な謎解いいと？
0141デフォルトの名無しさん
2011/10/18(火) 22:02:21.78ユーザ登録時とかのフォームです
Entry Form、SignUp Form、Register Fromとか考えたんですが・・・
0142デフォルトの名無しさん
2011/10/18(火) 22:34:19.540143デフォルトの名無しさん
2011/10/18(火) 22:42:36.370144デフォルトの名無しさん
2011/10/19(水) 13:58:22.95② registration form: 登録フォーム
どちらも同じ用途に使われるけど、「登録フォーム」を問われているんだから②だろうね
0145デフォルトの名無しさん
2011/10/19(水) 15:14:43.360146デフォルトの名無しさん
2011/10/19(水) 15:18:35.030147デフォルトの名無しさん
2011/10/19(水) 15:19:46.590148デフォルトの名無しさん
2011/10/19(水) 19:01:54.080149デフォルトの名無しさん
2011/10/19(水) 19:30:27.280150デフォルトの名無しさん
2011/10/19(水) 21:14:40.630151デフォルトの名無しさん
2011/10/19(水) 21:19:00.060152デフォルトの名無しさん
2011/10/19(水) 21:54:19.900153デフォルトの名無しさん
2011/10/20(木) 12:45:31.290154デフォルトの名無しさん
2011/10/20(木) 13:27:10.650155デフォルトの名無しさん
2011/10/20(木) 13:52:58.100156デフォルトの名無しさん
2011/10/20(木) 14:39:11.900157デフォルトの名無しさん
2011/10/20(木) 16:05:58.870158デフォルトの名無しさん
2011/10/20(木) 18:28:03.67ConfirmedFormData: 確認済みフォームデータ
ValidFormData: 有効であるフォームデータ
■ このスレッドは過去ログ倉庫に格納されています