ohden 2015-01-23 02: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 2015-06-03 06:53:41  

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

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

Twitter認証

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

[Java] 連番

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

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

...

package com.renban.erq053.czp008;

/**...

鑑賞する »

[VB.net] 教育的指導

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

このプロジェクトではレビューが行き届いて...

Public Function Keisan() As Object

...

鑑賞する »

[VB.net] 大工道具の置き忘れ

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

その昔、寺や神社の建立に携わった大工は「...

Public Function GetData(capCode As Strin...

鑑賞する »