ダブルバッファについて

質問

スタート画面にBitmapを用いたアニメーションを用いて、次の場面に移動したいのですが
実行すると画面がちかちかしてしまいます。どう直せばよいか教えてもらえると助かります
プログラムはVisualStudio2008のMFCでやっています

#include "stdafx.h"
#include "graph.h"
#include "graphDoc.h"
#include "graphView.h"
#ifdef _DEBUG
#define new DEBUG_NEW
#endif
//全域変数
CDC bkDC;
CBitmap bkBMP;
#define ID_REDRAW 30
#define TIMER_MS_FPS 1000/30
#define ID_BITMAP 60
#define TIMER_MS_BITMAP 300
#include <math.h>
int draw_state = 0;
float mouse_x = -1.0, mouse_y = -1.0;
// CgraphView

IMPLEMENT_DYNCREATE(CgraphView, CView)

BEGIN_MESSAGE_MAP(CgraphView, CView)
// 標準印刷コマンド
ON_COMMAND(ID_FILE_PRINT, &CView::OnFilePrint)
ON_COMMAND(ID_FILE_PRINT_DIRECT, &CView::OnFilePrint)
ON_COMMAND(ID_FILE_PRINT_PREVIEW, &CView::OnFilePrintPreview)

ON_COMMAND(ID_BK_WHITE, &CgraphView::OnBkWhite)
ON_COMMAND(ID_BK_BLUE, &CgraphView::OnBkBlue)
ON_COMMAND(ID_BK_PINK, &CgraphView::OnBkPink)
ON_WM_TIMER()
ON_WM_DESTROY()
ON_WM_LBUTTONDOWN()
END_MESSAGE_MAP()

// CgraphView コンストラクション/デストラクション

CgraphView::CgraphView()
: background_color(0)
, draw_state(0)
, Bmp_ID(0)
{
// TODO: 構築コードをここに追加します。
background_color = 0;
int draw_state = 0;

}

CgraphView::~CgraphView()
{
}

BOOL CgraphView::PreCreateWindow(CREATESTRUCT& cs)
{
// TODO: この位置で CREATESTRUCT cs を修正して Window クラスまたはスタイルを
// 修正してください。

return CView::PreCreateWindow(cs);
}

// CgraphView 描画

void CgraphView::OnDraw(CDC* pDC)
{

CgraphDoc* pDoc = GetDocument();
ASSERT_VALID(pDoc);
if (!pDoc)
return;

// TODO: この場所にネイティブ データ用の描画コードを追加します。
//ダブルバッファに関するコード
CRect rc;
GetClientRect(&rc);
bkDC.CreateCompatibleDC(pDC);
bkBMP.CreateCompatibleBitmap(pDC, rc.right, rc.bottom);
bkDC.SelectObject(&bkBMP);
//描画は、裏画面bkDCにおいて行う
bkDC.FillSolidRect(rc, RGB(153, 204, 255));
int x, y, sx, sy;

if(draw_state == 0)
{
bkDC.SetTextColor(RGB(255, 255, 255));
bkDC.TextOut(480, 580, _T("Start"));

sx = 128;
sy = 128;
y = 340;

x = 200;
draw_school(x, y, sx, sy);//定義は省略

x = 400;
draw_health(x, y, sx, sy);//定義は省略

x = 600;
draw_environment(x, y, sx, sy);//定義は省略
}
else if(draw_state == 1)
{
CRect myRect;
GetClientRect(myRect);
pDC->FillSolidRect(myRect, RGB(0, 0, 0));
}

//裏画面を表画面に送る
//pDC->BitBlt(0, 0, rc.right, rc.bottom, &bkDC, 0, 0, SRCCOPY);
//裏画面を消去
bkBMP.DeleteObject();
bkDC.DeleteDC();
//実行画面のプログラムを省略
　　　void CgraphView::OnInitialUpdate()
{
CView::OnInitialUpdate();

// TODO: ここに特定なコードを追加するか、もしくは基本クラスを呼び出してください。
bitmap[0].LoadBitmap(IDB_BITMAP1);
bitmap[1].LoadBitmap(IDB_BITMAP2);
bitmap[2].LoadBitmap(IDB_BITMAP3);
bitmap[3].LoadBitmap(IDB_BITMAP4);
bitmap[4].LoadBitmap(IDB_BITMAP5);
bitmap[5].LoadBitmap(IDB_BITMAP6);
//タイマーをスタート
SetTimer(ID_BITMAP, TIMER_MS_BITMAP, NULL);
//画面再描画のタイマーをスタートする
SetTimer(ID_REDRAW, TIMER_MS_FPS, NULL);
Bmp_ID = 0;

}

void CgraphView::OnTimer(UINT_PTR nIDEvent)
{
// TODO: ここにメッセージ ハンドラ コードを追加するか、既定の処理を呼び出します。
if(nIDEvent == ID_BITMAP)
Bmp_ID = (Bmp_ID+1) % 2;

if(nIDEvent == ID_REDRAW)
InvalidateRect(NULL, FALSE);

CView::OnTimer(nIDEvent);
}

void CgraphView::OnDestroy()
{
CView::OnDestroy();

// TODO: ここにメッセージ ハンドラ コードを追加します。
KillTimer(ID_BITMAP);
KillTimer(ID_REDRAW);
}

void CgraphView::OnLButtonDown(UINT nFlags, CPoint point)
{
// TODO: ここにメッセージ ハンドラ コードを追加するか、既定の処理を呼び出します。
mouse_x = point.x;
mouse_y - point.y;

if(draw_state == 0)
{
if(mouse_x > 420 && mouse_y > 500)
{
draw_state = 1;
}
}

CView::OnLButtonDown(nFlags, point);
}

//以下は関数の定義なので省略

回答No.7

m-take0220

＞ pOldBitmap = bkDC.SelectObject(&Bitmap);
＞ bkDC.SelectObject(&bkBMP);

この部分は、

pOldBitmap = bkDC.SelectObject(&bkBMP);

でいいです。(当然、Bitmapは必要ありません)

ちらつきの件は、最初回答にあるもう一つの部分が原因の可能性が高いです。

補足

最初の回答とはWM_ERASEBACKGROUNDの部分ですよね

回答No.6

m-take0220

だから、元に戻してるんですか?
作成したビットマップを削除しても、元に戻るわけではありませんよ。

補足

下記のように選択したビットマップを元に戻しましたがうまくいきませんでした
//ダブルバッファに関するコード
CRect rc;
GetClientRect(&rc);
bkDC.CreateCompatibleDC(pDC);
bkBMP.CreateCompatibleBitmap(pDC, rc.right, rc.bottom);
CBitmap Bitmap, *pOldBitmap;
pOldBitmap = bkDC.SelectObject(&Bitmap);
bkDC.SelectObject(&bkBMP);

//裏画面を表画面に送る
pDC->BitBlt(0, 0, rc.right, rc.bottom, &bkDC, 0, 0, SRCCOPY);
bkDC.SelectObject(pOldBitmap);
//裏画面を消去
bkBMP.DeleteObject();
bkDC.DeleteDC();

回答No.5

m-take0220

間違ってるから指摘してるんですが。

＞ 必ず元の（既定）オブジェクトに置き換えるべきです。

の部分を実行しているコードがあるなら、抜き出してみてください。

補足

SelectObjectを使用しているのはbkDC.SelectObject(&bkBMP);だけだと思うのですが違いますか

回答No.4

m-take0220

デバイスコンテキストに選択された状態のビットマップを削除してしまうと、そのデバイスコンテキストはどうなるのでしょうか?

また、SelectObjectの説明には

＞ この関数は、指定されたのと同じタイプで、それまで選択されていたオブジェクトを返します。
＞ アプリケーションは新しいオブジェクトを使い終えたら、必ず元の（既定）オブジェクトに置き換
＞ えるべきです。

http://msdn.microsoft.com/ja-jp/library/cc410576.aspx

という記述がありますが、その通りにしていますか?

このようなことは、デバイスコンテキストを使った描画の解説には、必ず書かれているはずの事柄です。だから、もう一度確認しましょうと言っているのです。

補足

使い方は間違っていないと思うのですがどこかおかしいですかね

回答No.3

m-take0220

ビットマップに限らず、デバイスコンテキストに選択したGDIオブジェクト(ペン・ブラシ・フォントなど)は、使用し終わったら選択解除する(元々デバイスコンテキストに選択されていたオブジェクトを選択しなおす)必要があります。
デバイスコンテキストを使用した描画の解説では、必ず出てくる部分なので、もう一度確認してみてください。

補足

選択したGDIオブジェクトはDeleteObject()を使ってちゃんと削除しているのですがどこが違うのですか

回答No.2

m-take0220

ビットマップを削除している部分と、具体的に指摘してるんですが。
どのコードかまで指摘しなければならないとすると、あなたがこれらのコードを理解していないということになります。それでは修正は難しいでしょう。

補足

ビットマップを削除する部分は分かるのですが、選択したビットマップを解除している部分がどういうことなのか分かりません

回答No.1

m-take0220

選択したビットマップを解除せずに削除しようとしてたり、問題が結構あると思います。
もう一度、描画コードの基本を確認してみては?
bkDC、bmBMPは関数内で定義しても問題ないと思いますし。

あと、WM_ERASEBACKGROUNDをどう処理しているかも確認したほうがいいでしょう。

補足

具体的にどの部分を修正すればよいか
上のプログラムの場所で教えてもらえますか？