Entries

スポンサーサイト

上記の広告は1ヶ月以上更新のないブログに表示されています。
新しい記事を書く事で広告が消せます。

本日見かけた酷いコード9

タグ: C++ 酷いコード
// 非静的メンバ関数
void Hoge::Loop(int size)
{
	static int i = 0;
	for(i = 0; 0 < size; i++)

なぜかstaticなループカウンタ。
まさかこんなもののためにインスタンス単位のスレッド安全性が放棄されていようとは、呼び出し側は夢にも思うまい。

// 非静的メンバ関数
void Fuga::SetData(const Data& data)
{
	static int size = 0;
	
	if(size == 0)
	{
		size = data.size;
	}
	
	for(int i = 0; i < size; i++)

さすがにこれは実際のコードを簡略化しているのだが……マルチスレッドどころか、シングルスレッドで2回呼んだだけでもうどうなることか分からない。

スポンサーサイト

コメント

コメントの投稿

コメントの投稿
管理者にだけ表示を許可する

トラックバック

トラックバック URL
http://idlysphere.blog66.fc2.com/tb.php/101-0f9913a9
この記事にトラックバックする(FC2ブログユーザー)

Appendix

タグ

Blog内検索

上記広告は1ヶ月以上更新のないブログに表示されています。新しい記事を書くことで広告を消せます。