mikecat_mixc mikecat_mixc 2013-08-25 16:20:37

[C++] 関数化しようよ(提案) このエントリーをはてなブックマークに追加

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

DXライブラリである座標を中央にして文字列を描画し、ボタンとするコード。同じ文字列リテラルを3個ずつ使用してしまっていてよくない。

DrawGraph(50,300,
	buttonImage[loadStatus==GAME_STATUS_MAINGAME_INIT?1:0],
	FALSE);
DrawString(
	50+75-GetDrawStringWidth("スタート",
		lstrlen("スタート"))/2,
	300+20-GetFontSize()/2,
	"スタート",clBlack);
DrawGraph(245,300,
	buttonImage[loadStatus==GAME_STATUS_PRACTICE_INIT?1:0],
	FALSE);
DrawString(
	245+75-GetDrawStringWidth("練習スタート",
		lstrlen("練習スタート"))/2,
	300+20-GetFontSize()/2,
	"練習スタート",clBlack);
DrawGraph(440,300,
	buttonImage[loadStatus==GAME_STATUS_TUTORIAL_INIT?1:0],
	FALSE);
DrawString(
	440+75-GetDrawStringWidth("ルール説明",
		lstrlen("ルール説明"))/2,
	300+20-GetFontSize()/2,
	"ルール説明",clBlack);
DrawGraph(50,380,
	buttonImage[loadStatus==GAME_STATUS_RANKING_INIT?1:0],
	FALSE);
DrawString(
	50+75-GetDrawStringWidth("ランキング",
		lstrlen("ランキング"))/2,
	380+20-GetFontSize()/2,
	"ランキング",clBlack);
DrawGraph(245,380,
	buttonImage[loadStatus==GAME_STATUS_OMAKE_INIT?1:0],
	FALSE);
DrawString(
	245+75-GetDrawStringWidth("おまけ",
		lstrlen("おまけ"))/2,
	380+20-GetFontSize()/2,
	"おまけ",clBlack);
DrawGraph(440,380,
	buttonImage[loadStatus==GAME_STATUS_END?1:0],
	FALSE);
DrawString(
	440+75-GetDrawStringWidth("終了",
		lstrlen("終了"))/2,
	380+20-GetFontSize()/2,
	"終了",clBlack);

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

コメント(4)

#1 c_9_l c_9_l 2013-08-31 12:48:51  

似たようなあちらこちらに部分がいっぱい…

#2 tkcomcom1is tkcomcom1is 2013-08-31 16:03:33  

この問題は「同じ文字列を3個ずつ使用している」という問題だけ?
アピール文中の「同じものを使用している」という要素は他にあるはずだと思います。

#3 panzer_jagdiron panzer_jagdiron 2013-09-02 18:42:00  

・メソッドの改行のやりかたが気に食わない
・文字列はどこかで宣言して使いまわしていいんじゃない?
・ルーチン化しよう(提案)

#4 YanaPIIDXer YanaPIIDXer 2014-04-10 21:40:59  

文字列もそうだけど座標がマジックナンバーなのが一番問題

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

Twitter認証

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

[その他] [PL/SQL]自動生成?いいえ、手書き...

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

処理本体はあえて除外したくなるほどの衝撃...

PROCEDURE my_annonymous_arguments (
   ...

鑑賞する »

[PHP] 値を返すかどうかはあなた次第

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

某Wordpressのプラグインより。 ...

function get_item_custom( $post_id, $typ...

鑑賞する »

[PHP] データベースに・・・・うわぁぁぁぁぁあ

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

例外って何だろう。SQLってなんだろう。...

// DBに接続
$pdo = new PDO($dsn,$user,$pas...

鑑賞する »