ohden ohden 2015-01-23 11:28:22

[C#] なぜ関数化する... このエントリーをはてなブックマークに追加

投稿者からのアピールポイント

この際、SQLを文字列で作ってるのには目をつぶったとして... なぜStringBuilder使って結合する?SQL作るだけの関数作る? アプリケーション内で使用するSQLの数だけ関数が有る...そして、makeSql関数内にロジックが入ってるものは1つも無い...

private static readonly string SQL_SELECT_TO_TARGET_HOGE = makeSqlSelectToHoge();

private static string makeSqlSelectToHoge()
{
	StringBuilder sb = new StringBuilder();

	sb.Append(" SELECT ");
	sb.Append(" HOGE_ID, ");
	sb.Append(" HOGE_CODE ");
	sb.Append(" FROM ");
	sb.Append(" HOGE_MST ");
	sb.Append(" WHERE ");
	sb.Append(" HOGE_ID = @Value ");
	sb.Append(" AND DEL = '0'; ");

	return sb.ToString();
}

使い方ヒント: 「これは臭う」という行を見付けたら、各行のsmellをクリックしてマーキングしておきましょう(要Twitter OAuth認証)

コメント(1)

#1 9YdxOr 9YdxOr 2015-06-03 15:53:41  

しかし、すべての SQL 文を回収したいということはありえます

コメント投稿には、twitter認証が必要です。

Twitter認証

このウンコードに臭った人は、こちらのウンコードにも臭ってます

[Java] 連番

このエントリーをはてなブックマークに追加

もはや人間が読むものではない。

...

package com.renban.erq053.czp008;

/**...

鑑賞する »

[C] ( 0][0 )/

このエントリーをはてなブックマークに追加

とある組み込み系のコード。元気に動いては...

#define NUM_PORT 8
#define NUM_CHANNEL ...

鑑賞する »

[JavaScript] ちょwww画像URLでHANTEI!?w...

このエントリーをはてなブックマークに追加

数々のPGが散っていった案件を引き継いだ...

var c_icon = new Array;
c[0] = "http://...

鑑賞する »