Разбиение программы на модули

[Neolit1819]

Добрый день пытаюсь из программы вынести созанный мной класс в отдельный заголовочный файл. создал файл "WorkToBMP.h" где описываю класс

#pragma once
class WorkToBMP
{
public:
	BITMAPFILEHEADER bmfHeader; //заголовок файла
	BITMAPINFOHEADER bmiHeader; // структура файла
	HANDLE FileHandle;//файл для записи BMP
	DWORD bf,bf1;

	void openfile(const wchar_t* filepath);
	void scanfile();
	void closefile();


	
};

создал файл "WorkToBMP.cpp" где описывается реализация класса

#pragma once
#include "StdAfx.h"
#include "WorkToBMP.h"
#include "MFC_BMP_READDlg.cpp"


	BITMAPFILEHEADER bmfHeader; //заголовок файла
	BITMAPINFOHEADER bmiHeader; // структура файла
	HANDLE FileHandle;//файл для записи BMP
	DWORD bf,bf1;

void openfile(const wchar_t* filepath)  
	{ 
	  FileHandle=CreateFile(filepath,GENERIC_READ |
      GENERIC_WRITE, FILE_SHARE_WRITE | FILE_SHARE_READ, NULL, OPEN_EXISTING,
	  FILE_ATTRIBUTE_NORMAL, 0);
	 
	  if(INVALID_HANDLE_VALUE == FileHandle)
		Error = "File not find ";  //присваиваю глобальной переменной сообщение об ошибке
   
	}
	
	
void closefile()
	{		CloseHandle(FileHandle);	}


void scanfile()
	{
		 // читаем заголовок файла
   if (!(ReadFile(FileHandle,&bmfHeader,sizeof(bmfHeader),&bf,NULL) )) 
	  Error = "Eror header reads  "+ GetLastError(); //присваиваю глобальной переменной сообщение об ошибке
	  
  
         // читаем структура файла
    if (!(ReadFile(FileHandle,&bmiHeader,sizeof(bmiHeader),&bf1,NULL) )) 
	   Error = "Eror info reads  "+ GetLastError(); //присваиваю глобальной переменной сообщение об ошибке
       
   
   char buf[20];
   Size   = itoa(bmfHeader.bfSize,buf,10);
   Width  = itoa(bmiHeader.biWidth,buf,10);    
   Height = itoa(bmiHeader.biHeight,buf,10);  
   Compression = itoa(bmiHeader.biCompression,buf,10);
	                    	}

в главном модуле подключаю все 
#include "stdafx.h"
#include "MFC_BMP_READ.h"
#include "MFC_BMP_READDlg.h"
#include "afxdialogex.h"
#include "WorkToBMP.h"
#include "WorkToBMP.cpp"

при комплияции сбой построения
d:\мои документы\visual studio 2010\projects\mfc_bmp_read\mfc_bmp_ read\mfc_bmp_readdlg.h(34): см. предыдущее определение "OnBnClickedButtonOpen"
1>d:\мои документы\visual studio 2010\projects\mfc_bmp_read\mfc_bmp_ read\mfc_bmp_readdlg.cpp(195): error C2084: функция "void CMFC_BMP_READDlg::OnBnClickedButton Info(void)" уже имеет текст реализации
1> d:\мои документы\visual studio 2010\projects\mfc_bmp_read\mfc_bmp_ read\mfc_bmp_readdlg.h(40): см. предыдущее определение "OnBnClickedButtonInfo"

[Ivan_32]

// WorkToBMP.h
class WorkToBMP
{
public:
	BITMAPFILEHEADER bmfHeader; //заголовок файла
	BITMAPINFOHEADER bmiHeader; // структура файла
	HANDLE FileHandle;//файл для записи BMP
	DWORD bf,bf1;
	
	WorkToBMP(const wchar_t* filepath);
	~WorkToBMP();
	void openfile(const wchar_t* filepath);
	void scanfile();
	void closefile();	
};

// WorkToBMP.cpp

WorkToBMP::WorkToBMP(const wchar_t* filepath)
{
    this.openfile(filepath);
}
WorkToBMP::WorkToBMP()
{
    this.closefile();
}

WorkToBMP::openfile(const wchar_t* filepath)
{
   this.FileHandle=CreateFile(filepath,GENERIC_READ |
      GENERIC_WRITE, FILE_SHARE_WRITE | FILE_SHARE_READ, NULL, OPEN_EXISTING,
	  FILE_ATTRIBUTE_NORMAL, 0);
	 
	  if(INVALID_HANDLE_VALUE == this.FileHandle)
		Error = "File not find ";  //присваиваю глобальной переменной сообщение об ошибке
}
WorkToBMP::closefile()
{
     CloseHandle(this.FileHandle);
}

Дальше думаю все понятно. Почитайте про создание классов и к слову, захват ресурса и освобождение ресурса лучше делать в конструкторе и деструкторе соответственно, а сами объекты класса, если они не подразумеваются как общие(т.е. используются в определенный момент времени, после чего уничтожаются), лучше создавать на стеке что бы не морочится потом с уничтожением( думаю, 99% программистов хоть раз в жизни забывали уничтожать объекты ).

[Neolit1819]

Да согласен, конструктор с деструктором лишними не будут)
Переделал как вы сказали, добавив подобным образом в файл // WorkToBMP.cpp реализацию функиции сканирования файла.

При построение выдает d:\мои документы\visual studio 2010\projects\mfc_bmp_read\mfc_bmp_ read\mfc_bmp_readdlg.cpp(9): fatal error C1014: слишком много включаемых файлов: глубина = 1024
1> MFC_BMP_READDlg.cpp
1>d:\мои документы\visual studio 2010\projects\mfc_bmp_read\mfc_bmp_ read\mfc_bmp_readdlg.cpp(9): fatal error C1014: слишком много включаемых файлов: глубина = 1024
1> Создание кода...

файл WorkToBMP.cpp включает
#include "StdAfx.h"
#include "WorkToBMP.h"
#include "MFC_BMP_READDlg.cpp"

файл "MFC_BMP_READDlg.cpp"
#include "stdafx.h"
#include "MFC_BMP_READ.h"
#include "MFC_BMP_READDlg.h"
#include "afxdialogex.h"
#include "WorkToBMP.h"
#include "WorkToBMP.cpp"

Видимо, что то лишнее пробывал изменять, одно/другое убрать, добавить, все равно собрать не получается

[Ivan_32]

.cpp подключать не нужно кстати.

Что именно сбоит сказать сложно так как вы все же выложили код всего одного класса и в нем вроде бы ничего такого нет, для верности можно обернуть весь код в .h -файле класса в такую конструкцию:

#ifndef WORKTOBMP_H
#define WORKTOBMP_H

class WorkToBMP
{
public:
	BITMAPFILEHEADER bmfHeader; //заголовок файла
	BITMAPINFOHEADER bmiHeader; // структура файла
	HANDLE FileHandle;//файл для записи BMP
	DWORD bf,bf1;
	
	WorkToBMP(const wchar_t* filepath);
	~WorkToBMP();
	void openfile(const wchar_t* filepath);
	void scanfile();
	void closefile();	
};

#endif

А стоп! У вас взаимный инклуд:
WorkToBMP.h - включает MFC_BMP_READDlg.cpp
а, он свою очередь WorkToBMP.h

Вообщем уберите все инклуды .cpp файлов и все станет замечательно.

[Neolit1819]

Ivan_32 благодарю