レビューに回ってきたコード。実際には各クラスは別ファイルに分かれていてこの順番に読んでいった。
MyClass1 を見て ⇒ 実装漏れか?作っちゃった消し忘れたごみかな?
MyClass2 を見て ⇒ またかよ・・・
MyClass3 を見て ⇒ もしかしてBaseClassを継承したクラスを必ず実装すべしみたいな開発標準でもあるのかな?
BaseClassを見て ⇒ (゚Д゚;)
このレベルのコーダーがGetTypeメソッドを知っていることがむしろ驚きだった。
public class BizClass1 : BaseClass {
//省略してるんじゃなくてホントに継承してるだけ(なんかコメントはあった)
}
public class BizClass2 : BaseClass {
//これも省略してるんじゃなくてホントに継承してるだけ(なんかコメントはあった)
}
public class BizClass3 : BaseClass {
//これも省略してるんじゃなくてホントに継承してるだけ(なんかコメントはあった)
}
public abstract class BaseClass {
void Method(object paramA, object paramB) {
// なんか共通的な処理
if( this.GetType().ToString() == "BizClass1" ) {
int intA = (int)paramA;
//Aを使った処理
} else if ( this.GetType().ToString() == "BizClass2" ){
string strA = (string)paramA;
string strB = (string)paramB;
//AとBを使った処理
}
}
// メソッドはこれひとつ
}
私が若かったころは、GetTypeなんて便利なものは無くて。。。。。 と、古き良き時代のころの話を延々とやって、時代遅れの先輩っていうイメージを植えつけてしまうことになりそうだ
typeofは、知らないのかな?
あ、書いた人はそんなこと言ってました、しかも自慢げに。僕はトラブル対応に駆り出されてレビューしてるんですけどね?