デバイスコンテキスト上で描画した画像を.icoファイルに出力

Question

実現したいこと

デバイスコンテキスト上のビットマップに画像をロードして図形とテキストを追記した後、.icoファイルとして出力したいです。

発生している問題・エラーメッセージ

出力した画像の画質が荒くなり、画質の荒れ具合も毎回変化して不安定。
元となる画像のビット数は24bitなのに、出力される画像のビット数が32bitになってしまう。

調査内容

以下のようにCImageクラスを利用しないパターンも実装してみましたが、出力されるファイルが真っ黒の何も描画されていない画像になってしまいます。

void CTalkListDlg::MakeUnreadCountIcon( int nUnreadCount )
{
	HICON hIcon = (HICON)LoadImage( AfxGetApp( )->m_hInstance, MAKEINTRESOURCE( アイコンファイルのリソースID ), IMAGE_ICON, 32, 32, LR_DEFAULTCOLOR );

	HDC hDC = ::GetDC( m_hWnd );
	HDC hMemDC = CreateCompatibleDC( hDC ); // メモリデバイスコンテキスト作成
	HBITMAP hBmp = CreateCompatibleBitmap( hMemDC, 32, 32 ); // サイズ指定してビットマップ作成
	::ReleaseDC( m_hWnd, hDC );
	SelectObject( hMemDC, hBmp );
	DrawIconEx( hMemDC, 0, 0, hIcon, 32, 32, 0, NULL, DI_NORMAL | DI_DEFAULTSIZE );

	// 赤い円を描画
　　　省略
	Ellipse( hMemDC, nCircleLeft, nCircleTop, nCircleRight, nCircleBottom );
	省略
	//

	// 未読件数を描画
	CString strUnreadCount;
	省略
	DrawText( hMemDC, strUnreadCount, strUnreadCount.GetLength( ), rcText, DT_SINGLELINE | DT_CENTER | DT_VCENTER );
	DeleteObject( hNewFont );
	//

	CString strOutputFile = _T( "アイコンファイルの出力先パス" );
	HBITMAP hBitmap = (HBITMAP)GetCurrentObject( hMemDC, OBJ_BITMAP );
	SaveIconFile( hBitmap, strOutputFile );

	DeleteObject( hIcon );
	::ReleaseDC( m_hWnd, hDC );
}

結果

元の画像

出力される画像
※外側の黒い枠はスクリーンショットをトリミングした際の名残です。

該当のソースコード

typedef struct _ICONDIR
	{
		WORD    wReserved;			// 予約 常に0
		WORD    wResourceType;		// リソースタイプ　1:アイコン　2:カーソル
		WORD	wResourceCount;	// アイコンの数
	} ICONDIR;

	typedef struct _ICONDIRENTRY
	{
		BYTE	bWidth;		// アイコンの幅
		BYTE	bHeight;		// アイコンの高さ
		BYTE	bColorCount;	// カラー数
		BYTE	bReserved;	// 予約
		WORD	wColorPlanes;	// カラープレーン数（カーソル時、Ｘホットスポット）
		WORD	wPixcelBit;	// ピクセルのビット深度（カーソル時、Ｙホットスポット）
		DWORD	dwIconFileSize;	// アイコンファイルのサイズ
		DWORD	dwIconOffset;	// ファイルの先頭からピクセルデータまでのオフセット
	} ICONDIRENTRY;

void CTalkListDlg::ChangeIconUnreadState()
{
	int nUnreadCount = CountUnreadMessage(); // 未読コメント数を取得

	if ( 0 < nUnreadCount ) { // 未読コメントがある場合
		// 未読件数付きのアイコンを作成
		MakeUnreadCountIcon( nUnreadCount );
		
		// タスクバーアイコン変更
		theApp.m_hTaskBarIcon = (HICON)LoadImage( AfxGetApp()->m_hInstance, _T( "アイコンファイルのパス" ), IMAGE_ICON, 0, 0, LR_LOADFROMFILE );
		HICON hIcon = (HICON)LoadImage( AfxGetApp()->m_hInstance, MAKEINTRESOURCE( アイコンファイルのリソースID ), IMAGE_ICON, 16, 16, LR_DEFAULTCOLOR );
		SetIcon( hIcon, FALSE );
		SetIcon( theApp.m_hTaskBarIcon, TRUE );
        }
}

//// 未読件数を表示した画像を作成
void CTalkListDlg::MakeUnreadCountIcon( int nUnreadCount )
{
	HICON hIcon = (HICON)LoadImage( AfxGetApp( )->m_hInstance, MAKEINTRESOURCE( アイコンファイルのリーソースID ), IMAGE_ICON, 32, 32, LR_DEFAULTCOLOR );

	CImage ciLoadImage;
	CImage ciBackImage;
	CDC *pDC = GetDC();
　　  CString strIconPath;
	strIconPath.Format( _T( "下地となるアイコンファイルのパス" ) );
	ciLoadImage.Load( strIconPath );
	ciBackImage.Create( 32, 32, 24, 0 );
	HDC hDC = ciBackImage.GetDC( );
	DrawIconEx( hDC, 0, 0, hIcon, 32, 32, 0, NULL, DI_NORMAL );
	ciLoadImage.Draw( hDC, 0, 0, 32, 32 );
	ciLoadImage.Destroy();

	//HDC hDC = ::GetDC( m_hWnd );
	//HDC hMemDC = CreateCompatibleDC( hDC ); // メモリデバイスコンテキスト作成
	//HBITMAP hBmp = CreateCompatibleBitmap( hMemDC, 32, 32 ); // サイズ指定してビットマップ作成
	//::ReleaseDC( m_hWnd, hDC );
	//SelectObject( hMemDC, hBmp );
	//DrawIconEx( hMemDC, 0, 0, hIcon, 32, 32, 0, NULL, DI_NORMAL | DI_DEFAULTSIZE );

	// 赤い円を描画
	int nCircleLeft = 13;
	int nCircleTop = 13;
	int nCircleRight = 32;
	int nCircleBottom = 32;
	HPEN   hOldPen = (HPEN)SelectObject( hDC, GetStockObject( NULL_PEN ) );
	HBRUSH hNewBrush = (HBRUSH)CreateSolidBrush( RGB( 255, 0, 0 ) );
	HBRUSH hOldBrush = (HBRUSH)SelectObject( hDC, hNewBrush );
	Ellipse( hDC, nCircleLeft, nCircleTop, nCircleRight, nCircleBottom );
	SelectObject( hDC, hOldPen );
	SelectObject( hDC, hOldBrush );
	DeleteObject( hNewBrush );
	//

	// 未読件数を描画
	CString strUnreadCount;
	HFONT hNewFont;
	if ( 99 < nUnreadCount ) {
		strUnreadCount = _T( "99+" );
		hNewFont = CreateFont( 16, 7, 0, 0, FW_MEDIUM, FALSE, FALSE, 0, SHIFTJIS_CHARSET, OUT_DEFAULT_PRECIS, CLIP_DEFAULT_PRECIS, DEFAULT_QUALITY, DEFAULT_PITCH | FF_DONTCARE, _T( "メイリオ" ) );
	}
	else if ( 9 < nUnreadCount ) {
		strUnreadCount.Format( _T( "%d" ), nUnreadCount );
		hNewFont = CreateFont( 16, 9, 0, 0, FW_MEDIUM, FALSE, FALSE, 0, SHIFTJIS_CHARSET, OUT_DEFAULT_PRECIS, CLIP_DEFAULT_PRECIS, DEFAULT_QUALITY, DEFAULT_PITCH | FF_DONTCARE, _T( "メイリオ" ) );
	}
	else {
		strUnreadCount.Format( _T( "%d" ), nUnreadCount );
		hNewFont = CreateFont( 18, 11, 0, 0, FW_MEDIUM, FALSE, FALSE, 0, SHIFTJIS_CHARSET, OUT_DEFAULT_PRECIS, CLIP_DEFAULT_PRECIS, DEFAULT_QUALITY, DEFAULT_PITCH | FF_DONTCARE, _T( "メイリオ" ) );
	}
	HGDIOBJ hObj = SelectObject( hDC, hNewFont );

	int nDifference = 4;
	CRect rcText;
	rcText.left = nCircleLeft + nDifference;
	rcText.top = nCircleTop + nDifference;
	rcText.right = nCircleRight - nDifference;
	rcText.bottom = nCircleBottom - nDifference;
	SetTextColor( hDC, RGB( 255, 255, 255 ) );
	SetBkColor( hDC, RGB( 255, 0, 0 ) );
	DrawText( hDC, strUnreadCount, strUnreadCount.GetLength( ), rcText, DT_SINGLELINE | DT_CENTER | DT_VCENTER );
	DeleteObject( hNewFont );
	//

	CString strOutputFile = _T( "出力するアイコンファイルの保存先パス" );
	HBITMAP hBitmap = (HBITMAP)GetCurrentObject( hDC, OBJ_BITMAP );
	SaveIconFile( hBitmap, strOutputFile );

	//ciBackImage.Draw( *pDC, 0, 0, 32, 32 ); // 描画する画像の確認用
	ciBackImage.ReleaseDC();
	ciBackImage.Destroy();

	DeleteObject( hIcon );
	::ReleaseDC( m_hWnd, hDC );
}

int sizebits( BITMAPINFO *lpbi )
{
	if ( 1 != lpbi->bmiHeader.biPlanes ) {
		return -1;
	}

	switch ( lpbi->bmiHeader.biBitCount ) {
		default:
			return -1;
		case 1:
		case 4:
		case 8:
		case 16:
		case 24:
		case 32:
			break;
	}
	int szbits = ( ( ( ( lpbi->bmiHeader.biWidth * lpbi->bmiHeader.biBitCount + 7 ) / 8 + 3 ) / 4 ) * 4 ) * ( lpbi->bmiHeader.biHeight );
	return szbits;
}

// Bitmapヘッダー作成、Bitmapファイル出力
void CTalkListDlg::SaveIconFile( HBITMAP hBmp, CString strFileName )
{
	ICONDIR			icDirHeader = { 0 };	// アイコンファイルヘッダー 　　　6byte
	ICONDIRENTRY	icDirEntry = { 0 };		// アイコン情報				16byte
	BITMAPINFO		*pBmpInfo;		// ビットマップ情報			40byte
	
	BITMAP			bmp = { 0 };	// ビットマップ構造体
	LONG			imageSize;　　　// 画像のサイズ
	LONG			pixcels;		// 画素数
	LPBYTE			bits;			// 画像ビット
	HDC				hDC, hMemDC;

	// BITMAP情報を取得する
	GetObject( hBmp, sizeof( bmp ), &bmp );
	hDC = ::GetDC( 0 );
	hMemDC = CreateCompatibleDC( hDC );
	::ReleaseDC( 0, hDC );
	HBITMAP hBmpOld = (HBITMAP)SelectObject( hMemDC, hBmp );

	// ファイルサイズ計算
	imageSize = bmp.bmWidthBytes * bmp.bmHeight + sizeof( ICONDIR ) + sizeof( ICONDIRENTRY ) + sizeof( BITMAPINFOHEADER );
	switch ( bmp.bmBitsPixel ) {
		case 2:
			pixcels = 2;
			break;
		case 4:
			pixcels = 16;
			break;
		case 8:
			pixcels = 256;
			break;
		default:
			pixcels = 0;
	}
	imageSize += ( sizeof( RGBQUAD ) * pixcels );

	// ICONDIR出力
	ZeroMemory( &icDirHeader, sizeof( icDirHeader ) );
	icDirHeader.wReserved = 0;
	icDirHeader.wResourceType = 1;
	icDirHeader.wResourceCount = 1;

	// BITMAPINFOHEADER 出力
	pBmpInfo = new BITMAPINFO[sizeof( BITMAPINFOHEADER ) + sizeof( RGBQUAD ) * pixcels];
	ZeroMemory( pBmpInfo, sizeof( BITMAPINFOHEADER ) );
	pBmpInfo->bmiHeader.biSize = sizeof( BITMAPINFOHEADER );
	pBmpInfo->bmiHeader.biWidth = bmp.bmWidth;
	pBmpInfo->bmiHeader.biHeight = bmp.bmHeight;
	pBmpInfo->bmiHeader.biPlanes = 1; // 1
	pBmpInfo->bmiHeader.biBitCount = bmp.bmBitsPixel;
	pBmpInfo->bmiHeader.biCompression = BI_RGB; // 非圧縮
	if ( 0 != pixcels ) {
		GetDIBColorTable( hMemDC, 0, pixcels, pBmpInfo->bmiColors ); // カラーDIB情報取得
	}

	// 画像データ取得
	bits = new BYTE[bmp.bmWidthBytes * bmp.bmHeight];
	GetDIBits( hMemDC, hBmp, 0, bmp.bmHeight, bits, pBmpInfo, DIB_RGB_COLORS );

	// ICONDIRENTRY出力
	char chWidth = bmp.bmWidth;
	char chHeight = bmp.bmHeight;

	icDirEntry.bWidth = chWidth;
	icDirEntry.bHeight = chHeight;
	icDirEntry.bColorCount = 1 << pBmpInfo->bmiHeader.biBitCount;
	icDirEntry.bReserved = 0;
	icDirEntry.wColorPlanes = pBmpInfo->bmiHeader.biPlanes;
	icDirEntry.wPixcelBit = pBmpInfo->bmiHeader.biBitCount;
	
	int palettesize = pBmpInfo->bmiHeader.biBitCount <= 8 ? ( 1 << pBmpInfo->bmiHeader.biBitCount ) : 0;
	icDirEntry.dwIconFileSize = palettesize + sizebits( pBmpInfo ) + sizebits( pBmpInfo ); // 対応するビットマップデータのバイト数
	icDirEntry.dwIconOffset = sizeof( ICONDIR ) + sizeof( ICONDIRENTRY ); // 22

	// バイナリファイル書き込み
	CString strErrorMsg;
	WriteIconBinary( strFileName, icDirHeader, icDirEntry, *pBmpInfo, bits, strErrorMsg );

	delete[] bits;
	delete[] pBmpInfo;
	SelectObject( hMemDC, hBmpOld );
	DeleteDC( hMemDC );
}

補足情報（FW/ツールのバージョンなど）

開発環境　Windows　VisualStudio 2013 C++

Answer 1

WriteIconBinary()の内容が分からないので、ソースからわかる部分だけ。

• GetDIBits()の説明に書かれている「アプリケーションがこの関数を呼び出すときに、 hbmp パラメーターで識別されるビットマップをデバイス コンテキストに選択しないでください。」が守られていない。
  • SaveIconFile()内部だけでなく、それを呼び出している部分でもGetCurrentObject()で選択中のビットマップハンドルを取得している。
• SaveIconFile()では描画しているわけではないからメモリDCを作成する必要はないのでは。
• pBmpInfoは配列として確保する必要はないのでは。
• デバイスコンテキストにビットマップ・フォントを選択した後、元に戻していない。
  • フォントをDeleteObject()しても、選択中は失敗する。
• LoadImage()でファイルを読み込む場合は、hInstにはNULLを設定する。
• CImage::GetDC()で取得したhDCを、CImage::ReleaseDC()で解放したあとにさらに::ReleaseDC()に渡している。
• SetIcon()に指定したアイコンハンドルって、メンバー変数に保存しておくなどして、使わなくなったら削除する必要があるのでは。
• 構造体の定義部分は#pragma packを使ってアラインメントを指定したほうが良さそう。もしデフォルトのアラインメントでパディングが挿入されると、正しいファイルが出力されない。